)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"83299724be9e4e66f9b109cdab66292529812fea","unresolved":false,"context_lines":[{"line_number":5,"context_line":"CommitDate: 2019-10-17 15:28:14 +0200"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Make sure tox install requirements.txt with upper-constraints"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Change-Id: Iba797243d2a137b551223165a1af1a8676bcea02"},{"line_number":10,"context_line":"Closes-Bug: #1848499"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"3fa7e38b_04e7f6ae","line":8,"updated":"2019-10-17 13:36:44.000000000","message":"Could we have more details here? I think what happened is Ieb4ab13cf8ca5683fcd7b18ed669e8a26659bff1 means upper-constraints were no longer applied to requirements.txt which means we pull in networkx uncapped and v2.4 of that library breaks taskflow which we depend on for the powervm driver. This fixes the issue by explicitly including requirements.txt in deps so upper-constraints will be applied on it and its transitive dependencies.","commit_id":"e972f4c41faf6ffa3cf70df50eebad998eebf17c"},{"author":{"_account_id":9708,"name":"Balazs Gibizer","display_name":"gibi","email":"gibizer@gmail.com","username":"gibi"},"change_message_id":"5b042587b88ae9d6bafe6514b42d5f970d000799","unresolved":false,"context_lines":[{"line_number":5,"context_line":"CommitDate: 2019-10-17 15:28:14 +0200"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Make sure tox install requirements.txt with upper-constraints"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Change-Id: Iba797243d2a137b551223165a1af1a8676bcea02"},{"line_number":10,"context_line":"Closes-Bug: #1848499"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"3fa7e38b_67302ca0","line":8,"in_reply_to":"3fa7e38b_04e7f6ae","updated":"2019-10-17 14:03:54.000000000","message":"Done","commit_id":"e972f4c41faf6ffa3cf70df50eebad998eebf17c"}],"tox.ini":[{"author":{"_account_id":11904,"name":"Sean McGinnis","email":"sean.mcginnis@gmail.com","username":"SeanM"},"change_message_id":"d83397321da3800cfa1b37cb53c9271bc3324786","unresolved":false,"context_lines":[{"line_number":6,"context_line":"# ignore_basepython_conflict."},{"line_number":7,"context_line":"ignore_basepython_conflict \u003d True"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"[testenv]"},{"line_number":10,"context_line":"basepython \u003d python3"},{"line_number":11,"context_line":"usedevelop \u003d True"},{"line_number":12,"context_line":"whitelist_externals \u003d"}],"source_content_type":"text/x-properties","patch_set":1,"id":"3fa7e38b_243d52dd","line":9,"updated":"2019-10-17 13:42:16.000000000","message":"Would be good to add back in something like:\n\ninstall_command \u003d pip install {opts} {packages}\n\nThe key part is keeping the -c and -r bits separate in deps that can be overridden in different targets depending on the need. So setting the deps below here in the [testenv] section sets a default for all jobs to use, then in jobs like [testenv:lower-constraints] and [testenv:docs] you can override to different dependencies.","commit_id":"e972f4c41faf6ffa3cf70df50eebad998eebf17c"},{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"ad62bebf6364a7688874fa97b0f711f0e29193d5","unresolved":false,"context_lines":[{"line_number":6,"context_line":"# ignore_basepython_conflict."},{"line_number":7,"context_line":"ignore_basepython_conflict \u003d True"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"[testenv]"},{"line_number":10,"context_line":"basepython \u003d python3"},{"line_number":11,"context_line":"usedevelop \u003d True"},{"line_number":12,"context_line":"whitelist_externals \u003d"}],"source_content_type":"text/x-properties","patch_set":1,"id":"3fa7e38b_071ff85b","line":9,"in_reply_to":"3fa7e38b_243d52dd","updated":"2019-10-17 13:45:56.000000000","message":"That shouldn\u0027t matter since it\u0027s the default in tox [1] (well, essentially)\n\n[1] https://github.com/tox-dev/tox/blob/1883946bd3f233f58928864700eb0e6007332ffd/src/tox/config/__init__.py#L232","commit_id":"e972f4c41faf6ffa3cf70df50eebad998eebf17c"},{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"c2bf403c19ec3959a172d868d095b1910db8e566","unresolved":false,"context_lines":[{"line_number":153,"context_line":"[testenv:venv]"},{"line_number":154,"context_line":"deps \u003d"},{"line_number":155,"context_line":"  {[testenv]deps}"},{"line_number":156,"context_line":"  -r{toxinidir}/doc/requirements.txt"},{"line_number":157,"context_line":"commands \u003d"},{"line_number":158,"context_line":"  {posargs}"},{"line_number":159,"context_line":""}],"source_content_type":"text/x-properties","patch_set":1,"id":"3fa7e38b_44edee8c","line":156,"updated":"2019-10-17 13:36:36.000000000","message":"You need it here also (also, test requirements?)","commit_id":"e972f4c41faf6ffa3cf70df50eebad998eebf17c"},{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"88e3f3b699fa91313fc76846fe6c8eb3754d5aa3","unresolved":false,"context_lines":[{"line_number":153,"context_line":"[testenv:venv]"},{"line_number":154,"context_line":"deps \u003d"},{"line_number":155,"context_line":"  {[testenv]deps}"},{"line_number":156,"context_line":"  -r{toxinidir}/doc/requirements.txt"},{"line_number":157,"context_line":"commands \u003d"},{"line_number":158,"context_line":"  {posargs}"},{"line_number":159,"context_line":""}],"source_content_type":"text/x-properties","patch_set":1,"id":"3fa7e38b_c400fe38","line":156,"in_reply_to":"3fa7e38b_44edee8c","updated":"2019-10-17 13:37:49.000000000","message":"Why, we use {[testenv]deps} here.","commit_id":"e972f4c41faf6ffa3cf70df50eebad998eebf17c"},{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"2a65acfc71f81a0af4a9f0517ad0ffc2cbc1d93d","unresolved":false,"context_lines":[{"line_number":153,"context_line":"[testenv:venv]"},{"line_number":154,"context_line":"deps \u003d"},{"line_number":155,"context_line":"  {[testenv]deps}"},{"line_number":156,"context_line":"  -r{toxinidir}/doc/requirements.txt"},{"line_number":157,"context_line":"commands \u003d"},{"line_number":158,"context_line":"  {posargs}"},{"line_number":159,"context_line":""}],"source_content_type":"text/x-properties","patch_set":1,"id":"3fa7e38b_04b5b67e","line":156,"in_reply_to":"3fa7e38b_c400fe38","updated":"2019-10-17 13:38:42.000000000","message":"Correct. Ignore me here","commit_id":"e972f4c41faf6ffa3cf70df50eebad998eebf17c"},{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"88e3f3b699fa91313fc76846fe6c8eb3754d5aa3","unresolved":false,"context_lines":[{"line_number":161,"context_line":"description \u003d"},{"line_number":162,"context_line":"  Build main documentation."},{"line_number":163,"context_line":"deps \u003d"},{"line_number":164,"context_line":"  -c{env:UPPER_CONSTRAINTS_FILE:https://releases.openstack.org/constraints/upper/master}"},{"line_number":165,"context_line":"  -r{toxinidir}/doc/requirements.txt"},{"line_number":166,"context_line":"commands \u003d"},{"line_number":167,"context_line":"  rm -rf doc/build/html doc/build/doctrees"}],"source_content_type":"text/x-properties","patch_set":1,"id":"3fa7e38b_6497caed","line":164,"range":{"start_line":164,"start_character":2,"end_line":164,"end_character":88},"updated":"2019-10-17 13:37:49.000000000","message":"Why not just replace this with {[testenv]deps}?","commit_id":"e972f4c41faf6ffa3cf70df50eebad998eebf17c"},{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"2a65acfc71f81a0af4a9f0517ad0ffc2cbc1d93d","unresolved":false,"context_lines":[{"line_number":161,"context_line":"description \u003d"},{"line_number":162,"context_line":"  Build main documentation."},{"line_number":163,"context_line":"deps \u003d"},{"line_number":164,"context_line":"  -c{env:UPPER_CONSTRAINTS_FILE:https://releases.openstack.org/constraints/upper/master}"},{"line_number":165,"context_line":"  -r{toxinidir}/doc/requirements.txt"},{"line_number":166,"context_line":"commands \u003d"},{"line_number":167,"context_line":"  rm -rf doc/build/html doc/build/doctrees"}],"source_content_type":"text/x-properties","patch_set":1,"id":"3fa7e38b_e4abba1b","line":164,"range":{"start_line":164,"start_character":2,"end_line":164,"end_character":88},"in_reply_to":"3fa7e38b_6497caed","updated":"2019-10-17 13:38:42.000000000","message":"+1","commit_id":"e972f4c41faf6ffa3cf70df50eebad998eebf17c"},{"author":{"_account_id":9708,"name":"Balazs Gibizer","display_name":"gibi","email":"gibizer@gmail.com","username":"gibi"},"change_message_id":"5b042587b88ae9d6bafe6514b42d5f970d000799","unresolved":false,"context_lines":[{"line_number":161,"context_line":"description \u003d"},{"line_number":162,"context_line":"  Build main documentation."},{"line_number":163,"context_line":"deps \u003d"},{"line_number":164,"context_line":"  -c{env:UPPER_CONSTRAINTS_FILE:https://releases.openstack.org/constraints/upper/master}"},{"line_number":165,"context_line":"  -r{toxinidir}/doc/requirements.txt"},{"line_number":166,"context_line":"commands \u003d"},{"line_number":167,"context_line":"  rm -rf doc/build/html doc/build/doctrees"}],"source_content_type":"text/x-properties","patch_set":1,"id":"3fa7e38b_8747684d","line":164,"range":{"start_line":164,"start_character":2,"end_line":164,"end_character":88},"in_reply_to":"3fa7e38b_6497caed","updated":"2019-10-17 14:03:54.000000000","message":"Done","commit_id":"e972f4c41faf6ffa3cf70df50eebad998eebf17c"},{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"c2bf403c19ec3959a172d868d095b1910db8e566","unresolved":false,"context_lines":[{"line_number":162,"context_line":"  Build main documentation."},{"line_number":163,"context_line":"deps \u003d"},{"line_number":164,"context_line":"  -c{env:UPPER_CONSTRAINTS_FILE:https://releases.openstack.org/constraints/upper/master}"},{"line_number":165,"context_line":"  -r{toxinidir}/doc/requirements.txt"},{"line_number":166,"context_line":"commands \u003d"},{"line_number":167,"context_line":"  rm -rf doc/build/html doc/build/doctrees"},{"line_number":168,"context_line":"  # Check that all JSON files don\u0027t have \\r\\n in line."}],"source_content_type":"text/x-properties","patch_set":1,"id":"3fa7e38b_04ce162d","line":165,"updated":"2019-10-17 13:36:36.000000000","message":"You need to provide it here too, because we install nova as part of our documentation (for apidoc stuff)","commit_id":"e972f4c41faf6ffa3cf70df50eebad998eebf17c"},{"author":{"_account_id":9708,"name":"Balazs Gibizer","display_name":"gibi","email":"gibizer@gmail.com","username":"gibi"},"change_message_id":"5b042587b88ae9d6bafe6514b42d5f970d000799","unresolved":false,"context_lines":[{"line_number":162,"context_line":"  Build main documentation."},{"line_number":163,"context_line":"deps \u003d"},{"line_number":164,"context_line":"  -c{env:UPPER_CONSTRAINTS_FILE:https://releases.openstack.org/constraints/upper/master}"},{"line_number":165,"context_line":"  -r{toxinidir}/doc/requirements.txt"},{"line_number":166,"context_line":"commands \u003d"},{"line_number":167,"context_line":"  rm -rf doc/build/html doc/build/doctrees"},{"line_number":168,"context_line":"  # Check that all JSON files don\u0027t have \\r\\n in line."}],"source_content_type":"text/x-properties","patch_set":1,"id":"3fa7e38b_67782c8f","line":165,"in_reply_to":"3fa7e38b_04ce162d","updated":"2019-10-17 14:03:54.000000000","message":"Adding {[testenv]deps} solve this too","commit_id":"e972f4c41faf6ffa3cf70df50eebad998eebf17c"},{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"10cc25bece89c674b0b19dfa3c7a88f0cd1b2f5f","unresolved":false,"context_lines":[{"line_number":27,"context_line":"  PYTHONWARNINGS \u003d ignore::UserWarning:psycopg2"},{"line_number":28,"context_line":"deps \u003d"},{"line_number":29,"context_line":"  -c{env:UPPER_CONSTRAINTS_FILE:https://releases.openstack.org/constraints/upper/master}"},{"line_number":30,"context_line":"  -r{toxinidir}/requirements.txt"},{"line_number":31,"context_line":"  -r{toxinidir}/test-requirements.txt"},{"line_number":32,"context_line":"passenv \u003d"},{"line_number":33,"context_line":"  OS_DEBUG GENERATE_HASHES"}],"source_content_type":"text/x-properties","patch_set":2,"id":"3fa7e38b_883857d3","line":30,"updated":"2019-10-25 16:25:02.000000000","message":"This...","commit_id":"be09b73796b2aa22d0888d98eb29315f2f4786ab"},{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"10cc25bece89c674b0b19dfa3c7a88f0cd1b2f5f","unresolved":false,"context_lines":[{"line_number":161,"context_line":"description \u003d"},{"line_number":162,"context_line":"  Build main documentation."},{"line_number":163,"context_line":"deps \u003d"},{"line_number":164,"context_line":"  {[testenv]deps}"},{"line_number":165,"context_line":"  -r{toxinidir}/doc/requirements.txt"},{"line_number":166,"context_line":"commands \u003d"},{"line_number":167,"context_line":"  rm -rf doc/build/html doc/build/doctrees"}],"source_content_type":"text/x-properties","patch_set":2,"id":"3fa7e38b_e84c6b77","line":164,"updated":"2019-10-25 16:25:02.000000000","message":"is wrong here because we don\u0027t want to install requirements.txt in the docs job because it means we\u0027re trying to install things like psycopg2 which is failing:\n\nhttps://zuul.opendev.org/t/openstack/build/10344924bca44849b06b8fae29b5494f/log/job-output.txt#652\n\n2019-10-25 13:58:32.599585 | ubuntu-bionic |   Downloading http://mirror.gra1.ovh.openstack.org/pypifiles/packages/12/67/5d953cb20497d4f56965bc5bcf03134244be7bae4eb2b1f7ca5cf31b245f/os_xenapi-0.3.4-py2.py3-none-any.whl (137kB)\n\n2019-10-25 13:58:32.599697 | ubuntu-bionic | Collecting psycopg2\u003d\u003d\u003d2.8.4 (from -c /home/zuul/src/opendev.org/openstack/requirements/upper-constraints.txt (line 97))\n\n2019-10-25 13:58:32.599851 | ubuntu-bionic |   Downloading http://mirror.gra1.ovh.openstack.org/pypifiles/packages/84/d7/6a93c99b5ba4d4d22daa3928b983cec66df4536ca50b22ce5dcac65e4e71/psycopg2-2.8.4.tar.gz (377kB)\n\n2019-10-25 13:58:32.599905 | ubuntu-bionic |     ERROR: Command errored out with exit status 1:\n\n2019-10-25 13:58:32.600316 | ubuntu-bionic |      command: /home/zuul/src/opendev.org/openstack/nova/.tox/docs/bin/python -c \u0027import sys, setuptools, tokenize; sys.argv[0] \u003d \u0027\"\u0027\"\u0027/tmp/pip-install-ki5_j1sg/psycopg2/setup.py\u0027\"\u0027\"\u0027; __file__\u003d\u0027\"\u0027\"\u0027/tmp/pip-install-ki5_j1sg/psycopg2/setup.py\u0027\"\u0027\"\u0027;f\u003dgetattr(tokenize, \u0027\"\u0027\"\u0027open\u0027\"\u0027\"\u0027, open)(__file__);code\u003df.read().replace(\u0027\"\u0027\"\u0027\\r\\n\u0027\"\u0027\"\u0027, \u0027\"\u0027\"\u0027\\n\u0027\"\u0027\"\u0027);f.close();exec(compile(code, __file__, \u0027\"\u0027\"\u0027exec\u0027\"\u0027\"\u0027))\u0027 egg_info --egg-base pip-egg-info\n\n2019-10-25 13:58:32.600384 | ubuntu-bionic |          cwd: /tmp/pip-install-ki5_j1sg/psycopg2/\n\n2019-10-25 13:58:32.600421 | ubuntu-bionic |     Complete output (7 lines):\n\n2019-10-25 13:58:32.600449 | ubuntu-bionic |     running egg_info\n\n2019-10-25 13:58:32.600496 | ubuntu-bionic |     creating pip-egg-info/psycopg2.egg-info\n\n2019-10-25 13:58:32.600551 | ubuntu-bionic |     writing pip-egg-info/psycopg2.egg-info/PKG-INFO\n\n2019-10-25 13:58:32.600631 | ubuntu-bionic |     writing dependency_links to pip-egg-info/psycopg2.egg-info/dependency_links.txt\n\n2019-10-25 13:58:32.600706 | ubuntu-bionic |     writing top-level names to pip-egg-info/psycopg2.egg-info/top_level.txt\n\n2019-10-25 13:58:32.600780 | ubuntu-bionic |     writing manifest file \u0027pip-egg-info/psycopg2.egg-info/SOURCES.txt\u0027\n\n2019-10-25 13:58:32.600936 | ubuntu-bionic |     Error: b\u0027You need to install postgresql-server-dev-X.Y for building a server-side extension or libpq-dev for building a client-side application.\\n\u0027","commit_id":"be09b73796b2aa22d0888d98eb29315f2f4786ab"}]}