* chore(provider): zone cache provider interface
Signed-off-by: ivan katliarchuk <ivan.katliarchuk@gmail.com>
* chore(provider): zone cache provider interface
Signed-off-by: ivan katliarchuk <ivan.katliarchuk@gmail.com>
* chore(provider): zone cache provider interface
Signed-off-by: ivan katliarchuk <ivan.katliarchuk@gmail.com>
* chore(provider): zone cache provider interface
Signed-off-by: ivan katliarchuk <ivan.katliarchuk@gmail.com>
* chore(provider): zone cache provider interface
Signed-off-by: ivan katliarchuk <ivan.katliarchuk@gmail.com>
* chore(provider): zone cache provider interface
Signed-off-by: ivan katliarchuk <ivan.katliarchuk@gmail.com>
* chore(provider): zone cache provider interface
Signed-off-by: ivan katliarchuk <ivan.katliarchuk@gmail.com>
* chore(provider): zone cache provider interface
Signed-off-by: ivan katliarchuk <ivan.katliarchuk@gmail.com>
---------
Signed-off-by: ivan katliarchuk <ivan.katliarchuk@gmail.com>
* test(provider/exoscale): improve test coverage from 62.8 to 80.4%
* Revert "test(provider/exoscale): improve test coverage from 62.8 to 80.4%"
This reverts commit 56aa8b1dc87bc45f49fb247213d26f8883969865.
* test(provider/exoscale): improve test coverage from 62.8 to 80.4%
Since ef62107, it is now possible to enable support for NAPTR records in
the AWS provider. This patch does so and adds some tests for it.
Co-authored-by: woltere <wolter.eldering@quandago.com>
* fix(pdns): ensure trailing dot for NS records
* Fix formatting test typos, make trailing types to list, refer to types with const
* Simplify trailing dot condition
* Move trailing types comment to correct location
* feat(coredns): use managed-by to separate records
Signed-off-by: Jan Jansen <jan.jansen@gdata.de>
* feat(coredns): use txt-owner-id to strictly separated external-dns instances
Signed-off-by: Jan Jansen <jan.jansen@gdata.de>
* fix tests
Signed-off-by: Jan Jansen <jan.jansen@gdata.de>
* fix reviewer comments
Signed-off-by: Jan Jansen <jan.jansen@gdata.de>
* answer review comments
* fix deletion behavior and remove extra function
* fix markdown
* fix tests again
---------
Signed-off-by: Jan Jansen <jan.jansen@gdata.de>
* feat(annotations): add custom annotation prefix support for split horizon DNS
Add --annotation-prefix flag to allow customizing the annotation prefix
used by external-dns. This enables split horizon DNS scenarios where
multiple instances process different sets of annotations from the same
Kubernetes resources.
Changes:
- Add AnnotationPrefix field to Config with validation
- Convert annotation constants to variables that can be reconfigured
- Add SetAnnotationPrefix() function to rebuild annotation keys
- Integrate annotation prefix setting in controller startup
- Update Helm chart with annotationPrefix value
- Add comprehensive split horizon DNS documentation
- Update FAQ with annotation prefix examples
This maintains full backward compatibility - the default prefix remains
"external-dns.alpha.kubernetes.io/".
Co-Authored-By: Claude <noreply@anthropic.com>
* docs(advanced): fix markdown formatting in split-horizon guide
Add blank lines before code blocks to improve markdown rendering
and comply with markdownlint rules.
Co-Authored-By: Claude <noreply@anthropic.com>
* docs(advanced): fix markdown formatting in split-horizon guide
Co-Authored-By: Claude <noreply@anthropic.com>
* docs(charts): regenerate Helm chart documentation
Co-Authored-By: Claude <noreply@anthropic.com>
* test: add AnnotationPrefix field to test configs
Add missing AnnotationPrefix field to minimalConfig and overriddenConfig
test configurations to match the new default value set in NewConfig().
Co-Authored-By: Claude <noreply@anthropic.com>
* test(charts): update error pattern in json-schema test
Update expected error message pattern to match current Helm validation
output format.
Co-Authored-By: Claude <noreply@anthropic.com>
* refactor(annotations): remove init() for explicit initialization
- Remove init() function from annotations package
- Add explicit SetAnnotationPrefix() call in controller/execute.go
- Remove annotation key aliases from source/source.go
- Replace all alias usages with annotations.* references (348 changes in 28 files)
- Add TestMain to existing test files (service_test.go, cloudflare_test.go)
This change makes annotation initialization explicit and predictable,
avoiding hidden global state initialization at import time.
Co-Authored-By: Claude <noreply@anthropic.com>
* docs: update changelog and mkdocs to include annotationPrefix and split horizon DNS
Signed-off-by: Aleksei Sviridkin <f@lex.la>
* docs(split-horizon): fix linting
Signed-off-by: Aleksei Sviridkin <f@lex.la>
* refactor(annotations): replace hardcoded annotation prefix with constant
Replace all hardcoded "external-dns.alpha.kubernetes.io/" strings
with annotations.DefaultAnnotationPrefix constant to establish
a single source of truth.
Changes:
- Add DefaultAnnotationPrefix constant in source/annotations/annotations.go
- Replace hardcoded string in controller/execute.go with constant reference
- Replace hardcoded strings in pkg/apis/externaldns/types.go (2 occurrences)
- Add helm unit tests for annotationPrefix value
This eliminates string duplication and makes future changes easier.
Co-Authored-By: Claude <noreply@anthropic.com>
---------
Signed-off-by: Aleksei Sviridkin <f@lex.la>
Co-authored-by: Claude <noreply@anthropic.com>
Right now we get a fatal due to dual DNS records.
Instead do a softError and try again next reconcile.
Signed-off-by: Edvin Norling <edvin.norling@kognic.com>
* Refactor pihole tests to reduce cyclomatic complexity and preserve coverage
* refactor: split TestProvider into focused test functions to reduce cyclomatic complexity
- Split monolithic TestProvider into four focused test functions:
- TestProvider_InitialState: tests empty provider state
- TestProvider_CreateRecords: tests record creation
- TestProvider_DeleteRecords: tests record deletion
- TestProvider_UpdateRecords: tests record updates
- Each function tests a single scenario, reducing complexity
- Preserves all original test logic and maintains 89% coverage
- Passes cyclop linter with complexity threshold of 10
* coredns: fix debug message
* consistent spacing in debug message
Co-authored-by: Ivan Ka <5395690+ivankatliarchuk@users.noreply.github.com>
---------
Co-authored-by: Pascal Bachor <bachorp@users.noreply.github.com>
Co-authored-by: Ivan Ka <5395690+ivankatliarchuk@users.noreply.github.com>
* fix(idna): fix handling of domains
Signed-off-by: ivan katliarchuk <ivan.katliarchuk@gmail.com>
* fix(idna): fix handling of domains
Signed-off-by: ivan katliarchuk <ivan.katliarchuk@gmail.com>
* fix(idna): fix handling of domains
Signed-off-by: ivan katliarchuk <ivan.katliarchuk@gmail.com>
---------
Signed-off-by: ivan katliarchuk <ivan.katliarchuk@gmail.com>
