Skip to content
This repository has been archived by the owner on Oct 28, 2024. It is now read-only.

feat: Update OpenTelemetry SDK to 1.11.0 #1664

Merged
merged 1 commit into from
Apr 21, 2022

Conversation

matschaffer
Copy link
Contributor

@matschaffer matschaffer commented Apr 21, 2022

What does this PR do?

  • It updates the OpenTelemetry SDK to 1.11.0

Why is it important?

I ran into the issue reported on open-telemetry/opentelemetry-python#2288 when attempting to run filebeat module integration tests (process on https://www.elastic.co/guide/en/beats/devguide/current/filebeat-modules-devguide.html#_test). This occurred while reviewing elastic/beats#31286

Updating to 1.11.0 resolves the error.

Testing notes

To run using this copy against my filebeat integration tests I had to:

  1. activate the python env
  2. pip uninstall pytest-otel; pip uninstall opentelemetry-sdk
  3. pip install pip install ../../apm-pipeline-library/resources/scripts/pytest_otel

I was able to confirm the correct version was being used via pip freeze | grep opentelemetry

At one point I also did a pip install -r requirements.txt from the pytest_otel directory. I'm uncertain if this is required on a clean virtual env or not.

@matschaffer matschaffer requested a review from a team as a code owner April 21, 2022 05:31
@matschaffer matschaffer self-assigned this Apr 21, 2022
@matschaffer matschaffer requested a review from a team as a code owner April 21, 2022 05:31
@apmmachine
Copy link
Contributor

💚 Build Succeeded

the below badges are clickable and redirect to their specific view in the CI or DOCS
Pipeline View Test View Changes Artifacts preview preview

Expand to view the summary

Build stats

  • Start Time: 2022-04-21T05:31:22.714+0000

  • Duration: 8 min 35 sec

Test stats 🧪

Test Results
Failed 0
Passed 1420
Skipped 2
Total 1422

💚 Flaky test report

Tests succeeded.

🤖 GitHub comments

To re-run your PR in the CI, just comment with:

  • /test : Re-trigger the build.

  • run infra tests : Run the test-infra test.

  • /test all : Re-trigger the build for all the stages.

  • run elasticsearch-ci/docs : Re-trigger the docs validation. (use unformatted text in the comment!)

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants