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

[ci] pin docker client version #47785

Merged
merged 1 commit into from
Sep 22, 2024
Merged

[ci] pin docker client version #47785

merged 1 commit into from
Sep 22, 2024

Conversation

aslonnie
Copy link
Collaborator

so that docker client upgrade won't break us

@aslonnie
Copy link
Collaborator Author

related issue: docker/compose#12140

@aslonnie
Copy link
Collaborator Author

and the revert: docker/compose#12141

seems that they are planning to get the changes again in some way, not sure if they will keep the backwards compatibility or not.

so that docker client upgrade won't break us

Signed-off-by: Lonnie Liu <lonnie@anyscale.com>
@aslonnie aslonnie added the go add ONLY when ready to merge, run all tests label Sep 22, 2024
aslonnie added a commit that referenced this pull request Sep 22, 2024
pin docker client version

Signed-off-by: Lonnie Liu <lonnie@anyscale.com>
@aslonnie aslonnie enabled auto-merge (squash) September 22, 2024 07:51
@aslonnie aslonnie merged commit 18b2d94 into master Sep 22, 2024
7 checks passed
@aslonnie aslonnie deleted the lonnie-240921-dockerver branch September 22, 2024 08:52
aslonnie added a commit that referenced this pull request Sep 22, 2024
pin docker client version

Signed-off-by: Lonnie Liu <lonnie@anyscale.com>
ujjawal-khare pushed a commit to ujjawal-khare-27/ray that referenced this pull request Oct 15, 2024
so that docker client upgrade won't break us

Signed-off-by: Lonnie Liu <lonnie@anyscale.com>
Signed-off-by: ujjawal-khare <ujjawal.khare@dream11.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
go add ONLY when ready to merge, run all tests
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants