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: Upgrade to google.golang.org/protobuf/cmd/[email protected] #6541

Draft
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

aknuds1
Copy link
Contributor

@aknuds1 aknuds1 commented Nov 2, 2023

What this PR does

Upgrade build image to google.golang.org/protobuf/cmd/[email protected], since github.com/golang/protobuf/protoc-gen-go is deprecated.

Which issue(s) this PR fixes or relates to

Checklist

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

Copy link
Contributor

github-actions bot commented Nov 2, 2023

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:pr6541-d2a96ab505`. This can take up to 1 hour.

@pstibrany
Copy link
Member

Can we do all the changes in single PR (thinking of #6540), otherwise there will be lot of rebasing and image rebuilding going on.

@aknuds1
Copy link
Contributor Author

aknuds1 commented Nov 2, 2023

@pstibrany I wanted to get this image built and try to regenerate *.pb.go files with it. If you prefer it, I can fuse into one PR, sure.

@pstibrany
Copy link
Member

@pstibrany I wanted to get this image built and try to regenerate *.pb.go files with it. If you prefer it, I can fuse into one PR, sure.

I think it would have been easier, but feel free to do it your way.

@aknuds1
Copy link
Contributor Author

aknuds1 commented Nov 2, 2023

I was hoping to get rid of github.com/golang/protobuf references in *.pb.go files, but no dice. Theory is these are generated by github.com/gogo/protobuf/protoc-gen-gogoslick, so we'd have to migrate off gogo also.

Copy link
Contributor

github-actions bot commented Nov 2, 2023

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:pr6541-9a9d7c46fb`. This can take up to 1 hour.

Copy link
Contributor

github-actions bot commented Feb 6, 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:pr6541-07eb79ae37. This can take up to 1 hour.

Copy link
Contributor

github-actions bot commented Oct 7, 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:pr6541-2edf665d50. This can take up to 1 hour.

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.

2 participants