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

Update kubernetes-zfs-provisioner RBAC API version to v1 #116

Merged
merged 1 commit into from
Jan 17, 2022
Merged

Update kubernetes-zfs-provisioner RBAC API version to v1 #116

merged 1 commit into from
Jan 17, 2022

Conversation

Timmy1e
Copy link
Contributor

@Timmy1e Timmy1e commented Jan 15, 2022

What this PR does / why we need it:

In Kubernetes v1.22 rbac.authorization.k8s.io/v1beta1 was deprecated, you can read more about it here.

This causes the following error when the chart is ran on any version newer then v1.22, and the current version is v1.26:

Error: UPGRADE FAILED: unable to recognize "": no matches for kind "ClusterRoleBinding" in version "rbac.authorization.k8s.io/v1beta1"

This PR resolves this with the following changes:

  • Update rbac.authorization.k8s.io/v1beta1 to rbac.authorization.k8s.io/v1 in kubernetes-zfs-provisioner.
  • Bump kubernetes-zfs-provisioner chart version from 1.1.2 to 1.1.3.

Checklist

  • Chart Version bumped
  • make docs lint passes
  • Variables are documented in the values.yaml as required in Helm-Docs.
  • PR contains the label that identifies the chart, e.g. chart/[chart]
  • PR contains the label that identifies the type of change, which is one of
    bug, enhancement, documentation, change, breaking, dependency

Update `rbac.authorization.k8s.io/v1beta1` to `rbac.authorization.k8s.io/v1` in `kubernetes-zfs-provisioner`.
Bump `kubernetes-zfs-provisioner` chart version from `1.1.2` to `1.1.3`.
@ccremer ccremer merged commit f5745d5 into ccremer:master Jan 17, 2022
@ccremer
Copy link
Owner

ccremer commented Jan 17, 2022

Thanks for the contribution!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants