Skip to content

feat(pihole)!: deprecate v5 API support#6123

Merged
k8s-ci-robot merged 1 commit intokubernetes-sigs:masterfrom
gofogo:remove-pihole-5-support
Jan 25, 2026
Merged

feat(pihole)!: deprecate v5 API support#6123
k8s-ci-robot merged 1 commit intokubernetes-sigs:masterfrom
gofogo:remove-pihole-5-support

Conversation

@ivankatliarchuk
Copy link
Copy Markdown
Member

@ivankatliarchuk ivankatliarchuk commented Jan 17, 2026

What does it do ?

  • Add deprecation warning when using Pi-hole v5 API (the default)
  • Guide users to migrate by setting --pihole-api-version="6"

Pihole v5 was last released with hot fix in Dec 29, 2024 aka 2+ years ago. Those who want to stay on software that is working but not patched and no longer supported, they should pin current version of external-dns. There is no point to support both versions. Release page https://github.com/pi-hole/pi-hole/releases?page=2

After the next release, we will:

  • Remove the --pihole-api-version flag
  • Remove support for v5 API

Motivation

Pi-hole v5 is end-of-life. Dropping v5 support simplifies maintenance and lets us focus on the current Pi-hole v6 API.

More

  • Yes, this PR title follows Conventional Commits
  • Yes, I added unit tests
  • Yes, I updated end user documentation accordingly

Signed-off-by: ivan katliarchuk <ivan.katliarchuk@gmail.com>
@k8s-ci-robot k8s-ci-robot added the provider Issues or PRs related to a provider label Jan 17, 2026
@k8s-ci-robot k8s-ci-robot added size/M Denotes a PR that changes 30-99 lines, ignoring generated files. cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. labels Jan 17, 2026
@coveralls
Copy link
Copy Markdown

Pull Request Test Coverage Report for Build 21092247306

Details

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • 11 unchanged lines in 1 file lost coverage.
  • Overall coverage increased (+0.001%) to 79.07%

Files with Coverage Reduction New Missed Lines %
pihole/pihole.go 11 84.21%
Totals Coverage Status
Change from base Build 21073002251: 0.001%
Covered Lines: 16067
Relevant Lines: 20320

💛 - Coveralls

Copy link
Copy Markdown
Member

@u-kai u-kai left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

/lgtm

@k8s-ci-robot
Copy link
Copy Markdown
Contributor

@u-kai: changing LGTM is restricted to collaborators

Details

In response to this:

/lgtm

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository.

@vflaux
Copy link
Copy Markdown
Contributor

vflaux commented Jan 20, 2026

/lgtm

@k8s-ci-robot k8s-ci-robot added the lgtm "Looks good to me", indicates that a PR is ready to be merged. label Jan 20, 2026
@ivankatliarchuk
Copy link
Copy Markdown
Member Author

/approve

@k8s-ci-robot
Copy link
Copy Markdown
Contributor

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: ivankatliarchuk, u-kai

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Details Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@k8s-ci-robot k8s-ci-robot added the approved Indicates a PR has been approved by an approver from all required OWNERS files. label Jan 25, 2026
@k8s-ci-robot k8s-ci-robot merged commit 7579ce2 into kubernetes-sigs:master Jan 25, 2026
20 checks passed
@ivankatliarchuk ivankatliarchuk deleted the remove-pihole-5-support branch January 25, 2026 09:15
ivankatliarchuk added a commit to gofogo/k8s-sigs-external-dns-fork that referenced this pull request Feb 1, 2026
…_total

* master:
  chore(deps): bump mkdocs-git-revision-date-localized-plugin (kubernetes-sigs#6161)
  fix(annotations): initialize annotation keys at declaration time (kubernetes-sigs#6159)
  chore(linter): unused params and functions linter (kubernetes-sigs#6142)
  docs(fqdn): use correct arguments order in FQDN Templating custom functions (kubernetes-sigs#6144)
  chore(cloudflare): improve tests (kubernetes-sigs#6150)
  refactor(kubeclient): consolidate duplicate code (kubernetes-sigs#6076)
  remove call to get latest kubectl (kubernetes-sigs#6148)
  refactor(aws): extract and restructure alias-handling logic to enable safe upcoming fixes (kubernetes-sigs#6021)
  feat(pihole): deprecate v5 API support (kubernetes-sigs#6123)
  chore(cloudflare): move custom hostnames logic to dedicated files (kubernetes-sigs#6114)
  chore(provider): zone cache provider interface (kubernetes-sigs#6120)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

approved Indicates a PR has been approved by an approver from all required OWNERS files. cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. lgtm "Looks good to me", indicates that a PR is ready to be merged. provider Issues or PRs related to a provider size/M Denotes a PR that changes 30-99 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants