Skip to content

Replace OpenCensus with OpenTelemetry#2828

Draft
pink-milk wants to merge 1 commit into
microsoft:mainfrom
pink-milk:otel-migration
Draft

Replace OpenCensus with OpenTelemetry#2828
pink-milk wants to merge 1 commit into
microsoft:mainfrom
pink-milk:otel-migration

Conversation

@pink-milk

@pink-milk pink-milk commented Jul 18, 2026

Copy link
Copy Markdown

Adds the internal/ot tracing package and switches instrumentation from OpenCensus (internal/oc, go.opencensus.io) to OpenTelemetry (go.opentelemetry.io/otel, github.com/containerd/otelttrpc, otelgrpc).

  • internal/oc and pkg/octtrpc are retained for backward compatibility during the transition.

Forward-port of the OTEL migration from the internal ADO fork onto upstream
main. Adds the internal/ot tracing package and switches instrumentation from
OpenCensus (internal/oc, go.opencensus.io) to OpenTelemetry (go.opentelemetry.io/otel,
containerd/otelttrpc, otelgrpc). internal/oc and pkg/octtrpc are retained for
backward compatibility during the transition.

Signed-off-by: Olivia Vuong <vuongolivia@gmail.com>
@pink-milk

Copy link
Copy Markdown
Author

@microsoft-github-policy-service agree company="Microsoft"

Comment thread .github/workflows/ci.yml
working-directory: test

- uses: actions/upload-artifact@v7
- uses: actions/upload-artifact@v6

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

This should remain at V7.

Comment thread .github/workflows/ci.yml
name: Build zapdir.exe

- uses: actions/upload-artifact@v7
- uses: actions/upload-artifact@v6

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

This should remain at V7.


- name: Upload SARIF Results as Build Artifact
uses: actions/upload-artifact@v7
uses: actions/upload-artifact@v6

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

This should remain at V7.

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