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

Docker-Compose no more #5190

Merged
merged 4 commits into from
Sep 18, 2024
Merged

Conversation

mnaamani
Copy link
Member

@mnaamani mnaamani commented Sep 18, 2024

Update all scripts, docs and automation to not install or use docker-compose, replaced with docker compose command. It is no longer recommended, and it is not found on the github runners so some github workflows are failing.

@freakstatic
Copy link
Contributor

@mnaamani you missed the README.md of the QN
https://github.com/mnaamani/joystream/blob/bee4f6b6b6d3c1204ac11f432f24125e12a33478/query-node/README.md#L137

ln -s /usr/local/bin/docker-compose /usr/bin/docker-compose
"
creates: /usr/local/bin/docker-compose
# - name: Install docker-compose
Copy link
Contributor

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

Copy link
Member Author

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

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

Copy link
Member Author

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.

Copy link
Contributor

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 👍

Copy link
Contributor

@freakstatic freakstatic left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

@mnaamani mnaamani merged commit f98ab20 into Joystream:master Sep 18, 2024
23 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants