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

*: Attempt to fix multiarch builds by updating remote Docker engine version #5083

Merged
merged 1 commit into from
Jan 20, 2022

Conversation

matej-g
Copy link
Collaborator

@matej-g matej-g commented Jan 20, 2022

  • I added CHANGELOG entry for this change.
  • Change is not relevant to the end user.

Changes

This PR attempts to fix current issue on CI - #5077 - which causes the publish_* jobs not to pass on the CircleCI.

The first attempt is to update the remote Docker engine version (see: https://circleci.com/docs/2.0/building-docker-images/#docker-version)

Temporarily, the job has been allowed on all branches but pushing images has been commented out, we just need to make sure we pass the docker-test step

Verification

Waiting on CI to pass

Signed-off-by: Matej Gera <matejgera@gmail.com>
@matej-g
Copy link
Collaborator Author

matej-g commented Jan 20, 2022

@matej-g
Copy link
Collaborator Author

matej-g commented Jan 20, 2022

Tests are successful 👍 - https://app.circleci.com/pipelines/github/thanos-io/thanos/8646/workflows/fda3dfa9-d706-47c8-8df1-b32ec8c8976e/jobs/17975. Removing the test commit and opening this up for review.

@matej-g matej-g force-pushed the attempt-fix-multiarch-builds branch from 144db13 to b94392a Compare January 20, 2022 11:04
@matej-g matej-g changed the title [WIP] *: Attempt to fix multiarch builds *: Attempt to fix multiarch builds by updating remote Docker engine version Jan 20, 2022
Copy link
Member

@bwplotka bwplotka left a comment

Choose a reason for hiding this comment

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

Thanks!

@bwplotka bwplotka merged commit 5c11f24 into thanos-io:main Jan 20, 2022
@GiedriusS
Copy link
Member

publish_main works 🎉

Nicholaswang pushed a commit to Nicholaswang/thanos that referenced this pull request Mar 6, 2022
Signed-off-by: Matej Gera <matejgera@gmail.com>
Signed-off-by: Nicholaswang <wzhever@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants