[ObsUX][HostsOtel] Remove hostsOtelEnabled feature flag#232497
[ObsUX][HostsOtel] Remove hostsOtelEnabled feature flag#232497rmyz merged 11 commits intoelastic:mainfrom
Conversation
|
Pinging @elastic/obs-ux-infra_services-team (Team:obs-ux-infra_services) |
rmyz
left a comment
There was a problem hiding this comment.
code LGTM, added a small nit
x-pack/solutions/observability/plugins/infra/public/alerting/inventory/index.tsx
Outdated
Show resolved
Hide resolved
135f26c to
c6f0081
Compare
…elEnabled-feature-flag
azasypkin
left a comment
There was a problem hiding this comment.
Changes in src/platform/test/plugin_functional/test_suites/core_plugins/rendering.ts LGMT from AppEx Security side (removed exposed boolean variable 🥳).
41f910c to
ca95cd1
Compare
🤖 GitHub commentsExpand to view the GitHub comments
Just comment with:
|
|
/oblt-deploy |
|
Hey @MiriamAparicio @crespocarlos, I found a few bugs/enhancements but I'll raise these as follow up issues in an upcoming deliverable. Overall, looking great! |
|
Making this a draft as we need to address some bugs before removing the feature flag |
|
After #233619 is merged, we can merge this one too |
💚 Build Succeeded
Metrics [docs]Async chunks
Page load bundle
History
|
Co-authored-by: Sergi Romeu <sergi.romeu@elastic.co>
Co-authored-by: Sergi Romeu <sergi.romeu@elastic.co>
Co-authored-by: Sergi Romeu <sergi.romeu@elastic.co>
Summary
Closes https://github.com/elastic/observability-dev/issues/4689
This PR removes the feature flag
hostsOtelEnabledadded for the development during implementation of the Schema Selector for Hosts UI to work for Otel