Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

[Cloud Posture] Remove k8s instructions for cspm policies #146843

Merged
merged 2 commits into from
Dec 5, 2022

Conversation

kfirpeled
Copy link
Contributor

@kfirpeled kfirpeled commented Dec 2, 2022

Summary

When installing CSPM the kubernetes deployment is not relevant and shouldn't be visible.
This PR is fixing that

Screen Shot 2022-12-04 at 15 41 00

How to test it

In order to install CSPM integration you use cloud_security_posture 1.1.0 which is recently available elastic/integrations#4752

@kfirpeled kfirpeled added the Team:Cloud Security Cloud Security team related label Dec 2, 2022
@kfirpeled kfirpeled requested a review from JordanSh December 4, 2022 13:43
@kfirpeled kfirpeled marked this pull request as ready for review December 4, 2022 13:44
@kfirpeled kfirpeled requested a review from a team as a code owner December 4, 2022 13:44
@elasticmachine
Copy link
Contributor

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

@kfirpeled kfirpeled added release_note:skip Skip the PR/issue when compiling release notes v8.7.0 labels Dec 4, 2022
@botelastic botelastic bot added the Team:Fleet Team label for Observability Data Collection Fleet team label Dec 4, 2022
@elasticmachine
Copy link
Contributor

Pinging @elastic/fleet (Team:Fleet)

@@ -66,4 +70,14 @@ export function useIsK8sPolicy(agentPolicy?: AgentPolicy) {
return { isK8s };
}

const isK8sPackage = (pkg: PackagePolicy) => K8S_PACKAGES.has(pkg.package?.name as string);
Copy link
Contributor

Choose a reason for hiding this comment

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

this is a bit problematic since pkg.package? can return undefined and will throw when used in .has().

@kibana-ci
Copy link
Collaborator

💚 Build Succeeded

Metrics [docs]

Public APIs missing comments

Total count of every public API that lacks a comment. Target amount is 0. Run node scripts/build_api_docs --plugin [yourplugin] --stats comments for more detailed information.

id before after diff
fleet 920 921 +1

Async chunks

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

id before after diff
fleet 891.1KB 891.2KB +117.0B

Page load bundle

Size of the bundles that are downloaded on every page load. Target size is below 100kb

id before after diff
fleet 119.9KB 120.0KB +131.0B
Unknown metric groups

API count

id before after diff
fleet 1025 1026 +1

ESLint disabled in files

id before after diff
osquery 1 2 +1

ESLint disabled line counts

id before after diff
enterpriseSearch 19 21 +2
fleet 59 65 +6
osquery 109 115 +6
securitySolution 443 449 +6
total +20

Total ESLint disabled count

id before after diff
enterpriseSearch 20 22 +2
fleet 68 74 +6
osquery 110 117 +7
securitySolution 520 526 +6
total +21

History

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

Copy link
Contributor

@criamico criamico left a comment

Choose a reason for hiding this comment

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

LGTM

@kfirpeled kfirpeled merged commit f72e50f into elastic:main Dec 5, 2022
@kibanamachine kibanamachine added the backport:skip This commit does not require backporting label Dec 5, 2022
@kfirpeled kfirpeled deleted the cspm/hide_k8s_option_for_cspm branch December 5, 2022 11:52
# for free to join this conversation on GitHub. Already have an account? # 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 Team:Fleet Team label for Observability Data Collection Fleet team v8.7.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Cloud Posture] CSPM integration - remove kubernetes deployment option
6 participants