)]}'
{"scripts/run-upgrade.sh":[{"author":{"_account_id":7353,"name":"Kevin Carter","email":"kevin@cloudnull.com","username":"cloudnull"},"change_message_id":"084dd6cc437ce1a2429136ef1f05b622446393ac","unresolved":false,"context_lines":[{"line_number":299,"context_line":"# Remove containers that we no longer need"},{"line_number":300,"context_line":"pushd playbooks"},{"line_number":301,"context_line":"  # Ensure that apt-transport-https is installed everywhere before doing anything else."},{"line_number":302,"context_line":"  ansible \u0027hosts:all_containers:!nova_console:!repo_container:!cinder_scheduler:!rsyslog_all\u0027 \\"},{"line_number":303,"context_line":"          -m shell \\"},{"line_number":304,"context_line":"          -a \"apt-get update \u0026\u0026 apt-get install -y apt-transport-https\""},{"line_number":305,"context_line":""}],"source_content_type":"text/x-sh","patch_set":3,"id":"ba3cc151_8c9dd60b","line":302,"updated":"2015-07-03 15:17:54.000000000","message":"Why are we not running this section on these containers? It seems like if we move line 318-325 above this we should be able to simply execute this as is because the items in inventory will be otherwise removed. Additionally we want/need to make sure that the apt-transport-https package is everywhere before running the upgrade.","commit_id":"11776a7ef3891ccce35262558fb91a976edf82a0"},{"author":{"_account_id":7353,"name":"Kevin Carter","email":"kevin@cloudnull.com","username":"cloudnull"},"change_message_id":"084dd6cc437ce1a2429136ef1f05b622446393ac","unresolved":false,"context_lines":[{"line_number":433,"context_line":"# and new inventory which will have containers that have not been created yet."},{"line_number":434,"context_line":"cat \u003e /tmp/fix_container_interfaces.yml \u003c\u003cEOF"},{"line_number":435,"context_line":"- name: Fix container things"},{"line_number":436,"context_line":"  hosts: \"all_containers:!nova_console:!repo_container:!cinder_scheduler:!rsyslog_all\""},{"line_number":437,"context_line":"  max_fail_percentage: 100"},{"line_number":438,"context_line":"  gather_facts: false"},{"line_number":439,"context_line":"  user: root"}],"source_content_type":"text/x-sh","patch_set":3,"id":"ba3cc151_8c42b666","line":436,"updated":"2015-07-03 15:17:54.000000000","message":"Why are the excludes needed here?","commit_id":"11776a7ef3891ccce35262558fb91a976edf82a0"},{"author":{"_account_id":7353,"name":"Kevin Carter","email":"kevin@cloudnull.com","username":"cloudnull"},"change_message_id":"084dd6cc437ce1a2429136ef1f05b622446393ac","unresolved":false,"context_lines":[{"line_number":508,"context_line":""},{"line_number":509,"context_line":"  # Hunt for and remove any rpc_release link files from pip"},{"line_number":510,"context_line":"  ansible \u0027hosts:all_containers:!nova_console:!repo_container:!cinder_scheduler:!rsyslog_all\u0027 \\"},{"line_number":511,"context_line":"          -m \"shell\" \\"},{"line_number":512,"context_line":"          -a \"rm /root/.pip/links.d/rpc_release.link\""},{"line_number":513,"context_line":""},{"line_number":514,"context_line":"  # Run the fix adjustments play."}],"source_content_type":"text/x-sh","patch_set":3,"id":"ba3cc151_6c23b2bd","line":511,"updated":"2015-07-03 15:17:54.000000000","message":"Same question as before.","commit_id":"11776a7ef3891ccce35262558fb91a976edf82a0"}]}