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

Backport of PRs #328 and #332, plus go module refresh #340

Conversation

andreas-kupries
Copy link
Contributor

@andreas-kupries andreas-kupries commented Mar 11, 2024

Issue:

Fix rancher/rancher#44356 ; backport the solution for rancher/rancher#40584

Original solution and a fix are PRs

Problem

See rancher/rancher#40584

Solution

See #328 and #332.

⚠️ Checking the NamespacedRules was removed from the backport. This field is not present in the definition of GlobalRole used by the 0.4.x release series of webhook.

CheckList

  • Test
  • Docs

Proper validation of GlobalRole `rules`,
plus fix (TomL) to make validation not namespaced.
@tomleb
Copy link
Contributor

tomleb commented Mar 19, 2024

I think we should wait before merging this since we're in code freeze for 2.8

@andreas-kupries andreas-kupries merged commit 6d40ab3 into rancher:release/v0.4 Apr 3, 2024
1 check passed
@andreas-kupries andreas-kupries deleted the ak-gh40584-rules-validation-backport branch April 3, 2024 14:06
@samjustus
Copy link

rancher/rancher#44356

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.

5 participants