)]}'
{"ansible/roles/kolla-ansible/templates/overcloud-components.j2":[{"author":{"_account_id":17669,"name":"Doug Szumski","email":"doug@stackhpc.com","username":"DougSzumski"},"change_message_id":"33301464fe81e833ac720c6450c8f19d59e795db","unresolved":false,"context_lines":[{"line_number":83,"context_line":"[qdrouterd:children]"},{"line_number":84,"context_line":"control"},{"line_number":85,"context_line":""},{"line_number":86,"context_line":"[monasca-agent:children]"},{"line_number":87,"context_line":"compute"},{"line_number":88,"context_line":"control"},{"line_number":89,"context_line":"monitoring"}],"source_content_type":"text/x-jinja2","patch_set":22,"id":"5fc1f717_128f5ccd","line":86,"range":{"start_line":86,"start_character":1,"end_line":86,"end_character":8},"updated":"2019-04-10 10:07:37.000000000","message":"Thanks!","commit_id":"eda8246b7ba0512774f8988b7dc96fb5a5c3578b"}],"tox.ini":[{"author":{"_account_id":15197,"name":"Pierre Riteau","email":"pierre@stackhpc.com","username":"priteau","status":"StackHPC"},"change_message_id":"86cdfae800f30db1a6c59381374c11e65b494771","unresolved":false,"context_lines":[{"line_number":59,"context_line":"[testenv:molecule]"},{"line_number":60,"context_line":"# Molecule only supports python 2.7 and 3.6."},{"line_number":61,"context_line":"basepython \u003d python2"},{"line_number":62,"context_line":"# Use stable/rocky upper-constraints for molecule since pytest 3.10 fails with"},{"line_number":63,"context_line":"# the following error:"},{"line_number":64,"context_line":"# AttributeError: \u0027Config\u0027 object has no attribute \u0027cache\u0027"},{"line_number":65,"context_line":"deps \u003d"},{"line_number":66,"context_line":"    -c{env:UPPER_CONSTRAINTS_FILE:https://git.openstack.org/cgit/openstack/requirements/plain/upper-constraints.txt?h\u003dstable/rocky}"},{"line_number":67,"context_line":"    -r{toxinidir}/test-requirements.txt"},{"line_number":68,"context_line":"    -r{toxinidir}/doc/requirements.txt"},{"line_number":69,"context_line":"whitelist_externals \u003d"},{"line_number":70,"context_line":"    bash"},{"line_number":71,"context_line":"commands \u003d"}],"source_content_type":"text/x-properties","patch_set":22,"id":"3fce034c_b05e29e4","line":68,"range":{"start_line":62,"start_character":0,"end_line":68,"end_character":38},"updated":"2019-04-12 18:06:59.000000000","message":"I don\u0027t understand why we need this here, but it\u0027s removed later in https://review.openstack.org/#/c/652090/1/tox.ini\n\nIt is a remnant from earlier testing?","commit_id":"eda8246b7ba0512774f8988b7dc96fb5a5c3578b"},{"author":{"_account_id":14826,"name":"Mark Goddard","email":"markgoddard86@gmail.com","username":"mgoddard"},"change_message_id":"5ff7f6ee561d0fec56ae09e0be8b763cd7144500","unresolved":false,"context_lines":[{"line_number":59,"context_line":"[testenv:molecule]"},{"line_number":60,"context_line":"# Molecule only supports python 2.7 and 3.6."},{"line_number":61,"context_line":"basepython \u003d python2"},{"line_number":62,"context_line":"# Use stable/rocky upper-constraints for molecule since pytest 3.10 fails with"},{"line_number":63,"context_line":"# the following error:"},{"line_number":64,"context_line":"# AttributeError: \u0027Config\u0027 object has no attribute \u0027cache\u0027"},{"line_number":65,"context_line":"deps \u003d"},{"line_number":66,"context_line":"    -c{env:UPPER_CONSTRAINTS_FILE:https://git.openstack.org/cgit/openstack/requirements/plain/upper-constraints.txt?h\u003dstable/rocky}"},{"line_number":67,"context_line":"    -r{toxinidir}/test-requirements.txt"},{"line_number":68,"context_line":"    -r{toxinidir}/doc/requirements.txt"},{"line_number":69,"context_line":"whitelist_externals \u003d"},{"line_number":70,"context_line":"    bash"},{"line_number":71,"context_line":"commands \u003d"}],"source_content_type":"text/x-properties","patch_set":22,"id":"3fce034c_6d9c555d","line":68,"range":{"start_line":62,"start_character":0,"end_line":68,"end_character":38},"in_reply_to":"3fce034c_b05e29e4","updated":"2019-04-15 11:39:23.000000000","message":"When I first added this patch it was necessary, but when I recently created the other one I thought I\u0027d try removing it and it worked. Given that the net result will be the same, shall we keep it and save a respin?","commit_id":"eda8246b7ba0512774f8988b7dc96fb5a5c3578b"}]}