Skip to content

[Security Solution][THI] - remove obsolete timelineEsqlTabDisabled feature flag#205175

Merged
PhilippeOberti merged 2 commits intoelastic:mainfrom
PhilippeOberti:remove-timelineEsqlTabDisabled-ff
Jan 11, 2025
Merged

[Security Solution][THI] - remove obsolete timelineEsqlTabDisabled feature flag#205175
PhilippeOberti merged 2 commits intoelastic:mainfrom
PhilippeOberti:remove-timelineEsqlTabDisabled-ff

Conversation

@PhilippeOberti
Copy link
Copy Markdown
Contributor

@PhilippeOberti PhilippeOberti commented Dec 26, 2024

Summary

This PR removes the timelineEsqlTabDisabled feature flag that was introduced a long time ago and has been in disabled: false state for many months.
I noticed that the line was moved in this PR over 6 months ago but the introduction of the feature precedes that.

No UI changes introduced!

@PhilippeOberti PhilippeOberti added release_note:skip Skip the PR/issue when compiling release notes v9.0.0 Team:Threat Hunting:Investigations Security Solution Threat Hunting Investigations Team backport:version Backport to applied version labels v8.18.0 labels Dec 26, 2024
@PhilippeOberti PhilippeOberti requested review from a team as code owners December 26, 2024 21:34
@elasticmachine
Copy link
Copy Markdown
Contributor

Pinging @elastic/security-threat-hunting-investigations (Team:Threat Hunting:Investigations)

@PhilippeOberti PhilippeOberti force-pushed the remove-timelineEsqlTabDisabled-ff branch from 9370bf8 to 01cabfe Compare January 10, 2025 15:31
@PhilippeOberti PhilippeOberti enabled auto-merge (squash) January 10, 2025 16:35
@PhilippeOberti PhilippeOberti merged commit 3d6f452 into elastic:main Jan 11, 2025
@kibanamachine
Copy link
Copy Markdown
Contributor

Starting backport for target branches: 8.x

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

@elasticmachine
Copy link
Copy Markdown
Contributor

💛 Build succeeded, but was flaky

Failed CI Steps

Test Failures

  • [job] [logs] FTR Configs #91 / InfraOps App Metrics UI Hosts View #With data #Permissions: Read Only User - Single Host Flyout Dashboards Tab should render dashboards tab splash screen with disabled option to add dashboard

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
securitySolution 21.1MB 21.1MB -1.7KB

Page load bundle

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

id before after diff
securitySolution 87.3KB 87.3KB -27.0B

History

kibanamachine pushed a commit to kibanamachine/kibana that referenced this pull request Jan 11, 2025
…ature flag (elastic#205175)

## Summary

This PR removes the `timelineEsqlTabDisabled` feature flag that was
introduced a long time ago and has been in `disabled: false` state for
many months.
I noticed that the line was moved in [this
PR](elastic#176064) over 6 months ago but
the introduction of the feature precedes that.

No UI changes introduced!

(cherry picked from commit 3d6f452)
@kibanamachine
Copy link
Copy Markdown
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 Jan 11, 2025
…led feature flag (#205175) (#206364)

# Backport

This will backport the following commits from `main` to `8.x`:
- [[Security Solution][THI] - remove obsolete timelineEsqlTabDisabled
feature flag (#205175)](#205175)

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

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

<!--BACKPORT [{"author":{"name":"Philippe
Oberti","email":"philippe.oberti@elastic.co"},"sourceCommit":{"committedDate":"2025-01-11T00:14:19Z","message":"[Security
Solution][THI] - remove obsolete timelineEsqlTabDisabled feature flag
(#205175)\n\n## Summary\r\n\r\nThis PR removes the
`timelineEsqlTabDisabled` feature flag that was\r\nintroduced a long
time ago and has been in `disabled: false` state for\r\nmany
months.\r\nI noticed that the line was moved in
[this\r\nPR](#176064) over 6
months ago but\r\nthe introduction of the feature precedes
that.\r\n\r\nNo UI changes
introduced!","sha":"3d6f45292f794b10449c96a1446517490ce8168e","branchLabelMapping":{"^v9.0.0$":"main","^v8.18.0$":"8.x","^v(\\d+).(\\d+).\\d+$":"$1.$2"}},"sourcePullRequest":{"labels":["release_note:skip","v9.0.0","Team:Threat
Hunting:Investigations","backport:version","v8.18.0"],"title":"[Security
Solution][THI] - remove obsolete timelineEsqlTabDisabled feature
flag","number":205175,"url":"https://github.com/elastic/kibana/pull/205175","mergeCommit":{"message":"[Security
Solution][THI] - remove obsolete timelineEsqlTabDisabled feature flag
(#205175)\n\n## Summary\r\n\r\nThis PR removes the
`timelineEsqlTabDisabled` feature flag that was\r\nintroduced a long
time ago and has been in `disabled: false` state for\r\nmany
months.\r\nI noticed that the line was moved in
[this\r\nPR](#176064) over 6
months ago but\r\nthe introduction of the feature precedes
that.\r\n\r\nNo UI changes
introduced!","sha":"3d6f45292f794b10449c96a1446517490ce8168e"}},"sourceBranch":"main","suggestedTargetBranches":["8.x"],"targetPullRequestStates":[{"branch":"main","label":"v9.0.0","branchLabelMappingKey":"^v9.0.0$","isSourceBranch":true,"state":"MERGED","url":"https://github.com/elastic/kibana/pull/205175","number":205175,"mergeCommit":{"message":"[Security
Solution][THI] - remove obsolete timelineEsqlTabDisabled feature flag
(#205175)\n\n## Summary\r\n\r\nThis PR removes the
`timelineEsqlTabDisabled` feature flag that was\r\nintroduced a long
time ago and has been in `disabled: false` state for\r\nmany
months.\r\nI noticed that the line was moved in
[this\r\nPR](#176064) over 6
months ago but\r\nthe introduction of the feature precedes
that.\r\n\r\nNo UI changes
introduced!","sha":"3d6f45292f794b10449c96a1446517490ce8168e"}},{"branch":"8.x","label":"v8.18.0","branchLabelMappingKey":"^v8.18.0$","isSourceBranch":false,"state":"NOT_CREATED"}]}]
BACKPORT-->

Co-authored-by: Philippe Oberti <philippe.oberti@elastic.co>
@PhilippeOberti PhilippeOberti deleted the remove-timelineEsqlTabDisabled-ff branch January 21, 2025 21:32
viduni94 pushed a commit to viduni94/kibana that referenced this pull request Jan 23, 2025
…ature flag (elastic#205175)

## Summary

This PR removes the `timelineEsqlTabDisabled` feature flag that was
introduced a long time ago and has been in `disabled: false` state for
many months.
I noticed that the line was moved in [this
PR](elastic#176064) over 6 months ago but
the introduction of the feature precedes that.

No UI changes introduced!
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport:version Backport to applied version labels release_note:skip Skip the PR/issue when compiling release notes Team:Threat Hunting:Investigations Security Solution Threat Hunting Investigations Team v8.18.0 v9.0.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants