Skip to content

Improve opentelemetry-proto#3415

Merged
keith merged 1 commit intobazelbuild:mainfrom
mering:improve-opentelemetry-proto
Dec 14, 2024
Merged

Improve opentelemetry-proto#3415
keith merged 1 commit intobazelbuild:mainfrom
mering:improve-opentelemetry-proto

Conversation

@mering
Copy link
Copy Markdown
Contributor

@mering mering commented Dec 14, 2024

Add Python and Go proto bindings

@mering
Copy link
Copy Markdown
Contributor Author

mering commented Dec 14, 2024

@bazel-io skip_check unstable_url

@bazel-io
Copy link
Copy Markdown
Member

Hello @keith, modules you maintain (opentelemetry-proto) have been updated in this PR. Please review the changes.

@bazel-io bazel-io added the skip-url-stability-check Skip the URL stability check for the PR label Dec 14, 2024
@mering mering force-pushed the improve-opentelemetry-proto branch from 6975e1f to c823bd8 Compare December 14, 2024 16:00
@mering mering marked this pull request as draft December 14, 2024 16:11
@mering mering force-pushed the improve-opentelemetry-proto branch from c823bd8 to 2da4492 Compare December 14, 2024 17:11
@mering mering marked this pull request as ready for review December 14, 2024 17:11
keith
keith previously approved these changes Dec 14, 2024
@keith
Copy link
Copy Markdown
Member

keith commented Dec 14, 2024

ci issue

@mering mering force-pushed the improve-opentelemetry-proto branch from 2da4492 to 613198d Compare December 14, 2024 18:17
@bazel-io bazel-io dismissed keith’s stale review December 14, 2024 18:17

Require module maintainers' approval for newly pushed changes.

@mering
Copy link
Copy Markdown
Contributor Author

mering commented Dec 14, 2024

@keith can you trigger the CI again please?

@mering mering force-pushed the improve-opentelemetry-proto branch from 613198d to 06236b4 Compare December 14, 2024 18:23
@mering
Copy link
Copy Markdown
Contributor Author

mering commented Dec 14, 2024

@keith same here, one more time please.

@keith
Copy link
Copy Markdown
Member

keith commented Dec 14, 2024

done

Add Python and Go proto bindings
@mering mering force-pushed the improve-opentelemetry-proto branch from 06236b4 to f48a813 Compare December 14, 2024 18:29
@mering
Copy link
Copy Markdown
Contributor Author

mering commented Dec 14, 2024

@keith again please? Thx.

@keith keith enabled auto-merge (squash) December 14, 2024 18:35
@keith keith merged commit d7bb3f5 into bazelbuild:main Dec 14, 2024
@mering mering deleted the improve-opentelemetry-proto branch December 14, 2024 19:11
meteorcloudy pushed a commit 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

skip-url-stability-check Skip the URL stability check for the PR

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants