[Enhancement] Add flag to export rules via KQL search on name#4594
[Enhancement] Add flag to export rules via KQL search on name#4594eric-forte-elastic merged 10 commits intoelastic:mainfrom
Conversation
Enhancement - GuidelinesThese guidelines serve as a reminder set of considerations when addressing adding a feature to the code. Documentation and Context
Code Standards and Practices
Testing
Additional Checks
|
|
👋 Great find on the incorrect filter. Aligns with the docs. Also, if you lint, this should be g2g. You will need to do a pyproject.toml patch bump to |
|
Also tested the code changes locally and performs as expected 🚀 In this case, I expect the KQL to return DAC Test Rules 1-4 which occurs. |
Co-authored-by: Eric Forte <119343520+eric-forte-elastic@users.noreply.github.com>
470e88e to
0bebe0b
Compare
|
@eric-forte-elastic @Mikaayenson I added those changes:
|
|
@eric-forte-elastic Anything else or would it be ready to merge? :) |
@frederikb96 since there are code changes in both the Kibana lib and kbwrap in Detection rules, the detection rules pyproject.toml should have a patch version bump too. Otherwise looks good to me 👍 |
|
Same here, bumped it up once more 🙂 👍 |
|
Will be merging this one in too following #4593 |
|
@eric-forte-elastic Oh just merged main into this again, since conflict was there. But yes, definitely feel free to merge further as fits :) |
|
No worries :) Just having the battling pytoml versions lol, will approve momentarily, (depending on when you read this message we may need to bump it one more time) |
Pull Request
Summary - What I changed
For the CLI:
Add the possibility to use a new flag
--rule-namewhich allows searching for rules via their name and then exporting those from Kibana to local custom rules directory.How To Test
Use
Checklist
bug,enhancement,schema,maintenance,Rule: New,Rule: Deprecation,Rule: Tuning,Hunt: New, orHunt: Tuningso guidelines can be generatedmeta:rapid-mergelabel if planning to merge within 24 hoursContributor checklist