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

[bug] MetricReaderExt.RemoveMetric is failing in certain unit tests #5907

Open
TimothyMothra opened this issue Oct 16, 2024 · 0 comments · May be fixed by #5908
Open

[bug] MetricReaderExt.RemoveMetric is failing in certain unit tests #5907

TimothyMothra opened this issue Oct 16, 2024 · 0 comments · May be fixed by #5908
Assignees
Labels
bug Something isn't working metrics Metrics signal related pkg:OpenTelemetry Issues related to OpenTelemetry NuGet package

Comments

@TimothyMothra
Copy link
Contributor

TimothyMothra commented Oct 16, 2024

Package

OpenTelemetry

Package Version

main branch

Runtime Version

net462

Description

Some tests are failing when running locally.

  • OpenTelemetry.Extensions.Hosting.Tests (net462)
    • OpenTelemetryMetricsBuilderExtensionsTests.ReloadOfMetricsViaIConfigurationWithoutExportCleanupTest
      This is failing on every run.
  • OpenTelemetry.Tests (net462)
    I haven't identified the exact test that is failing here. Seems to be intermittent.

image

Seems to be hitting this condition:

var result = this.instrumentIdentityToMetric.TryRemove(metric.InstrumentIdentity, out var _);
Debug.Assert(result, "result was false");

Steps to Reproduce

Run unit test locally

Expected Result

N/A

Actual Result

N/A

Additional Context

This was first reported here #5890

@TimothyMothra TimothyMothra added bug Something isn't working needs-triage New issues which have not been classified or triaged by a community member labels Oct 16, 2024
@github-actions github-actions bot added the pkg:OpenTelemetry Issues related to OpenTelemetry NuGet package label Oct 16, 2024
@CodeBlanch CodeBlanch added metrics Metrics signal related and removed needs-triage New issues which have not been classified or triaged by a community member labels Oct 17, 2024
@CodeBlanch CodeBlanch self-assigned this Oct 17, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working metrics Metrics signal related pkg:OpenTelemetry Issues related to OpenTelemetry NuGet package
Projects
None yet
2 participants