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

Refactoring Kubernetes recommanded labels for Falco Sidekick #588

Merged
merged 7 commits into from
Dec 18, 2023

Conversation

mmadoo
Copy link
Contributor

@mmadoo mmadoo commented Dec 18, 2023

Created from #563 I just update CHANGELOG and bump chart version

What type of PR is this?
/kind cleanup

/kind chart-release

/area falcosidekick-chart

What this PR does / why we need it:

Checklist

  • Chart Version bumped
  • CHANGELOG.md updated

@poiana poiana added kind/cleanup dco-signoff: yes kind/chart-release Add this label when the chart version has been bumped labels Dec 18, 2023
@poiana poiana requested review from alacuku and bencer December 18, 2023 13:35
@poiana
Copy link
Contributor

poiana commented Dec 18, 2023

Welcome @mmadoo! It looks like this is your first PR to falcosecurity/charts 🎉

nlamirault and others added 7 commits December 18, 2023 14:46
Signed-off-by: Nicolas Lamirault <[email protected]>
Signed-off-by: Nicolas Lamirault <[email protected]>
Signed-off-by: Nicolas Lamirault <[email protected]>
Signed-off-by: Nicolas Lamirault <[email protected]>
- Use the recommanded label app.kubernetes.io/component for Core and UI
resources.
- Fix: UI Rbac and secrets resources only generate when webui.enabled is
  true

Signed-off-by: Nicolas Lamirault <[email protected]>
Signed-off-by: Nicolas Trangosi <[email protected]>
@mmadoo mmadoo force-pushed the fix/falcosidekick-labels branch from fcc5e34 to c680694 Compare December 18, 2023 13:47
@Issif Issif modified the milestone: falco-chart-2.0.0 Dec 18, 2023
@poiana
Copy link
Contributor

poiana commented Dec 18, 2023

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: Issif, mmadoo

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@poiana
Copy link
Contributor

poiana commented Dec 18, 2023

LGTM label has been added.

Git tree hash: 5f39900c9b784623c825525eac27741204ef2e96

@poiana poiana merged commit 46b578d into falcosecurity:master Dec 18, 2023
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants