-
Notifications
You must be signed in to change notification settings - Fork 471
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
Envoy 1.26 bump (includes inja bump) #8440
Conversation
Issues linked to changelog: |
Visit the preview URL for this PR (updated for commit 8c09ed7): https://gloo-edge--pr8440-envoy-gloo-bump-for-pf3tvz0t.web.app (expires Fri, 21 Jul 2023 18:46:42 GMT) 🔥 via Firebase Hosting GitHub Action 🌎 Sign: 77c2b86e287749579b7ff9cadb81e099042ef677 |
…to envoy-gloo-bump-for-inja
Can we test this with 1.25? |
…loo into envoy-gloo-bump-for-inja
…loo into envoy-gloo-bump-for-inja
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@solo-io/solo-docs sorry about all of the noise in this file. The real changes are on L172, L174, L385, L388
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think next time we do a seperate pr for the protobuf bump to remove it from functional code XD
Description
Bump envoy-gloo-ee to 1.26 version. The investigation around this change can be found in this doc. This bump will also include latest version of Inja.
This is really noisy due to protoc-gen-go dep bump, but the real changes are in
Checklist:
make -B install-go-tools generated-code
to ensure there will be no code diff