Skip to content

Conversation

@alex1702
Copy link

Updatet minimum docker-compose version to 1.29.2.

I saw the new version 21.8 and just tried to update on my server. Unfortunately the install.sh script broke without a proper error message. Then I saw that the docker-compose version must be higher on the release, updated it on my server and the installer then ran through.
So that a proper error message should come here is my patch for the 21.8.0 .

Updatet minimum docker-compose version to 1.29.2
@chadwhitacre
Copy link
Member

Unfortunately the install.sh script broke without a proper error message.

My hunch is that the script broke due to #1062. Can you review that ticket and confirm, @alex1702?

If so, then I don't believe that requiring an even newer version of docker-compose is the right solution.

Copy link
Member

@chadwhitacre chadwhitacre left a comment

Choose a reason for hiding this comment

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

Not sure we want to increase MIN_COMPOSE_VERSION in light of #1062.

@BYK
Copy link
Member

BYK commented Aug 17, 2021

@chadwhitacre I think there's a point here: we must have a test case with the minimum supported version of docker-compose. We may decide to have a matrix or just sticking with the oldest we support.

BYK added a commit that referenced this pull request Aug 17, 2021
@chadwhitacre
Copy link
Member

@BYK So close this in favor of #1066, yes?

@BYK
Copy link
Member

BYK commented Aug 17, 2021

@BYK So close this in favor of #1066, yes?

Yup, let's do that. Thanks a lot @alex1702 for putting this up!

@BYK BYK closed this Aug 17, 2021
BYK added a commit that referenced this pull request Aug 18, 2021
@github-actions github-actions bot locked and limited conversation to collaborators Sep 2, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants