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

fix: use different tests for the demos #1523

Merged
merged 1 commit into from
Feb 4, 2022

Conversation

kuisathaverat
Copy link
Contributor

What does this PR do?

  • It uses a new set of tests for the demos.

Why is it important?

Before this change we use the pytest-otel tests for the demos, we can no longer use those tests because we use the In-Memory Otel exporter and a file to check the plugin behavior, this causes not all the traces to be reported to the demo. Using a simpler test suit for the demos fixes the issue.

@kuisathaverat kuisathaverat added bug Something isn't working Team:Automation Label for the Observability productivity team labels Feb 1, 2022
@kuisathaverat kuisathaverat requested a review from a team February 1, 2022 12:20
@kuisathaverat kuisathaverat self-assigned this Feb 1, 2022
@kuisathaverat kuisathaverat requested a review from a team as a code owner February 1, 2022 12:20
@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

  • Reason: null

  • Start Time: 2022-02-01T12:20:44.015+0000

  • Duration: 12 min 33 sec

  • Commit: 23d769a

Test stats 🧪

Test Results
Failed 0
Passed 1336
Skipped 2
Total 1338

💚 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.

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

@kuisathaverat kuisathaverat merged commit bc2f737 into elastic:main Feb 4, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working Team:Automation Label for the Observability productivity team
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants