Skip to content

Align target names for opentelemetry-proto with BCR#37662

Merged
phlax merged 1 commit intoenvoyproxy:mainfrom
mering:align-opentelemetry-proto-bcr
Dec 16, 2024
Merged

Align target names for opentelemetry-proto with BCR#37662
phlax merged 1 commit intoenvoyproxy:mainfrom
mering:align-opentelemetry-proto-bcr

Conversation

@mering
Copy link
Contributor

@mering mering commented Dec 14, 2024

Align target names for opentelemetry-proto with Bazel Central Registry.

This is a requirement for compatibility with Bzlmod.

@repokitteh-read-only
Copy link

Hi @mering, welcome and thank you for your contribution.

We will try to review your Pull Request as quickly as possible.

In the meantime, please take a look at the contribution guidelines if you have not done so already.

🐱

Caused by: #37662 was opened by mering.

see: more, trace.

@repokitteh-read-only repokitteh-read-only bot added api deps Approval required for changes to Envoy's external dependencies labels Dec 14, 2024
@repokitteh-read-only
Copy link

CC @envoyproxy/api-shepherds: Your approval is needed for changes made to (api/envoy/|docs/root/api-docs/).
envoyproxy/api-shepherds assignee is @markdroth
CC @envoyproxy/api-watchers: FYI only for changes made to (api/envoy/|docs/root/api-docs/).
CC @envoyproxy/dependency-shepherds: Your approval is needed for changes made to (bazel/.*repos.*\.bzl)|(bazel/dependency_imports\.bzl)|(api/bazel/.*\.bzl)|(.*/requirements\.txt)|(.*\.patch).
envoyproxy/dependency-shepherds assignee is @phlax

🐱

Caused by: #37662 was opened by mering.

see: more, trace.

@mering mering marked this pull request as draft December 14, 2024 16:11
This is a requirement for compatibility with Bzlmod.

Signed-off-by: maleo <maleo@google.com>
@mering mering marked this pull request as ready for review December 14, 2024 18:58
Copy link
Member

@phlax phlax left a comment

Choose a reason for hiding this comment

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

/lgtm deps

thanks @mering

@repokitteh-read-only repokitteh-read-only bot removed the deps Approval required for changes to Envoy's external dependencies label Dec 16, 2024
@markdroth
Copy link
Contributor

/lgtm api

@phlax phlax merged commit e4d5ceb into envoyproxy:main Dec 16, 2024
@mering mering deleted the align-opentelemetry-proto-bcr branch December 16, 2024 16:11
meteorcloudy pushed a commit to bazelbuild/bazel-central-registry that referenced this pull request Dec 19, 2024
- Based on envoyproxy/envoy#34355
- Requires
#3420 and
#3423
- Upstreaming target name alignment in
#3415 and
envoyproxy/envoy#37662
- Issue for patched Protobuf:
envoyproxy/envoy#37669
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.

3 participants