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

[Search Sessions] Enable Search Sessions #91097

Merged
merged 8 commits into from
Feb 13, 2021

Conversation

Dosant
Copy link
Contributor

@Dosant Dosant commented Feb 11, 2021

Summary

❗ Waiting for #84975 merged first

Enables Search Sessions by default

I had to tweak a bunch of feature_controls tests. In the most of the cases changed global_dashboard_all -> global_dashboard_read. Those tests are testing the visibility of specific management sections and assertions broke because as per #90818 dashboard_all and discover_all now enable search sessions and search session management becomes available.

Release Notes

Search Sessions in Discover and Dashboard are available

@Dosant Dosant changed the title enable search sessions [Search Sessions] Enable Search Sessions Feb 11, 2021
@Dosant Dosant added release highlight Team:AppServices v7.12.0 v8.0.0 Feature:Search Querying infrastructure in Kibana release_note:feature Makes this part of the condensed release notes and removed release highlight labels Feb 11, 2021
@Dosant Dosant marked this pull request as ready for review February 11, 2021 16:42
@Dosant Dosant requested review from a team as code owners February 11, 2021 16:42
@elasticmachine
Copy link
Contributor

Pinging @elastic/kibana-app-services (Team:AppServices)

Copy link
Member

@pheyos pheyos left a comment

Choose a reason for hiding this comment

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

Transform test changes LGTM

Copy link
Member

@legrego legrego left a comment

Choose a reason for hiding this comment

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

Test changes LGTM - thank you for the ping on this PR, AND for the added functional tests in #90818!

Edit: I'd like to see the user filtering PR merged ahead of this one. We shouldn't turn this on by default until we have the per-user restrictions in place

Copy link

@elastic-jb elastic-jb left a comment

Choose a reason for hiding this comment

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

Ship it!

@lizozom
Copy link
Contributor

lizozom commented Feb 13, 2021

@elasticmachine merge upstream

@kibanamachine
Copy link
Contributor

💚 Build Succeeded

Metrics [docs]

Page load bundle

Size of the bundles that are downloaded on every page load. Target size is below 100kb

id before after diff
dataEnhanced 38.7KB 38.7KB +3.0B

History

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

@lizozom lizozom merged commit 5fb5be6 into elastic:master Feb 13, 2021
lizozom pushed a commit to lizozom/kibana that referenced this pull request Feb 13, 2021
* enable search sessions

* adjust tests

* adjust tests

* adjust tests

* adjust tests

* make order consistent

Co-authored-by: Kibana Machine <42973632+kibanamachine@users.noreply.github.com>
lizozom added a commit that referenced this pull request Feb 14, 2021
* [Search Sessions] Enable Search Sessions (#91097)

* enable search sessions

* adjust tests

* adjust tests

* adjust tests

* adjust tests

* make order consistent

Co-authored-by: Kibana Machine <42973632+kibanamachine@users.noreply.github.com>

* adjust tests

that is skipped in master and also needs adjustment in master to pass with search sessions

Co-authored-by: Anton Dosov <anton.dosov@elastic.co>
Co-authored-by: Kibana Machine <42973632+kibanamachine@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Feature:Search Querying infrastructure in Kibana release_note:feature Makes this part of the condensed release notes v7.12.0 v8.0.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants