)]}'
{"tox.ini":[{"author":{"_account_id":5196,"name":"Matthew Treinish","email":"mtreinish@kortar.org","username":"treinish"},"change_message_id":"fa56f6722c6da83c1a43051e1e8c2c6e5bf7309c","unresolved":false,"context_lines":[{"line_number":13,"context_line":"usedevelop \u003d True"},{"line_number":14,"context_line":""},{"line_number":15,"context_line":"[testenv:py26]"},{"line_number":16,"context_line":"setenv \u003d PYTHONHASHSEED\u003d0"},{"line_number":17,"context_line":"commands \u003d python setup.py test --slowest --testr-arg\u003d\u0027tempest\\.tests {posargs}\u0027"},{"line_number":18,"context_line":""},{"line_number":19,"context_line":"[testenv:py33]"}],"source_content_type":"text/x-properties","patch_set":1,"id":"1abeadc6_19f83dc2","line":16,"updated":"2014-08-21 14:30:23.000000000","message":"You actually don\u0027t need this line. \n\nOn master we have to set an extra env variable, OS_TEST_PATH, to specify the unit test test suite instead of tempest. \n\nIn havana the test suites weren\u0027t separated yet so they didn\u0027t need to set an extra env variable. By using setenv here you\u0027re overrriding the default one in the testenv section to use an option that is already specified.","commit_id":"7ee83215cb7881460308b13e4d7fb4509587ec78"},{"author":{"_account_id":9656,"name":"Ihar Hrachyshka","email":"ihrachys@redhat.com","username":"ihrachys","status":"Red Hat Networking Systems Engineer"},"change_message_id":"1f70e05d29ba7c7d6c28b003793bf376dffc9c8a","unresolved":false,"context_lines":[{"line_number":13,"context_line":"usedevelop \u003d True"},{"line_number":14,"context_line":""},{"line_number":15,"context_line":"[testenv:py26]"},{"line_number":16,"context_line":"setenv \u003d PYTHONHASHSEED\u003d0"},{"line_number":17,"context_line":"commands \u003d python setup.py test --slowest --testr-arg\u003d\u0027tempest\\.tests {posargs}\u0027"},{"line_number":18,"context_line":""},{"line_number":19,"context_line":"[testenv:py33]"}],"source_content_type":"text/x-properties","patch_set":1,"id":"1abeadc6_f97d411d","line":16,"in_reply_to":"1abeadc6_19f83dc2","updated":"2014-08-21 14:34:02.000000000","message":"Ah, I now see how it really works. So there\u0027s inheritance involved. ;) Ok, I\u0027ll fix that right now.","commit_id":"7ee83215cb7881460308b13e4d7fb4509587ec78"},{"author":{"_account_id":5196,"name":"Matthew Treinish","email":"mtreinish@kortar.org","username":"treinish"},"change_message_id":"fa56f6722c6da83c1a43051e1e8c2c6e5bf7309c","unresolved":false,"context_lines":[{"line_number":17,"context_line":"commands \u003d python setup.py test --slowest --testr-arg\u003d\u0027tempest\\.tests {posargs}\u0027"},{"line_number":18,"context_line":""},{"line_number":19,"context_line":"[testenv:py33]"},{"line_number":20,"context_line":"setenv \u003d PYTHONHASHSEED\u003d0"},{"line_number":21,"context_line":"commands \u003d python setup.py test --slowest --testr-arg\u003d\u0027tempest\\.tests {posargs}\u0027"},{"line_number":22,"context_line":""},{"line_number":23,"context_line":"[testenv:py27]"}],"source_content_type":"text/x-properties","patch_set":1,"id":"1abeadc6_b9c829a9","line":20,"updated":"2014-08-21 14:30:23.000000000","message":"Same here","commit_id":"7ee83215cb7881460308b13e4d7fb4509587ec78"},{"author":{"_account_id":5196,"name":"Matthew Treinish","email":"mtreinish@kortar.org","username":"treinish"},"change_message_id":"fa56f6722c6da83c1a43051e1e8c2c6e5bf7309c","unresolved":false,"context_lines":[{"line_number":21,"context_line":"commands \u003d python setup.py test --slowest --testr-arg\u003d\u0027tempest\\.tests {posargs}\u0027"},{"line_number":22,"context_line":""},{"line_number":23,"context_line":"[testenv:py27]"},{"line_number":24,"context_line":"setenv \u003d PYTHONHASHSEED\u003d0"},{"line_number":25,"context_line":"commands \u003d python setup.py test --slowest --testr-arg\u003d\u0027tempest\\.tests {posargs}\u0027"},{"line_number":26,"context_line":""},{"line_number":27,"context_line":"[testenv:all]"}],"source_content_type":"text/x-properties","patch_set":1,"id":"1abeadc6_f9c5418e","line":24,"updated":"2014-08-21 14:30:23.000000000","message":"and here","commit_id":"7ee83215cb7881460308b13e4d7fb4509587ec78"},{"author":{"_account_id":5196,"name":"Matthew Treinish","email":"mtreinish@kortar.org","username":"treinish"},"change_message_id":"fa56f6722c6da83c1a43051e1e8c2c6e5bf7309c","unresolved":false,"context_lines":[{"line_number":97,"context_line":"   sh tools/pretty_tox_serial.sh \u0027(?!.*\\[.*\\bslow\\b.*\\])((smoke)|(^tempest\\.scenario)) {posargs}\u0027"},{"line_number":98,"context_line":""},{"line_number":99,"context_line":"[testenv:coverage]"},{"line_number":100,"context_line":"setenv \u003d PYTHONHASHSEED\u003d0"},{"line_number":101,"context_line":"sitepackages \u003d True"},{"line_number":102,"context_line":"commands \u003d"},{"line_number":103,"context_line":"   python -m tools/tempest_coverage -c start --combine"}],"source_content_type":"text/x-properties","patch_set":1,"id":"1abeadc6_f9b0e144","line":100,"updated":"2014-08-21 14:30:23.000000000","message":"So this line is not needed either. The coverage job for havana relied on the now non-existent nova coverage extension and was a full tempest run. The ENV line in testenv covers this job as well. By using setenv here you\u0027re overridding the other env used for the tempest jobs. (although I\u0027m not sure they\u0027re actually needed)\n\nOn the master branch the coverage job is a unit test coverage which is why we needed this line sperately for the same reason on the jobs above.","commit_id":"7ee83215cb7881460308b13e4d7fb4509587ec78"}]}