)]}'
{"scripts/gate-check-commit.sh":[{"author":{"_account_id":12898,"name":"Tony Breeds","email":"tony@bakeyournoodle.com","username":"tonyb"},"change_message_id":"0e1b8be7092796f8e1582eb937996d637aecbc97","unresolved":false,"context_lines":[{"line_number":111,"context_line":"  # This installs from a git checkout"},{"line_number":112,"context_line":"  # PIP_COMMAND and PIP_OPTS are exported by the bootstrap-ansible script."},{"line_number":113,"context_line":"  # PIP_OPTS contains the whole set of constraints that need to be applied."},{"line_number":114,"context_line":"  ${PIP_COMMAND} install --isolated ${PIP_OPTS} --requirement ${ARA_SRC_HOME}/requirements.txt ${ARA_SRC_HOME} \"${ANSIBLE_PACKAGE:-ansible}\""},{"line_number":115,"context_line":"else"},{"line_number":116,"context_line":"  # This installs from pypi"},{"line_number":117,"context_line":"  # PIP_COMMAND and PIP_OPTS are exported by the bootstrap-ansible script."}],"source_content_type":"text/x-sh","patch_set":2,"id":"5f7c97a3_e3427568","line":114,"range":{"start_line":114,"start_character":36,"end_line":114,"end_character":46},"updated":"2018-07-03 02:13:46.000000000","message":"Just use: ${PIP_OPTS%%--constraint *} here and then you don\u0027t need to maniplate/save the original PIP_OPTS","commit_id":"368b0f0b0056b304a6b4dc36fb402cd5d7725b29"},{"author":{"_account_id":12898,"name":"Tony Breeds","email":"tony@bakeyournoodle.com","username":"tonyb"},"change_message_id":"b0248b1be641baa3b15ff8fa335c947765a995d1","unresolved":false,"context_lines":[{"line_number":109,"context_line":"  # This installs from a git checkout"},{"line_number":110,"context_line":"  # PIP_COMMAND and PIP_OPTS are exported by the bootstrap-ansible script."},{"line_number":111,"context_line":"  # PIP_OPTS contains the whole set of constraints that need to be applied."},{"line_number":112,"context_line":"  ${PIP_COMMAND} install --isolated ${PIP_OPTS%%--constraint *} --requirement ${ARA_SRC_HOME}/requirements.txt ${ARA_SRC_HOME} \"${ANSIBLE_PACKAGE:-ansible}\""},{"line_number":113,"context_line":"else"},{"line_number":114,"context_line":"  # This installs from pypi"},{"line_number":115,"context_line":"  # PIP_COMMAND and PIP_OPTS are exported by the bootstrap-ansible script."}],"source_content_type":"text/x-sh","patch_set":3,"id":"5f7c97a3_fea1acf8","line":112,"updated":"2018-07-03 02:36:22.000000000","message":"If you\u0027re now *not* using constraints, what does manually specifying the requirements file give you?  It\u0027ll be process the same when read from ara.","commit_id":"b792a306aa8d63b771a9879086ac142e82c85d6c"},{"author":{"_account_id":12898,"name":"Tony Breeds","email":"tony@bakeyournoodle.com","username":"tonyb"},"change_message_id":"b0248b1be641baa3b15ff8fa335c947765a995d1","unresolved":false,"context_lines":[{"line_number":114,"context_line":"  # This installs from pypi"},{"line_number":115,"context_line":"  # PIP_COMMAND and PIP_OPTS are exported by the bootstrap-ansible script."},{"line_number":116,"context_line":"  # PIP_OPTS contains the whole set of constraints that need to be applied."},{"line_number":117,"context_line":"  ${PIP_COMMAND} install --isolated ${PIP_OPTS%%--constraint *} --requirement https://git.openstack.org/cgit/openstack/ara/plain/requirements.txt ara \"${ANSIBLE_PACKAGE:-ansible}\""},{"line_number":118,"context_line":"fi"},{"line_number":119,"context_line":"# Dynamically retrieve the location of the ARA callback so we are able to find"},{"line_number":120,"context_line":"# it on both py2 and py3"}],"source_content_type":"text/x-sh","patch_set":3,"id":"5f7c97a3_beab34d5","line":117,"range":{"start_line":117,"start_character":78,"end_line":117,"end_character":145},"updated":"2018-07-03 02:36:22.000000000","message":"This will probably get out of syn with the version on pypi.  ie if a change is merged on master that un-pins Flask, until that\u0027s released you\u0027ll be broken.","commit_id":"b792a306aa8d63b771a9879086ac142e82c85d6c"},{"author":{"_account_id":12898,"name":"Tony Breeds","email":"tony@bakeyournoodle.com","username":"tonyb"},"change_message_id":"d7ac28d734fbb101a360b000a5a7688f4cd2c70d","unresolved":false,"context_lines":[{"line_number":98,"context_line":""},{"line_number":99,"context_line":"# Bootstrap Ansible"},{"line_number":100,"context_line":"source \"${OSA_CLONE_DIR}/scripts/bootstrap-ansible.sh\""},{"line_number":101,"context_line":"# PIP_COMMAND is now set within the ansible venv and PIP_OPTS is now set with constraints/proxy."},{"line_number":102,"context_line":""},{"line_number":103,"context_line":"# Install ARA and add it to the callback path provided by bootstrap-ansible.sh/openstack-ansible.rc"},{"line_number":104,"context_line":"# This is added *here* instead of bootstrap-ansible so it\u0027s used for CI purposes only."}],"source_content_type":"text/x-sh","patch_set":4,"id":"5f7c97a3_de6b9083","line":101,"range":{"start_line":101,"start_character":2,"end_line":101,"end_character":49},"updated":"2018-07-03 02:47:48.000000000","message":"Ahh okay I missed that no more duelling flask","commit_id":"fc6a134bea6cc40598733cdaebb63997074f29c6"}]}