Skip to content

Commit

Permalink
Merge pull request #47170 from ktsakalozos/bug/restart
Browse files Browse the repository at this point in the history
Automatic merge from submit-queue (batch tested with PRs 47860, 47170)

Fix restart action on juju kubernetes-master

**What this PR does / why we need it**: Restart action of kubernetes-master of Juju is not functioning. 

**Which issue this PR fixes** *(optional, in `fixes #<issue number>(, fixes #<issue_number>, ...)` format, will close that issue when PR gets merged)*: fixes https://github.com/juju-solutions/bundle-canonical-kubernetes/issues/299

**Special notes for your reviewer**:

**Release note**:

```
Fix: Restart action of juju's kubernetes-master restarts the respective snap based services
```
  • Loading branch information
Kubernetes Submit Queue authored Jun 26, 2017
2 parents 93ef2f2 + 1b88d08 commit ec1fcbe
Showing 1 changed file with 6 additions and 9 deletions.
15 changes: 6 additions & 9 deletions cluster/juju/layers/kubernetes-master/actions/restart
Original file line number Diff line number Diff line change
Expand Up @@ -4,14 +4,11 @@ set +ex

# Restart the apiserver, controller-manager, and scheduler

systemctl restart kube-apiserver
systemctl restart snap.kube-apiserver.daemon
action-set apiserver.status='restarted'

action-set 'apiserver.status' 'restarted'
systemctl restart snap.kube-controller-manager.daemon
action-set controller-manager.status='restarted'

systemctl restart kube-controller-manager

action-set 'controller-manager.status' 'restarted'

systemctl restart kube-scheduler

action-set 'kube-scheduler.status' 'restarted'
systemctl restart snap.kube-scheduler.daemon
action-set kube-scheduler.status='restarted'

0 comments on commit ec1fcbe

Please sign in to comment.