-
Notifications
You must be signed in to change notification settings - Fork 4
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
geen change status bij ldap-wijzigingen #360
Comments
Laten we alsjeblieft voor eens en voor altijd onthouden dat dit niet gaat lukken tot dit issue opgelost is: ldapmodify geeft altijd exitcode 0 als er geen error opgetreden is, zowel voor een modify die iets verandert als een waar niks gebeurt, omdat dat een valide uitkomst is (voor ldapmodify). |
Ja, maar in dit geval gebruik je ldap_ttrs helemaal niet, maar je eigen ldap python script. Daar kunnen we het prima goed doen volgens mij. |
Maar kunnen we dan ldap_attrs en ldap_entry niet beter fixen en PR'en? |
Dat is nog een beter idee idd |
Eens kijken hoe ver we komen |
confirmed. |
Probleem is dat deze task https://github.com/SURFscz/SRAM-deploy/blob/main/roles/ldap/tasks/main.yml#L112-L123 nooit changes aangeeft. Als je bv het
ldap_loglevel
wijzigt, zou hij dat moeten laten zijn tijden de deploy. Dat gebeurt nu niet.Idealiter laat hij bij
ansible --diff
ook zijn wat de wijziging is (maar ik weet niet of dat mogelijk is).The text was updated successfully, but these errors were encountered: