Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

docker-compose -> docker compose #2690

Merged
merged 1 commit into from
Feb 19, 2024
Merged

docker-compose -> docker compose #2690

merged 1 commit into from
Feb 19, 2024

Conversation

GaryPWhite
Copy link
Contributor

Description

Notes for Reviewers

Signed commits

  • Yes, I signed my commits.

Signed-off-by: Gary White Jr <7660110+GaryPWhite@users.noreply.github.com>
@GaryPWhite GaryPWhite requested a review from sgoggins as a code owner February 19, 2024 19:48
@@ -24,7 +24,6 @@ missingModules=""
#Check everything that needs to be in the $PATH is in there.
#Bash doesn't let this work if this is in an if statement for some reason it has to be chained
type -P "docker" &>/dev/null && echo "docker found..." || missingModules="${missingModules} docker"
type -P "docker-compose" &>/dev/null && echo "docker-compose found..." || missingModules="${missingModules} docker-compose"
Copy link
Contributor Author

Choose a reason for hiding this comment

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

I can understand why this might be problematic / scary. The reason I filed this PR is because on a modern release of RHEL we must use backports / unstable downloads to get an unsupported version of docker-compose -- happy to discuss alternatives.

Copy link
Member

@sgoggins sgoggins left a comment

Choose a reason for hiding this comment

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

Very helpful updates! Thanks, @GaryPWhite !!

@sgoggins sgoggins merged commit 047889e into chaoss:main Feb 19, 2024
5 checks passed
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants