Skip to content

[Security Solution][THI] - remove obsolete analyzerDatePickersAndSourcererDisabled feature flag#205176

Merged
PhilippeOberti merged 3 commits intoelastic:mainfrom
PhilippeOberti:remove-analyzerDatePickersAndSourcererDisabled-ff
Jan 8, 2025
Merged

[Security Solution][THI] - remove obsolete analyzerDatePickersAndSourcererDisabled feature flag#205176
PhilippeOberti merged 3 commits intoelastic:mainfrom
PhilippeOberti:remove-analyzerDatePickersAndSourcererDisabled-ff

Conversation

@PhilippeOberti
Copy link
Copy Markdown
Contributor

@PhilippeOberti PhilippeOberti commented Dec 26, 2024

Summary

This PR removes the analyzerDatePickersAndSourcererDisabled 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:39
@elasticmachine
Copy link
Copy Markdown
Contributor

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

@PhilippeOberti PhilippeOberti force-pushed the remove-analyzerDatePickersAndSourcererDisabled-ff branch from 63345cb to c7f0f6f Compare December 26, 2024 22:04
Copy link
Copy Markdown
Contributor

@christineweng christineweng left a comment

Choose a reason for hiding this comment

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

@PhilippeOberti Thanks for cleaning this up! Left a nit comment, else LGTM

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

the wrapper can be removed as well

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

yup, not sure how I missed that... done!

@PhilippeOberti PhilippeOberti force-pushed the remove-analyzerDatePickersAndSourcererDisabled-ff branch from c7f0f6f to 976a4dd Compare January 6, 2025 23:25
@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
securitySolution 22.1MB 22.1MB -2.6KB

Page load bundle

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

id before after diff
securitySolution 88.2KB 88.1KB -43.0B

History

@PhilippeOberti PhilippeOberti merged commit 3fd987c into elastic:main Jan 8, 2025
@PhilippeOberti PhilippeOberti deleted the remove-analyzerDatePickersAndSourcererDisabled-ff branch January 8, 2025 16:00
@kibanamachine
Copy link
Copy Markdown
Contributor

Starting backport for target branches: 8.x

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

kibanamachine pushed a commit to kibanamachine/kibana that referenced this pull request Jan 8, 2025
…cererDisabled feature flag (elastic#205176)

## Summary

This PR removes the `analyzerDatePickersAndSourcererDisabled` 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 3fd987c)
@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 8, 2025
…ndSourcererDisabled feature flag (#205176) (#205927)

# Backport

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

<!--- 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-08T15:59:58Z","message":"[Security
Solution][THI] - remove obsolete analyzerDatePickersAndSourcererDisabled
feature flag (#205176)\n\n## Summary\r\n\r\nThis PR removes the
`analyzerDatePickersAndSourcererDisabled` feature\r\nflag that was
introduced a long time ago and has been in `disabled:\r\nfalse` state
for many 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":"3fd987caa8f932881992b846e63ee6516a5c74be","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 analyzerDatePickersAndSourcererDisabled
feature
flag","number":205176,"url":"https://github.com/elastic/kibana/pull/205176","mergeCommit":{"message":"[Security
Solution][THI] - remove obsolete analyzerDatePickersAndSourcererDisabled
feature flag (#205176)\n\n## Summary\r\n\r\nThis PR removes the
`analyzerDatePickersAndSourcererDisabled` feature\r\nflag that was
introduced a long time ago and has been in `disabled:\r\nfalse` state
for many 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":"3fd987caa8f932881992b846e63ee6516a5c74be"}},"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/205176","number":205176,"mergeCommit":{"message":"[Security
Solution][THI] - remove obsolete analyzerDatePickersAndSourcererDisabled
feature flag (#205176)\n\n## Summary\r\n\r\nThis PR removes the
`analyzerDatePickersAndSourcererDisabled` feature\r\nflag that was
introduced a long time ago and has been in `disabled:\r\nfalse` state
for many 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":"3fd987caa8f932881992b846e63ee6516a5c74be"}},{"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>
viduni94 pushed a commit to viduni94/kibana that referenced this pull request Jan 23, 2025
…cererDisabled feature flag (elastic#205176)

## Summary

This PR removes the `analyzerDatePickersAndSourcererDisabled` 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.

4 participants