{"action":"created","issue":{"url":"https://api.github.com/repos/alphagov/govuk-puppet/issues/4177","repository_url":"https://api.github.com/repos/alphagov/govuk-puppet","labels_url":"https://api.github.com/repos/alphagov/govuk-puppet/issues/4177/labels{/name}","comments_url":"https://api.github.com/repos/alphagov/govuk-puppet/issues/4177/comments","events_url":"https://api.github.com/repos/alphagov/govuk-puppet/issues/4177/events","html_url":"https://github.com/alphagov/govuk-puppet/pull/4177","id":138133287,"number":4177,"title":"Allow jumpboxes to reboot unattended overnight","user":{"login":"mattbostock","id":4348848,"avatar_url":"https://avatars.githubusercontent.com/u/4348848?v=3","gravatar_id":"","url":"https://api.github.com/users/mattbostock","html_url":"https://github.com/mattbostock","followers_url":"https://api.github.com/users/mattbostock/followers","following_url":"https://api.github.com/users/mattbostock/following{/other_user}","gists_url":"https://api.github.com/users/mattbostock/gists{/gist_id}","starred_url":"https://api.github.com/users/mattbostock/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/mattbostock/subscriptions","organizations_url":"https://api.github.com/users/mattbostock/orgs","repos_url":"https://api.github.com/users/mattbostock/repos","events_url":"https://api.github.com/users/mattbostock/events{/privacy}","received_events_url":"https://api.github.com/users/mattbostock/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":1,"created_at":"2016-03-03T10:00:23Z","updated_at":"2016-03-03T11:19:12Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/alphagov/govuk-puppet/pulls/4177","html_url":"https://github.com/alphagov/govuk-puppet/pull/4177","diff_url":"https://github.com/alphagov/govuk-puppet/pull/4177.diff","patch_url":"https://github.com/alphagov/govuk-puppet/pull/4177.patch"},"body":"The reason we had not previously allowed the jumpboxes to reboot\r\novernight was to avoid exacerbating an incident if someone was\r\ninvestigating an issue out of hours (the time when the unattended\r\nreboots occur) and was logged off mid-incident because the jumpbox was\r\nrebooted.\r\n\r\nThis script checks if there are any users logged on so that the jumpbox\r\ncan reboot overnight.\r\n\r\nNote that there's still the potential that the jumpbox never returns\r\nfrom the reboot, or that it's still rebooting when someone needs to log\r\nin, however I think that that scenario is rare enough that it's not\r\nsomething we should worry about. We also have a second jumpbox.\r\n\r\n* * *\r\n\r\nI plan to refactor the way we're installing the check scripts to use a defined type rather than a new class for each check script but I'll do that in a later PR."},"comment":{"url":"https://api.github.com/repos/alphagov/govuk-puppet/issues/comments/191713172","html_url":"https://github.com/alphagov/govuk-puppet/pull/4177#issuecomment-191713172","issue_url":"https://api.github.com/repos/alphagov/govuk-puppet/issues/4177","id":191713172,"user":{"login":"surminus","id":5610276,"avatar_url":"https://avatars.githubusercontent.com/u/5610276?v=3","gravatar_id":"","url":"https://api.github.com/users/surminus","html_url":"https://github.com/surminus","followers_url":"https://api.github.com/users/surminus/followers","following_url":"https://api.github.com/users/surminus/following{/other_user}","gists_url":"https://api.github.com/users/surminus/gists{/gist_id}","starred_url":"https://api.github.com/users/surminus/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/surminus/subscriptions","organizations_url":"https://api.github.com/users/surminus/orgs","repos_url":"https://api.github.com/users/surminus/repos","events_url":"https://api.github.com/users/surminus/events{/privacy}","received_events_url":"https://api.github.com/users/surminus/received_events","type":"User","site_admin":false},"created_at":"2016-03-03T11:19:12Z","updated_at":"2016-03-03T11:19:12Z","body":"Nice solution! I think also it was problematic because the jumpboxes are used heavily in env-syncs for data transfers? For a server like this it might even be worth pushing forward the time they do maintenance tasks (such as rebooting) to 11pm or something. \r\n\r\nI can just see them rebooting when the env-sync scripts try to connect which means the sync will probably timeout, abort and skip doing a sync completely that morning."}}