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

[tests-only][full-ci] report tracings of failing tests only #11571

Merged
merged 4 commits into from
Sep 13, 2024

Conversation

saw-jan
Copy link
Member

@saw-jan saw-jan commented Sep 12, 2024

Description

Currently, we get the tracing reports of all the tests that have been retried regardless of pass or fail. And with keycloak tracing is enabled which causes all tracing to be reported for all tests.

Screenshot from 2024-09-13 10-02-17

Report only the tracings of failed tests. With this PR, now we should get the tracing reports of failed tests only.

Related Issue

Motivation and Context

How Has This Been Tested?

Screenshots (if appropriate):

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)
  • Technical debt
  • Tests
  • Documentation
  • Maintenance (e.g. dependency updates or tooling)

Open tasks:

  • ...

.drone.star Show resolved Hide resolved
tests/e2e/cucumber/environment/index.ts Show resolved Hide resolved
@saw-jan saw-jan force-pushed the tests/tracings-on-fail-only branch from 153d26e to cb32c86 Compare September 13, 2024 04:25
Copy link

Copy link
Contributor

@ScharfViktor ScharfViktor left a comment

Choose a reason for hiding this comment

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

👍

@saw-jan saw-jan merged commit 9a93ca2 into master Sep 13, 2024
3 checks passed
@saw-jan saw-jan deleted the tests/tracings-on-fail-only branch September 13, 2024 08:41
saw-jan added a commit that referenced this pull request Sep 13, 2024
* test: save the tracing reports of failed tests only

* test: bump ocis

* test: disable tracing by default

* bump web commit for tests
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants