open-telemetry / opentelemetry-python

OpenTelemetry Python API and SDK
https://opentelemetry.io
Apache License 2.0
1.66k stars 568 forks source link

Flaky opencensus shim test #3975

Open xrmx opened 1 week ago

xrmx commented 1 week ago

Describe your environment

OS: Windows Python version: (e.g., Python 3.8.10) SDK version: (e.g., 1.25.0) API version: (e.g., 1.25.0)

What happened?

____________ TestPatch.test_install_shim_affects_existing_tracers _____________

self = <tests.test_patch.TestPatch testMethod=test_install_shim_affects_existing_tracers>

    def test_install_shim_affects_existing_tracers(self):
        # Initially the shim is not installed. A OC Tracer instance should have a NoopTracer
        oc_tracer = Tracer()
>       self.assertIsInstance(oc_tracer.tracer, NoopTracer)
E       AssertionError: <opencensus.trace.tracers.context_tracer.ContextTracer object at 0x00000262F02C29D0> is not an instance of <class 'opencensus.trace.tracers.noop_tracer.NoopTracer'>

Steps to Reproduce

tox -e py38-opentelemetry-opencensus-shim in CI

Expected Result

test pass

Actual Result

test failure

Additional context

No response

Would you like to implement a fix?

None

ocelotl commented 2 days ago

Did this happen in CI? Do you have a link to the actual run that failed?

xrmx commented 1 day ago

Did this happen in CI? Do you have a link to the actual run that failed?

It's from CI, I don't have a link