)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":5263,"name":"Jeremy Stanley","display_name":"fungi","email":"fungi@yuggoth.org","username":"fungi","status":"missing, presumed fed"},"change_message_id":"c53a2c2634e0768bd9669453f868e4434149a4b0","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"29017e50_63cb4896","updated":"2023-01-17 17:33:31.000000000","message":"My main concern with this approach is that to run the lower-constraints, py27 or py36 testenvs, you\u0027d need to export something like TOX_CONSTRAINTS_FILE\u003d/dev/null in order to not end up with conflicting master branch constraints applied.","commit_id":"b19a9fa4298f54867b1074804ee13441ec3604d7"}],"tox.ini":[{"author":{"_account_id":15343,"name":"Tim Burke","email":"tburke@nvidia.com","username":"tburke"},"change_message_id":"61a9a9a9275a78d1b8401597b6a68b02de0667e7","unresolved":true,"context_lines":[{"line_number":7,"context_line":""},{"line_number":8,"context_line":"[testenv]"},{"line_number":9,"context_line":"usedevelop \u003d True"},{"line_number":10,"context_line":"install_command \u003d pip install -U {opts} {packages} -c{env:TOX_CONSTRAINTS_FILE:https://releases.openstack.org/constraints/upper/master}"},{"line_number":11,"context_line":"setenv \u003d VIRTUAL_ENV\u003d{envdir}"},{"line_number":12,"context_line":"deps \u003d"},{"line_number":13,"context_line":"  -c{env:TOX_CONSTRAINTS_FILE:https://releases.openstack.org/constraints/upper/master}"}],"source_content_type":"text/x-properties","patch_set":1,"id":"136d8866_d01c30c8","line":10,"updated":"2023-01-17 17:16:39.000000000","message":"Is this going to cause issues with any of the other tox jobs that specify constraints files?\n\nIs the issue maybe the -U in install_command? I remember running into similar issues with the lower-constraints job in https://review.opendev.org/c/openstack/swift/+/868732 ...","commit_id":"b19a9fa4298f54867b1074804ee13441ec3604d7"},{"author":{"_account_id":5263,"name":"Jeremy Stanley","display_name":"fungi","email":"fungi@yuggoth.org","username":"fungi","status":"missing, presumed fed"},"change_message_id":"1ad7000636f2ea58bdeafec410c6e720b38280f2","unresolved":true,"context_lines":[{"line_number":7,"context_line":""},{"line_number":8,"context_line":"[testenv]"},{"line_number":9,"context_line":"usedevelop \u003d True"},{"line_number":10,"context_line":"install_command \u003d pip install -U {opts} {packages} -c{env:TOX_CONSTRAINTS_FILE:https://releases.openstack.org/constraints/upper/master}"},{"line_number":11,"context_line":"setenv \u003d VIRTUAL_ENV\u003d{envdir}"},{"line_number":12,"context_line":"deps \u003d"},{"line_number":13,"context_line":"  -c{env:TOX_CONSTRAINTS_FILE:https://releases.openstack.org/constraints/upper/master}"}],"source_content_type":"text/x-properties","patch_set":1,"id":"e545678f_296930f9","line":10,"in_reply_to":"136d8866_d01c30c8","updated":"2023-01-17 17:28:20.000000000","message":"Seems like it should be, but dropping the -U from install_command doesn\u0027t seem to fix the problem for me (in local testing at least).","commit_id":"b19a9fa4298f54867b1074804ee13441ec3604d7"},{"author":{"_account_id":5263,"name":"Jeremy Stanley","display_name":"fungi","email":"fungi@yuggoth.org","username":"fungi","status":"missing, presumed fed"},"change_message_id":"c53a2c2634e0768bd9669453f868e4434149a4b0","unresolved":false,"context_lines":[{"line_number":7,"context_line":""},{"line_number":8,"context_line":"[testenv]"},{"line_number":9,"context_line":"usedevelop \u003d True"},{"line_number":10,"context_line":"install_command \u003d pip install -U {opts} {packages} -c{env:TOX_CONSTRAINTS_FILE:https://releases.openstack.org/constraints/upper/master}"},{"line_number":11,"context_line":"setenv \u003d VIRTUAL_ENV\u003d{envdir}"},{"line_number":12,"context_line":"deps \u003d"},{"line_number":13,"context_line":"  -c{env:TOX_CONSTRAINTS_FILE:https://releases.openstack.org/constraints/upper/master}"}],"source_content_type":"text/x-properties","patch_set":1,"id":"e7ebd524_36439b31","line":10,"in_reply_to":"e545678f_296930f9","updated":"2023-01-17 17:33:31.000000000","message":"Done","commit_id":"b19a9fa4298f54867b1074804ee13441ec3604d7"},{"author":{"_account_id":15343,"name":"Tim Burke","email":"tburke@nvidia.com","username":"tburke"},"change_message_id":"61a9a9a9275a78d1b8401597b6a68b02de0667e7","unresolved":true,"context_lines":[{"line_number":10,"context_line":"install_command \u003d pip install -U {opts} {packages} -c{env:TOX_CONSTRAINTS_FILE:https://releases.openstack.org/constraints/upper/master}"},{"line_number":11,"context_line":"setenv \u003d VIRTUAL_ENV\u003d{envdir}"},{"line_number":12,"context_line":"deps \u003d"},{"line_number":13,"context_line":"  -c{env:TOX_CONSTRAINTS_FILE:https://releases.openstack.org/constraints/upper/master}"},{"line_number":14,"context_line":"  -r{toxinidir}/requirements.txt"},{"line_number":15,"context_line":"  -r{toxinidir}/test-requirements.txt"},{"line_number":16,"context_line":"commands \u003d"}],"source_content_type":"text/x-properties","patch_set":1,"id":"39860ae6_9a18fa58","line":13,"updated":"2023-01-17 17:16:39.000000000","message":"Do we still need this part?","commit_id":"b19a9fa4298f54867b1074804ee13441ec3604d7"},{"author":{"_account_id":5263,"name":"Jeremy Stanley","display_name":"fungi","email":"fungi@yuggoth.org","username":"fungi","status":"missing, presumed fed"},"change_message_id":"1ad7000636f2ea58bdeafec410c6e720b38280f2","unresolved":true,"context_lines":[{"line_number":10,"context_line":"install_command \u003d pip install -U {opts} {packages} -c{env:TOX_CONSTRAINTS_FILE:https://releases.openstack.org/constraints/upper/master}"},{"line_number":11,"context_line":"setenv \u003d VIRTUAL_ENV\u003d{envdir}"},{"line_number":12,"context_line":"deps \u003d"},{"line_number":13,"context_line":"  -c{env:TOX_CONSTRAINTS_FILE:https://releases.openstack.org/constraints/upper/master}"},{"line_number":14,"context_line":"  -r{toxinidir}/requirements.txt"},{"line_number":15,"context_line":"  -r{toxinidir}/test-requirements.txt"},{"line_number":16,"context_line":"commands \u003d"}],"source_content_type":"text/x-properties","patch_set":1,"id":"98a7bffb_e98b6d8e","line":13,"in_reply_to":"39860ae6_9a18fa58","updated":"2023-01-17 17:28:20.000000000","message":"It\u0027s redundant, I agree. Other projects which add it in install_command don\u0027t include it in deps.","commit_id":"b19a9fa4298f54867b1074804ee13441ec3604d7"},{"author":{"_account_id":5263,"name":"Jeremy Stanley","display_name":"fungi","email":"fungi@yuggoth.org","username":"fungi","status":"missing, presumed fed"},"change_message_id":"c53a2c2634e0768bd9669453f868e4434149a4b0","unresolved":false,"context_lines":[{"line_number":10,"context_line":"install_command \u003d pip install -U {opts} {packages} -c{env:TOX_CONSTRAINTS_FILE:https://releases.openstack.org/constraints/upper/master}"},{"line_number":11,"context_line":"setenv \u003d VIRTUAL_ENV\u003d{envdir}"},{"line_number":12,"context_line":"deps \u003d"},{"line_number":13,"context_line":"  -c{env:TOX_CONSTRAINTS_FILE:https://releases.openstack.org/constraints/upper/master}"},{"line_number":14,"context_line":"  -r{toxinidir}/requirements.txt"},{"line_number":15,"context_line":"  -r{toxinidir}/test-requirements.txt"},{"line_number":16,"context_line":"commands \u003d"}],"source_content_type":"text/x-properties","patch_set":1,"id":"e452469b_78bccf32","line":13,"in_reply_to":"98a7bffb_e98b6d8e","updated":"2023-01-17 17:33:31.000000000","message":"Done","commit_id":"b19a9fa4298f54867b1074804ee13441ec3604d7"}]}