Skip to content

🐛 fix/helm: refactor values, fix validation webhook, rbac #430

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

Merged
merged 1 commit into from
Feb 5, 2025

Conversation

jfbus
Copy link
Contributor

@jfbus jfbus commented Feb 4, 2025

To follow Helm best practices, flatten values tree & use camelCase.

A validation webhook did not point to the right service and some rbac rules were missing.

@jfbus jfbus marked this pull request as ready for review February 4, 2025 16:32
@jfbus jfbus force-pushed the fix/helm_validation branch from 55e0f73 to 66ed30f Compare February 4, 2025 16:43
@jfbus jfbus changed the title 🐛 fix/helm: fix validation webhook + fix typo in label 🐛 fix/helm: fix validation webhook/typo in label/rbac Feb 4, 2025
@jfbus jfbus force-pushed the fix/helm_validation branch from 66ed30f to f010ebd Compare February 5, 2025 10:13
@jfbus jfbus force-pushed the fix/helm_validation branch from f010ebd to a625609 Compare February 5, 2025 10:34
@jfbus jfbus changed the title 🐛 fix/helm: fix validation webhook/typo in label/rbac 🐛 fix/helm: refactor values, fix validation webhook, rbac Feb 5, 2025
@jfbus jfbus merged commit 068c646 into main Feb 5, 2025
2 checks passed
@jfbus jfbus deleted the fix/helm_validation branch February 5, 2025 12:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants