)]}'
{"test-requirements.txt":[{"author":{"_account_id":16168,"name":"Tomasz Trębski","email":"kornicameister@gmail.com","username":"trebskit"},"change_message_id":"aa2722f0e78c01dc544c4d85df51df34d6336075","unresolved":false,"context_lines":[{"line_number":1,"context_line":"coverage\u003e\u003d3.6 # Apache-2.0"},{"line_number":2,"context_line":"hacking\u003e\u003d0.10.2"},{"line_number":3,"context_line":"flake8\u003e\u003d2.2.4,\u003c2.6.0"},{"line_number":4,"context_line":"nose\u003d\u003d1.3.0"}],"source_content_type":"text/plain","patch_set":1,"id":"1a95cdbc_32e98d1b","line":1,"updated":"2016-10-05 11:04:41.000000000","message":"https://github.com/openstack/requirements/blob/master/upper-constraints.txt#L94.\n\nI think we should have it equal to the upper-constraints.","commit_id":"c3699b68e4fad56a80ca92fd9062e063dfd50411"},{"author":{"_account_id":13702,"name":"yushiro","username":"yushiro","inactive":true},"change_message_id":"4e5b8977c8a7c312c1e937a5debcca77751b223b","unresolved":false,"context_lines":[{"line_number":1,"context_line":"coverage\u003e\u003d3.6 # Apache-2.0"},{"line_number":2,"context_line":"hacking\u003e\u003d0.10.2"},{"line_number":3,"context_line":"flake8\u003e\u003d2.2.4,\u003c2.6.0"},{"line_number":4,"context_line":"nose\u003d\u003d1.3.0"}],"source_content_type":"text/plain","patch_set":1,"id":"da4df55a_c0a2ccad","line":1,"in_reply_to":"1a95cdbc_32e98d1b","updated":"2016-12-27 15:51:18.000000000","message":"Hi Tomasz,  you\u0027re right.  I\u0027ll update it.","commit_id":"c3699b68e4fad56a80ca92fd9062e063dfd50411"}],"tox.ini":[{"author":{"_account_id":16168,"name":"Tomasz Trębski","email":"kornicameister@gmail.com","username":"trebskit"},"change_message_id":"94a22eed70b3d9f128b8205500ee7171b3b108b7","unresolved":false,"context_lines":[{"line_number":19,"context_line":""},{"line_number":20,"context_line":"[testenv:cover]"},{"line_number":21,"context_line":"commands \u003d"},{"line_number":22,"context_line":"  nosetests -w tests/ -e tests_to_fix --with-coverage \\"},{"line_number":23,"context_line":"  --cover-package\u003dmonasca_agent --cover-erase"},{"line_number":24,"context_line":"  coverage html --include \u0027monasca_agent/*\u0027 --omit \u0027*/tests/*\u0027"},{"line_number":25,"context_line":""}],"source_content_type":"text/x-properties","patch_set":2,"id":"ba5201f7_c1527712","line":22,"updated":"2017-01-09 06:53:39.000000000","message":"In you other change you seemed to used testr to run tests, why not here as well ?","commit_id":"490ab3ebc0cb591b8ac117bb42c9c3d6e8228243"},{"author":{"_account_id":20033,"name":"Artur Basiak","email":"artur.basiak@ts.fujitsu.com","username":"abasiak"},"change_message_id":"532e2c8cf52f5a190ec95cc9f18e8198047a3d2a","unresolved":false,"context_lines":[{"line_number":19,"context_line":""},{"line_number":20,"context_line":"[testenv:cover]"},{"line_number":21,"context_line":"commands \u003d"},{"line_number":22,"context_line":"  nosetests -w tests/ -e tests_to_fix --with-coverage \\"},{"line_number":23,"context_line":"  --cover-package\u003dmonasca_agent --cover-erase"},{"line_number":24,"context_line":"  coverage html --include \u0027monasca_agent/*\u0027 --omit \u0027*/tests/*\u0027"},{"line_number":25,"context_line":""}],"source_content_type":"text/x-properties","patch_set":2,"id":"ba5201f7_626d1bcc","line":22,"range":{"start_line":22,"start_character":12,"end_line":22,"end_character":14},"updated":"2017-01-10 05:46:37.000000000","message":"There is no such option -w in nosetests. There was a change which removed that option from tox.ini:\nhttps://review.openstack.org/#/c/398344/","commit_id":"490ab3ebc0cb591b8ac117bb42c9c3d6e8228243"},{"author":{"_account_id":13702,"name":"yushiro","username":"yushiro","inactive":true},"change_message_id":"08bee5716c3c67bb7c37bb5cd40ec6a1c4b4787f","unresolved":false,"context_lines":[{"line_number":19,"context_line":""},{"line_number":20,"context_line":"[testenv:cover]"},{"line_number":21,"context_line":"commands \u003d"},{"line_number":22,"context_line":"  nosetests -w tests/ -e tests_to_fix --with-coverage \\"},{"line_number":23,"context_line":"  --cover-package\u003dmonasca_agent --cover-erase"},{"line_number":24,"context_line":"  coverage html --include \u0027monasca_agent/*\u0027 --omit \u0027*/tests/*\u0027"},{"line_number":25,"context_line":""}],"source_content_type":"text/x-properties","patch_set":2,"id":"ba5201f7_e75e7675","line":22,"in_reply_to":"ba5201f7_21f5ce82","updated":"2017-01-10 09:00:22.000000000","message":"Thanks for your link and invitation as a reviewer, Tomasz.","commit_id":"490ab3ebc0cb591b8ac117bb42c9c3d6e8228243"},{"author":{"_account_id":16168,"name":"Tomasz Trębski","email":"kornicameister@gmail.com","username":"trebskit"},"change_message_id":"b76d6e08c566dd5284132959435cedd3fc4c8f8d","unresolved":false,"context_lines":[{"line_number":19,"context_line":""},{"line_number":20,"context_line":"[testenv:cover]"},{"line_number":21,"context_line":"commands \u003d"},{"line_number":22,"context_line":"  nosetests -w tests/ -e tests_to_fix --with-coverage \\"},{"line_number":23,"context_line":"  --cover-package\u003dmonasca_agent --cover-erase"},{"line_number":24,"context_line":"  coverage html --include \u0027monasca_agent/*\u0027 --omit \u0027*/tests/*\u0027"},{"line_number":25,"context_line":""}],"source_content_type":"text/x-properties","patch_set":2,"id":"ba5201f7_21f5ce82","line":22,"in_reply_to":"ba5201f7_47e109d0","updated":"2017-01-10 08:21:41.000000000","message":"Here\u0027s the example where testr is used: https://review.openstack.org/#/c/358182/","commit_id":"490ab3ebc0cb591b8ac117bb42c9c3d6e8228243"},{"author":{"_account_id":13702,"name":"yushiro","username":"yushiro","inactive":true},"change_message_id":"08bee5716c3c67bb7c37bb5cd40ec6a1c4b4787f","unresolved":false,"context_lines":[{"line_number":19,"context_line":""},{"line_number":20,"context_line":"[testenv:cover]"},{"line_number":21,"context_line":"commands \u003d"},{"line_number":22,"context_line":"  nosetests -w tests/ -e tests_to_fix --with-coverage \\"},{"line_number":23,"context_line":"  --cover-package\u003dmonasca_agent --cover-erase"},{"line_number":24,"context_line":"  coverage html --include \u0027monasca_agent/*\u0027 --omit \u0027*/tests/*\u0027"},{"line_number":25,"context_line":""}],"source_content_type":"text/x-properties","patch_set":2,"id":"ba5201f7_276a3e1b","line":22,"range":{"start_line":22,"start_character":12,"end_line":22,"end_character":14},"in_reply_to":"ba5201f7_626d1bcc","updated":"2017-01-10 09:00:22.000000000","message":"Thanks for your information, Artur.","commit_id":"490ab3ebc0cb591b8ac117bb42c9c3d6e8228243"},{"author":{"_account_id":13702,"name":"yushiro","username":"yushiro","inactive":true},"change_message_id":"e95619fcd0918e96f5757c0343b64797f6c056d7","unresolved":false,"context_lines":[{"line_number":19,"context_line":""},{"line_number":20,"context_line":"[testenv:cover]"},{"line_number":21,"context_line":"commands \u003d"},{"line_number":22,"context_line":"  nosetests -w tests/ -e tests_to_fix --with-coverage \\"},{"line_number":23,"context_line":"  --cover-package\u003dmonasca_agent --cover-erase"},{"line_number":24,"context_line":"  coverage html --include \u0027monasca_agent/*\u0027 --omit \u0027*/tests/*\u0027"},{"line_number":25,"context_line":""}],"source_content_type":"text/x-properties","patch_set":2,"id":"ba5201f7_47e109d0","line":22,"in_reply_to":"ba5201f7_c1527712","updated":"2017-01-10 05:19:13.000000000","message":"Hi Tomasz, I wanted to give comments from monasca members whether this command can change from \u0027nosetests\u0027 to \u0027testr\u0027.  I\u0027d like to align with \u0027testr\u0027 for testing.  I\u0027ll align \u0027testr\u0027 for next PS.","commit_id":"490ab3ebc0cb591b8ac117bb42c9c3d6e8228243"}]}