Skip to content

Commit

Permalink
Merge pull request #2866 from stevehipwell/fix-chart-namespaces
Browse files Browse the repository at this point in the history
Remove namespace from cluster scoped resources
  • Loading branch information
k8s-ci-robot authored Jul 13, 2022
2 parents 579c8a8 + d35f7a2 commit 6b40e70
Show file tree
Hide file tree
Showing 4 changed files with 10 additions and 12 deletions.
9 changes: 8 additions & 1 deletion charts/external-dns/CHANGELOG.md
Original file line number Diff line number Diff line change
Expand Up @@ -10,10 +10,17 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
<!-- ## [UNRELEASED]
### Added
### Changed
### Fixed
### Deprecated
### Removed -->

## [v1.10.0] - 2022-07-04
## [v1.10.1] - 2022-07-11

### Fixed

- Fixed incorrect addition of `namespace` to `ClusterRole` & `ClusterRoleBinding`. [@stevehipwell](https://github.com/stevehipwell)

## [v1.10.0] - 2022-07-08

### Added

Expand Down
11 changes: 2 additions & 9 deletions charts/external-dns/Chart.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,7 @@ apiVersion: v2
name: external-dns
description: ExternalDNS synchronizes exposed Kubernetes Services and Ingresses with DNS providers.
type: application
version: 1.10.0
version: 1.10.1
appVersion: 0.12.0
keywords:
- kubernetes
Expand Down Expand Up @@ -30,11 +30,4 @@ annotations:
- name: Process Namespace Sharing
url: https://kubernetes.io/docs/tasks/configure-pod-container/share-process-namespace/
- kind: changed
description: "Update ExternalDNS version to v0.12.0"
- kind: changed
description: "Set resource namespaces to {{ .Release.Namespace }} in the templates instead of waiting until apply time for inference."
- kind: fixed
description: "Fixed rbac.additionalPermissions default value."
links:
- name: GitHub Issue #2796
url: https://github.com/kubernetes-sigs/external-dns/pull/2796
description: "Fixed incorrect addition of `namespace` to `ClusterRole` & `ClusterRoleBinding`."
1 change: 0 additions & 1 deletion charts/external-dns/templates/clusterrole.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -3,7 +3,6 @@ apiVersion: rbac.authorization.k8s.io/v1
kind: ClusterRole
metadata:
name: {{ template "external-dns.fullname" . }}
namespace: {{ .Release.Namespace }}
labels:
{{- include "external-dns.labels" . | nindent 4 }}
rules:
Expand Down
1 change: 0 additions & 1 deletion charts/external-dns/templates/clusterrolebinding.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -3,7 +3,6 @@ apiVersion: rbac.authorization.k8s.io/v1
kind: ClusterRoleBinding
metadata:
name: {{ printf "%s-viewer" (include "external-dns.fullname" .) }}
namespace: {{ .Release.Namespace }}
labels:
{{- include "external-dns.labels" . | nindent 4 }}
roleRef:
Expand Down

0 comments on commit 6b40e70

Please sign in to comment.