)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":17685,"name":"Elod Illes","email":"elod.illes@est.tech","username":"elod.illes"},"change_message_id":"9d8eb243c052efadd04481bdab937b8941ac569c","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"0f478ac9_331e424f","updated":"2026-02-17 13:50:04.000000000","message":"finally a new virtualenv release arrived [1] with setuptools 82.0.0 bundled in it 😭\n\n[1] https://virtualenv.pypa.io/en/latest/changelog.html","commit_id":"c88da99b417282303e2230ab721fb8856bfc1761"},{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"2d2da2040929ce809090cdca6cd7edc85edd4cbb","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"52ed4911_5d1a0332","updated":"2026-02-12 16:42:51.000000000","message":"lgtm, thanks","commit_id":"c88da99b417282303e2230ab721fb8856bfc1761"},{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"6c5d3b6bf564f0167caf428d859c3ca1ce8ac782","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"0d2e3698_46a40e80","updated":"2026-02-17 17:48:25.000000000","message":"ok this is passing ci\n\ni woudl prefer not to regress the configuration but since it unblocks that gate for now i guess this is ok for stable given this branch will be eol soon","commit_id":"961ff612e92057cb98e68b34dcab2acf46290026"},{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"e88930efe85f17dc64977959e6b2edee7e2564a9","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"692a4316_566777b6","updated":"2026-02-17 17:12:06.000000000","message":"you shoudl be fixing thie for all of the targets not defining sperate ones for 3.12","commit_id":"961ff612e92057cb98e68b34dcab2acf46290026"}],"tox.ini":[{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"89d1a81fcb178c63e652a47d737e53bd397b530e","unresolved":true,"context_lines":[{"line_number":64,"context_line":"# by making greenlet leaks a failure."},{"line_number":65,"context_line":"  NOVA_RAISE_ON_GREENLET_LEAK\u003dTrue"},{"line_number":66,"context_line":""},{"line_number":67,"context_line":"[testenv:py312]"},{"line_number":68,"context_line":"setenv \u003d"},{"line_number":69,"context_line":"  {[testenv]setenv}"},{"line_number":70,"context_line":"# we do not have any greenlet leaks in unit tests so enforce that"}],"source_content_type":"text/x-properties","patch_set":2,"id":"d6444d56_30a4877f","line":67,"range":{"start_line":67,"start_character":9,"end_line":67,"end_character":14},"updated":"2026-02-17 17:09:31.000000000","message":"why are you ding this speratly.?","commit_id":"961ff612e92057cb98e68b34dcab2acf46290026"},{"author":{"_account_id":17685,"name":"Elod Illes","email":"elod.illes@est.tech","username":"elod.illes"},"change_message_id":"d9b7182132fe41175d1dddccec7c20ffc20723ad","unresolved":true,"context_lines":[{"line_number":64,"context_line":"# by making greenlet leaks a failure."},{"line_number":65,"context_line":"  NOVA_RAISE_ON_GREENLET_LEAK\u003dTrue"},{"line_number":66,"context_line":""},{"line_number":67,"context_line":"[testenv:py312]"},{"line_number":68,"context_line":"setenv \u003d"},{"line_number":69,"context_line":"  {[testenv]setenv}"},{"line_number":70,"context_line":"# we do not have any greenlet leaks in unit tests so enforce that"}],"source_content_type":"text/x-properties","patch_set":2,"id":"072cd7b5_4883441e","line":67,"range":{"start_line":67,"start_character":9,"end_line":67,"end_character":14},"in_reply_to":"d6444d56_30a4877f","updated":"2026-02-17 17:43:45.000000000","message":"well, based on my tests, py312 based jobs does not have virtualenv installed, hence they need the pinning in \u0027deps\u0027\n\nwhile all the others need the pin via \u0027requires \u003d virtualenv\u003c20.37.0\u0027 as they use virtualenv and things would fail without that.\n\nso in short: this is a bit awkward, but afaik, this is how it works.","commit_id":"961ff612e92057cb98e68b34dcab2acf46290026"},{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"89d1a81fcb178c63e652a47d737e53bd397b530e","unresolved":true,"context_lines":[{"line_number":108,"context_line":"  stestr --test-path\u003d./nova/tests/functional run {posargs}"},{"line_number":109,"context_line":"  stestr slowest"},{"line_number":110,"context_line":""},{"line_number":111,"context_line":"[testenv:functional-py312]"},{"line_number":112,"context_line":"description \u003d"},{"line_number":113,"context_line":"  Run functional tests."},{"line_number":114,"context_line":"setenv \u003d"}],"source_content_type":"text/x-properties","patch_set":2,"id":"3f36aafd_a9d8e581","line":111,"updated":"2026-02-17 17:09:31.000000000","message":"note that this is techinially optional in 2024.2\n\n\nhttps://github.com/openstack/governance/blob/master/reference/runtimes/2024.2.rst#advanceunstable-testing\n\n\nbut its also unfortuneate that we are not doign this properly by adding -py312\non line 81","commit_id":"961ff612e92057cb98e68b34dcab2acf46290026"},{"author":{"_account_id":17685,"name":"Elod Illes","email":"elod.illes@est.tech","username":"elod.illes"},"change_message_id":"d9b7182132fe41175d1dddccec7c20ffc20723ad","unresolved":true,"context_lines":[{"line_number":108,"context_line":"  stestr --test-path\u003d./nova/tests/functional run {posargs}"},{"line_number":109,"context_line":"  stestr slowest"},{"line_number":110,"context_line":""},{"line_number":111,"context_line":"[testenv:functional-py312]"},{"line_number":112,"context_line":"description \u003d"},{"line_number":113,"context_line":"  Run functional tests."},{"line_number":114,"context_line":"setenv \u003d"}],"source_content_type":"text/x-properties","patch_set":2,"id":"c7835c3f_848f613c","line":111,"in_reply_to":"3f36aafd_a9d8e581","updated":"2026-02-17 17:43:45.000000000","message":"as i explained in the commit message, i was surprised, too, that py312 was added in 2024.2 Dalmatian as it was not officially required.\n\nsee my explanation for L67 about why i moved it to a separate place.","commit_id":"961ff612e92057cb98e68b34dcab2acf46290026"}]}
