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

fix: add an "or" condition for configmap-ui #857

Conversation

Aliloya-git
Copy link
Contributor

What type of PR is this?
/kind bug
/kind chart-release
/area falcosidekick-chart

What this PR does / why we need it:
Fixes #856

Which issue(s) this PR fixes:
Fixes #856

Special notes for your reviewer:
I am not an expert so please verify the issue, review my PR carefully, and let me know if you have any feedback/comment for me.
Thank you for your review in advance :)

Checklist

  • Chart Version bumped
  • Variables are documented in the README.md
  • CHANGELOG.md updated

@poiana poiana added dco-signoff: yes kind/bug Something isn't working kind/chart-release Add this label when the chart version has been bumped area/falcosidekick-chart labels Mar 25, 2025
@poiana
Copy link
Contributor

poiana commented Mar 25, 2025

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

@poiana poiana requested review from bencer and leogr March 25, 2025 22:25
@poiana poiana added the size/XS label Mar 25, 2025
Copy link
Contributor

@IgorEulalio IgorEulalio left a comment

Choose a reason for hiding this comment

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

@Aliloya-eng I see it's your first contribution, thanks for your PR!

Just a minor change to highlight it will be only there if one of them are enabled.

Co-authored-by: Igor Eulalio <[email protected]>
Signed-off-by: Aliloya <[email protected]>
@Aliloya-git Aliloya-git changed the title fix: remove faulty condition for configmap fix: add an "or" condition for configmap-ui Mar 26, 2025
Signed-off-by: Aliloya <[email protected]>
@Aliloya-git Aliloya-git requested a review from IgorEulalio March 26, 2025 16:11
@poiana
Copy link
Contributor

poiana commented Mar 27, 2025

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: Aliloya-eng, leogr

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 Mar 27, 2025

LGTM label has been added.

Git tree hash: aed465ccef56de7b79337042d55424bf7f6e68e4

@poiana poiana merged commit e648b42 into falcosecurity:master Mar 27, 2025
6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
approved area/falcosidekick-chart dco-signoff: yes kind/bug Something isn't working kind/chart-release Add this label when the chart version has been bumped lgtm size/XS
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Falcosidekick - missing configmap when externalredis is enabled
4 participants