Skip to content

Commit

Permalink
πŸ› Fix namespace override which was missing on ServiceAccount (#731)
Browse files Browse the repository at this point in the history
  • Loading branch information
mloiseleur authored Nov 21, 2022
1 parent c394f05 commit f4a7bec
Show file tree
Hide file tree
Showing 4 changed files with 21 additions and 3 deletions.
11 changes: 9 additions & 2 deletions traefik/Changelog.md
Original file line number Diff line number Diff line change
@@ -1,10 +1,17 @@
# Change Log

## 20.3.1 ![AppVersion: v2.9.4](https://img.shields.io/static/v1?label=AppVersion&message=v2.9.4&color=success&logo=) ![Helm: v3](https://img.shields.io/static/v1?label=Helm&message=v3&color=informational&logo=helm)

**Release date:** 2022-11-21

* πŸ› Fix namespace override which was missing on `ServiceAccount`


## 20.3.0 ![AppVersion: v2.9.4](https://img.shields.io/static/v1?label=AppVersion&message=v2.9.4&color=success&logo=) ![Helm: v3](https://img.shields.io/static/v1?label=Helm&message=v3&color=informational&logo=helm)

**Release date:** 2022-11-16
**Release date:** 2022-11-17

* Add option to override instance label value
* Add overwrite option for instance label value (#725)

### Default value changes

Expand Down
4 changes: 3 additions & 1 deletion traefik/Chart.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,7 @@ apiVersion: v2
name: traefik
description: A Traefik based Kubernetes ingress controller
type: application
version: 20.3.0
version: 20.3.1
appVersion: v2.9.4
keywords:
- traefik
Expand All @@ -25,3 +25,5 @@ maintainers:
icon: https://raw.githubusercontent.com/traefik/traefik/v2.3/docs/content/assets/img/traefik.logo.png
annotations:
artifacthub.io/changes: "- Adds support for namespace overrides in subchart use\n- Document recent changes in the README (#717)\n"
artifacthub.io/changes: |
- πŸ› Fix namespace override which was missing on `ServiceAccount`
1 change: 1 addition & 0 deletions traefik/templates/rbac/serviceaccount.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -3,6 +3,7 @@ kind: ServiceAccount
apiVersion: v1
metadata:
name: {{ include "traefik.serviceAccountName" . }}
namespace: {{ template "traefik.namespace" . }}
labels:
{{- include "traefik.labels" . | nindent 4 }}
annotations:
Expand Down
8 changes: 8 additions & 0 deletions traefik/tests/rbac-config_test.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -173,6 +173,10 @@ tests:
path: metadata.namespace
value: NAMESPACE
template: rbac/role.yaml
- equal:
path: metadata.namespace
value: NAMESPACE
template: rbac/serviceaccount.yaml
- it: should accept overridden namespace
set:
namespaceOverride: "traefik-ns-override"
Expand All @@ -183,3 +187,7 @@ tests:
path: metadata.namespace
value: "traefik-ns-override"
template: rbac/role.yaml
- equal:
path: metadata.namespace
value: "traefik-ns-override"
template: rbac/serviceaccount.yaml

0 comments on commit f4a7bec

Please sign in to comment.