-
Notifications
You must be signed in to change notification settings - Fork 514
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
[FR] Bundle KQL & Kibana libs into base dependencies #3662
Conversation
"semver==3.0.2" | ||
"semver==3.0.2", | ||
"detection-rules-kql @ git+https://github.com/elastic/detection-rules.git#subdirectory=lib/kql", | ||
"detection-rules-kibana @ git+https://github.com/elastic/detection-rules.git#subdirectory=lib/kibana" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
🔥
Reminder to update export-rules to |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
🟢 Manual review, looks good to me! 👍
(cherry picked from commit 7883754)
(cherry picked from commit 7883754)
(cherry picked from commit 7883754)
(cherry picked from commit 7883754)
@@ -24,7 +24,6 @@ jobs: | |||
python -m pip install --upgrade pip | |||
pip cache purge | |||
pip install .[dev] | |||
pip install lib/kql lib/kibana |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
If needed, we can also move this line up instead of removing it so that it installs from the local environment.
(cherry picked from commit 7883754)
(cherry picked from commit 7883754)
(cherry picked from commit 7883754)
(cherry picked from commit 7883754)
(cherry picked from commit 7883754)
(cherry picked from commit 7883754)
(cherry picked from commit 7883754)
Issues
Related to #3514
Summary
Instead of bundling kql/kibana packages separately, add the packages to the code pyproject.toml using the git+ url.
Note: This implies that kql/kibana will pull from
main
git. For local development you will need to uninstall the git version and install locally for testing/dev.Bug Fixes
Updates the test cli script to use the new export command.
Testing
make test-cli
andmake test-remote-cli
should pass.make
should work as expected.Make output
3662_output.txt