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

[Reporting/Tests] Consolidate test archives, move kbn objects to kbn_archiver #116528

Merged

Conversation

tsullivan
Copy link
Member

@tsullivan tsullivan commented Oct 27, 2021

Summary

Closes #96268

This PR consolidates all the Kibana saved objects that were in various ES Archives and moves them to a single Kbn Archive referred to as reporting/logs in the code.

To access any of these saved objects from a test, use reportingApi.initLogs() to install into the test cluster, and reportingApi.teardownLogs() to remove.

@tsullivan tsullivan force-pushed the reporting/test-archives-consolidations branch from f22511b to a663bd6 Compare October 28, 2021 00:54
@tsullivan tsullivan force-pushed the reporting/test-archives-consolidations branch from a663bd6 to 0133637 Compare October 28, 2021 01:12
@tsullivan tsullivan changed the title Reporting/test archives consolidations [Reporting/Tests] Consolidate test archives, move kbn objects to kbn_archiver Nov 1, 2021
@tsullivan tsullivan marked this pull request as ready for review November 1, 2021 23:00
@tsullivan tsullivan requested review from a team as code owners November 1, 2021 23:00
@tsullivan tsullivan added SharedUX/fix-it-week Bugs that have been groomed and queued up for the team's next fix it week release_note:skip Skip the PR/issue when compiling release notes Team:AppServicesUx labels Nov 1, 2021
@elasticmachine
Copy link
Contributor

Pinging @elastic/kibana-reporting-services (Team:Reporting Services)

Copy link
Contributor

@jloleysens jloleysens left a comment

Choose a reason for hiding this comment

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

LGTM @tsullivan , left a question about one of the data .load calls to check before merging 👍🏻

@tsullivan tsullivan enabled auto-merge (squash) November 2, 2021 19:05
@tsullivan
Copy link
Member Author

@elasticmachine merge upstream

@tsullivan tsullivan merged commit 488f112 into elastic:main Nov 3, 2021
@kibanamachine
Copy link
Contributor

💚 Build Succeeded

Metrics [docs]

✅ unchanged

History

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

tsullivan added a commit to tsullivan/kibana that referenced this pull request Nov 3, 2021
…archiver (elastic#116528)

* remove unused

* remove kibana objects from reporting es_archives

* import objects using kibanaServer for tests

* consolidate ecommerce_kibana_spaces

* self-review

* fix nanos test

* fix loading of reporting/ecommerce_kibana_spaces

* fix csv snapshots

* fix more csv tests

* archive rename

* consolidate canvas_disallowed_url archive

* clean up snapshots

* fix CSV tests

* polish

* remove unused

* Update x-pack/test/reporting_api_integration/reporting_and_security/network_policy.ts
tsullivan added a commit that referenced this pull request Nov 4, 2021
…archiver (#116528) (#117440)

* remove unused

* remove kibana objects from reporting es_archives

* import objects using kibanaServer for tests

* consolidate ecommerce_kibana_spaces

* self-review

* fix nanos test

* fix loading of reporting/ecommerce_kibana_spaces

* fix csv snapshots

* fix more csv tests

* archive rename

* consolidate canvas_disallowed_url archive

* clean up snapshots

* fix CSV tests

* polish

* remove unused

* Update x-pack/test/reporting_api_integration/reporting_and_security/network_policy.ts
@tsullivan tsullivan deleted the reporting/test-archives-consolidations branch February 11, 2022 17:47
@sophiec20 sophiec20 added the (Deprecated) Feature:Reporting Use Reporting:Screenshot, Reporting:CSV, or Reporting:Framework instead label Aug 21, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
(Deprecated) Feature:Reporting Use Reporting:Screenshot, Reporting:CSV, or Reporting:Framework instead release_note:skip Skip the PR/issue when compiling release notes SharedUX/fix-it-week Bugs that have been groomed and queued up for the team's next fix it week v8.0.0 v8.1.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Reporting/Tech Debt] Consolidate es_archives for Reporting tests
5 participants