Skip to content

Commit

Permalink
tls-init and tls-init-cleanup jobs use GA RBAC API instead of beta (h…
Browse files Browse the repository at this point in the history
  • Loading branch information
ishustava authored Jan 27, 2021
1 parent 153f65e commit 1e2330e
Show file tree
Hide file tree
Showing 5 changed files with 6 additions and 4 deletions.
2 changes: 2 additions & 0 deletions CHANGELOG.md
Original file line number Diff line number Diff line change
Expand Up @@ -18,6 +18,8 @@ BREAKING CHANGES:
for more information on how to upgrade.

BUG FIXES:
* Use `rbac.authorization.k8s.io/v1` instead of `rbac.authorization.k8s.io/v1beta1` API version for the `roles` and `rolebindings` used by the `tls-init`
and `tls-init-cleanup` jobs. [[GH-789](https://github.com/hashicorp/consul-helm/issues/789)]
* Fix API version of Ingress resource for Consul UI. [[GH-786](https://github.com/hashicorp/consul-helm/pull/786)]

## 0.29.0 (Jan 22, 2021)
Expand Down
2 changes: 1 addition & 1 deletion templates/tls-init-cleanup-role.yaml
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
{{- if (or (and (ne (.Values.server.enabled | toString) "-") .Values.server.enabled) (and (eq (.Values.server.enabled | toString) "-") .Values.global.enabled)) }}
{{- if .Values.global.tls.enabled }}
apiVersion: rbac.authorization.k8s.io/v1beta1
apiVersion: rbac.authorization.k8s.io/v1
kind: Role
metadata:
name: {{ template "consul.fullname" . }}-tls-init-cleanup
Expand Down
2 changes: 1 addition & 1 deletion templates/tls-init-cleanup-rolebinding.yaml
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
{{- if (or (and (ne (.Values.server.enabled | toString) "-") .Values.server.enabled) (and (eq (.Values.server.enabled | toString) "-") .Values.global.enabled)) }}
{{- if .Values.global.tls.enabled }}
apiVersion: rbac.authorization.k8s.io/v1beta1
apiVersion: rbac.authorization.k8s.io/v1
kind: RoleBinding
metadata:
name: {{ template "consul.fullname" . }}-tls-init-cleanup
Expand Down
2 changes: 1 addition & 1 deletion templates/tls-init-role.yaml
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
{{- if (or (and (ne (.Values.server.enabled | toString) "-") .Values.server.enabled) (and (eq (.Values.server.enabled | toString) "-") .Values.global.enabled)) }}
{{- if .Values.global.tls.enabled }}
apiVersion: rbac.authorization.k8s.io/v1beta1
apiVersion: rbac.authorization.k8s.io/v1
kind: Role
metadata:
name: {{ template "consul.fullname" . }}-tls-init
Expand Down
2 changes: 1 addition & 1 deletion templates/tls-init-rolebinding.yaml
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
{{- if (or (and (ne (.Values.server.enabled | toString) "-") .Values.server.enabled) (and (eq (.Values.server.enabled | toString) "-") .Values.global.enabled)) }}
{{- if .Values.global.tls.enabled }}
apiVersion: rbac.authorization.k8s.io/v1beta1
apiVersion: rbac.authorization.k8s.io/v1
kind: RoleBinding
metadata:
name: {{ template "consul.fullname" . }}-tls-init
Expand Down

0 comments on commit 1e2330e

Please sign in to comment.