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][Onboarding] Enable search indices & gate with a feature flag #195802

Merged

Conversation

TattdCodeMonkey
Copy link
Contributor

@TattdCodeMonkey TattdCodeMonkey commented Oct 10, 2024

Summary

This PR enables the search_indices plugin in serverless search. But then gates it with a UI settings feature flag until we are ready to ship the new onboarding experience to all users.

Testing

Locally you can add this to your kibana.dev.yml to enable the FF:

uiSettings.overrides.searchIndices:globalEmptyStateEnabled: true

Or you can enable the ui setting via Dev Tools and refresh the browser:

POST kbn:/internal/kibana/settings/searchIndices:globalEmptyStateEnabled
{"value": true}

Checklist

@TattdCodeMonkey TattdCodeMonkey added release_note:skip Skip the PR/issue when compiling release notes v9.0.0 Team:Search backport:prev-minor Backport to (8.x) the previous minor version (i.e. one version back from main) labels Oct 10, 2024
@TattdCodeMonkey TattdCodeMonkey requested review from a team as code owners October 10, 2024 15:48
@kibanamachine
Copy link
Contributor

Flaky Test Runner Stats

🎉 All tests passed! - kibana-flaky-test-suite-runner#7123

[✅] x-pack/test_serverless/functional/test_suites/search/config.feature_flags.ts: 25/25 tests passed.
[✅] x-pack/test_serverless/functional/test_suites/search/config.ts: 25/25 tests passed.

see run history

Copy link
Member

@joemcelroy joemcelroy left a comment

Choose a reason for hiding this comment

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

tested with and without the ui setting config value set. Works as expected.

@TattdCodeMonkey
Copy link
Contributor Author

@elasticmachine merge upstream

@jloleysens jloleysens added the ci:project-deploy-elasticsearch Create an Elasticsearch Serverless project label Oct 11, 2024
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.

Approving to unblock progress, but I think it's worth checking this PR against a project in QA before merging. I just added a tag and merged main.

@elasticmachine
Copy link
Contributor

elasticmachine commented Oct 11, 2024

💚 Build Succeeded

Metrics [docs]

Module Count

Fewer modules leads to a faster build time

id before after diff
searchIndices 220 221 +1

Public APIs missing comments

Total count of every public API that lacks a comment. Target amount is 0. Run node scripts/build_api_docs --plugin [yourplugin] --stats comments for more detailed information.

id before after diff
searchIndices 18 19 +1

Page load bundle

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

id before after diff
searchIndices 6.8KB 7.1KB +267.0B
serverlessSearch 21.5KB 21.5KB +9.0B
total +276.0B
Unknown metric groups

API count

id before after diff
searchIndices 18 19 +1

History

@kc13greiner kc13greiner self-requested a review October 11, 2024 14:28
Copy link
Contributor

@kc13greiner kc13greiner left a comment

Choose a reason for hiding this comment

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

LGTM

@TattdCodeMonkey TattdCodeMonkey merged commit 8577d13 into elastic:main Oct 11, 2024
27 checks passed
@TattdCodeMonkey TattdCodeMonkey deleted the onboarding/enable-and-gate branch October 11, 2024 17:21
@kibanamachine
Copy link
Contributor

Starting backport for target branches: 8.x

https://github.com/elastic/kibana/actions/runs/11296621299

kibanamachine pushed a commit to kibanamachine/kibana that referenced this pull request Oct 11, 2024
…elastic#195802)

## Summary

This PR enables the `search_indices` plugin in serverless search. But
then gates it with a UI settings feature flag until we are ready to ship
the new onboarding experience to all users.

### Testing
Locally you can add this to your `kibana.dev.yml` to enable the FF:
```
uiSettings.overrides.searchIndices:globalEmptyStateEnabled: true
```

Or you can enable the ui setting via Dev Tools and refresh the browser:

```
POST kbn:/internal/kibana/settings/searchIndices:globalEmptyStateEnabled
{"value": true}
```

### Checklist

- [x] [Unit or functional
tests](https://www.elastic.co/guide/en/kibana/master/development-tests.html)
were updated or added to match the most common scenarios
- [x] [Flaky Test
Runner](https://ci-stats.kibana.dev/trigger_flaky_test_runner/1) was
used on any tests changed

Co-authored-by: Elastic Machine <[email protected]>
Co-authored-by: Jean-Louis Leysens <[email protected]>
(cherry picked from commit 8577d13)
@kibanamachine
Copy link
Contributor

💚 All backports created successfully

Status Branch Result
8.x

Note: Successful backport PRs will be merged automatically after passing CI.

Questions ?

Please refer to the Backport tool documentation

kibanamachine added a commit that referenced this pull request Oct 11, 2024
…ature flag (#195802) (#195972)

# Backport

This will backport the following commits from `main` to `8.x`:
- [[Search][Onboarding] Enable search indices &amp; gate with a feature
flag (#195802)](#195802)

<!--- Backport version: 9.4.3 -->

### Questions ?
Please refer to the [Backport tool
documentation](https://github.com/sqren/backport)

<!--BACKPORT [{"author":{"name":"Rodney
Norris","email":"[email protected]"},"sourceCommit":{"committedDate":"2024-10-11T17:21:47Z","message":"[Search][Onboarding]
Enable search indices & gate with a feature flag (#195802)\n\n##
Summary\r\n\r\nThis PR enables the `search_indices` plugin in serverless
search. But\r\nthen gates it with a UI settings feature flag until we
are ready to ship\r\nthe new onboarding experience to all
users.\r\n\r\n### Testing\r\nLocally you can add this to your
`kibana.dev.yml` to enable the
FF:\r\n```\r\nuiSettings.overrides.searchIndices:globalEmptyStateEnabled:
true\r\n```\r\n\r\nOr you can enable the ui setting via Dev Tools and
refresh the browser:\r\n\r\n```\r\nPOST
kbn:/internal/kibana/settings/searchIndices:globalEmptyStateEnabled\r\n{\"value\":
true}\r\n```\r\n\r\n### Checklist\r\n\r\n- [x] [Unit or
functional\r\ntests](https://www.elastic.co/guide/en/kibana/master/development-tests.html)\r\nwere
updated or added to match the most common scenarios\r\n- [x] [Flaky
Test\r\nRunner](https://ci-stats.kibana.dev/trigger_flaky_test_runner/1)
was\r\nused on any tests changed\r\n\r\nCo-authored-by: Elastic Machine
<[email protected]>\r\nCo-authored-by: Jean-Louis
Leysens
<[email protected]>","sha":"8577d1357e2c59f66e397a480ec39b2c405787db","branchLabelMapping":{"^v9.0.0$":"main","^v8.16.0$":"8.x","^v(\\d+).(\\d+).\\d+$":"$1.$2"}},"sourcePullRequest":{"labels":["release_note:skip","v9.0.0","Team:Search","backport:prev-minor","ci:project-deploy-elasticsearch"],"title":"[Search][Onboarding]
Enable search indices & gate with a feature
flag","number":195802,"url":"https://github.com/elastic/kibana/pull/195802","mergeCommit":{"message":"[Search][Onboarding]
Enable search indices & gate with a feature flag (#195802)\n\n##
Summary\r\n\r\nThis PR enables the `search_indices` plugin in serverless
search. But\r\nthen gates it with a UI settings feature flag until we
are ready to ship\r\nthe new onboarding experience to all
users.\r\n\r\n### Testing\r\nLocally you can add this to your
`kibana.dev.yml` to enable the
FF:\r\n```\r\nuiSettings.overrides.searchIndices:globalEmptyStateEnabled:
true\r\n```\r\n\r\nOr you can enable the ui setting via Dev Tools and
refresh the browser:\r\n\r\n```\r\nPOST
kbn:/internal/kibana/settings/searchIndices:globalEmptyStateEnabled\r\n{\"value\":
true}\r\n```\r\n\r\n### Checklist\r\n\r\n- [x] [Unit or
functional\r\ntests](https://www.elastic.co/guide/en/kibana/master/development-tests.html)\r\nwere
updated or added to match the most common scenarios\r\n- [x] [Flaky
Test\r\nRunner](https://ci-stats.kibana.dev/trigger_flaky_test_runner/1)
was\r\nused on any tests changed\r\n\r\nCo-authored-by: Elastic Machine
<[email protected]>\r\nCo-authored-by: Jean-Louis
Leysens
<[email protected]>","sha":"8577d1357e2c59f66e397a480ec39b2c405787db"}},"sourceBranch":"main","suggestedTargetBranches":[],"targetPullRequestStates":[{"branch":"main","label":"v9.0.0","branchLabelMappingKey":"^v9.0.0$","isSourceBranch":true,"state":"MERGED","url":"https://github.com/elastic/kibana/pull/195802","number":195802,"mergeCommit":{"message":"[Search][Onboarding]
Enable search indices & gate with a feature flag (#195802)\n\n##
Summary\r\n\r\nThis PR enables the `search_indices` plugin in serverless
search. But\r\nthen gates it with a UI settings feature flag until we
are ready to ship\r\nthe new onboarding experience to all
users.\r\n\r\n### Testing\r\nLocally you can add this to your
`kibana.dev.yml` to enable the
FF:\r\n```\r\nuiSettings.overrides.searchIndices:globalEmptyStateEnabled:
true\r\n```\r\n\r\nOr you can enable the ui setting via Dev Tools and
refresh the browser:\r\n\r\n```\r\nPOST
kbn:/internal/kibana/settings/searchIndices:globalEmptyStateEnabled\r\n{\"value\":
true}\r\n```\r\n\r\n### Checklist\r\n\r\n- [x] [Unit or
functional\r\ntests](https://www.elastic.co/guide/en/kibana/master/development-tests.html)\r\nwere
updated or added to match the most common scenarios\r\n- [x] [Flaky
Test\r\nRunner](https://ci-stats.kibana.dev/trigger_flaky_test_runner/1)
was\r\nused on any tests changed\r\n\r\nCo-authored-by: Elastic Machine
<[email protected]>\r\nCo-authored-by: Jean-Louis
Leysens
<[email protected]>","sha":"8577d1357e2c59f66e397a480ec39b2c405787db"}}]}]
BACKPORT-->

Co-authored-by: Rodney Norris <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport:prev-minor Backport to (8.x) the previous minor version (i.e. one version back from main) ci:project-deploy-elasticsearch Create an Elasticsearch Serverless project release_note:skip Skip the PR/issue when compiling release notes Team:Search v8.16.0 v9.0.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants