Skip to content

Commit fe83c0d

Browse files
chore(source): code cleanup
2 parents 9f427e5 + 0cc2c50 commit fe83c0d

File tree

128 files changed

+3426
-1841
lines changed

Some content is hidden

Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.

128 files changed

+3426
-1841
lines changed

.github/workflows/ci.yaml

Lines changed: 8 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -17,7 +17,11 @@ jobs:
1717
checks: write # to create a new check based on the results (shogo82148/actions-goveralls)
1818

1919
name: Build
20-
runs-on: ubuntu-latest
20+
runs-on: ${{ matrix.os }}
21+
strategy:
22+
matrix:
23+
# tests for target OS
24+
os: [ubuntu-latest, macos-latest]
2125
steps:
2226

2327
- name: Check out code into the Go module directory
@@ -27,6 +31,7 @@ jobs:
2731
uses: actions/setup-go@v5
2832
with:
2933
go-version-file: go.mod
34+
check-latest: true
3035
id: go
3136

3237
- name: Install CI
@@ -37,7 +42,7 @@ jobs:
3742
run: |
3843
apt update
3944
apt install -y make gcc libc-dev git
40-
if: github.actor == 'nektos/act'
45+
if: github.actor == 'nektos/act' && matrix.os == 'ubuntu-latest'
4146

4247
- name: Test
4348
run: make test
@@ -46,5 +51,5 @@ jobs:
4651
uses: shogo82148/actions-goveralls@v1
4752
with:
4853
path-to-profile: profile.cov
49-
if: github.actor != 'nektos/act'
54+
if: github.actor != 'nektos/act' && matrix.os == 'ubuntu-latest'
5055
continue-on-error: true

.github/workflows/dependency-update.yaml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -17,7 +17,7 @@ jobs:
1717
uses: actions/checkout@v4.2.2
1818
# https://github.com/renovatebot/github-action
1919
- name: self-hosted renovate
20-
uses: renovatebot/github-action@v41.0.18
20+
uses: renovatebot/github-action@v41.0.22
2121
with:
2222
# https://docs.github.com/en/actions/security-for-github-actions/security-guides/automatic-token-authentication
2323
token: ${{ secrets.GITHUB_TOKEN }}

.github/workflows/docs.yaml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -19,7 +19,7 @@ jobs:
1919
with:
2020
fetch-depth: 0
2121

22-
- uses: actions/setup-python@8d9ed9ac5c53483de85588cdf95a591a75ab9f55 # v5.5.0
22+
- uses: actions/setup-python@a26af69be951a213d495a4c3e4e4022e16d87065 # v5.6.0
2323
with:
2424
python-version: "3.12"
2525
cache: "pip"

.github/workflows/json-yaml-validate.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -17,7 +17,7 @@ jobs:
1717
- uses: actions/checkout@11bd71901bbe5b1630ceea73d27597364c9af683 # v4.2.2
1818

1919
- name: json-yaml-validate
20-
uses: GrantBirki/json-yaml-validate@v3.2.1
20+
uses: GrantBirki/json-yaml-validate@v3.3.0
2121
with:
2222
comment: "true" # enable comment mode
2323
yaml_exclude_regex: "(charts/external-dns/templates.*|mkdocs.yml)"

.github/workflows/lint-test-chart.yaml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -78,7 +78,7 @@ jobs:
7878
run: ah lint --kind helm || exit 1
7979

8080
- name: Install Python
81-
uses: actions/setup-python@8d9ed9ac5c53483de85588cdf95a591a75ab9f55 # v5.5.0
81+
uses: actions/setup-python@a26af69be951a213d495a4c3e4e4022e16d87065 # v5.6.0
8282
with:
8383
token: ${{ github.token }}
8484
python-version: "3.x"

.github/workflows/lint.yaml

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -26,7 +26,7 @@ jobs:
2626
files: '.'
2727
config_file: ".markdownlint.json"
2828

29-
- name: Set up Go 1.x
29+
- name: Set up Go
3030
uses: actions/setup-go@v5
3131
with:
3232
go-version-file: go.mod
@@ -44,11 +44,11 @@ jobs:
4444
4545
# https://github.com/golangci/golangci-lint-action?tab=readme-ov-file#verify
4646
- name: Verify linter configuration and Lint go code
47-
uses: golangci/golangci-lint-action@v7
47+
uses: golangci/golangci-lint-action@v8
4848
with:
4949
verify: true
5050
args: --timeout=30m
51-
version: v2.0
51+
version: v2.1
5252

5353
# Run Spectral
5454
- name: Lint OpenAPI spec

.golangci.yml

Lines changed: 5 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,8 @@
1+
# https://golangci-lint.run/usage/configuration/
12
version: "2"
23
linters:
34
default: none
4-
enable:
5+
enable: # golangci-lint help linters
56
- dogsled
67
- goprintffuncname
78
- govet
@@ -13,6 +14,9 @@ linters:
1314
- unconvert
1415
- unused
1516
- whitespace
17+
- predeclared # Find code that shadows one of Go's predeclared identifiers
18+
- sloglint # Ensure consistent code style when using log/slog
19+
- asciicheck # Checks that all code identifiers does not have non-ASCII symbols in the name
1620
settings:
1721
exhaustive:
1822
default-signifies-exhaustive: false

CONTRIBUTING.md

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -12,6 +12,8 @@ We have full documentation on how to get started contributing here:
1212
- [Kubernetes Contributor Guide](https://git.k8s.io/community/contributors/guide) - Main contributor documentation, or you can just jump directly to the [contributing section](https://git.k8s.io/community/contributors/guide#contributing)
1313
- [Contributor Cheat Sheet](https://git.k8s.io/community/contributors/guide/contributor-cheatsheet) - Common resources for existing developers
1414

15+
This project follows the [Conventional Commits](https://www.conventionalcommits.org/en/v1.0.0/) specification on PR title. The explicit commit history is used, among other things, to provide a readable changelog in release notes.
16+
1517
## Mentorship
1618

1719
- [Mentoring Initiatives](https://git.k8s.io/community/mentoring) - We have a diverse set of mentorship programs available that are always looking for volunteers!

Makefile

Lines changed: 7 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -32,10 +32,6 @@ else
3232
CONTROLLER_GEN=$(shell which controller-gen)
3333
endif
3434

35-
#? controller-gen-install: download controller-gen if necessary
36-
controller-gen-install:
37-
@scripts/install-tools.sh --generator
38-
3935
#? golangci-lint-install: Install golangci-lint tool
4036
golangci-lint-install:
4137
@scripts/install-tools.sh --golangci
@@ -67,10 +63,11 @@ oas-lint:
6763
.PHONY: lint
6864
lint: licensecheck go-lint oas-lint
6965

70-
#? crd: Generates CRD using controller-gen
66+
#? crd: Generates CRD using controller-gen and copy it into chart
7167
.PHONY: crd
7268
crd: controller-gen-install
73-
${CONTROLLER_GEN} crd:crdVersions=v1 paths="./endpoint/..." output:crd:stdout > docs/contributing/crd-source/crd-manifest.yaml
69+
${CONTROLLER_GEN} crd:crdVersions=v1 paths="./endpoint/..." output:crd:stdout > config/crd/standard/dnsendpoint.yaml
70+
cp -f config/crd/standard/dnsendpoint.yaml charts/external-dns/crds/dnsendpoint.yaml
7471

7572
#? test: The verify target runs tasks similar to the CI tasks, but without code coverage
7673
.PHONY: test
@@ -201,3 +198,7 @@ helm-template:
201198
helm-lint:
202199
scripts/helm-tools.sh --schema
203200
scripts/helm-tools.sh --docs
201+
202+
.PHONY: go-dependency
203+
go-dependency: ## Dependency maintanance
204+
go mod tidy

charts/external-dns/CHANGELOG.md

Lines changed: 8 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -18,6 +18,14 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
1818

1919
## [UNRELEASED]
2020

21+
### Changed
22+
23+
- Allow extraArgs to also be a map enabling overrides of individual values ([#5293](https://github.com/kubernetes-sigs/external-dns/pull/5293)) _@frittentheke
24+
25+
### Fixed
26+
27+
- Fixed wrong type definitions for webhook probes. ([#5297](https://github.com/kubernetes-sigs/external-dns/pull/5297)) _@semnell_
28+
2129
## [v1.16.1] - 2025-04-10
2230

2331
### Changed

0 commit comments

Comments
 (0)