{"action":"created","issue":{"url":"https://api.github.com/repos/contiv/ansible/issues/23","labels_url":"https://api.github.com/repos/contiv/ansible/issues/23/labels{/name}","comments_url":"https://api.github.com/repos/contiv/ansible/issues/23/comments","events_url":"https://api.github.com/repos/contiv/ansible/issues/23/events","html_url":"https://github.com/contiv/ansible/pull/23","id":125685544,"number":23,"title":"Misc fixes and enhancements","user":{"login":"mapuri","id":9538438,"avatar_url":"https://avatars.githubusercontent.com/u/9538438?v=3","gravatar_id":"","url":"https://api.github.com/users/mapuri","html_url":"https://github.com/mapuri","followers_url":"https://api.github.com/users/mapuri/followers","following_url":"https://api.github.com/users/mapuri/following{/other_user}","gists_url":"https://api.github.com/users/mapuri/gists{/gist_id}","starred_url":"https://api.github.com/users/mapuri/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/mapuri/subscriptions","organizations_url":"https://api.github.com/users/mapuri/orgs","repos_url":"https://api.github.com/users/mapuri/repos","events_url":"https://api.github.com/users/mapuri/events{/privacy}","received_events_url":"https://api.github.com/users/mapuri/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":5,"created_at":"2016-01-08T20:16:56Z","updated_at":"2016-01-09T06:18:04Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/contiv/ansible/pulls/23","html_url":"https://github.com/contiv/ansible/pull/23","diff_url":"https://github.com/contiv/ansible/pull/23.diff","patch_url":"https://github.com/contiv/ansible/pull/23.patch"},"body":"- install ansible, packer, vagrant and virtualbox in respective roles\r\n  - ansible is a base package as it will be used for cluster provisioning\r\n  - packer, vagrant and virtualbox are dev environment related tools and will enable us to setup or baremetal host environments.\r\n- fixed openvswitch installation for redhat\r\n  - a task got leftover in `base` role as part of `dev` role change, which causes base role to fail.\r\n- fixed yum package installation by separating out epel-release into it's own task\r\n  - otherwise the packages that depend on it fail\r\n- fixed apt package update task to just update the list and no try to update all installed packages\r\n  - otherwise the task fails with conflicts\r\n- install bzip2 in base packages (fixes #22)\r\n\r\n/cc @erikh "},"comment":{"url":"https://api.github.com/repos/contiv/ansible/issues/comments/170201091","html_url":"https://github.com/contiv/ansible/pull/23#issuecomment-170201091","issue_url":"https://api.github.com/repos/contiv/ansible/issues/23","id":170201091,"user":{"login":"jainvipin","id":9005681,"avatar_url":"https://avatars.githubusercontent.com/u/9005681?v=3","gravatar_id":"","url":"https://api.github.com/users/jainvipin","html_url":"https://github.com/jainvipin","followers_url":"https://api.github.com/users/jainvipin/followers","following_url":"https://api.github.com/users/jainvipin/following{/other_user}","gists_url":"https://api.github.com/users/jainvipin/gists{/gist_id}","starred_url":"https://api.github.com/users/jainvipin/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/jainvipin/subscriptions","organizations_url":"https://api.github.com/users/jainvipin/orgs","repos_url":"https://api.github.com/users/jainvipin/repos","events_url":"https://api.github.com/users/jainvipin/events{/privacy}","received_events_url":"https://api.github.com/users/jainvipin/received_events","type":"User","site_admin":false},"created_at":"2016-01-09T06:18:03Z","updated_at":"2016-01-09T06:18:03Z","body":"@mapuri, @erikh \r\n\r\n> > we should be comparing versions of tools instead of existence of binaries.\r\n\r\n> yes, that is a valid point for installing specific versions. I think with apt/yum packages it is easier to do that in same task, but with binaries that we download (like packer) and manually install we end requiring more task to stat and then get version. I think we can write a custom module or something to address this pattern. For now, I will add a separate task to check for packer version than checking the binary.\r\n\r\nIf we don't compare the versions it is a non-starter IMO because if someone uses ansible to update to latest, it will never work. This happened to someone using our ansible while docker was already installed on one of the servers, however the version wasn't correct and thus created an issue.\r\n\r\nping @jojimt @DivyaVavili - as they ran into similar issue where upgrade didn't work"}}