{"action":"created","issue":{"url":"https://api.github.com/repos/rcbops/rpc-openstack/issues/2732","repository_url":"https://api.github.com/repos/rcbops/rpc-openstack","labels_url":"https://api.github.com/repos/rcbops/rpc-openstack/issues/2732/labels{/name}","comments_url":"https://api.github.com/repos/rcbops/rpc-openstack/issues/2732/comments","events_url":"https://api.github.com/repos/rcbops/rpc-openstack/issues/2732/events","html_url":"https://github.com/rcbops/rpc-openstack/pull/2732","id":282246465,"number":2732,"title":"[RO-3268] Bootstrap rpco without breaking artifacts","user":{"login":"cloudnull","id":2066292,"avatar_url":"https://avatars2.githubusercontent.com/u/2066292?v=4","gravatar_id":"","url":"https://api.github.com/users/cloudnull","html_url":"https://github.com/cloudnull","followers_url":"https://api.github.com/users/cloudnull/followers","following_url":"https://api.github.com/users/cloudnull/following{/other_user}","gists_url":"https://api.github.com/users/cloudnull/gists{/gist_id}","starred_url":"https://api.github.com/users/cloudnull/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/cloudnull/subscriptions","organizations_url":"https://api.github.com/users/cloudnull/orgs","repos_url":"https://api.github.com/users/cloudnull/repos","events_url":"https://api.github.com/users/cloudnull/events{/privacy}","received_events_url":"https://api.github.com/users/cloudnull/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":3,"created_at":"2017-12-14T21:35:33Z","updated_at":"2017-12-15T14:32:37Z","closed_at":null,"author_association":"MEMBER","pull_request":{"url":"https://api.github.com/repos/rcbops/rpc-openstack/pulls/2732","html_url":"https://github.com/rcbops/rpc-openstack/pull/2732","diff_url":"https://github.com/rcbops/rpc-openstack/pull/2732.diff","patch_url":"https://github.com/rcbops/rpc-openstack/pull/2732.patch"},"body":"Because APT artifacts can not be used at kick time it's possible that\r\nthe basic installation of OSA can bring an environment ahead of our\r\ncurrent artifacted release. This change makes it possible for APT\r\nartifacts to be present when bootstrapping the environment. When a\r\ndeployer runs a basic installation of RPC-OpenStack they have the choice\r\nto enable or disable artifacts however if they chose to enable them all\r\npackage installations should come from our artifacted repo assuming\r\nthey're available.\r\n\r\nThis change ensures only the minimum requirements are installed on the system\r\nprior to setting up the release and configuring our artifacted solutions.\r\nTo keep everything as idempotent as possible and to ensure we're using\r\nplaybooks to configure the deployment, the initial install will create an\r\n*rpc-ansible* virtual environment in \"/opt\" using a cached version of ansible\r\nfrom [ rpc-repo.rackspace.com ]. This simple ansible installation runs only\r\nthe required playbooks to get the deployment online. Once OSA has been\r\nbootstrapped all future interactions can be run through the\r\n`openstack-ansible` cli tool. \r\n\r\nNon-Idempotent commands in scripts were have been moved into\r\nplaybooks where possible. Modules have replaced most of the\r\nfunctions and if no suitable module was available the command module\r\nwas used but in the most idempotent way possible by leveraging local\r\nfacts.\r\n\r\nSigned-off-by: Kevin Carter <kevin.carter@rackspace.com>\r\n\r\nIssue: [RO-3268](https://rpc-openstack.atlassian.net/browse/RO-3268)"},"comment":{"url":"https://api.github.com/repos/rcbops/rpc-openstack/issues/comments/352019866","html_url":"https://github.com/rcbops/rpc-openstack/pull/2732#issuecomment-352019866","issue_url":"https://api.github.com/repos/rcbops/rpc-openstack/issues/2732","id":352019866,"user":{"login":"odyssey4me","id":2152106,"avatar_url":"https://avatars2.githubusercontent.com/u/2152106?v=4","gravatar_id":"","url":"https://api.github.com/users/odyssey4me","html_url":"https://github.com/odyssey4me","followers_url":"https://api.github.com/users/odyssey4me/followers","following_url":"https://api.github.com/users/odyssey4me/following{/other_user}","gists_url":"https://api.github.com/users/odyssey4me/gists{/gist_id}","starred_url":"https://api.github.com/users/odyssey4me/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/odyssey4me/subscriptions","organizations_url":"https://api.github.com/users/odyssey4me/orgs","repos_url":"https://api.github.com/users/odyssey4me/repos","events_url":"https://api.github.com/users/odyssey4me/events{/privacy}","received_events_url":"https://api.github.com/users/odyssey4me/received_events","type":"User","site_admin":false},"created_at":"2017-12-15T14:32:37Z","updated_at":"2017-12-15T14:32:37Z","author_association":"CONTRIBUTOR","body":"@cloudnull Suit yourself, I think this is madness. This does not help us implement a properly repeatable build and is a massive amount of tech debt to carry in comparison. This is over 300 lines added, plus a new venv, plus a wholly different way of doing the setup. The point of this rewrite was to reduce the wrappers and flatten the build. This is just adding layers, adding more possibilities for variation in our builds and making it harder to debug what's going on.\r\n\r\nIf this merges, I will work with it only because I have to. But, this is not an approach that I support."}}
{"actor":{"display_login":"odyssey4me"}}