{"action":"created","issue":{"url":"https://api.github.com/repos/openshift/openshift-ansible/issues/3028","repository_url":"https://api.github.com/repos/openshift/openshift-ansible","labels_url":"https://api.github.com/repos/openshift/openshift-ansible/issues/3028/labels{/name}","comments_url":"https://api.github.com/repos/openshift/openshift-ansible/issues/3028/comments","events_url":"https://api.github.com/repos/openshift/openshift-ansible/issues/3028/events","html_url":"https://github.com/openshift/openshift-ansible/issues/3028","id":198393025,"number":3028,"title":"If upgrade fails, and user re-runs, master node is marked unschedulable","user":{"login":"smarterclayton","id":1163175,"avatar_url":"https://avatars.githubusercontent.com/u/1163175?v=3","gravatar_id":"","url":"https://api.github.com/users/smarterclayton","html_url":"https://github.com/smarterclayton","followers_url":"https://api.github.com/users/smarterclayton/followers","following_url":"https://api.github.com/users/smarterclayton/following{/other_user}","gists_url":"https://api.github.com/users/smarterclayton/gists{/gist_id}","starred_url":"https://api.github.com/users/smarterclayton/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/smarterclayton/subscriptions","organizations_url":"https://api.github.com/users/smarterclayton/orgs","repos_url":"https://api.github.com/users/smarterclayton/repos","events_url":"https://api.github.com/users/smarterclayton/events{/privacy}","received_events_url":"https://api.github.com/users/smarterclayton/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":1,"created_at":"2017-01-03T00:00:10Z","updated_at":"2017-01-03T00:03:29Z","closed_at":null,"body":"```\r\n- name: Set node schedulability\r\n    command: >\r\n      {{ openshift.common.client_binary }} adm manage-node {{ openshift.common.hostname | lower }} --schedulable=true\r\n    delegate_to: \"{{ groups.oo_first_master.0 }}\"\r\n    when: openshift.node.schedulable | bool\r\n    when: l_docker_upgrade is defined and l_docker_upgrade | bool and inventory_hostname in groups.oo_nodes_to_upgrade and openshift.node.schedulable | bool\r\n```\r\n\r\nmeans that a re-run of the upgrade will leave the node unschedulable.  In my case, the cluster I'm testing has the master as an infra node, so marking it unschedulable failed the upgrade.\r\n\r\nI'm not sure this *has* to be fixed, but it's something that could result in confusion for a user during upgrade."},"comment":{"url":"https://api.github.com/repos/openshift/openshift-ansible/issues/comments/270034804","html_url":"https://github.com/openshift/openshift-ansible/issues/3028#issuecomment-270034804","issue_url":"https://api.github.com/repos/openshift/openshift-ansible/issues/3028","id":270034804,"user":{"login":"smarterclayton","id":1163175,"avatar_url":"https://avatars.githubusercontent.com/u/1163175?v=3","gravatar_id":"","url":"https://api.github.com/users/smarterclayton","html_url":"https://github.com/smarterclayton","followers_url":"https://api.github.com/users/smarterclayton/followers","following_url":"https://api.github.com/users/smarterclayton/following{/other_user}","gists_url":"https://api.github.com/users/smarterclayton/gists{/gist_id}","starred_url":"https://api.github.com/users/smarterclayton/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/smarterclayton/subscriptions","organizations_url":"https://api.github.com/users/smarterclayton/orgs","repos_url":"https://api.github.com/users/smarterclayton/repos","events_url":"https://api.github.com/users/smarterclayton/events{/privacy}","received_events_url":"https://api.github.com/users/smarterclayton/received_events","type":"User","site_admin":false},"created_at":"2017-01-03T00:03:29Z","updated_at":"2017-01-03T00:03:29Z","body":"It might be sufficient for a preflight check to verify that all infra-nodes are marked schedulable."}}
{"actor":{"display_login":"smarterclayton"}}