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 Posture] CIS AWS - Multi-dashboards links and labels #148941

Merged
merged 7 commits into from
Jan 15, 2023

Conversation

JordanSh
Copy link
Contributor

@JordanSh JordanSh commented Jan 15, 2023

Summary

Resolves #146590

image

  • Dashboards will now link to the findings page with the proper filter
  • Some labels were changed for the cloud dashboard
  • dashboard will now display the AWS logo when benchmark.id is cis_aws

For now, we will not split the dashboards completely, the only difference ATM is just some labels, we decided that its better to keep it simple for now and replace those labels where its needed

@JordanSh JordanSh self-assigned this Jan 15, 2023
@JordanSh JordanSh added release_note:skip Skip the PR/issue when compiling release notes Team:Cloud Security Cloud Security team related labels Jan 15, 2023
@JordanSh JordanSh requested a review from orouz January 15, 2023 12:02
@JordanSh JordanSh marked this pull request as ready for review January 15, 2023 12:02
@JordanSh JordanSh requested a review from a team as a code owner January 15, 2023 12:02
@elasticmachine
Copy link
Contributor

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

@kibanamachine kibanamachine requested a review from a team as a code owner January 15, 2023 12:08
Copy link
Contributor

@orouz orouz left a comment

Choose a reason for hiding this comment

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

LGTM 👍

left some minor comments

});
};

const handleViewAllClick = (clusterId: string) => {
navToFindings({ cluster_id: clusterId, 'result.evaluation': RULE_FAILED });
navToFindings({
cluster_id: clusterId,
Copy link
Contributor

Choose a reason for hiding this comment

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

is cluster_id relevant for CSPM? @CohenIdo

Copy link
Contributor Author

Choose a reason for hiding this comment

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

there will be cluster ids and account ids in cspm, but until we actually know whats going on here, we rely on cluster id for now

@JordanSh JordanSh enabled auto-merge (squash) January 15, 2023 13:24
@JordanSh JordanSh merged commit 535d27b into elastic:main Jan 15, 2023
@kibana-ci
Copy link
Collaborator

💚 Build Succeeded

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 139.0KB 139.6KB +613.0B

History

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

cc @JordanSh

@kibanamachine kibanamachine added v8.7.0 backport:skip This commit does not require backporting labels Jan 15, 2023
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:skip Skip the PR/issue when compiling release notes Team:Cloud Security Cloud Security team related v8.7.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Cloud Posture] Multiple integrations support - Dashboard per integration
5 participants