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.

Sign up for GitHub

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

Jump to bottom

Update kube-dns version to 1.14.2 #45684

Merged
merged 1 commit into from
May 12, 2017
Merged

Update kube-dns version to 1.14.2 #45684

merged 1 commit into from
May 12, 2017

Conversation

bowei
Copy link
Member

@bowei bowei commented May 11, 2017 •
edited by enisoc
Loading 

Update kube-dns to 1.14.2.
  • Support kube-master-url flag without kubeconfig
  • Fix concurrent R/Ws in dns.go
  • Fix confusing logging when initialize server
  • Fix printf in cmd/kube-dns/app/server.go
  • Fix version on startup and --version flag
  • Support specifying port number for nameserver in stubDomains

@bowei
Update kube-dns version to 1.14.2
1c223c8 
Changes:

- Support kube-master-url flag without kubeconfig
- Fix concurrent R/Ws in dns.go
- Fix confusing logging when initialize server
- Fix printf in cmd/kube-dns/app/server.go
- Fix version on startup and --version flag
- Support specifying port number for nameserver in stubDomains
@k8s-ci-robot k8s-ci-robot added the cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. label May 11, 2017
@bowei
Copy link
Member Author

bowei commented May 11, 2017

/assign @MrHohn

@k8s-github-robot k8s-github-robot added size/S Denotes a PR that changes 10-29 lines, ignoring generated files. release-note Denotes a PR that will be considered when it comes time to generate release notes. labels May 11, 2017
@bowei bowei mentioned this pull request May 11, 2017
Closed
@MrHohn
Copy link
Member

MrHohn commented May 11, 2017

/lgtm

@k8s-ci-robot k8s-ci-robot added the lgtm "Looks good to me", indicates that a PR is ready to be merged. label May 11, 2017
@MrHohn
Copy link
Member

MrHohn commented May 11, 2017

@k8s-bot kops aws e2e test this

@MrHohn
Copy link
Member

MrHohn commented May 11, 2017

@k8s-bot bazel test this

@bowei
Copy link
Member Author

bowei commented May 11, 2017

/assign @mikedanese for approval

@bowei
Copy link
Member Author

bowei commented May 11, 2017

/assign @mikedanese

@mikedanese
Copy link
Member

/approve

@k8s-github-robot k8s-github-robot added the approved Indicates a PR has been approved by an approver from all required OWNERS files. label May 11, 2017
@luxas
Copy link
Member

luxas commented May 12, 2017

/lgtm from me as well

@k8s-github-robot
Copy link

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: MrHohn, bowei, luxas, mikedanese

Needs approval from an approver in each of these OWNERS Files:

You can indicate your approval by writing /approve in a comment
You can cancel your approval by writing /approve cancel in a comment

@k8s-github-robot
Copy link

Automatic merge from submit-queue

@k8s-github-robot k8s-github-robot merged commit 3b9a90a into kubernetes:master May 12, 2017
@erikgrinaker
Copy link

@bowei Any chance of having this backported to 1.6? We are seeing multiple kube-dns crashes every day.

@bowei
Copy link
Member Author

bowei commented May 30, 2017

I'll add it to the point release.

@gmarek gmarek mentioned this pull request Jun 12, 2017
Merged
k8s-github-robot pushed a commit that referenced this pull request Jun 12, 2017
Merge pull request #47328 from gmarek/automated-cherry-pick-of-#45684…
584f2b6 
…-upstream-release-1.6

Automatic merge from submit-queue

Automated cherry pick of #45684

Cherry pick of #45684 on release-1.6.

#45684: Update kube-dns version to 1.14.2
@seungkyua seungkyua mentioned this pull request Jun 22, 2017
Merged
seungkyua added a commit to seungkyua/k8s-kubespray that referenced this pull request Jun 27, 2017
@seungkyua
Make kubedns up to date
d5516a4 
Update kube-dns version to 1.14.2
kubernetes/kubernetes#45684
@bowei bowei deleted the kube-dns-update branch September 5, 2017 20:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@MrHohn MrHohn MrHohn approved these changes

Assignees

@justinsb justinsb

@mikedanese mikedanese

@luxas luxas

@MrHohn MrHohn

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. release-note Denotes a PR that will be considered when it comes time to generate release notes. size/S Denotes a PR that changes 10-29 lines, ignoring generated files.
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
8 participants
@bowei @MrHohn @mikedanese @luxas @k8s-github-robot @erikgrinaker @justinsb @k8s-ci-robot