)]}'
{"tox.ini":[{"author":{"_account_id":6159,"name":"Flavio Percoco Premoli","display_name":"flaper87","email":"flavio.percoco@flyrlabs.com","username":"flaper87"},"change_message_id":"01f524b34218fb0d69c6aaf3667e33c172554c4a","unresolved":false,"context_lines":[{"line_number":44,"context_line":"setenv \u003d TRANSPORT_URL\u003drabbit://guest:password@localhost"},{"line_number":45,"context_line":"commands \u003d python setup.py testr --slowest --testr-args\u003d\u0027tests.functional\u0027"},{"line_number":46,"context_line":""},{"line_number":47,"context_line":"[testenv:py27-amqp1]"},{"line_number":48,"context_line":"setenv \u003d TRANSPORT_URL\u003damqp://guest:password@localhost"},{"line_number":49,"context_line":"deps \u003d -r{toxinidir}/amqp1-requirements.txt"},{"line_number":50,"context_line":"        {[testenv]deps}"}],"source_content_type":"text/x-properties","patch_set":3,"id":"7a8c0949_43133a66","line":47,"updated":"2014-10-31 18:13:41.000000000","message":"Should this mention `functional` - or just `func` - somewhere? There\u0027s already a testenv for amqp1 and it could be quite confusing for folks to see this.\n\nAlso, it\u0027d be nice to add a comment explaining why we need this.","commit_id":"47c407ddd10995da70e54fec5548e21708b91580"},{"author":{"_account_id":8770,"name":"Ken Giusti","email":"kgiusti@gmail.com","username":"kgiusti"},"change_message_id":"9217b266ced17a9ef2469a67bcb89236f9305454","unresolved":false,"context_lines":[{"line_number":45,"context_line":"commands \u003d python setup.py testr --slowest --testr-args\u003d\u0027tests.functional\u0027"},{"line_number":46,"context_line":""},{"line_number":47,"context_line":"[testenv:py27-amqp1]"},{"line_number":48,"context_line":"setenv \u003d TRANSPORT_URL\u003damqp://guest:password@localhost"},{"line_number":49,"context_line":"deps \u003d -r{toxinidir}/amqp1-requirements.txt"},{"line_number":50,"context_line":"        {[testenv]deps}"},{"line_number":51,"context_line":"commands \u003d python setup.py testr --slowest --testr-args\u003d\u0027tests.functional\u0027"}],"source_content_type":"text/x-properties","patch_set":3,"id":"7a8c0949_c3364d03","line":48,"updated":"2014-10-31 15:56:25.000000000","message":"Oops - I forgot: the checked in AMQP 1.0 driver doesn\u0027t support user credentials in the URL at the moment.\n\nI\u0027ve got a patch to enable this [1], but until it gets merged, you can leave guest:password@ off the URL.\n\n[1] https://review.openstack.org/#/c/132286/","commit_id":"47c407ddd10995da70e54fec5548e21708b91580"},{"author":{"_account_id":2813,"name":"Mehdi Abaakouk","email":"sileht@sileht.net","username":"sileht"},"change_message_id":"305b14da9024ad885b31a53d4439f2f2265e7680","unresolved":false,"context_lines":[{"line_number":45,"context_line":"commands \u003d python setup.py testr --slowest --testr-args\u003d\u0027tests.functional\u0027"},{"line_number":46,"context_line":""},{"line_number":47,"context_line":"[testenv:py27-amqp1]"},{"line_number":48,"context_line":"setenv \u003d TRANSPORT_URL\u003damqp://guest:password@localhost"},{"line_number":49,"context_line":"deps \u003d -r{toxinidir}/amqp1-requirements.txt"},{"line_number":50,"context_line":"        {[testenv]deps}"},{"line_number":51,"context_line":"commands \u003d python setup.py testr --slowest --testr-args\u003d\u0027tests.functional\u0027"}],"source_content_type":"text/x-properties","patch_set":3,"id":"7a8c0949_fe87620d","line":48,"in_reply_to":"7a8c0949_c3364d03","updated":"2014-10-31 16:18:15.000000000","message":"It\u0027s OK to keep the usename/password, this is just to prepare the futur of oslo.messaging gating :) \n\nI don\u0027t really know the infra policy about experimental vs non-voting jobs, but I think theses jobs will not vote until we are sure everything works and don\u0027t result into false positive or negative review.\n\nSo, we have time to make some adjustment latter if needed.","commit_id":"47c407ddd10995da70e54fec5548e21708b91580"}]}