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

[Security Solution][Endpoint] Use a feature flag to use the new pending actions logic #117219

Merged
merged 7 commits into from
Nov 4, 2021
Merged

[Security Solution][Endpoint] Use a feature flag to use the new pending actions logic #117219

merged 7 commits into from
Nov 4, 2021

Conversation

ashokaditya
Copy link
Member

@ashokaditya ashokaditya commented Nov 2, 2021

Summary

Adds a feature flag to check pending actions logic. This is by default set to true for now. We might need to set it to false in case issues with cloud instances are not resolved. Then we would fall back to the old logic for showing pending actions

Checklist

@ashokaditya ashokaditya added v8.0.0 release_note:skip Skip the PR/issue when compiling release notes Team:Defend Workflows “EDR Workflows” sub-team of Security Solution auto-backport Deprecated - use backport:version if exact versions are needed v7.16.0 v8.1.0 labels Nov 2, 2021
@ashokaditya ashokaditya self-assigned this Nov 2, 2021
@ashokaditya ashokaditya marked this pull request as ready for review November 2, 2021 19:11
@ashokaditya ashokaditya requested review from a team as code owners November 2, 2021 19:11
@ashokaditya ashokaditya requested review from academo and pzl November 2, 2021 19:11
@elasticmachine
Copy link
Contributor

Pinging @elastic/security-onboarding-and-lifecycle-mgt (Team:Onboarding and Lifecycle Mgt)

@ashokaditya ashokaditya requested a review from kevinlog November 2, 2021 19:11
@ashokaditya
Copy link
Member Author

@elasticmachine merge upstream

@ashokaditya ashokaditya requested a review from pzl November 3, 2021 17:09
@ashokaditya
Copy link
Member Author

jenkins build this

@ashokaditya
Copy link
Member Author

@elasticmachine merge upstream

`parseExperimentalConfigValue` method sets feature flag key values to `true` if passed as arg
@ashokaditya
Copy link
Member Author

@elasticmachine merge upstream

@ashokaditya ashokaditya enabled auto-merge (squash) November 4, 2021 10:11
@ashokaditya ashokaditya merged commit d422da5 into elastic:main Nov 4, 2021
@kibanamachine
Copy link
Contributor

💚 Build Succeeded

Metrics [docs]

Page load bundle

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

id before after diff
securitySolution 252.9KB 252.9KB +33.0B

History

To update your PR or re-run it, just comment with:
@elasticmachine merge upstream

cc @ashokaditya

@ashokaditya ashokaditya deleted the task/olm-pending_actions_with_feature_flag branch November 4, 2021 12:09
kibanamachine added a commit to kibanamachine/kibana that referenced this pull request Nov 4, 2021
…ng actions logic (elastic#117219)

* use a feature flag to use the new pending actions logic

refs elastic/issues/116715

* switch off pending actions for endpoints when feature flag is disabled

review suggestions

* update/add tests to use FF

* correctly override the FF

`parseExperimentalConfigValue` method sets feature flag key values to `true` if passed as arg

Co-authored-by: Kibana Machine <[email protected]>
kibanamachine added a commit to kibanamachine/kibana that referenced this pull request Nov 4, 2021
…ng actions logic (elastic#117219)

* use a feature flag to use the new pending actions logic

refs elastic/issues/116715

* switch off pending actions for endpoints when feature flag is disabled

review suggestions

* update/add tests to use FF

* correctly override the FF

`parseExperimentalConfigValue` method sets feature flag key values to `true` if passed as arg

Co-authored-by: Kibana Machine <[email protected]>
@kibanamachine
Copy link
Contributor

💚 Backport successful

Status Branch Result
8.0
7.16

The backport PRs will be merged automatically after passing CI.

kibanamachine added a commit that referenced this pull request Nov 4, 2021
…ng actions logic (#117219) (#117497)

* use a feature flag to use the new pending actions logic

refs /issues/116715

* switch off pending actions for endpoints when feature flag is disabled

review suggestions

* update/add tests to use FF

* correctly override the FF

`parseExperimentalConfigValue` method sets feature flag key values to `true` if passed as arg

Co-authored-by: Kibana Machine <[email protected]>

Co-authored-by: Ashokaditya <[email protected]>
kibanamachine added a commit that referenced this pull request Nov 4, 2021
…ng actions logic (#117219) (#117498)

* use a feature flag to use the new pending actions logic

refs /issues/116715

* switch off pending actions for endpoints when feature flag is disabled

review suggestions

* update/add tests to use FF

* correctly override the FF

`parseExperimentalConfigValue` method sets feature flag key values to `true` if passed as arg

Co-authored-by: Kibana Machine <[email protected]>

Co-authored-by: Ashokaditya <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
auto-backport Deprecated - use backport:version if exact versions are needed release_note:skip Skip the PR/issue when compiling release notes Team:Defend Workflows “EDR Workflows” sub-team of Security Solution v7.16.0 v8.0.0 v8.1.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants