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

deployment: Update rbac versions from v1beta1 to v1 #3015

Merged
merged 1 commit into from
Oct 9, 2020

Conversation

narahari92
Copy link
Contributor

Update the apiVersions of resources in rbac group such as ClusterRoleBinding, Role etc. to
v1 version from v1beta1 as it's deprecated since kubernetes 1.17 version and will be removed
in 1.22 version.

Fixes #2991

Signed-off-by: hnarahari hnarahari@vmware.com

Update the apiVersions of resources in rbac group such as  ClusterRoleBinding, Role etc. to
v1 version from  v1beta1 as it's deprecated since kubernetes 1.17 version and will be removed
in 1.22 version.

Fixes projectcontour#2991

Signed-off-by: hnarahari <hnarahari@vmware.com>
@narahari92
Copy link
Contributor Author

I've tested with k8s versions 1.16.1 and 1.19.1 in kind cluster and rbac resources get created with v1 version in both.

@codecov
Copy link

codecov bot commented Oct 9, 2020

Codecov Report

Merging #3015 into main will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##             main    #3015   +/-   ##
=======================================
  Coverage   74.98%   74.98%           
=======================================
  Files          92       92           
  Lines        5920     5920           
=======================================
  Hits         4439     4439           
  Misses       1383     1383           
  Partials       98       98           

Copy link
Member

@skriss skriss left a comment

Choose a reason for hiding this comment

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

This LGTM, thanks @narahari92!

FYI, the integration test failure is a known test flake.

@skriss skriss requested a review from stevesloka October 9, 2020 15:01
Copy link
Member

@stevesloka stevesloka left a comment

Choose a reason for hiding this comment

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

/lgtm

@stevesloka stevesloka merged commit 1e99a20 into projectcontour:main Oct 9, 2020
@jpeach jpeach added this to the 1.10.0 milestone Oct 9, 2020
@narahari92
Copy link
Contributor Author

Thank you @skriss. I was confused why a test was failing for this change :)

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.

Move to v1 ClusterRoleBinding for deployment RBAC
4 participants