{"action":"created","issue":{"url":"https://api.github.com/repos/openshift/openshift-ansible/issues/2406","repository_url":"https://api.github.com/repos/openshift/openshift-ansible","labels_url":"https://api.github.com/repos/openshift/openshift-ansible/issues/2406/labels{/name}","comments_url":"https://api.github.com/repos/openshift/openshift-ansible/issues/2406/comments","events_url":"https://api.github.com/repos/openshift/openshift-ansible/issues/2406/events","html_url":"https://github.com/openshift/openshift-ansible/pull/2406","id":174826364,"number":2406,"title":"Record schedulability of node prior to upgrade and re-set it to that","user":{"login":"sdodson","id":48161,"avatar_url":"https://avatars.githubusercontent.com/u/48161?v=3","gravatar_id":"","url":"https://api.github.com/users/sdodson","html_url":"https://github.com/sdodson","followers_url":"https://api.github.com/users/sdodson/followers","following_url":"https://api.github.com/users/sdodson/following{/other_user}","gists_url":"https://api.github.com/users/sdodson/gists{/gist_id}","starred_url":"https://api.github.com/users/sdodson/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/sdodson/subscriptions","organizations_url":"https://api.github.com/users/sdodson/orgs","repos_url":"https://api.github.com/users/sdodson/repos","events_url":"https://api.github.com/users/sdodson/events{/privacy}","received_events_url":"https://api.github.com/users/sdodson/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":6,"created_at":"2016-09-02T18:57:00Z","updated_at":"2016-09-02T20:02:57Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/openshift/openshift-ansible/pulls/2406","html_url":"https://github.com/openshift/openshift-ansible/pull/2406","diff_url":"https://github.com/openshift/openshift-ansible/pull/2406.diff","patch_url":"https://github.com/openshift/openshift-ansible/pull/2406.patch"},"body":"Previously we were setting schedulability to the state defined in the inventory\r\nwithout regard to whether or not it was manually made schedulable or\r\nunschedulable. The right thing seems to be to record the state prior to upgrade\r\nand set it back.\r\n\r\nAlso, the bug reports that masters were being set schedulable when they shouldn't have been but I couldn't replicate that behavior but I think this would fix that too.\r\n\r\nBased on https://github.com/openshift/openshift-ansible/blob/master/playbooks/common/openshift-cluster/redeploy-certificates.yml#L202-L236\r\nFixes Bug 1372594"},"comment":{"url":"https://api.github.com/repos/openshift/openshift-ansible/issues/comments/244474409","html_url":"https://github.com/openshift/openshift-ansible/pull/2406#issuecomment-244474409","issue_url":"https://api.github.com/repos/openshift/openshift-ansible/issues/2406","id":244474409,"user":{"login":"dgoodwin","id":51265,"avatar_url":"https://avatars.githubusercontent.com/u/51265?v=3","gravatar_id":"","url":"https://api.github.com/users/dgoodwin","html_url":"https://github.com/dgoodwin","followers_url":"https://api.github.com/users/dgoodwin/followers","following_url":"https://api.github.com/users/dgoodwin/following{/other_user}","gists_url":"https://api.github.com/users/dgoodwin/gists{/gist_id}","starred_url":"https://api.github.com/users/dgoodwin/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/dgoodwin/subscriptions","organizations_url":"https://api.github.com/users/dgoodwin/orgs","repos_url":"https://api.github.com/users/dgoodwin/repos","events_url":"https://api.github.com/users/dgoodwin/events{/privacy}","received_events_url":"https://api.github.com/users/dgoodwin/received_events","type":"User","site_admin":false},"created_at":"2016-09-02T20:02:57Z","updated_at":"2016-09-02T20:02:57Z","body":"Status is preserved with your change @sdodson. LGTM.\r\n"}}
{"actor":{"display_login":"dgoodwin"}}