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

mimir-build-image: update to Debian Bookworm #6980

Merged
merged 10 commits into from
Jan 2, 2024
Merged

Conversation

colega
Copy link
Contributor

@colega colega commented Dec 21, 2023

What this PR does

Current image has an old sed 4.7 version installed, which fails to work properly under certain conditions on OSX, failing as:

/bin/sed: couldn't open temporary file ...: Permission denied

This seems to be fixed in 4.8 1, which is available in Bookworm, and I could workaround by bringing the backports, but it's just easier to update the build image.

Which issue(s) this PR fixes or relates to

Not reported I think.

Checklist

  • Tests updated.
  • Documentation added.
  • CHANGELOG.md updated - the order of entries should be [CHANGE], [FEATURE], [ENHANCEMENT], [BUGFIX].
  • about-versioning.md updated with experimental features.

Current image has an old sed 4.7 version installed, which fails to work
properly under certain conditions on OSX, failing as:

/bin/sed: couldn't open temporary file ...: Permission denied

This seems to be fixed in 4.8 [1], which is available in Bookworm, and I
could workaround by bringing the backports, but it's just easier to
update the build image.

[1]: https://forums.docker.com/t/sed-couldnt-open-temporary-file-xyz-permission-denied-when-using-virtiofs/125473

Signed-off-by: Oleg Zaytsev <mail@olegzaytsev.com>
@colega colega requested a review from a team as a code owner December 21, 2023 11:03
@ying-jeanne
Copy link
Contributor

Push mimir-build-image / build_and_push is skipped... don't see why

Copy link
Contributor

Building new version of mimir-build-image. After image is built and pushed to Docker Hub, a new commit will automatically be added to this PR with new image version grafana/mimir-build-image:pr6980-478aa31e45. This can take up to 1 hour.

@ying-jeanne
Copy link
Contributor

pushed a commit to solve the build issue

@colega
Copy link
Contributor Author

colega commented Dec 21, 2023

Thank you, @ying-jeanne !

@colega
Copy link
Contributor Author

colega commented Dec 21, 2023

What's wrong now? Why the build doesn't run?

@aknuds1
Copy link
Contributor

aknuds1 commented Dec 21, 2023

@colega I think it's due to a shortcoming in GitHub Actions. It doesn't re-trigger when the workflow pushes its image updating commit. I work around it by pushing a commit manually (e.g. by rebasing).

Signed-off-by: Oleg Zaytsev <mail@olegzaytsev.com>
@colega
Copy link
Contributor Author

colega commented Dec 21, 2023

Oh I see, I guess github actions commits don't trigger github actions to avoid infinite loops. I've pushed an empty commit.

Thank you @aknuds1

Copy link
Contributor

Not building new version of mimir-build-image. This PR modifies mimir-build-image/Dockerfile, but the image grafana/mimir-build-image:pr6980-478aa31e45 already exists.

Signed-off-by: Oleg Zaytsev <mail@olegzaytsev.com>
Copy link
Contributor

Not building new version of mimir-build-image. This PR modifies mimir-build-image/Dockerfile, but the image grafana/mimir-build-image:pr6980-478aa31e45 already exists.

Seems that asome String() methods have changed

Signed-off-by: Oleg Zaytsev <mail@olegzaytsev.com>
@colega colega requested a review from a team as a code owner December 21, 2023 15:36
Copy link
Contributor

Not building new version of mimir-build-image. This PR modifies mimir-build-image/Dockerfile, but the image grafana/mimir-build-image:pr6980-478aa31e45 already exists.

Signed-off-by: Oleg Zaytsev <mail@olegzaytsev.com>
Copy link
Contributor

Not building new version of mimir-build-image. This PR modifies mimir-build-image/Dockerfile, but the image grafana/mimir-build-image:pr6980-478aa31e45 already exists.

Copy link
Contributor

@aldernero aldernero left a comment

Choose a reason for hiding this comment

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

LGTM

@colega
Copy link
Contributor Author

colega commented Jan 2, 2024

Merged main with an updated build image, I guess this should build another one now?

Copy link
Contributor

github-actions bot commented Jan 2, 2024

Building new version of mimir-build-image. After image is built and pushed to Docker Hub, a new commit will automatically be added to this PR with new image version grafana/mimir-build-image:pr6980-2a47130a83. This can take up to 1 hour.

colega and others added 2 commits January 2, 2024 09:55
Signed-off-by: Oleg Zaytsev <mail@olegzaytsev.com>
Copy link
Contributor

github-actions bot commented Jan 2, 2024

Not building new version of mimir-build-image. This PR modifies mimir-build-image/Dockerfile, but the image grafana/mimir-build-image:pr6980-2a47130a83 already exists.

@colega colega merged commit e1e9461 into main Jan 2, 2024
29 checks passed
@colega colega deleted the mimir-build-image-bookworm branch January 2, 2024 10:38
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.

None yet

4 participants