Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Deploys dns-controller with --watch-ingress=true #2468

Merged
merged 1 commit into from
May 1, 2017
Merged

Deploys dns-controller with --watch-ingress=true #2468

merged 1 commit into from
May 1, 2017

Conversation

geojaz
Copy link
Member

@geojaz geojaz commented May 1, 2017

Flips the default watch-ingress on dns-controller to true.

Closes #1413, #2360 and #551


This change is Reviewable

@k8s-ci-robot k8s-ci-robot added the cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. label May 1, 2017
Copy link
Contributor

@chrislovecnm chrislovecnm left a comment

Choose a reason for hiding this comment

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

Awesome

Copy link
Contributor

@chrislovecnm chrislovecnm left a comment

Choose a reason for hiding this comment

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

Awesome

@chrislovecnm chrislovecnm self-assigned this May 1, 2017
@chrislovecnm
Copy link
Contributor

/lgtm

@chrislovecnm chrislovecnm merged commit ac5a488 into kubernetes:master May 1, 2017
@geojaz geojaz deleted the dns-controller-enable-watch-ingress branch May 1, 2017 05:24
@kenden
Copy link
Contributor

kenden commented Jun 1, 2017

This got reverted by #2560

k8s-github-robot pushed a commit that referenced this pull request Aug 30, 2017
Automatic merge from submit-queue

Adds DNSControllerSpec and WatchIngress flag

This PR is in reference to #2496, #2468 and the issues referenced in there relating to use of the watch-ingress flag. 

This PR attempts to rectify this situation and gives users who want it, the option to turn on watch-ingress without forcing it on them. Also spits out a warning to the logs about potential side effects.

Includes notes in `docs/cluster_spec.md` to explain.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cncf-cla: yes Indicates the PR's author has signed the CNCF CLA.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants