Skip to content
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

Closed
baszoetekouw opened this issue Oct 18, 2022 · 7 comments · Fixed by #363
Closed

geen change status bij ldap-wijzigingen #360

baszoetekouw opened this issue Oct 18, 2022 · 7 comments · Fixed by #363
Milestone

Comments

@baszoetekouw
Copy link
Member

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

@baszoetekouw baszoetekouw changed the title geen chaneg status bij ldap-wijzigingen geen change status bij ldap-wijzigingen Oct 18, 2022
@mrvanes mrvanes moved this to Todo in SRAM development Oct 18, 2022
@mrvanes
Copy link
Contributor

mrvanes commented Oct 18, 2022

Laten we alsjeblieft voor eens en voor altijd onthouden dat dit niet gaat lukken tot dit issue opgelost is:
ansible-collections/community.general#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).

@mrvanes mrvanes moved this from Todo to In progress in SRAM development Oct 18, 2022
@baszoetekouw
Copy link
Member Author

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
Copy link
Contributor

mrvanes commented Oct 18, 2022

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

@baszoetekouw
Copy link
Member Author

Dat is nog een beter idee idd

@mrvanes
Copy link
Contributor

mrvanes commented Oct 19, 2022

Eens kijken hoe ver we komen
ansible-collections/community.general#5385

@mrvanes
Copy link
Contributor

mrvanes commented Oct 31, 2022

@baszoetekouw
Copy link
Member Author

confirmed.

@baszoetekouw baszoetekouw moved this from To be tested to To be deployed in SRAM development Nov 15, 2022
@sram-deploy-tools-automation sram-deploy-tools-automation bot moved this from To be deployed to Done in SRAM development Jan 18, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Archived in project
Development

Successfully merging a pull request may close this issue.

2 participants