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

[Discover] Unskip date_nanos and shard links functional tests #82878

Merged

Conversation

kertal
Copy link
Member

@kertal kertal commented Nov 6, 2020

Summary

PR originally used for investigation of #81457 adding logs for flaky test runner . However in the meantime lot's of the underlying index pattern API changed, so the failure that originally led to the skipping of this tests is no longer possible. So what this PR just takes care of the unskipping. And here are 150 flaky test runner runs to prove that there's no flakiness involved:

https://kibana-ci.elastic.co/job/kibana+flaky-test-suite-runner/1030/

Fixes #82035
Fixes #80104

Checklist

Delete any items that are not applicable to this PR.

For maintainers

@kertal kertal self-assigned this Nov 6, 2020
@kertal kertal added the Feature:Discover Discover Application label Nov 6, 2020
@kertal kertal changed the title [Discover] Unskip date_nanos [Discover] Unskip date_nanos functional test Nov 6, 2020
@kertal
Copy link
Member Author

kertal commented Nov 24, 2020

@elasticmachine merge upstream

@kibanamachine
Copy link
Contributor

merge conflict between base and head

@kibanamachine
Copy link
Contributor

💚 Build Succeeded

Metrics [docs]

✅ unchanged

History

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

@kertal kertal added this to In progress in Discover via automation Nov 30, 2020
@kertal kertal changed the title [Discover] Unskip date_nanos functional test [Discover] Unskip date_nanos and shard links functional tests Nov 30, 2020
@kertal kertal added Team:Visualizations Visualization editors, elastic-charts and infrastructure release_note:skip Skip the PR/issue when compiling release notes labels Nov 30, 2020
@kertal kertal removed this from In progress in Discover Nov 30, 2020
@kertal kertal marked this pull request as ready for review November 30, 2020 15:50
@elasticmachine
Copy link
Contributor

Pinging @elastic/kibana-app (Team:KibanaApp)

@kertal kertal merged commit 0017c2d into elastic:master Nov 30, 2020
kertal added a commit to kertal/kibana that referenced this pull request Nov 30, 2020
gmmorris added a commit to gmmorris/kibana that referenced this pull request Nov 30, 2020
* master: (25 commits)
  [Alerting] fixes buggy default message behaviour (elastic#84202)
  [Graph] Use new ES client and change license API (elastic#84398)
  [DOCS] Adds redirect to known plugins page (elastic#84001)
  Update IndexPatternSelect to get fields from indexPatternService instead of savedObject attributes (elastic#84376)
  Adding timestamps to created events so the sorting is stable (elastic#84515)
  [DOCS] Redirects for drilldown links (elastic#83846)
  [Fleet] Support for showing an Integration Detail Custom (UI Extension) tab (elastic#83805)
  [Enterprise Search] Migrate shared Schema components (elastic#84381)
  [Discover] Unskip date_nanos and shard links functional tests (elastic#82878)
  [APM] ML anomaly detection integration: Displaying anomaly job results in the Transaction duration chart is not as intended  (elastic#84415)
  Support for painless language autocomplete within monaco (elastic#80577)
  [Lens] Time scale ui (elastic#83904)
  removing beta callouts (elastic#84510)
  [Lens] (Accessibility) add aria-label to chart type icon (elastic#84493)
  Trusted Apps signer API. (elastic#83661)
  increase stdout max listeners for legacy logging (elastic#84497)
  [APM] Service overview: Add throughput chart (elastic#84439)
  [Discover] Unskip main functional tests (elastic#84300)
  Uptime overview overhaul (elastic#83406)
  [APM] Adjust time formats based on the difference between start and end (elastic#84470)
  ...
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment