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] add securty capability to Cloud Defend package so it only shows up in Security projects in serverless #8208

Merged
merged 2 commits into from Oct 16, 2023

Conversation

maxcold
Copy link
Contributor

@maxcold maxcold commented Oct 16, 2023

Proposed commit message

Add security capability so that the cloud_defend package shows up in Security projects on serverless after Oct 17th so that cloud_defend is only available in Security projects (you cannot use it from Observability projects). More context in https://github.com/elastic/security-team/issues/7624

Checklist

  • I have reviewed tips for building integrations and this pull request is aligned with them.
  • I have verified that all data streams collect metrics or logs.
  • I have added an entry to my package's changelog.yml file.
  • I have verified that Kibana version constraints are current according to guidelines.

Author's Checklist

  • [ ]

How to test this PR locally

Related issues

Screenshots

@maxcold maxcold added enhancement New feature or request Team:Cloud Security Label for the Cloud Security team Integration:Defend for Containers Elastic Defend for Containers (D4C) (cloud-defend) labels Oct 16, 2023
@maxcold
Copy link
Contributor Author

maxcold commented Oct 16, 2023

I ran an elastic-package check to validate the package but haven't tested it manually as I haven't dealt with the cloud_defend plugin before and don't think the change requires extensive testing anyway as it only adds the capability to the manifest. But if anyone has the bandwidth to test the package, I'd appreciate it.

@maxcold maxcold marked this pull request as ready for review October 16, 2023 10:12
@maxcold maxcold requested a review from a team as a code owner October 16, 2023 10:12
@elasticmachine
Copy link

elasticmachine commented Oct 16, 2023

💚 Build Succeeded

the below badges are clickable and redirect to their specific view in the CI or DOCS
Pipeline View Test View Changes Artifacts preview preview

Expand to view the summary

Build stats

  • Start Time: 2023-10-16T10:04:01.374+0000

  • Duration: 14 min 21 sec

Test stats 🧪

Test Results
Failed 0
Passed 8
Skipped 0
Total 8

🤖 GitHub comments

Expand to view the GitHub comments

To re-run your PR in the CI, just comment with:

  • /test : Re-trigger the build.

@elasticmachine
Copy link

🌐 Coverage report

Name Metrics % (covered/total) Diff
Packages 100.0% (0/0) 💚
Files 100.0% (0/0) 💚 13.043
Classes 100.0% (0/0) 💚 13.043
Methods 27.778% (5/18) 👎 -56.722
Lines 100.0% (0/0) 💚 13.542
Conditionals 100.0% (0/0) 💚

@eyalkraft
Copy link
Contributor

LGTM, let's wait for @elastic/sec-linux-platform to test it.

@norrietaylor
Copy link
Member

LGTM

@maxcold maxcold merged commit 19d25db into elastic:main Oct 16, 2023
4 checks passed
@maxcold maxcold deleted the add-security-capability-to-cloud-defend branch October 16, 2023 17:23
@elasticmachine
Copy link

Package cloud_defend - 1.2.2 containing this change is available at https://epr.elastic.co/search?package=cloud_defend

@andrewkroh
Copy link
Member

andrewkroh commented Oct 23, 2023

so that the cloud_defend package shows up in Security projects on serverless

Did something change to require the security capability before cloud_defend would be advertised within "Serverless" to users? AFAIK cloud_defend doesn't require any special capabilities beyond your normal Fleet. Why was this change necessary? Is there something capability that only exists in security projects that it requires?

If this capability is required for a package to show up in security projects, then there are about 150 more packages that need changed.

@eyalkraft
Copy link
Contributor

Hi @andrewkroh! While an agent with cloud_defend installed might work on o11y projects, from a UI perspective (I'm pretty sure) all the relevant screens like session view or security k8s dashboard won't be available so the value is questionable.

Maybe we're wrong here to limit this integration on serverless only to security type projects - I guess that's mainly a product question.

@snehsach19 @nick-alayil do you think we should revert this change?

If this capability is required for a package to show up in security projects, then there are about 150 more packages that need changed.

I believe they will show up if they've been upgraded to spec v3. (doc) the capabilities part will just limit them to security serverless project only.

@andrewkroh
Copy link
Member

andrewkroh commented Oct 23, 2023

the capabilities part will just limit them to security serverless project only.

Yes, that was my understanding. The PR description used here is misleading, this change actually excludes cloud_defend from project types other than Security.

And I agree, this is a product management question on whether we want the cloud_defend integration to be exclusively available in Security projects.

@andrewkroh andrewkroh changed the title [Cloud Security] add securty capability to Cloud Defend package so it shows up in Security projects in serverless [Cloud Security] add securty capability to Cloud Defend package so it only shows up in Security projects in serverless Oct 23, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request Integration:Defend for Containers Elastic Defend for Containers (D4C) (cloud-defend) Team:Cloud Security Label for the Cloud Security team
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

6 participants