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] Render Setup Technology Selector based on deployment mode #194347

Conversation

seanrathier
Copy link
Contributor

@seanrathier seanrathier commented Sep 27, 2024

Summary

This change migrates the CSPM agentless setup to use the deployment_mode setting in the CSPM integration.

It also fixes

I discovered an issue with how we are converting to use the deployment_modes, currently Fleet checks if ANY integration has the deployment_mode agentless set to determine if agentless is available. However, we have KSPM and CNVM which is not agentless.

Issue created here to determine the best solution for all agentless integrations here

Related PRs

Checklist

@seanrathier seanrathier linked an issue Sep 27, 2024 that may be closed by this pull request
2 tasks
@seanrathier seanrathier self-assigned this Oct 1, 2024
@seanrathier seanrathier added Team:Cloud Security Cloud Security team related backport:prev-minor Backport to (8.x) the previous minor version (i.e. one version back from main) 8.16 candidate labels Oct 1, 2024
@seanrathier seanrathier marked this pull request as ready for review October 1, 2024 22:01
@seanrathier seanrathier requested review from a team as code owners October 1, 2024 22:01
@elasticmachine
Copy link
Contributor

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

@seanrathier seanrathier added the release_note:skip Skip the PR/issue when compiling release notes label Oct 1, 2024
@botelastic botelastic bot added the Team:Fleet Team label for Observability Data Collection Fleet team label Oct 1, 2024
@elasticmachine
Copy link
Contributor

Pinging @elastic/fleet (Team:Fleet)

@seanrathier seanrathier requested review from a team as code owners October 2, 2024 02:16
@kibana-ci
Copy link
Collaborator

kibana-ci commented Oct 2, 2024

💔 Build Failed

Failed CI Steps

Test Failures

  • [job] [logs] FTR Configs #20 / Alerting builtin alertTypes cancellable_rule rule that implements cancellation services throws an error if search runs longer than rule timeout
  • [job] [logs] FTR Configs #36 / Cloud Security Posture Agentless cloud should create agentless-agent
  • [job] [logs] FTR Configs #36 / Cloud Security Posture Agentless cloud should create agentless-agent
  • [job] [logs] FTR Configs #100 / cloud_security_posture Agentless API Serverless should create agentless-agent
  • [job] [logs] FTR Configs #100 / cloud_security_posture Agentless API Serverless should create agentless-agent
  • [job] [logs] FTR Configs #63 / cloud_security_posture Serverless - Agentless CIS Integration Page Serverless - Agentless CIS_AWS Single Account Launch Cloud formation should show CIS_AWS Launch Cloud formation button when credentials selector is direct access keys and package version is 1.10.0-preview01
  • [job] [logs] FTR Configs #63 / cloud_security_posture Serverless - Agentless CIS Integration Page Serverless - Agentless CIS_AWS Single Account Launch Cloud formation should show CIS_AWS Launch Cloud formation button when credentials selector is direct access keys and package version is 1.10.0-preview01

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 506.9KB 507.0KB +112.0B
fleet 1.7MB 1.7MB +4.0B
total +116.0B
Unknown metric groups

ESLint disabled line counts

id before after diff
@kbn/test-suites-xpack 722 724 +2

Total ESLint disabled count

id before after diff
@kbn/test-suites-xpack 746 748 +2

History

  • 💔 Build #238618 failed 96773c29b257c3142cd579b28fdae61776aa0565
  • 💔 Build #238599 failed c883111ff0c3e91265da6aaf6e482be3ffe1c506

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

cc @seanrathier

@seanrathier seanrathier force-pushed the 191041-render-setup-technology-deployment-mode branch from 1b7ae28 to 37f5619 Compare October 7, 2024 19:14
@Omolola-Akinleye Omolola-Akinleye self-requested a review October 7, 2024 19:24
Copy link
Contributor

@Omolola-Akinleye Omolola-Akinleye 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! Thanks for catching this!

@seanrathier seanrathier added the ci:cloud-deploy Create or update a Cloud deployment label Oct 7, 2024
@seanrathier
Copy link
Contributor Author

/ci

@seanrathier seanrathier force-pushed the 191041-render-setup-technology-deployment-mode branch from 21871f3 to 0fe3408 Compare October 8, 2024 21:10
@elasticmachine
Copy link
Contributor

elasticmachine commented Oct 9, 2024

💚 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 506.3KB 506.4KB +112.0B
fleet 1.7MB 1.7MB +21.0B
total +133.0B
Unknown metric groups

ESLint disabled line counts

id before after diff
@kbn/test-suites-xpack 723 725 +2

Total ESLint disabled count

id before after diff
@kbn/test-suites-xpack 747 749 +2

History

cc @seanrathier

@seanrathier seanrathier merged commit 2609a53 into elastic:main Oct 9, 2024
22 checks passed
@kibanamachine
Copy link
Contributor

Starting backport for target branches: 8.x

https://github.com/elastic/kibana/actions/runs/11255360120

@kibanamachine
Copy link
Contributor

💚 All backports created successfully

Status Branch Result
8.x

Note: Successful backport PRs will be merged automatically after passing CI.

Questions ?

Please refer to the Backport tool documentation

@kibanamachine kibanamachine mentioned this pull request Oct 9, 2024
2 tasks
kibanamachine added a commit that referenced this pull request Oct 9, 2024
…oyment mode (#194347) (#195591)

# Backport

This will backport the following commits from `main` to `8.x`:
- [[Cloud Security] Render Setup Technology Selector based on deployment
mode (#194347)](#194347)

<!--- Backport version: 9.4.3 -->

### Questions ?
Please refer to the [Backport tool
documentation](https://github.com/sqren/backport)

<!--BACKPORT
[{"author":{"name":"seanrathier","email":"[email protected]"},"sourceCommit":{"committedDate":"2024-10-09T12:52:03Z","message":"[Cloud
Security] Render Setup Technology Selector based on deployment mode
(#194347)","sha":"2609a533fa950b9e7974d704fa0c3f9986816dbe","branchLabelMapping":{"^v9.0.0$":"main","^v8.16.0$":"8.x","^v(\\d+).(\\d+).\\d+$":"$1.$2"}},"sourcePullRequest":{"labels":["release_note:skip","Team:Fleet","v9.0.0","Team:Cloud
Security","backport:prev-minor","ci:cloud-deploy","8.16
candidate"],"title":"[Cloud Security] Render Setup Technology Selector
based on deployment
mode","number":194347,"url":"https://github.com/elastic/kibana/pull/194347","mergeCommit":{"message":"[Cloud
Security] Render Setup Technology Selector based on deployment mode
(#194347)","sha":"2609a533fa950b9e7974d704fa0c3f9986816dbe"}},"sourceBranch":"main","suggestedTargetBranches":[],"targetPullRequestStates":[{"branch":"main","label":"v9.0.0","branchLabelMappingKey":"^v9.0.0$","isSourceBranch":true,"state":"MERGED","url":"https://github.com/elastic/kibana/pull/194347","number":194347,"mergeCommit":{"message":"[Cloud
Security] Render Setup Technology Selector based on deployment mode
(#194347)","sha":"2609a533fa950b9e7974d704fa0c3f9986816dbe"}}]}]
BACKPORT-->

Co-authored-by: seanrathier <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
8.16 candidate backport:prev-minor Backport to (8.x) the previous minor version (i.e. one version back from main) ci:cloud-deploy Create or update a Cloud deployment 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.16.0 v9.0.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Render Setup Technology Selector based on deployment mode
6 participants