Skip to content
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
6 changes: 3 additions & 3 deletions .github/workflows/lint-test-chart.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -38,8 +38,8 @@ jobs:
run: |
set -euo pipefail

helm plugin install https://github.com/losisin/helm-values-schema-json.git
helm plugin install https://github.com/helm-unittest/helm-unittest.git
helm plugin install https://github.com/losisin/helm-values-schema-json.git --verify=false
helm plugin install https://github.com/helm-unittest/helm-unittest.git --verify=false

- name: Run Helm Schema check
working-directory: charts/external-dns
Expand Down Expand Up @@ -78,7 +78,7 @@ jobs:
run: |
set -euo pipefail

helm unittest -f 'tests/*_test.yaml' --color charts/external-dns
helm unittest -f 'tests/*_test.yaml' charts/external-dns

- name: Install YQ
uses: action-stars/install-tool-from-github-release@1fa61c3bea52eca3bcdb1f5c961a3b113fe7fa54 # v0.2.6
Expand Down
1 change: 1 addition & 0 deletions charts/external-dns/CHANGELOG.md
Original file line number Diff line number Diff line change
Expand Up @@ -29,6 +29,7 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
### Fixed

- Fixed the missing schema for `.provider.webhook.serviceMonitor` configs ([#5932](https://github.com/kubernetes-sigs/external-dns/pull/5932)) _@chrisbsmith_
- Fixed incorrect indentation of selector labels under `spec.template.spec.topologySpreadConstraints` when `topologySpreadConstraints` is set. ([#6054](https://github.com/kubernetes-sigs/external-dns/pull/6054)) _@andylim0221_

## [v1.19.0] - 2025-09-08

Expand Down
2 changes: 1 addition & 1 deletion charts/external-dns/templates/deployment.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -294,7 +294,7 @@ spec:
{{- if not (hasKey . "labelSelector") }}
labelSelector:
matchLabels:
{{- include "external-dns.selectorLabels" $ | nindent 12 }}
{{- include "external-dns.selectorLabels" $ | nindent 14 }}
{{- end }}
{{- end }}
{{- end }}
Expand Down
21 changes: 21 additions & 0 deletions charts/external-dns/tests/deployment-scheduling_test.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -2,6 +2,7 @@ suite: Deployment scheduling configuration
templates:
- deployment.yaml
release:
name: external-dns
namespace: default
tests:
- it: should not provide defaults for affinities or tolerations
Expand Down Expand Up @@ -116,3 +117,23 @@ tests:
key: iam/scope
operator: Equal
value: namespace

- it: should include selector labels when setting topologySpreadConstraints
set:
topologySpreadConstraints:
- maxSkew: 1
topologyKey: topology.kubernetes.io/zone
whenUnsatisfiable: ScheduleAnyway
asserts:
- exists:
path: spec.template.spec.topologySpreadConstraints
- equal:
path: spec.template.spec.topologySpreadConstraints
value:
- maxSkew: 1
topologyKey: topology.kubernetes.io/zone
whenUnsatisfiable: ScheduleAnyway
labelSelector:
matchLabels:
app.kubernetes.io/name: external-dns
app.kubernetes.io/instance: external-dns
Loading