Skip to content

[monitor] Updating @opentelemetry packages and removing deprecated @opentelemetry/sdk-metrics-base#23307

Merged
mpodwysocki merged 3 commits intoAzure:mainfrom
mpodwysocki:bug/23260
Sep 26, 2022
Merged

[monitor] Updating @opentelemetry packages and removing deprecated @opentelemetry/sdk-metrics-base#23307
mpodwysocki merged 3 commits intoAzure:mainfrom
mpodwysocki:bug/23260

Conversation

@mpodwysocki
Copy link
Contributor

Packages impacted by this PR

  • [monitor-opentelemetry-exporter]
  • [monitor-query]
  • [opentelemetry-instrumentation-azure-sdk]
  • [core-http]
  • [core-rest-pipeline]
  • [test-utils]

Issues associated with this PR

Describe the problem that is addressed by this PR

Upgrades to the latest @opentelemetry releases, including removing the deprecated @opentelemetry/sdk-metrics-base package in favor of @opentelemetry/sdk-metrics.

What are the possible designs available to address the problem? If there are more than one possible design, why was the one in this PR chosen?

Are there test cases added in this PR? (If not, why?)

Provide a list of related PRs (if any)

Command used to generate this PR:**(Applicable only to SDK release request PRs)

Checklists

  • Added impacted package name to the issue description
  • Does this PR needs any fixes in the SDK Generator?** (If so, create an Issue in the Autorest/typescript repository and link it here)
  • Added a changelog (if necessary)

@azure-sdk
Copy link
Collaborator

API change check

API changes are not detected in this pull request.

Copy link
Member

@hectorhdzg hectorhdzg left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Member

@jeremymeng jeremymeng left a comment

Choose a reason for hiding this comment

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

LGTM

@mpodwysocki
Copy link
Contributor Author

/check-enforcer reset

@KarishmaGhiya
Copy link
Member

@mpodwysocki You can rebase this branch to update the recordings from #23320 .It should work.

@mpodwysocki mpodwysocki merged commit bc8fd02 into Azure:main Sep 26, 2022
@mpodwysocki mpodwysocki deleted the bug/23260 branch September 26, 2022 21:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Azure.Core Monitor Monitor, Monitor Ingestion, Monitor Query

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants