-
Notifications
You must be signed in to change notification settings - Fork 3.9k
xds: Envoy proto sync to 2024-07-06 #11401
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
Conversation
Update repositories.bzl as well. |
Thanks - this was the first step. I think I should've made it a draft to be clear. |
`envoyproxy/envoy`: Sync protos to the latest imported version envoyproxy/envoy@ab911ac (commit 2024-07-06, cl/651956889). Should be a noop, just a routine xDS proto update to make upcoming RLQS-related imports simpler.
5c4bd2d
to
6a749f4
Compare
@ejona86 - updated repositories.bzl. Few questions:
|
No. This specific dependency is still pulled from the old way, even with MODULE.bazel. So there's nothing extra in this specific case.
Yeah, you should be good to go. |
`cncf/xds`: Sync protos to the latest imported version cncf/xds@024c85f (commit 2024-07-23, cl/655545156). Should be a noop, just a routine xDS proto update to make upcoming RLQS-related imports simpler, see related #11401. Note that CEL is only added as a bazel dependency as now it's required to build cncf/xds. Actual third-party source import will be done in the follow up PR, where RLQS dependencies are added to the import scripts.
`envoyproxy/envoy`: Sync protos to the latest imported version envoyproxy/envoy@ab911ac (commit 2024-07-06, cl/651956889). Should be a noop, just a routine xDS proto update to make upcoming RLQS-related imports simpler.
envoyproxy/envoy
: Sync protos to the latest imported version envoyproxy/envoy@ab911ac (commit 2024-07-06, cl/651956889).Should be a noop, just a routine xDS proto update to make upcoming RLQS-related imports simpler.