Merge ~verterok/ols-jenkaas:reboot-is-reboot into ols-jenkaas:main

Proposed by Guillermo Gonzalez
Status: Merged
Approved by: Guillermo Gonzalez
Approved revision: 231f8f016c85f2b41e9c8296a897ee02f636c429
Merged at revision: 8d72939da43403ba3ca4fc35a91049bfa323333f
Proposed branch: ~verterok/ols-jenkaas:reboot-is-reboot
Merge into: ols-jenkaas:main
Diff against target: 11 lines (+2/-2)
1 file modified
slave/reboot (+2/-2)
Reviewer Review Type Date Requested Status
Gravity Yang (community) Approve
Maximiliano Bertacchini Approve
Review via email: mp+456205@code.launchpad.net

Commit message

Fix the reboot script to actually reboot the instance. Delay it 1min using `at` in order to allow the job to finish.

To post a comment you must log in.
Revision history for this message
Maximiliano Bertacchini (maxiberta) wrote :

+1 thanks!

review: Approve
Revision history for this message
Gravity Yang (gravi1984) wrote :

nice, thanks!

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1diff --git a/slave/reboot b/slave/reboot
2index 614e9d1..b38e11e 100755
3--- a/slave/reboot
4+++ b/slave/reboot
5@@ -1,4 +1,4 @@
6 #!/bin/sh
7
8-#shutdown -r +2
9-sudo service docker restart
10+echo "Rebooting in 1min"
11+echo sudo reboot | at now+1min

Subscribers

People subscribed via source and target branches