-
Notifications
You must be signed in to change notification settings - Fork 115
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
Docker-Compose no more #5190
Docker-Compose no more #5190
Conversation
@mnaamani you missed the README.md of the QN |
ln -s /usr/local/bin/docker-compose /usr/bin/docker-compose | ||
" | ||
creates: /usr/local/bin/docker-compose | ||
# - name: Install docker-compose |
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.
this part can probably be removed
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.
Agreed
@@ -43,7 +43,6 @@ | |||
- docker-compose |
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 think this line is no longer needed
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.
This Ansible task will actually uninstall docker-compose. See the later line state: absent
so the apt
module will remove it if found.
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.
Oh I see, makes sense then 👍
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.
LGTM!
Update all scripts, docs and automation to not install or use
docker-compose
, replaced withdocker compose
command. It is no longer recommended, and it is not found on the github runners so some github workflows are failing.