Movatterモバイル変換


[0]ホーム

URL:


Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings

Add cdr label to all cloud security integrations#9213

Merged
CohenIdo merged 6 commits intoelastic:mainfrom
CohenIdo:add-cdr-label
Feb 27, 2024
Merged

Add cdr label to all cloud security integrations#9213
CohenIdo merged 6 commits intoelastic:mainfrom
CohenIdo:add-cdr-label

Conversation

@CohenIdo
Copy link
Contributor

@CohenIdoCohenIdo commentedFeb 21, 2024
edited
Loading

Summary

Adding a new subcategory label,cloudsecurity_cdr to all cloud security-related integrations.

solves:

Demo

Screen.Recording.2024-02-21.at.14.33.02.mov

@CohenIdoCohenIdo changed the titleadd cdr label to all cloud security integrationsAdd cdr label to all cloud security integrationsFeb 21, 2024
@CohenIdoCohenIdo marked this pull request as ready for reviewFebruary 21, 2024 12:41
@CohenIdoCohenIdo requested review froma team ascode ownersFebruary 21, 2024 12:41
Copy link
Contributor

@kfirpeledkfirpeled left a comment

Choose a reason for hiding this comment

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

lgtm, I added minor tweaks

data_streams:
- guardduty
categories:
- cloudsecurity_cdr
Copy link
Contributor

Choose a reason for hiding this comment

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

Do you know if security category is missing here? I do see it in the demo but it would mean it will be missing from the Security filter. which I'm not sure it makes sense

- containers
- kubernetes
- security
- cloudsecurity_cdr
Copy link
Contributor

Choose a reason for hiding this comment

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

nit: you can add the tag to the root categories section (line 12)

multiple: true
categories:
- security
- cloudsecurity_cdr
Copy link
Contributor

Choose a reason for hiding this comment

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

nit: you can add the tag to the root categories section (line 12)
same goes with security - it shouldn't be repeated

Copy link
Contributor

Choose a reason for hiding this comment

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

and if you are there, can you please fix the root categories?

categories:  - aws  - kubernetes  - security

it should be only security and cloudsecurity_cdr
aws and kubernetes categories should be only in the relevant supported integrations

@elasticmachine
Copy link

🚀 Benchmarks report

Packagegoogle_scc 👍(1) 💚(1) 💔(2)

Expand to view
Data streamPrevious EPSNew EPSDiff (%)Result
asset1012.15834.72-177.43 (-17.53%)💔
audit1890.361420.45-469.91 (-24.86%)💔

Packageprisma_cloud 👍(2) 💚(0) 💔(3)

Expand to view
Data streamPrevious EPSNew EPSDiff (%)Result
alert1579.781340.48-239.3 (-15.15%)💔
audit4926.113597.12-1328.99 (-26.98%)💔
host_profile1231.531031.99-199.54 (-16.2%)💔

Packagesnyk 👍(1) 💚(0) 💔(1)

Expand to view
Data streamPrevious EPSNew EPSDiff (%)Result
vulnerabilities2724.81798.56-926.24 (-33.99%)💔

Packagewiz 👍(1) 💚(1) 💔(1)

Expand to view
Data streamPrevious EPSNew EPSDiff (%)Result
vulnerability1964.641364.26-600.38 (-30.56%)💔

To see the full report comment with/test benchmark fullreport

@elasticmachine
Copy link

💚 Build Succeeded

History

@elastic-sonarqube
Copy link

Quality Gate passedQuality Gate passed

Kudos, no new issues were introduced!

0 New issues
0 Security Hotspots
No Coverage information No data about Coverage
No Duplication information No data about Duplication

See analysis details on SonarQube

@CohenIdoCohenIdo merged commit558efe4 intoelastic:mainFeb 27, 2024
@elasticmachine
Copy link

Package aws - 2.12.1 containing this change is available athttps://epr.elastic.co/search?package=aws

@elasticmachine
Copy link

Package cloud_security_posture - 1.8.0-preview08 containing this change is available athttps://epr.elastic.co/search?package=cloud_security_posture

@elasticmachine
Copy link

Package google_scc - 1.1.1 containing this change is available athttps://epr.elastic.co/search?package=google_scc

@elasticmachine
Copy link

Package microsoft_defender_cloud - 1.1.1 containing this change is available athttps://epr.elastic.co/search?package=microsoft_defender_cloud

@elasticmachine
Copy link

Package prisma_cloud - 1.1.1 containing this change is available athttps://epr.elastic.co/search?package=prisma_cloud

@elasticmachine
Copy link

Package snyk - 1.20.1 containing this change is available athttps://epr.elastic.co/search?package=snyk

@elasticmachine
Copy link

Package wiz - 1.1.1 containing this change is available athttps://epr.elastic.co/search?package=wiz

@elasticmachine
Copy link

Package cloud_security_posture - 1.8.0 containing this change is available athttps://epr.elastic.co/search?package=cloud_security_posture

@andrewkrohandrewkroh added Integration:awsAWS Integration:cloud_security_postureSecurity Posture Management Integration:google_sccGoogle Security Command Center labelsJul 22, 2024
Sign up for freeto join this conversation on GitHub. Already have an account?Sign in to comment

Reviewers

@P1llusP1llusP1llus approved these changes

@kfirpeledkfirpeledkfirpeled approved these changes

@ShourieGShourieGShourieG approved these changes

Assignees

No one assigned

Labels

Integration:awsAWSIntegration:cloud_security_postureSecurity Posture ManagementIntegration:google_sccGoogle Security Command CenterIntegration:microsoft_defender_cloudMicrosoft Defender for CloudIntegration:prisma_cloudPalo Alto Prisma CloudIntegration:snykSnykIntegration:wizWiz

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

6 participants

@CohenIdo@elasticmachine@P1llus@kfirpeled@ShourieG@andrewkroh

Comments


[8]ページ先頭

©2009-2026 Movatter.jp