)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":13252,"name":"Dr. Jens Harbott","display_name":"Jens Harbott (frickler)","email":"frickler@offenerstapel.de","username":"jrosenboom"},"change_message_id":"7a6e61d0a6abe0b9c3a4d5035329872460e6320b","unresolved":true,"context_lines":[{"line_number":4,"context_line":"Commit:     Nate Johnston \u003cnate.johnston@redhat.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2022-10-11 14:25:24 -0400"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Restrict virtualenv to avoid suds-jurko dep issue"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Since an unconstrained virtualenv can cause breakage in downstream"},{"line_number":10,"context_line":"CI systems using much later versions of packages, constrain it in"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"2a8351bd_0657c4f3","line":7,"updated":"2022-10-13 09:48:38.000000000","message":"Could please create a bug report that shows the details of this issue? From what I gather so far, I would rather work to remove the broken suds-jurko dependency than implementing the proposed workaround.\n\nAnd yes, opendev CI setup hides this kind of failures due to cached wheel builds, that\u0027s an issue that also needs to be tackled. Contributions welcome.","commit_id":"ea8c75fe28fe810365edb85181dc3f616869e6ea"},{"author":{"_account_id":13995,"name":"Nate Johnston","email":"nate.johnston@redhat.com","username":"natejohnston"},"change_message_id":"e531f4c0c6be53cdfa57473b869676ddc14d591b","unresolved":false,"context_lines":[{"line_number":4,"context_line":"Commit:     Nate Johnston \u003cnate.johnston@redhat.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2022-10-11 14:25:24 -0400"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Restrict virtualenv to avoid suds-jurko dep issue"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Since an unconstrained virtualenv can cause breakage in downstream"},{"line_number":10,"context_line":"CI systems using much later versions of packages, constrain it in"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"00ff406b_f4bcdab9","line":7,"in_reply_to":"2a8351bd_0657c4f3","updated":"2022-10-13 16:03:11.000000000","message":"Documented: https://bugs.launchpad.net/designate/+bug/1992825","commit_id":"ea8c75fe28fe810365edb85181dc3f616869e6ea"}],"/PATCHSET_LEVEL":[{"author":{"_account_id":11628,"name":"Michael Johnson","email":"johnsomor@gmail.com","username":"johnsom"},"change_message_id":"322f24b76d763016dcf4b486de6a539af1e3dc05","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"8a863d39_0ddcd5d9","updated":"2022-10-11 16:30:53.000000000","message":"This is a bit of an odd one given this is already constrained in upper-constraints: https://github.com/openstack/requirements/blob/stable/wallaby/upper-constraints.txt#L586","commit_id":"e128fd7622dcc09f7de709818c7f6dd19cc29660"},{"author":{"_account_id":18551,"name":"Szymon Datko","email":"szymon.datko@gmail.com","username":"sdatko"},"change_message_id":"177e230d78a973350a0af0c891ae69479142d833","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"04c19257_b6b47e1a","in_reply_to":"8a863d39_0ddcd5d9","updated":"2022-10-11 16:41:22.000000000","message":"\u003e This is a bit of an odd one given this is already constrained in upper-constraints: https://github.com/openstack/requirements/blob/stable/wallaby/upper-constraints.txt#L586\n\nHey Michael, this is unfortunately a kind of broken-design of tox why the `requires` option is necessary.\n\nWhen calling tox, it first creates virtual environment with most recent virtualenv package by default and ignores the constraints (as those are only defined for the dependencies really). Then, when the actual deps are installed, they use wrong setuptools that was installed with virtualenv.\n\nTo make the constraints for setuptools effective, the install_command would have to be changed to always downgrade the setuptools first and then install what\u0027s specified in requirements.txt file.\n\nNOTE: the problem may not be visible in OpenDev Zuul instance, as the Zuul jobs here do not build the packages, but use the prebuilt wheels from some mirros instead (at least from what I checked).","commit_id":"e128fd7622dcc09f7de709818c7f6dd19cc29660"},{"author":{"_account_id":13995,"name":"Nate Johnston","email":"nate.johnston@redhat.com","username":"natejohnston"},"change_message_id":"1b72ee16dbe2d29ba33fc305a2aac7459fb55c1c","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":1,"id":"9fca100f_090c08e3","in_reply_to":"8a863d39_0ddcd5d9","updated":"2022-10-11 16:50:09.000000000","message":"I believe this is a very similar situation to the use_2to3 issue that was discussed on openstack-discuss in September 2021 [1], and the thread indicated pretty devent community support for pinning versions in ways similar to this change.  That support came at least partially because (quoting from the thread): \"Zuul\u0027s ensure-tox task[4] installs tox which installs the virtualenv package which bundles the latest setuptools package. This happens without applying any constraints.\"\n\n[1] https://lists.openstack.org/pipermail/openstack-discuss/2021-September/024987.html\n[4] dangling reference from the quote.","commit_id":"e128fd7622dcc09f7de709818c7f6dd19cc29660"},{"author":{"_account_id":11628,"name":"Michael Johnson","email":"johnsomor@gmail.com","username":"johnsom"},"change_message_id":"edd1de206349e605ffd07be686e23c003ab9f875","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":1,"id":"a551bf23_e9c2de81","in_reply_to":"9fca100f_090c08e3","updated":"2022-10-11 17:12:18.000000000","message":"Yeah, these stable branches here do not have any test failures as they enforce those upper constraints limits. That is why I commented that also pinning this in tox is a bit odd.","commit_id":"e128fd7622dcc09f7de709818c7f6dd19cc29660"},{"author":{"_account_id":13995,"name":"Nate Johnston","email":"nate.johnston@redhat.com","username":"natejohnston"},"change_message_id":"20646097acfe2d2c914f2b1064dc1d2623caa732","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":1,"id":"73182983_1cc32e5a","in_reply_to":"a551bf23_e9c2de81","updated":"2022-10-11 18:26:45.000000000","message":"Amended the commit message to make it clear that it is not issues with the upstream CI being remediated by this change.","commit_id":"e128fd7622dcc09f7de709818c7f6dd19cc29660"},{"author":{"_account_id":13995,"name":"Nate Johnston","email":"nate.johnston@redhat.com","username":"natejohnston"},"change_message_id":"6222ba7623187345edaa33b112e8801bd3d4c0d4","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"fd46a068_047460a5","updated":"2022-10-12 20:00:40.000000000","message":"recheck weird failed test only in the ipv6 bind job","commit_id":"ea8c75fe28fe810365edb85181dc3f616869e6ea"},{"author":{"_account_id":13252,"name":"Dr. Jens Harbott","display_name":"Jens Harbott (frickler)","email":"frickler@offenerstapel.de","username":"jrosenboom"},"change_message_id":"5c24f87c1b8905f41e43d31b8193ba67f2cb1089","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"db117aa4_8ba29057","updated":"2022-10-17 09:14:55.000000000","message":"I would rather see us drop the unconditional import of suds-jurko.\n\nMaybe we can even consider backporting the drop of the akamai backend, given that the service it is supposed to be talking to no longer exists?","commit_id":"2559524b710c6c85afc7915765c8eaff04b3392b"},{"author":{"_account_id":22623,"name":"Erik Olof Gunnar Andersson","email":"eandersson@blizzard.com","username":"eoandersson"},"change_message_id":"50993584cda88e0cb2c2e66cf8eeada28029b6aa","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":4,"id":"bf602b9e_7a1d197d","in_reply_to":"96cf38f4_1781155d","updated":"2022-10-17 18:07:27.000000000","message":"Not liking any of these solutions, my preferred path would be to drop the unsupported akamai backend.","commit_id":"2559524b710c6c85afc7915765c8eaff04b3392b"},{"author":{"_account_id":13995,"name":"Nate Johnston","email":"nate.johnston@redhat.com","username":"natejohnston"},"change_message_id":"5ecb6a36e8f0987dbdb2b68eded540c570b82de8","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":4,"id":"96cf38f4_1781155d","in_reply_to":"db117aa4_8ba29057","updated":"2022-10-17 14:54:03.000000000","message":"I have no objection to that approach.","commit_id":"2559524b710c6c85afc7915765c8eaff04b3392b"}],"tox.ini":[{"author":{"_account_id":18551,"name":"Szymon Datko","email":"szymon.datko@gmail.com","username":"sdatko"},"change_message_id":"27fd7cea8ab563f97eb1d9d69f2895a59315f121","unresolved":true,"context_lines":[{"line_number":3,"context_line":"envlist \u003d py37,flake8"},{"line_number":4,"context_line":"ignore_basepython_conflict \u003d True"},{"line_number":5,"context_line":"skipsdist \u003d True"},{"line_number":6,"context_line":"requires \u003d virtualenv\u003c20.8"},{"line_number":7,"context_line":""},{"line_number":8,"context_line":"[testenv]"},{"line_number":9,"context_line":"basepython \u003d python3"}],"source_content_type":"text/x-properties","patch_set":1,"id":"e7226c1a_adb410ad","line":6,"updated":"2022-10-11 16:31:07.000000000","message":"According to tox documentation, the `requires` options was added in version 3.2.0, so bumping also `minversion` in line 2 would be good.\n\ndetails: https://tox.wiki/en/latest/config.html#conf-requires","commit_id":"e128fd7622dcc09f7de709818c7f6dd19cc29660"},{"author":{"_account_id":13995,"name":"Nate Johnston","email":"nate.johnston@redhat.com","username":"natejohnston"},"change_message_id":"1b72ee16dbe2d29ba33fc305a2aac7459fb55c1c","unresolved":false,"context_lines":[{"line_number":3,"context_line":"envlist \u003d py37,flake8"},{"line_number":4,"context_line":"ignore_basepython_conflict \u003d True"},{"line_number":5,"context_line":"skipsdist \u003d True"},{"line_number":6,"context_line":"requires \u003d virtualenv\u003c20.8"},{"line_number":7,"context_line":""},{"line_number":8,"context_line":"[testenv]"},{"line_number":9,"context_line":"basepython \u003d python3"}],"source_content_type":"text/x-properties","patch_set":1,"id":"d63be54c_013073cf","line":6,"in_reply_to":"e7226c1a_adb410ad","updated":"2022-10-11 16:50:09.000000000","message":"Added, thanks.  I made sure that tox 3.2.0 was released in the Wallaby time frame, and it was (released 2018).","commit_id":"e128fd7622dcc09f7de709818c7f6dd19cc29660"},{"author":{"_account_id":18551,"name":"Szymon Datko","email":"szymon.datko@gmail.com","username":"sdatko"},"change_message_id":"568f3620ad516350553b64726c32ff30a372b0b2","unresolved":false,"context_lines":[{"line_number":3,"context_line":"envlist \u003d py37,flake8"},{"line_number":4,"context_line":"ignore_basepython_conflict \u003d True"},{"line_number":5,"context_line":"skipsdist \u003d True"},{"line_number":6,"context_line":"requires \u003d virtualenv\u003c20.8"},{"line_number":7,"context_line":""},{"line_number":8,"context_line":"[testenv]"},{"line_number":9,"context_line":"basepython \u003d python3"}],"source_content_type":"text/x-properties","patch_set":1,"id":"48f213be_8f322fee","line":6,"in_reply_to":"e7226c1a_adb410ad","updated":"2022-10-11 16:41:38.000000000","message":"Done","commit_id":"e128fd7622dcc09f7de709818c7f6dd19cc29660"}]}