)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"a160c00c1dd8e66b7a704b54569bb3b6d6a85cda","unresolved":false,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"add lower-constraints job"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Create a tox environment for running the unit tests against the lower"},{"line_number":10,"context_line":"bounds of the dependencies."},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"Create a lower-constraints.txt to be used to enforce the lower bounds"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"bf659307_c67540ce","line":9,"updated":"2018-04-06 15:18:10.000000000","message":"This isn\u0027t happening:\n\nhttp://logs.openstack.org/61/555961/3/check/openstack-tox-lower-constraints/91b6d17/job-output.txt.gz#_2018-04-06_13_19_18_865706\n\nNote the job completes in \u003c5 minutes.","commit_id":"06000e4bd84d36c10fcfd848114cbcb84bf52a3c"},{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"ef175ac4a38c05f3e552f8df39f679da84f0ffb2","unresolved":false,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"add lower-constraints job"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Create a tox environment for running the unit tests against the lower"},{"line_number":10,"context_line":"bounds of the dependencies."},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"Create a lower-constraints.txt to be used to enforce the lower bounds"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"bf659307_e984d7b3","line":9,"in_reply_to":"bf659307_c67540ce","updated":"2018-04-06 15:49:35.000000000","message":"Damn, I thought the idea was to simply attempt to install the packages side-by-side and see if anything crapped out. Missed this section.","commit_id":"06000e4bd84d36c10fcfd848114cbcb84bf52a3c"}],"lower-constraints.txt":[{"author":{"_account_id":14070,"name":"Eric Fried","email":"openstack@fried.cc","username":"efried"},"change_message_id":"6571991cc9ffe4be704aadc9e9a170e015618484","unresolved":false,"context_lines":[{"line_number":5,"context_line":"attrs\u003d\u003d17.4.0"},{"line_number":6,"context_line":"automaton\u003d\u003d1.14.0"},{"line_number":7,"context_line":"Babel\u003d\u003d2.3.4"},{"line_number":8,"context_line":"enum34\u003d\u003d1.0.4"},{"line_number":9,"context_line":"bandit\u003d\u003d1.1.0"},{"line_number":10,"context_line":"bcrypt\u003d\u003d3.1.4"},{"line_number":11,"context_line":"cachetools\u003d\u003d2.0.1"}],"source_content_type":"text/plain","patch_set":3,"id":"bf659307_25abd575","line":8,"updated":"2018-04-06 13:17:56.000000000","message":"NOTE: I added this based on the message:\n\nPackage u\u0027enum34\u0027 is used in requirements.txt but not in lower-constraints.txt\n\nI had to make a best guess as to what to put here.  The requirements.txt entry is:\n\nenum34\u003e\u003d1.0.4;python_version\u003d\u003d\u00272.7\u0027 or python_version\u003d\u003d\u00272.6\u0027 or python_version\u003d\u003d\u00273.3\u0027 # BSD\n\n...and the pattern seems to be \u003d\u003d[the lowest version possible within that constraint]","commit_id":"06000e4bd84d36c10fcfd848114cbcb84bf52a3c"},{"author":{"_account_id":2472,"name":"Doug Hellmann","email":"dhellmann@redhat.com","username":"doug-hellmann"},"change_message_id":"e5ef2e17203e47a41f9235ea9085c3e4ff5bc569","unresolved":false,"context_lines":[{"line_number":5,"context_line":"attrs\u003d\u003d17.4.0"},{"line_number":6,"context_line":"automaton\u003d\u003d1.14.0"},{"line_number":7,"context_line":"Babel\u003d\u003d2.3.4"},{"line_number":8,"context_line":"enum34\u003d\u003d1.0.4"},{"line_number":9,"context_line":"bandit\u003d\u003d1.1.0"},{"line_number":10,"context_line":"bcrypt\u003d\u003d3.1.4"},{"line_number":11,"context_line":"cachetools\u003d\u003d2.0.1"}],"source_content_type":"text/plain","patch_set":3,"id":"bf659307_46eff086","line":8,"in_reply_to":"bf659307_25abd575","updated":"2018-04-06 15:15:36.000000000","message":"This should work. You could also have a different constraint for different versions if you need that but it doesn\u0027t sound like you do.","commit_id":"06000e4bd84d36c10fcfd848114cbcb84bf52a3c"},{"author":{"_account_id":14070,"name":"Eric Fried","email":"openstack@fried.cc","username":"efried"},"change_message_id":"6571991cc9ffe4be704aadc9e9a170e015618484","unresolved":false,"context_lines":[{"line_number":150,"context_line":"statsd\u003d\u003d3.2.2"},{"line_number":151,"context_line":"stestr\u003d\u003d1.0.0"},{"line_number":152,"context_line":"stevedore\u003d\u003d1.20.0"},{"line_number":153,"context_line":"setuptools\u003d\u003d21.0.0"},{"line_number":154,"context_line":"suds-jurko\u003d\u003d0.6"},{"line_number":155,"context_line":"taskflow\u003d\u003d2.16.0"},{"line_number":156,"context_line":"Tempita\u003d\u003d0.5.2"}],"source_content_type":"text/plain","patch_set":3,"id":"bf659307_c5b539cf","line":153,"updated":"2018-04-06 13:17:56.000000000","message":"NOTE: I added this based on the message:\n\nPackage u\u0027setuptools\u0027 is used in requirements.txt but not in lower-constraints.txt\n\nI had to make a best guess as to what to put here.  The requirements.txt entry is:\n\nsetuptools!\u003d24.0.0,!\u003d34.0.0,!\u003d34.0.1,!\u003d34.0.2,!\u003d34.0.3,!\u003d34.1.0,!\u003d34.1.1,!\u003d34.2.0,!\u003d34.3.0,!\u003d34.3.1,!\u003d34.3.2,!\u003d36.2.0,\u003e\u003d21.0.0 # PSF/ZPL\n\n...and the pattern seems to be \u003d\u003d[the lowest version possible within that constraint]","commit_id":"06000e4bd84d36c10fcfd848114cbcb84bf52a3c"},{"author":{"_account_id":14070,"name":"Eric Fried","email":"openstack@fried.cc","username":"efried"},"change_message_id":"1c3e5fac83b590c86a23d829072efdd2fc1891ea","unresolved":false,"context_lines":[{"line_number":119,"context_line":"pyOpenSSL\u003d\u003d17.5.0"},{"line_number":120,"context_line":"pyparsing\u003d\u003d2.2.0"},{"line_number":121,"context_line":"pyperclip\u003d\u003d1.6.0"},{"line_number":122,"context_line":"pypowervm\u003d\u003d1.1.12"},{"line_number":123,"context_line":"pyroute2\u003d\u003d0.4.21"},{"line_number":124,"context_line":"pytest\u003d\u003d3.4.2"},{"line_number":125,"context_line":"python-barbicanclient\u003d\u003d4.5.2"}],"source_content_type":"text/plain","patch_set":6,"id":"9f6a8fd7_3e2244ba","line":122,"range":{"start_line":122,"start_character":0,"end_line":122,"end_character":17},"updated":"2018-04-25 16:41:35.000000000","message":"This needs to be 1.1.15 after https://review.openstack.org/#/q/topic:pypowervm-1.1.15","commit_id":"c6b123c99f9101a75795b97146e768e40a7adf76"}],"tox.ini":[{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"a160c00c1dd8e66b7a704b54569bb3b6d6a85cda","unresolved":false,"context_lines":[{"line_number":240,"context_line":"commands \u003d bindep test"},{"line_number":241,"context_line":"usedevelop \u003d False"},{"line_number":242,"context_line":""},{"line_number":243,"context_line":"[testenv:lower-constraints]"},{"line_number":244,"context_line":"basepython \u003d python3"},{"line_number":245,"context_line":"deps \u003d"},{"line_number":246,"context_line":"  -c{toxinidir}/lower-constraints.txt"}],"source_content_type":"text/x-properties","patch_set":3,"id":"bf659307_a6a7ec4f","line":243,"updated":"2018-04-06 15:18:10.000000000","message":"Is this supposed to actually run tests? Because it\u0027s not:\n\nhttp://logs.openstack.org/61/555961/3/check/openstack-tox-lower-constraints/91b6d17/job-output.txt.gz#_2018-04-06_13_19_18_865706","commit_id":"06000e4bd84d36c10fcfd848114cbcb84bf52a3c"},{"author":{"_account_id":10343,"name":"Jim Rollenhagen","email":"jim@jimrollenhagen.com","username":"jimrollenhagen"},"change_message_id":"7b9349ed2016dfa66d681135098cbaafe006e6cf","unresolved":false,"context_lines":[{"line_number":240,"context_line":"commands \u003d bindep test"},{"line_number":241,"context_line":"usedevelop \u003d False"},{"line_number":242,"context_line":""},{"line_number":243,"context_line":"[testenv:lower-constraints]"},{"line_number":244,"context_line":"basepython \u003d python3"},{"line_number":245,"context_line":"deps \u003d"},{"line_number":246,"context_line":"  -c{toxinidir}/lower-constraints.txt"}],"source_content_type":"text/x-properties","patch_set":3,"id":"bf659307_e62084b3","line":243,"in_reply_to":"bf659307_a6a7ec4f","updated":"2018-04-06 15:21:55.000000000","message":"Needs the test commands similar to the py27 or py35 jobs:\n\n    commands \u003d\n        {[testenv]commands}\n        stestr run --blacklist-file\u003dtests-py3.txt {posargs}\n        env TEST_OSPROFILER\u003d1 stestr run --combine --no-discover \u0027nova.tests.unit.test_profiler\u0027","commit_id":"06000e4bd84d36c10fcfd848114cbcb84bf52a3c"},{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"3f722374e7c4477aa978680ec646577b3cc4c3ed","unresolved":false,"context_lines":[{"line_number":249,"context_line":"commands \u003d"},{"line_number":250,"context_line":"    {[testenv]commands}"},{"line_number":251,"context_line":"    stestr run --blacklist-file\u003dtests-py3.txt {posargs}"},{"line_number":252,"context_line":"    env TEST_OSPROFILER\u003d1 stestr run --combine --no-discover \u0027nova.tests.unit.test_profiler\u0027"}],"source_content_type":"text/x-properties","patch_set":4,"id":"bf659307_df7b4189","line":252,"range":{"start_line":252,"start_character":0,"end_line":252,"end_character":92},"updated":"2018-04-12 11:24:06.000000000","message":"nit: I very much doubt we need to run osprofiler for this but eh","commit_id":"281c6c983f4c88306d98be7d9cdafcc0af57d7bc"},{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"a8d1864b113d130853c49ab28fb1a7f446ac4e7f","unresolved":false,"context_lines":[{"line_number":247,"context_line":"commands \u003d"},{"line_number":248,"context_line":"    {[testenv]commands}"},{"line_number":249,"context_line":"    stestr run --blacklist-file\u003dtests-py3.txt {posargs}"},{"line_number":250,"context_line":"    env TEST_OSPROFILER\u003d1 stestr run --combine --no-discover \u0027nova.tests.unit.test_profiler\u0027"}],"source_content_type":"text/x-properties","patch_set":7,"id":"5f7c97a3_c20b5827","line":250,"updated":"2018-05-14 16:46:54.000000000","message":"Agree with Stephen\u0027s earlier remark that we don\u0027t need to run osprofiler checks with this, we already do that in the unit test runs above, and they are basically for osprofiler coverage and don\u0027t really have anything to do with bounds checking on dependencies.","commit_id":"cc7fbec71fe5f8d028ebf15c0c0f1ee3c4e2572d"}]}