-
Notifications
You must be signed in to change notification settings - Fork 82
Address outstanding DNS review comments in #26694 #48
Comments
Comment by goltermann @quinton-hoole this is P2...time to punt to v1.5? |
Comment by quinton-hoole @golterman Yes, 1.5 is fine. I seem to have lost the authority to update the Milestone? Did you perhaps change permissions? |
Comment by goltermann I moved it - no idea why you wouldn't have permission on the kubernetes repo. |
Comment by dims ok to move this to 1.6? please holler if not appropriate |
Comment by quinton-hoole Reassigning to @nikhiljindal in case he wants to address this in future. @mfanjie would also be a suitable candidate. |
Comment by ethernetdan Moving to 1.7 as late to happen in 1.6. Feel free to switch back if this is incorrect. |
Comment by marun Not a release blocker for 1.7. |
Comment by k8s-merge-robot [MILESTONENOTIFIER] Milestone Removed Important: kind: Must specify exactly one of [ Removing it from the milestone. |
Issues go stale after 90d of inactivity. If this issue is safe to close now please do so with Send feedback to sig-testing, kubernetes/test-infra and/or fejta. |
The issues have been addressed in current code after it was service controller code was refactored. So this issue is not relevant anymore |
Issue by quinton-hoole
Monday Jun 06, 2016 at 23:42 GMT
Originally opened as kubernetes/kubernetes#26921
See kubernetes/kubernetes#26694 for details. Specifically:
cc: @mfanjie FYI
The text was updated successfully, but these errors were encountered: