Skip to content

[Detection Engine] Add guided onboarding rule#144065

Merged
xcrzx merged 2 commits intoelastic:mainfrom
brokensound77:detection-rules/add-guided-onboarding
Oct 27, 2022
Merged

[Detection Engine] Add guided onboarding rule#144065
xcrzx merged 2 commits intoelastic:mainfrom
brokensound77:detection-rules/add-guided-onboarding

Conversation

@brokensound77
Copy link
Contributor

Summary

Adds guided onboarding detection rule

Checklist

Delete any items that are not applicable to this PR.

Screenshot

image

@brokensound77 brokensound77 added release_note:skip Skip the PR/issue when compiling release notes v8.6.0 labels Oct 26, 2022
@brokensound77 brokensound77 requested a review from a team as a code owner October 26, 2022 19:01
@@ -10,713 +10,713 @@
// - detection-rules repo using CLI command build-release
// Do not hand edit. Run script/command to regenerate package information instead

import rule1 from './credential_access_access_to_browser_credentials_procargs.json';
Copy link
Contributor Author

Choose a reason for hiding this comment

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

I noticed that the scripts/regen_prepackage_rules_index.sh orders the import by filename, whereas the detection-rules Package orders by rule name, which explains the mass change. It should have no impact, but we will see it revert to the name ordering on the next package update.

Copy link
Contributor

@terrancedejesus terrancedejesus left a comment

Choose a reason for hiding this comment

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

Changes seem fine. My question is more towards the rule updates.

  • If a client updates their existing stack to 8.5.0, this rule should load from file system but will CDN package overwrite and thus this rule will not exist? -> 8.3.2 package
  • If client does not update and are on, say version 8.3.0 this rule does not exist in file system nor CDN?

@kibana-ci
Copy link

💚 Build Succeeded

Metrics [docs]

✅ unchanged

History

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

@xcrzx xcrzx merged commit a5411c5 into elastic:main Oct 27, 2022
@kibanamachine kibanamachine added the backport:skip This PR does not require backporting label Oct 27, 2022
@brokensound77 brokensound77 deleted the detection-rules/add-guided-onboarding branch October 31, 2022 19:51
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 v8.6.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants