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

feat(helm): Automatically create role & bindings for namespace containing platform secret #1935

Conversation

ThomasK33
Copy link
Contributor

What issue type does this pull request address? (keep at least one, remove the others)
/kind enhancement

What does this pull request do? Which issues does it resolve? (use resolves #<issue_number> if possible)
resolves ENG-4148

Please provide a short message that should be published in the vcluster release notes
Fixed an issue where vcluster ...

What else do we need to know?

Copy link

netlify bot commented Jul 16, 2024

Deploy Preview for vcluster-docs canceled.

Name Link
🔨 Latest commit cc0c731
🔍 Latest deploy log https://app.netlify.com/sites/vcluster-docs/deploys/6696770ccfb12c00085139b1

@ThomasK33 ThomasK33 force-pushed the thomaskosiewski/eng-4148-vcluster-helm-chart-auto-create-role-and-rolebinding-if branch from 9957906 to 6e0f7b8 Compare July 16, 2024 10:28
Copy link
Contributor

@zerbitx zerbitx left a comment

Choose a reason for hiding this comment

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

LGTM

chart/templates/platform-rbac.yaml Outdated Show resolved Hide resolved
@ThomasK33 ThomasK33 force-pushed the thomaskosiewski/eng-4148-vcluster-helm-chart-auto-create-role-and-rolebinding-if branch from 6e0f7b8 to f6de710 Compare July 16, 2024 12:11
@ThomasK33 ThomasK33 requested a review from FabianKramm July 16, 2024 12:13
chart/templates/platform-rbac.yaml Outdated Show resolved Hide resolved
@ThomasK33 ThomasK33 force-pushed the thomaskosiewski/eng-4148-vcluster-helm-chart-auto-create-role-and-rolebinding-if branch from f6de710 to 32bc977 Compare July 16, 2024 12:38
@ThomasK33 ThomasK33 requested a review from FabianKramm July 16, 2024 12:39
chart/templates/platform-rbac.yaml Outdated Show resolved Hide resolved
chart/templates/platform-rbac.yaml Outdated Show resolved Hide resolved
@ThomasK33 ThomasK33 force-pushed the thomaskosiewski/eng-4148-vcluster-helm-chart-auto-create-role-and-rolebinding-if branch 2 times, most recently from 24c1152 to f112a8f Compare July 16, 2024 13:32
…ning platform secret

Signed-off-by: Thomas Kosiewski <thomas.kosiewski@loft.sh>
@ThomasK33 ThomasK33 force-pushed the thomaskosiewski/eng-4148-vcluster-helm-chart-auto-create-role-and-rolebinding-if branch from f112a8f to cc0c731 Compare July 16, 2024 13:35
@FabianKramm FabianKramm merged commit d5475d4 into main Jul 16, 2024
73 checks passed
@ThomasK33 ThomasK33 deleted the thomaskosiewski/eng-4148-vcluster-helm-chart-auto-create-role-and-rolebinding-if branch July 16, 2024 13:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants