Skip to content
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

Bump OpenTelemetry to 1.32.0 #37819

Merged
merged 2 commits into from
Nov 27, 2023
Merged

Bump OpenTelemetry to 1.32.0 #37819

merged 2 commits into from
Nov 27, 2023

Conversation

jeanbisutti
Copy link
Contributor

No description provided.

@github-actions github-actions bot added Azure.Core azure-core Event Hubs OpenTelemetry OpenTelemetry instrumentation Service Bus labels Nov 27, 2023
@azure-sdk
Copy link
Collaborator

API change check

APIView has identified API level changes in this PR and created following API reviews.

spring-cloud-azure-starter-monitor

@jeanbisutti jeanbisutti marked this pull request as ready for review November 27, 2023 14:39
@@ -167,8 +167,7 @@ public void shouldMonitor() throws InterruptedException, MalformedURLException {
TelemetryItem firstLogTelemetry = logs.get(0);
MonitorDomain logBaseData = firstLogTelemetry.getData().getBaseData();
MessageData logData = (MessageData) logBaseData;
assertThat(logData.getMessage())
.isEqualTo("Initializing Spring DispatcherServlet 'dispatcherServlet'");
assertThat(logData.getMessage()).startsWith("Starting SpringMonitorTest using");
Copy link
Member

Choose a reason for hiding this comment

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

nice 👍

@jeanbisutti jeanbisutti merged commit 19683a8 into main Nov 27, 2023
52 checks passed
@jeanbisutti jeanbisutti deleted the jean/bump-otel-1.32.0 branch November 27, 2023 17:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Azure.Core azure-core azure-spring All azure-spring related issues Event Hubs OpenTelemetry OpenTelemetry instrumentation Service Bus
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants