Skip to content

[ObsUX][HostsOtel] Remove hostsOtelEnabled feature flag#232497

Merged
rmyz merged 11 commits intoelastic:mainfrom
MiriamAparicio:4689-remove-hostsOtelEnabled-feature-flag
Sep 1, 2025
Merged

[ObsUX][HostsOtel] Remove hostsOtelEnabled feature flag#232497
rmyz merged 11 commits intoelastic:mainfrom
MiriamAparicio:4689-remove-hostsOtelEnabled-feature-flag

Conversation

@MiriamAparicio
Copy link
Copy Markdown
Contributor

@MiriamAparicio MiriamAparicio commented Aug 21, 2025

Summary

Closes https://github.com/elastic/observability-dev/issues/4689

This PR removes the feature flag hostsOtelEnabled added for the development during implementation of the Schema Selector for Hosts UI to work for Otel

@MiriamAparicio MiriamAparicio requested a review from a team as a code owner August 21, 2025 14:25
@MiriamAparicio MiriamAparicio added the release_note:skip Skip the PR/issue when compiling release notes label Aug 21, 2025
@MiriamAparicio MiriamAparicio requested a review from a team as a code owner August 21, 2025 14:25
@MiriamAparicio MiriamAparicio requested a review from a team August 21, 2025 14:25
@MiriamAparicio MiriamAparicio added the backport:skip This PR does not require backporting label Aug 21, 2025
@MiriamAparicio MiriamAparicio requested a review from a team as a code owner August 21, 2025 14:25
@MiriamAparicio MiriamAparicio added Team:obs-ux-infra_services - DEPRECATED DEPRECATED - Use Team:obs-presentation. v9.2.0 labels Aug 21, 2025
@elasticmachine
Copy link
Copy Markdown
Contributor

Pinging @elastic/obs-ux-infra_services-team (Team:obs-ux-infra_services)

Copy link
Copy Markdown
Contributor

@rmyz rmyz left a comment

Choose a reason for hiding this comment

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

code LGTM, added a small nit

@MiriamAparicio MiriamAparicio force-pushed the 4689-remove-hostsOtelEnabled-feature-flag branch from 135f26c to c6f0081 Compare August 22, 2025 10:19
Copy link
Copy Markdown
Contributor

@cesco-f cesco-f left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Copy Markdown
Contributor

@azasypkin azasypkin left a comment

Choose a reason for hiding this comment

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

Changes in src/platform/test/plugin_functional/test_suites/core_plugins/rendering.ts LGMT from AppEx Security side (removed exposed boolean variable 🥳).

@MiriamAparicio MiriamAparicio force-pushed the 4689-remove-hostsOtelEnabled-feature-flag branch from 41f910c to ca95cd1 Compare August 25, 2025 10:43
@rmyz rmyz added the ci:project-deploy-observability Create an Observability project label Aug 26, 2025
@github-actions
Copy link
Copy Markdown
Contributor

🤖 GitHub comments

Expand to view the GitHub comments

Just comment with:

  • /oblt-deploy : Deploy a Kibana instance using the Observability test environments.
  • run docs-build : Re-trigger the docs validation. (use unformatted text in the comment!)

@rmyz
Copy link
Copy Markdown
Contributor

rmyz commented Aug 26, 2025

/oblt-deploy

@roshan-elastic
Copy link
Copy Markdown

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!

@rmyz rmyz marked this pull request as draft August 27, 2025 09:21
@rmyz
Copy link
Copy Markdown
Contributor

rmyz commented Aug 27, 2025

Making this a draft as we need to address some bugs before removing the feature flag

@rmyz rmyz changed the title [ObsUX][HostsOtel] Remove hostsOtelEnabled feature flag [DO NOT MERGE][ObsUX][HostsOtel] Remove hostsOtelEnabled feature flag Aug 27, 2025
@rmyz rmyz marked this pull request as ready for review September 1, 2025 11:19
@rmyz
Copy link
Copy Markdown
Contributor

rmyz commented Sep 1, 2025

After #233619 is merged, we can merge this one too

@rmyz rmyz changed the title [DO NOT MERGE][ObsUX][HostsOtel] Remove hostsOtelEnabled feature flag [ObsUX][HostsOtel] Remove hostsOtelEnabled feature flag Sep 1, 2025
@rmyz rmyz enabled auto-merge (squash) September 1, 2025 13:58
@rmyz rmyz merged commit cd6c448 into elastic:main Sep 1, 2025
13 checks passed
@elasticmachine
Copy link
Copy Markdown
Contributor

💚 Build Succeeded

Metrics [docs]

Async chunks

Total size of all lazy-loaded chunks that will be downloaded as the user navigates the app

id before after diff
infra 1.0MB 1.0MB -82.0B

Page load bundle

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

id before after diff
infra 51.6KB 51.4KB -188.0B

History

cc @rmyz @MiriamAparicio

ymao1 pushed a commit to ymao1/kibana that referenced this pull request Sep 2, 2025
Co-authored-by: Sergi Romeu <sergi.romeu@elastic.co>
MichelLosier pushed a commit to MichelLosier/kibana that referenced this pull request Sep 2, 2025
Co-authored-by: Sergi Romeu <sergi.romeu@elastic.co>
kowalczyk-krzysztof pushed a commit to kowalczyk-krzysztof/kibana that referenced this pull request Sep 3, 2025
Co-authored-by: Sergi Romeu <sergi.romeu@elastic.co>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport:skip This PR does not require backporting release_note:skip Skip the PR/issue when compiling release notes Team:obs-ux-infra_services - DEPRECATED DEPRECATED - Use Team:obs-presentation. v9.2.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

8 participants