{"action":"created","issue":{"url":"https://api.github.com/repos/openshift/openshift-ansible-contrib/issues/561","repository_url":"https://api.github.com/repos/openshift/openshift-ansible-contrib","labels_url":"https://api.github.com/repos/openshift/openshift-ansible-contrib/issues/561/labels{/name}","comments_url":"https://api.github.com/repos/openshift/openshift-ansible-contrib/issues/561/comments","events_url":"https://api.github.com/repos/openshift/openshift-ansible-contrib/issues/561/events","html_url":"https://github.com/openshift/openshift-ansible-contrib/pull/561","id":243999449,"number":561,"title":"During provisioning, make unnecessary packages optional under a switch","user":{"login":"Tlacenka","id":6306671,"avatar_url":"https://avatars2.githubusercontent.com/u/6306671?v=4","gravatar_id":"","url":"https://api.github.com/users/Tlacenka","html_url":"https://github.com/Tlacenka","followers_url":"https://api.github.com/users/Tlacenka/followers","following_url":"https://api.github.com/users/Tlacenka/following{/other_user}","gists_url":"https://api.github.com/users/Tlacenka/gists{/gist_id}","starred_url":"https://api.github.com/users/Tlacenka/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/Tlacenka/subscriptions","organizations_url":"https://api.github.com/users/Tlacenka/orgs","repos_url":"https://api.github.com/users/Tlacenka/repos","events_url":"https://api.github.com/users/Tlacenka/events{/privacy}","received_events_url":"https://api.github.com/users/Tlacenka/received_events","type":"User","site_admin":false},"labels":[{"id":517606489,"url":"https://api.github.com/repos/openshift/openshift-ansible-contrib/labels/osp","name":"osp","color":"c5def5","default":false}],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":7,"created_at":"2017-07-19T10:49:00Z","updated_at":"2017-07-19T14:19:57Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/openshift/openshift-ansible-contrib/pulls/561","html_url":"https://github.com/openshift/openshift-ansible-contrib/pull/561","diff_url":"https://github.com/openshift/openshift-ansible-contrib/pull/561.diff","patch_url":"https://github.com/openshift/openshift-ansible-contrib/pull/561.patch"},"body":"#### What does this PR do?\r\nThe `openshift-prep` role installs packages that are not necessary for the deployment, such as `bash-completion`n and `vim-enhanced`.\r\nThese are now optional under `install_debug_packages` switch, which is set to `false` by default.\r\n\r\n#### How should this be manually tested?\r\n1) Run provisioning with both `manage_packages` and `install_debug_packages` set to `true`.\r\n- Result: all packages will be installed and updated, even the debug ones.\r\n\r\n2) Run provisioning with `manage_packages` set to `true`, `install_debug_packages` set to `false`.\r\n- Result: only required packages will be installed and updated.\r\n\r\n#### Is there a relevant Issue open for this?\r\n--\r\n\r\n#### Who would you like to review this?\r\ncc: @tomassedovic @bogdando PTAL\r\n"},"comment":{"url":"https://api.github.com/repos/openshift/openshift-ansible-contrib/issues/comments/316402689","html_url":"https://github.com/openshift/openshift-ansible-contrib/pull/561#issuecomment-316402689","issue_url":"https://api.github.com/repos/openshift/openshift-ansible-contrib/issues/561","id":316402689,"user":{"login":"tomassedovic","id":104339,"avatar_url":"https://avatars1.githubusercontent.com/u/104339?v=4","gravatar_id":"","url":"https://api.github.com/users/tomassedovic","html_url":"https://github.com/tomassedovic","followers_url":"https://api.github.com/users/tomassedovic/followers","following_url":"https://api.github.com/users/tomassedovic/following{/other_user}","gists_url":"https://api.github.com/users/tomassedovic/gists{/gist_id}","starred_url":"https://api.github.com/users/tomassedovic/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/tomassedovic/subscriptions","organizations_url":"https://api.github.com/users/tomassedovic/orgs","repos_url":"https://api.github.com/users/tomassedovic/repos","events_url":"https://api.github.com/users/tomassedovic/events{/privacy}","received_events_url":"https://api.github.com/users/tomassedovic/received_events","type":"User","site_admin":false},"created_at":"2017-07-19T14:19:57Z","updated_at":"2017-07-19T14:19:57Z","body":"Oops, the \"CI\" didn't actually run the end to end test. How embarrassing!\r\n\r\nTake two: https://travis-ci.org/tomassedovic/openshift-ansible-contrib/jobs/255280243"}}
{"actor":{"display_login":"tomassedovic"}}