-
Notifications
You must be signed in to change notification settings - Fork 1
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
Favor docker compose v2 in ansible tasks #1835
Conversation
20c8409
to
23a9ef7
Compare
build: never | ||
pull: always | ||
state: present | ||
wait: true |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@mattwr18 this is a nice new feature that will ensure that all containers are healthy before proceeding. In our pair-programming today we were facing the issue of silent failures.
- name: docker-compose | ||
- name: passlib | ||
pip_install_packages: | ||
- name: passlib |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
passlib
is required for htpasswd
role. The other two seem not required.
- role: geerlingguy.docker | ||
docker_packages_state: latest |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'm willing to keep this variable. Opinions/thoughts?
23a9ef7
to
fb0cdd1
Compare
3324708
to
5802351
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Also checked your changes @roschaefer and they look good... |
Fixes #1834
I've tested running the
ansible/restore_locally
andansible/site
playbooks and nothing appears to be broken.