SURFscz / SRAM-deploy

Deploy scripts for the SCZ
Apache License 2.0
5 stars 5 forks source link

geen change status bij ldap-wijzigingen #360

Closed baszoetekouw closed 1 year ago

baszoetekouw commented 1 year ago

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).

mrvanes commented 1 year ago

Laten we alsjeblieft voor eens en voor altijd onthouden dat dit niet gaat lukken tot dit issue opgelost is: https://github.com/ansible-collections/community.general/issues/977

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).

baszoetekouw commented 1 year ago

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.

mrvanes commented 1 year ago

Maar kunnen we dan ldap_attrs en ldap_entry niet beter fixen en PR'en?

baszoetekouw commented 1 year ago

Dat is nog een beter idee idd

mrvanes commented 1 year ago

Eens kijken hoe ver we komen https://github.com/ansible-collections/community.general/pull/5385

mrvanes commented 1 year ago

https://github.com/ansible-collections/community.general/pull/5435

baszoetekouw commented 1 year ago

confirmed.