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.

Sign up for GitHub

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

Jump to bottom

Remove security-bundle apps #56

Merged
merged 1 commit into from
Mar 6, 2024
Merged

Remove security-bundle apps #56

merged 1 commit into from
Mar 6, 2024

Conversation

fhielpos
Copy link
Member

@fhielpos fhielpos commented Mar 6, 2024

Remove security-bundle apps since now they are included in the default-apps. This doesn't include Trivy and Falco since those are disabled by default

@fhielpos fhielpos self-assigned this Mar 6, 2024
@fhielpos fhielpos requested a review from a team March 6, 2024 11:45
stone-z
stone-z approved these changes Mar 6, 2024
View reviewed changes
Copy link
Contributor

@stone-z stone-z left a comment

Choose a reason for hiding this comment

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

I'd even go a step further and enable the others through the config for the bundle, since that's how customers would install now

@fhielpos
Copy link
Member Author

fhielpos commented Mar 6, 2024

Yeah, we could do that too. For now I will merge this one so we unbreak CAPA.

@fhielpos fhielpos merged commit 15f1648 into master Mar 6, 2024
1 check passed
@fhielpos fhielpos deleted the remove-kyverno branch March 6, 2024 11:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@stone-z stone-z stone-z approved these changes

Assignees

@fhielpos fhielpos

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@fhielpos @stone-z