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

[Fleet] Remove experimental indexing toggles #155516

Closed
2 tasks done
kpollich opened this issue Apr 21, 2023 · 7 comments
Closed
2 tasks done

[Fleet] Remove experimental indexing toggles #155516

kpollich opened this issue Apr 21, 2023 · 7 comments
Assignees
Labels
Team:Fleet Team label for Observability Data Collection Fleet team

Comments

@kpollich
Copy link
Member

kpollich commented Apr 21, 2023

Ref #132818

In elastic/ingest-docs#151, we're moving towards providing an API-first approach to toggling experimental data stream features like synthetic source, TSDB, etc.

Currently, these toggles are gated by a feature flag, and are only used internally for testing and experimentation. With the advent of direct package upload elastic/elastic-package#1084 for better integrations DX, these toggles are not necessary. So, we're opting to remove them entirely.

Implementation

Preview Give feedback

Blocked by elastic/ingest-docs#151. We need to make sure we have public docs available for disabling these features in case users (internal or otherwise) run into problems with this functionality.

@kpollich kpollich added the Team:Fleet Team label for Observability Data Collection Fleet team label Apr 21, 2023
@elasticmachine
Copy link
Contributor

Pinging @elastic/fleet (Team:Fleet)

@jillguyonnet
Copy link
Contributor

I'm unsure about the third requirement:

Remove feature flag from ECE Kibana user settings allowlist

I see that the xpack.fleet.enableExperimental setting is allowlisted in cloud, but I'm not finding a predefined list of features.

@juliaElastic If you have a moment, do you know the answer to this question?

@jlind23
Copy link
Contributor

jlind23 commented Jun 28, 2023

Maybe @nchaulet knows as he is the one who pushed: https://github.com/elastic/cloud/pull/111041

@nchaulet
Copy link
Member

Actually I am not sure we should remove the flag from ECE as it's not specific to that feature, probably better to keep it in the allowlist

@jillguyonnet
Copy link
Contributor

So was my understanding correct that it's the experimental features setting that is allowlisted, not the specific feature flags?

jillguyonnet added a commit that referenced this issue Jun 29, 2023
Part of #155516

## Summary

* Remove experimental indexing toggles in Fleet UI
* Remove `experimentalDataStreamSettings` feature flag which is gating
these toggles

## Screenshots

### Before

![Screenshot 2023-06-27 at 12 03
13](https://github.com/elastic/kibana/assets/23701614/4ed94091-8fe2-4ecd-9ace-c1cdb4ed4057)

### After

![Screenshot 2023-06-27 at 17 15
48](https://github.com/elastic/kibana/assets/23701614/cac7f266-5660-48a6-9c06-722ceb806bd3)
@juliaElastic
Copy link
Contributor

enableExperimental includes all Fleet feature flags, not just the experimental indexing feature, so we should leave it in ECE I think.

@jlind23
Copy link
Contributor

jlind23 commented Jun 29, 2023

As stated by @juliaElastic and @nchaulet over slack, it doesn't make sense to remove the ECE allowlist as the enableExperimental setting includes all Fleet feature flags:

* A list of allowed values that can be used in `xpack.fleet.enableExperimental`.

Thus, closing this issue as done.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Team:Fleet Team label for Observability Data Collection Fleet team
Projects
None yet
Development

No branches or pull requests

6 participants