Merge pull request #59920 from juju-solutions/bug/cleancredreq

Automatic merge from submit-queue (batch tested with PRs 57136, 59920). If you want to cherry-pick this change to another branch, please follow the instructions <a href="https://github.com/kubernetes/community/blob/master/contributors/devel/cherry-picks.md">here</a>. Clean-up not needed method in juju charms **What this PR does / why we need it**: Improve code quality. Remove code that is not offeringany functionality. **Which issue(s) this PR fixes** *(optional, in `fixes #<issue number>(, fixes #<issue_number>, ...)` format, will close the issue(s) when PR gets merged)*: Fixes # **Special notes for your reviewer**: **Release note**: ```release-note NONE ```

Merge pull request #59920 from juju-solutions/bug/cleancredreq
9586cd06 · Kubernetes Submit Queue · GitHub · eac5bc00 · e2399de9 · 9586cd06
Unverified Commit 9586cd06 authored Feb 16, 2018 by Kubernetes Submit Queue Committed by GitHub Feb 16, 2018
Hide whitespace changes
Inline Side-by-side

Showing with 0 additions and 9 deletions

kubernetes_worker.py ...ju/layers/kubernetes-worker/reactive/kubernetes_worker.py +0 -9

No files found.
--- a/cluster/juju/layers/kubernetes-worker/reactive/kubernetes_worker.py
+++ b/cluster/juju/layers/kubernetes-worker/reactive/kubernetes_worker.py
@@ -945,15 +945,6 @@ def notify_master_gpu_not_enabled(kube_control):
 @when('kube-control.connected')
-@when('config.changed.kubelet-extra-args')
-def maybe_request_new_credentials(kube_control):
-    kubelet_extra_args = parse_extra_args('kubelet-extra-args')
-    cloud_provider = kubelet_extra_args.get('cloud-provider', '')
-    if data_changed('cloud_provider', cloud_provider):
-        request_kubelet_and_proxy_credentials(kube_control)
-@when('kube-control.connected')
 def request_kubelet_and_proxy_credentials(kube_control):
    """ Request kubelet node authorization with a well formed kubelet user.
    This also implies that we are requesting kube-proxy auth. """