-
Notifications
You must be signed in to change notification settings - Fork 444
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
[Cloud Posture] rule_number mapping update #5054
[Cloud Posture] rule_number mapping update #5054
Conversation
🌐 Coverage report
|
…field # Conflicts: # packages/cloud_security_posture/changelog.yml # packages/cloud_security_posture/manifest.yml
/test |
/test |
1 similar comment
/test |
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.
LGTM, please fix conflicts (maintain the prereleased version 1.2.10-next
)
…field # Conflicts: # packages/cloud_security_posture/changelog.yml # packages/cloud_security_posture/data_stream/findings/fields/rule.yml # packages/cloud_security_posture/manifest.yml
Related to this: |
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.
Please review the comments before merging
@@ -1,18 +1,18 @@ | |||
{ | |||
"id": "1d3a468f-78ca-54ff-a43c-0d205ad832b7", | |||
"id": "04e01d1a-d7d4-5020-a398-8aadd3fe32ae", |
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.
Please test an upgrade from an earlier version of the integration to this version to ensure saved objects are deleted and re-installed properly since the IDs changed
@@ -33,4 +35,4 @@ | |||
"csp-rule-template": "8.7.0" | |||
}, | |||
"coreMigrationVersion": "8.7.0" | |||
} | |||
} |
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.
All of these are missing a line break at the end of the file, consider modifying the script and regenerating all the rules
@oren-zohar this also resolves elastic/csp-security-policies#142 right? |
yes :) |
Package cloud_security_posture - 1.2.10-beta1 containing this change is available at https://epr.elastic.co/search?package=cloud_security_posture |
What does this PR do?
rule_number
mappingRelated issues
rule.rule_number
torule.benchmark.rule_number
cloudbeat#644