theforeman / puppet-foreman

Puppet module for Foreman
GNU General Public License v3.0
104 stars 271 forks source link

Fixes #35832 - set default_domain_suffix in sssd.conf #1100

Closed snagoor closed 5 months ago

snagoor commented 1 year ago

Setting a default_domain_suffix for IDM based authentication would reduce an effort to type in the @domain while authenticating.

https://docs.theforeman.org/nightly/Administering_Project/index-foreman-el.html#Using_FreeIPA_admin

snagoor commented 1 year ago

Validations seems to have failed as below

manifests/config.pp:209:legacy_facts:WARNING:legacy fact
Error: Process completed with exit code 1.

this $facts['domain'] seems to be deprecated and I am not sure but reading the docs https://puppet.com/docs/puppet/6/core_facts.html#domain it seems the new format https://puppet.com/docs/puppet/6/core_facts.html#networking to be $facts['networking']['domain'] . Please correct me if I am wrong.

adamruzicka commented 9 months ago

@snagoor gentle ping

evgeni commented 5 months ago

CLosing as https://github.com/theforeman/puppet-foreman/pull/1160 was merged