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

[infra] Disable flaky ML job ID format test suite #171915

Merged

Conversation

miltonhultgren
Copy link
Contributor

@miltonhultgren miltonhultgren commented Nov 24, 2023

These tests rely on using browser performance timings to extract URLs used when resolving ML job IDs to check if the right format is used and updated when migrating to the new format.
Because this change is transparent to the user there isn't any way to test the behaviour through UI elements and due to the structure of the state management it's not straight forward to test it in isolation.

#171913 has been opened to refactor the state management which should allow for this tests to be updated and re-enabled.

@miltonhultgren miltonhultgren added release_note:skip Skip the PR/issue when compiling release notes backport:prev-minor Backport to (9.0) the previous minor version (i.e. one version back from main) Team:obs-ux-logs Observability Logs User Experience Team labels Nov 24, 2023
@miltonhultgren miltonhultgren requested review from a team as code owners November 24, 2023 10:35
@elasticmachine
Copy link
Contributor

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

@apmmachine
Copy link
Contributor

🤖 GitHub comments

Expand to view the GitHub comments

Just comment with:

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

@miltonhultgren miltonhultgren enabled auto-merge (squash) November 24, 2023 10:43
@kibana-ci
Copy link
Collaborator

💚 Build Succeeded

Metrics [docs]

✅ unchanged

To update your PR or re-run it, just comment with:
@elasticmachine merge upstream

@miltonhultgren miltonhultgren merged commit f555dbe into elastic:main Nov 24, 2023
@miltonhultgren miltonhultgren added backport:skip This commit does not require backporting and removed backport:prev-minor Backport to (9.0) the previous minor version (i.e. one version back from main) labels Nov 24, 2023
@elastic elastic deleted a comment from kibanamachine Nov 24, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport:skip This commit does not require backporting release_note:skip Skip the PR/issue when compiling release notes Team:obs-ux-logs Observability Logs User Experience Team v8.12.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants