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

[Cloud Security] updates to move from doc_root.vulnerability.package -> doc_root.package (ECS) #164651

Merged
merged 3 commits into from
Aug 24, 2023

Conversation

mitodrummer
Copy link
Contributor

Summary

Issue: #157674

This updates all instances of vulnerability.package to the ECS standard package fieldset.

The new field has been populated by cloudbeat since 8.8

Checklist

Delete any items that are not applicable to this PR.

@mitodrummer mitodrummer requested a review from a team as a code owner August 23, 2023 23:33
@mitodrummer mitodrummer added the Team:Cloud Security Cloud Security team related label Aug 23, 2023
@elasticmachine
Copy link
Contributor

Pinging @elastic/kibana-cloud-security-posture (Team:Cloud Security)

@mitodrummer mitodrummer added technical debt Improvement of the software architecture and operational architecture release_note:deprecation labels Aug 23, 2023
@mitodrummer mitodrummer linked an issue Aug 23, 2023 that may be closed by this pull request
Copy link
Contributor

@opauloh opauloh left a comment

Choose a reason for hiding this comment

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

Looks good

@kibana-ci
Copy link
Collaborator

💛 Build succeeded, but was flaky

Failed CI Steps

Metrics [docs]

Async chunks

Total size of all lazy-loaded chunks that will be downloaded as the user navigates the app

id before after diff
cloudSecurityPosture 264.1KB 263.6KB -502.0B

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

@mitodrummer mitodrummer merged commit fda98b0 into elastic:main Aug 24, 2023
@kibanamachine kibanamachine added v8.11.0 backport:skip This commit does not require backporting labels Aug 24, 2023
@opauloh
Copy link
Contributor

opauloh commented Oct 24, 2023

Hi, It is worth noting this is just a deprecation notice, so there’s no action needed by users as it’s a non-breaking change, our guidance is if there's any custom report or dashboard relying on vulnerability.package.* to be pointed to package.* as it will not be populated in future versions of Cloudbeat.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport:skip This commit does not require backporting release_note:deprecation Team:Cloud Security Cloud Security team related technical debt Improvement of the software architecture and operational architecture v8.11.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Cloud Security] Use new fields locations in CNVM findings
5 participants