Skip to content

install/kubernetes: remove deprecated tofqdns-enable-poller option#12618

Merged
christarazi merged 1 commit intomasterfrom
pr/tklauser/helm-remove-to-fqdns-poller-option
Jul 22, 2020
Merged

install/kubernetes: remove deprecated tofqdns-enable-poller option#12618
christarazi merged 1 commit intomasterfrom
pr/tklauser/helm-remove-to-fqdns-poller-option

Conversation

@tklauser
Copy link
Copy Markdown
Member

The DNS poller was deprecated in the 1.8 release and all documentation
on how to configure it should have been removed as per [1]

[1] #10629 (review)

However, the respective helm option (which disables the DNS poller by
default) wasn't removed. Do so now.

Related: #8604
Fixes: b8971ff ("Deprecate DNS Poller in v1.8")

The DNS poller was deprecated in the 1.8 release and all documentation
on how to configure it should have been removed as per [1]

[1] #10629 (review)

However, the respective helm option (which disables the DNS poller by
default) wasn't removed. Do so now.

Related: #8604
Fixes: b8971ff ("Deprecate DNS Poller in v1.8")

Signed-off-by: Tobias Klauser <tklauser@distanz.ch>
@tklauser tklauser added release-note/misc This PR makes changes that have no direct user impact. needs-backport/1.8 labels Jul 22, 2020
@tklauser tklauser requested review from a team as code owners July 22, 2020 13:13
@tklauser tklauser requested a review from a team July 22, 2020 13:13
@coveralls
Copy link
Copy Markdown

Coverage Status

Coverage decreased (-0.01%) to 37.07% when pulling e876fd2 on pr/tklauser/helm-remove-to-fqdns-poller-option into 0685c1b on master.

@christarazi christarazi merged commit 3591629 into master Jul 22, 2020
@christarazi christarazi deleted the pr/tklauser/helm-remove-to-fqdns-poller-option branch July 22, 2020 16:51
dghubble added a commit to poseidon/terraform-render-bootstrap that referenced this pull request Jul 25, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

release-note/misc This PR makes changes that have no direct user impact.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants