Rather than requiring --info-sssd-domain as an argument make it
an optional argument, defaulting to enabling all SSSD domains.
Convert the argument from a single value into a list so that multiple
invocations can be made and all domains in the list will be enabled.
There is still the possibility that failures in configuring a domain
will occur (no domain found, for example) and these are considered
"soft" failures. That is it won't abort the server installation.
https://fedorahosted.org/ipsilon/ticket/78
Signed-off-by: Rob Crittenden <rcritten@redhat.com>
Reviewed-by: Patrick Uiterwijk <puiterwijk@redhat.com>
default='no',
help='Use mod_lookup_identity and SSSD to populate'
' user attrs')
default='no',
help='Use mod_lookup_identity and SSSD to populate'
' user attrs')
- group.add_argument('--info-sssd-domain', action='store',
+ group.add_argument('--info-sssd-domain', action='append',
help='SSSD domain to enable mod_lookup_identity'
' for')
help='SSSD domain to enable mod_lookup_identity'
' for')
if opts['info_sssd'] != 'yes':
return
if opts['info_sssd'] != 'yes':
return
- if not opts['info_sssd_domain']:
- print 'info-identity-domain is required'
- return False
confopts = {'instance': opts['instance']}
confopts = {'instance': opts['instance']}
print 'Loading SSSD config failed: %s' % e
return False
print 'Loading SSSD config failed: %s' % e
return False
- try:
- domain = sssdconfig.get_domain(opts['info_sssd_domain'])
- except SSSDConfig.NoDomainError:
- print 'No domain %s' % opts['info_sssd_domain']
+ if not opts['info_sssd_domain']:
+ domains = sssdconfig.list_domains()
+ else:
+ domains = opts['info_sssd_domain']
+
+ for domain in domains:
+ try:
+ sssd_domain = sssdconfig.get_domain(domain)
+ except SSSDConfig.NoDomainError:
+ print 'No SSSD domain %s' % domain
+ continue
+ else:
+ sssd_domain.set_option(
+ 'ldap_user_extra_attrs', ', '.join(SSSD_ATTRS)
+ )
+ sssdconfig.save_domain(sssd_domain)
+ configured += 1
+ print "Configured SSSD domain %s" % domain
+
+ if configured == 0:
+ print 'No SSSD domains configured'
- domain.set_option('ldap_user_extra_attrs', ', '.join(SSSD_ATTRS))
-
try:
sssdconfig.new_service('ifp')
except SSSDConfig.ServiceAlreadyExists:
try:
sssdconfig.new_service('ifp')
except SSSDConfig.ServiceAlreadyExists:
ifp.set_option('user_attributes', '+' + ', +'.join(SSSD_ATTRS))
sssdconfig.save_service(ifp)
ifp.set_option('user_attributes', '+' + ', +'.join(SSSD_ATTRS))
sssdconfig.save_service(ifp)
- sssdconfig.save_domain(domain)
sssdconfig.write(SSSD_CONF)
# for selinux enabled platforms, ignore if it fails just report
sssdconfig.write(SSSD_CONF)
# for selinux enabled platforms, ignore if it fails just report