{"action":"created","issue":{"url":"https://api.github.com/repos/alphagov/tsuru-ansible/issues/128","labels_url":"https://api.github.com/repos/alphagov/tsuru-ansible/issues/128/labels{/name}","comments_url":"https://api.github.com/repos/alphagov/tsuru-ansible/issues/128/comments","events_url":"https://api.github.com/repos/alphagov/tsuru-ansible/issues/128/events","html_url":"https://github.com/alphagov/tsuru-ansible/pull/128","id":96073002,"number":128,"title":"Unify AWS and GCE node names ","user":{"login":"mtekel","id":483891,"avatar_url":"https://avatars.githubusercontent.com/u/483891?v=3","gravatar_id":"","url":"https://api.github.com/users/mtekel","html_url":"https://github.com/mtekel","followers_url":"https://api.github.com/users/mtekel/followers","following_url":"https://api.github.com/users/mtekel/following{/other_user}","gists_url":"https://api.github.com/users/mtekel/gists{/gist_id}","starred_url":"https://api.github.com/users/mtekel/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/mtekel/subscriptions","organizations_url":"https://api.github.com/users/mtekel/orgs","repos_url":"https://api.github.com/users/mtekel/repos","events_url":"https://api.github.com/users/mtekel/events{/privacy}","received_events_url":"https://api.github.com/users/mtekel/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":{"login":"keymon","id":280032,"avatar_url":"https://avatars.githubusercontent.com/u/280032?v=3","gravatar_id":"","url":"https://api.github.com/users/keymon","html_url":"https://github.com/keymon","followers_url":"https://api.github.com/users/keymon/followers","following_url":"https://api.github.com/users/keymon/following{/other_user}","gists_url":"https://api.github.com/users/keymon/gists{/gist_id}","starred_url":"https://api.github.com/users/keymon/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/keymon/subscriptions","organizations_url":"https://api.github.com/users/keymon/orgs","repos_url":"https://api.github.com/users/keymon/repos","events_url":"https://api.github.com/users/keymon/events{/privacy}","received_events_url":"https://api.github.com/users/keymon/received_events","type":"User","site_admin":false},"milestone":null,"comments":0,"created_at":"2015-07-20T14:11:44Z","updated_at":"2015-07-20T16:59:21Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/alphagov/tsuru-ansible/pulls/128","html_url":"https://github.com/alphagov/tsuru-ansible/pull/128","diff_url":"https://github.com/alphagov/tsuru-ansible/pull/128.diff","patch_url":"https://github.com/alphagov/tsuru-ansible/pull/128.patch"},"body":"There is no reason why we should have different node names between AWS and GCE. Having same node names makes our ansible and terraform configuration more clear and straightforward.\r\n\r\nDepends on https://github.com/alphagov/tsuru-terraform/pull/85"},"comment":{"url":"https://api.github.com/repos/alphagov/tsuru-ansible/issues/comments/122951113","html_url":"https://github.com/alphagov/tsuru-ansible/pull/128#issuecomment-122951113","issue_url":"https://api.github.com/repos/alphagov/tsuru-ansible/issues/128","id":122951113,"user":{"login":"keymon","id":280032,"avatar_url":"https://avatars.githubusercontent.com/u/280032?v=3","gravatar_id":"","url":"https://api.github.com/users/keymon","html_url":"https://github.com/keymon","followers_url":"https://api.github.com/users/keymon/followers","following_url":"https://api.github.com/users/keymon/following{/other_user}","gists_url":"https://api.github.com/users/keymon/gists{/gist_id}","starred_url":"https://api.github.com/users/keymon/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/keymon/subscriptions","organizations_url":"https://api.github.com/users/keymon/orgs","repos_url":"https://api.github.com/users/keymon/repos","events_url":"https://api.github.com/users/keymon/events{/privacy}","received_events_url":"https://api.github.com/users/keymon/received_events","type":"User","site_admin":false},"created_at":"2015-07-20T16:59:21Z","updated_at":"2015-07-20T16:59:21Z","body":"I had problems here. The register of the docker nodes was hanging, asking for a password running `tsuru-admin` in the task `Register node`\r\n\r\nThere were 2 problems:\r\n * First [we run `tsuru-admin` commands without closing the stdin](https://github.com/alphagov/tsuru-ansible/blob/master/common.yml#L50), which can potentially block the process asking for a password. We should add `< /dev/null` in each line to prevent that.\r\n\r\n * Second, the authentication [happens only in one random node of the api](https://github.com/alphagov/tsuru-ansible/blob/master/tsuru_api.yml#L49) because `run_once: true`. But this is state that is stored in the API server, so should be executed in both servers.\r\n\r\n * Finally, in the [common.yml when registering the node we force it to use the first host]((https://github.com/alphagov/tsuru-ansible/blob/master/common.yml) which maybe is the one which authenticated before, or maybe not.\r\n"}}