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

[Observability] Fix flaky test #142210

Merged
merged 3 commits into from
Oct 3, 2022
Merged

Conversation

shahzad31
Copy link
Contributor

@shahzad31 shahzad31 commented Sep 29, 2022

@shahzad31 shahzad31 marked this pull request as ready for review September 29, 2022 13:48
@shahzad31 shahzad31 requested review from a team as code owners September 29, 2022 13:48
@shahzad31 shahzad31 added release_note:skip Skip the PR/issue when compiling release notes v8.6.0 labels Sep 29, 2022
Copy link
Contributor

@awahab07 awahab07 left a comment

Choose a reason for hiding this comment

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

LGTM

@botelastic botelastic bot added the Team:Uptime - DEPRECATED Synthetics & RUM sub-team of Application Observability label Sep 29, 2022
@elasticmachine
Copy link
Contributor

Pinging @elastic/uptime (Team:uptime)

Copy link
Member

@maryam-saeidi maryam-saeidi left a comment

Choose a reason for hiding this comment

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

LGTM!

search: `?rangeFrom=${rangeFrom}&rangeTo=${rangeTo}`,
});

await PageObjects.header.waitUntilLoadingHasFinished();
Copy link
Member

Choose a reason for hiding this comment

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

Shall we move waitUntilLoadingHasFinished inside navigateToApp?

@@ -48,16 +42,28 @@ export default function ({ getService, getPageObjects }: FtrProviderContext) {
await esArchiver.unload(
Path.join('x-pack/test/apm_api_integration/common/fixtures/es_archiver', 'rum_8.0.0')
);
await esArchiver.unload(
Copy link
Member

Choose a reason for hiding this comment

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

Just curious: if in loadIfNeeded the archive is not loaded, how does unload work in that case?

@maryam-saeidi maryam-saeidi added the failed-test A test failure on a tracked branch, potentially flaky-test label Sep 30, 2022
@shahzad31 shahzad31 added the ci:reuse-kibana-build Experimental label Oct 3, 2022
@kibana-ci
Copy link
Collaborator

💚 Build Succeeded

Metrics [docs]

Async chunks

Total size of all lazy-loaded chunks that will be downloaded as the user navigates the app

id before after diff
observability 523.6KB 523.7KB +34.0B

History

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

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 ci:reuse-kibana-build Experimental failed-test A test failure on a tracked branch, potentially flaky-test release_note:skip Skip the PR/issue when compiling release notes Team:Uptime - DEPRECATED Synthetics & RUM sub-team of Application Observability v8.6.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants