Fix SRAM conditional container restart#708
Conversation
c2ffc24 to
94f03fd
Compare
mrvanes
changed the title
| name: "sram-ldap" | ||
| restart: true | ||
| state: started | ||
| when: ldap_container is success and ldap_container is not change |
There was a problem hiding this comment.
Don't you want to restart the ldap container when it is changed? Not when it is not changed?
There was a problem hiding this comment.
Hmm... dit komt letterlijk zo uit de invite rol, maar je hebt wel een punt?
There was a problem hiding this comment.
Misschien dat de container sowieso wordt herstart in die andere taak als er een nieuwe image beschikbaar is, en dat je hiermee zorgt dat een container ook herstart wordt als er geen nieuwe image is? Zodat nieuwe ansible configuratie altijd in de containers doorkomen bij een deploy?
There was a problem hiding this comment.
@EricvanSchaik @mrvanes
volgens mij herstart docker_container task zelf containers als er iets wijzigt
- name: Create and start the server container
community.docker.docker_container:
en de restart in de handler doe je dan als wij iets wijzigen aan bv /opt/openconext/invite/configfile
we hadden eerder nl dubbele restarts
ik heb niet echt een handige plek gevonden voor een comment omdat we dit op 20 plekken zo doen
3 participants
No description provided.