URL: https://github.com/SSSD/sssd/pull/884
Title: #884: ipa: support disabled domains
pbrezina commented:
"""
* `master`
* fa3e53bb9ad18358989d625af4a0d8cbeb428458 - ipa: delete content of
disabled domains
* b12e7a495408635016a2ebf645448a630222d1be - sysdb: add
URL: https://github.com/SSSD/sssd/pull/884
Title: #884: ipa: support disabled domains
pbrezina commented:
"""
Thank you. Ack.
"""
See the full comment at
https://github.com/SSSD/sssd/pull/884#issuecomment-533021518
___
sssd-devel mailing list --
URL: https://github.com/SSSD/sssd/pull/884
Title: #884: ipa: support disabled domains
sumit-bose commented:
"""
Hi,
I added 2 new patches, one with a new sysdb call, the other to use it if a
domain gets disabled.
bye,
Sumit
"""
See the full comment at
URL: https://github.com/SSSD/sssd/pull/884
Title: #884: ipa: support disabled domains
sumit-bose commented:
"""
Ah, you mean the user and group data, I was thinking about the domain object.
Yes, I'll update the patch to remove user and group data.
"""
See the full comment at
URL: https://github.com/SSSD/sssd/pull/884
Title: #884: ipa: support disabled domains
pbrezina commented:
"""
Perhaps remove domain data but keep the domain object?
"""
See the full comment at
https://github.com/SSSD/sssd/pull/884#issuecomment-531744513
URL: https://github.com/SSSD/sssd/pull/884
Title: #884: ipa: support disabled domains
pbrezina commented:
"""
Can't we have both? Domain removed from the cache and message that it is
disabled in the logs? Because the domain data may take up significant space on
the disc.
"""
See the full
URL: https://github.com/SSSD/sssd/pull/884
Title: #884: ipa: support disabled domains
sumit-bose commented:
"""
Hi,
thank you for the review. I'd prefer to keep the domain around even in the
cache. Even if there is no technical reason I can currently think of, having
the message in the logs
URL: https://github.com/SSSD/sssd/pull/884
Title: #884: ipa: support disabled domains
pbrezina commented:
"""
Shouldn't you also delete the disabled subdomain with
`sysdb_subdomain_delete()`?
"""
See the full comment at
https://github.com/SSSD/sssd/pull/884#issuecomment-531729650