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

[aws-load-balancer-controller] Fix RBAC permissions on leader election role #301

Merged
merged 1 commit into from
Oct 26, 2020

Conversation

kishorj
Copy link
Contributor

@kishorj kishorj commented Oct 25, 2020

Issue #, if available:

Description of changes:
Undo PR #298 changes since we cannot restrict create request by resourceName. For create, this limitation is because the object name is not known at authorization time.

Tests

  • Installed chart without existing configmap aws-load-balancer-controller-leader in kube-system. Verified chart installed successfully and leader election worked.
  • Installed chart with existing configmap aws-load-balancer-controller-leader

For further reference https://kubernetes.io/docs/reference/access-authn-authz/rbac/

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

@kishorj kishorj changed the title Undo tightening leader election role Undo tightening of leader election role Oct 25, 2020
@kishorj kishorj changed the title Undo tightening of leader election role [aws-load-balancer-controller] Fix RBAC permissions on leader election role Oct 26, 2020
Copy link

@fawadkhaliq fawadkhaliq left a comment

Choose a reason for hiding this comment

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

/lgtm

@fawadkhaliq fawadkhaliq merged commit 0f5e8f1 into aws:master Oct 26, 2020
@kishorj kishorj deleted the lb-rbac-rollback branch May 11, 2021 16:31
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.

2 participants