Skip to content

[ObsUX] Unskip failing test#196727

Merged
MiriamAparicio merged 6 commits intoelastic:mainfrom
MiriamAparicio:191961-fix-skipped-tests
Oct 24, 2024
Merged

[ObsUX] Unskip failing test#196727
MiriamAparicio merged 6 commits intoelastic:mainfrom
MiriamAparicio:191961-fix-skipped-tests

Conversation

@MiriamAparicio
Copy link
Copy Markdown
Contributor

Closes #191961

@MiriamAparicio MiriamAparicio added release_note:skip Skip the PR/issue when compiling release notes backport:skip This PR does not require backporting v8.16.0 labels Oct 17, 2024
@MiriamAparicio MiriamAparicio requested a review from a team October 17, 2024 15:26
@botelastic botelastic bot added ci:project-deploy-observability Create an Observability project Team:obs-ux-infra_services - DEPRECATED DEPRECATED - Use Team:obs-presentation. labels Oct 17, 2024
@elasticmachine
Copy link
Copy Markdown
Contributor

Pinging @elastic/obs-ux-infra_services-team (Team:obs-ux-infra_services)

@github-actions
Copy link
Copy Markdown
Contributor

🤖 GitHub comments

Expand to view the GitHub comments

Just comment with:

  • /oblt-deploy : Deploy a Kibana instance using the Observability test environments.
  • run docs-build : Re-trigger the docs validation. (use unformatted text in the comment!)

@MiriamAparicio
Copy link
Copy Markdown
Contributor Author

@kibanamachine
Copy link
Copy Markdown
Contributor

Flaky Test Runner Stats

🟠 Some tests failed. - kibana-flaky-test-suite-runner#7168

[❌] APM - Cypress: 0/50 tests passed.

see run history

@kibanamachine
Copy link
Copy Markdown
Contributor

Flaky Test Runner Stats

🟠 Some tests failed. - kibana-flaky-test-suite-runner#7181

[❌] APM - Cypress: 8/50 tests passed.

see run history

@elasticmachine
Copy link
Copy Markdown
Contributor

elasticmachine commented Oct 23, 2024

💚 Build Succeeded

  • Buildkite Build
  • Commit: 5051614
  • Kibana Serverless Image: docker.elastic.co/kibana-ci/kibana-serverless:pr-196727-5051614e380c

Metrics [docs]

✅ unchanged

History


describe('Transaction details', () => {
// flaky
describe.skip('Transaction details', () => {
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Is this needed? Yesterday I skipped 2 flaky tests from this suite

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

I think it's better to skip the whole test suite as we don't know exactly what's the issue here

Copy link
Copy Markdown
Contributor

@cauemarcondes cauemarcondes left a comment

Choose a reason for hiding this comment

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

LGTM

@MiriamAparicio MiriamAparicio merged commit bd11d1a into elastic:main Oct 24, 2024
@MiriamAparicio MiriamAparicio deleted the 191961-fix-skipped-tests branch October 24, 2024 10:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport:skip This PR does not require backporting ci:project-deploy-observability Create an Observability project release_note:skip Skip the PR/issue when compiling release notes Team:obs-ux-infra_services - DEPRECATED DEPRECATED - Use Team:obs-presentation. v8.16.0 v9.0.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Failed Tests][Unsupported FTRs] APM Cypress && Synthetics Tests

5 participants