)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":36080,"name":"Erkin Mussurmankulov","display_name":"Eric","email":"erkin.mussurmankulov@ps.kz","username":"mongoose404","status":"PS Cloud services employee"},"change_message_id":"c0ee5baa17adbb1e5d6286f7f8e8d83be63a1a38","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"3ec3fe82_44d6f9fc","updated":"2026-08-06 09:18:47.000000000","message":"Hello! Thank you for the contribution, I also noted slow archive.ubuntu.com mirror yesterday. But I prefer [this](https://review.opendev.org/c/openstack/trove/+/999890) approach instead.","commit_id":"7a7053a5dd76014998b03a864949bb058999b85a"},{"author":{"_account_id":37598,"name":"Ivan Anfimov","display_name":"Ivan Anfimov","email":"lazekteam@gmail.com","username":"anfimovir"},"change_message_id":"868eb5bd75e5af50232ce0e8e72ad6fbd48fa04d","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"be43b0f4_89adea74","updated":"2026-08-10 14:00:27.000000000","message":"I think, we can backport it in 2026.1","commit_id":"7a7053a5dd76014998b03a864949bb058999b85a"},{"author":{"_account_id":5733,"name":"Kieran Spear","email":"kispear@gmail.com","username":"kspear"},"change_message_id":"071d2c61a3e2ad2be109a7c63688f64e452e2ea2","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"d80c8021_592f214e","updated":"2026-08-06 23:48:54.000000000","message":"recheck","commit_id":"7a7053a5dd76014998b03a864949bb058999b85a"},{"author":{"_account_id":36080,"name":"Erkin Mussurmankulov","display_name":"Eric","email":"erkin.mussurmankulov@ps.kz","username":"mongoose404","status":"PS Cloud services employee"},"change_message_id":"4ee06637f346c58883608d23bbd07b8ac654f921","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"060e69bd_4390f977","in_reply_to":"0d5bd246_225ea4f9","updated":"2026-08-11 10:16:12.000000000","message":"I meant \"improve execution time\", of course 🙂","commit_id":"7a7053a5dd76014998b03a864949bb058999b85a"},{"author":{"_account_id":5733,"name":"Kieran Spear","email":"kispear@gmail.com","username":"kspear"},"change_message_id":"eddb4315b554a03907eb2dea367da0b6ad720dd4","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":1,"id":"6d8025a9_ab9dfb22","in_reply_to":"1b888838_7436fb3a","updated":"2026-08-07 00:48:46.000000000","message":"Yeah, looks like Anubis is blocking things on releases.openstack as well:\n pip._vendor.packaging.markers.InvalidMarker: Expected a marker variable or quoted string\n    re not a bot!\u003c/title\u003e\u003clink rel\u003d\"stylesheet\" href\u003d\"/.within.website/x/xess/xess.min.css?cachebuster\u003dv1.26.2\"\u003e\u003cmeta name\u003d\"viewport\" content\u003d\"width\u003ddevice-width, initial-scale\u003d1.0\"\u003e\u003cmeta name\u003d\"robots\" content\u003d\"noindex,nofollow\"\u003e\u003cstyle\u003e\n    ^\n\n \u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d log end \u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\n ERROR: could not install deps [-chttps://releases.openstack.org/constraints/upper/master, -r/opt/stack/tempest/requirements.txt]","commit_id":"7a7053a5dd76014998b03a864949bb058999b85a"},{"author":{"_account_id":36080,"name":"Erkin Mussurmankulov","display_name":"Eric","email":"erkin.mussurmankulov@ps.kz","username":"mongoose404","status":"PS Cloud services employee"},"change_message_id":"cc721719b8fb8abb511435ac78e6acd9c2ca2c39","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"1b888838_7436fb3a","in_reply_to":"3ec3fe82_44d6f9fc","updated":"2026-08-06 09:20:45.000000000","message":"Oh, sorry, it\u0027s a different problem though 🤔","commit_id":"7a7053a5dd76014998b03a864949bb058999b85a"},{"author":{"_account_id":36080,"name":"Erkin Mussurmankulov","display_name":"Eric","email":"erkin.mussurmankulov@ps.kz","username":"mongoose404","status":"PS Cloud services employee"},"change_message_id":"f7073160e960798164025c0d55096b5db10abc0f","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":1,"id":"6e97b72e_5e9a197b","in_reply_to":"6d8025a9_ab9dfb22","updated":"2026-08-10 18:17:49.000000000","message":"Did you figure out whether this error was induced by this patch, or was it a temporary problem related to partial opendev.org or openstack.org availability?\nProbably we should pass a correct public url to the upper constraints instead, [this one](https://opendev.org/openstack/requirements/raw/branch/master/upper-constraints.txt). It\u0027s also located behind Anubis, but probably there are some exceptions for Zuul CI.\n\nOverall, it LGTM, but we need additional tests to figure out why Anubis blocked some requests. I\u0027m eager to improve the reliability of the tests, and this one looks like some kind of intermittent bug to me. 😄\n\n______________\n\nBtw, I ran into a [similar](https://review.opendev.org/c/openstack/trove/+/992028) problem addressed by this MR when I needed to add something to the requirements.txt file. You just can\u0027t do it easily.\n\nBut the approach in this MR is more universal and looks better than mine, so I\u0027m in favor of this MR.","commit_id":"7a7053a5dd76014998b03a864949bb058999b85a"},{"author":{"_account_id":5733,"name":"Kieran Spear","email":"kispear@gmail.com","username":"kspear"},"change_message_id":"241467fc05af6b162dbb7b6994547e682e85331e","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":1,"id":"78d6036d_d539c74b","in_reply_to":"6e97b72e_5e9a197b","updated":"2026-08-11 01:06:52.000000000","message":"Yeah it doesn\u0027t seem to be related, since the patch only overrides the DIB constraints file and the failures seem to be in the tempest install itself. It is odd to me why tempest isn\u0027t already using a local constraints though, I\u0027ll do some digging:\n\n    ++ lib/tempest:set_tempest_venv_constraints:142 :   export UPPER_CONSTRAINTS_FILE\u003dhttps://releases.openstack.org/constraints/upper/master\n    ++ lib/tempest:set_tempest_venv_constraints:142 :   UPPER_CONSTRAINTS_FILE\u003dhttps://releases.openstack.org/constraints/upper/master\n    ++ lib/tempest:set_tempest_venv_constraints:143 :   export TOX_CONSTRAINTS_FILE\u003dhttps://releases.openstack.org/constraints/upper/master\n    ++ lib/tempest:set_tempest_venv_constraints:143 :   TOX_CONSTRAINTS_FILE\u003dhttps://releases.openstack.org/constraints/upper/master\n    \n    \n    \nOn the requirements.txt issue, I _think_ the way this is supposed to work in Zuul is that if you make sure the node-local constraints/requirements file is used, you can push up a global requirements review and then use `Depends-On` in your trove change, and Zuul will make sure your requirements change is on the test node. That is from my memory of how things used to work, so possibly outdated information.","commit_id":"7a7053a5dd76014998b03a864949bb058999b85a"},{"author":{"_account_id":5733,"name":"Kieran Spear","email":"kispear@gmail.com","username":"kspear"},"change_message_id":"f0a8d098d82f47c3af25cdd47b101c1eb6e72f7f","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"9923ab2a_38145567","in_reply_to":"78d6036d_d539c74b","updated":"2026-08-11 01:31:43.000000000","message":"Ok I looked into the error more closely.\n\nhttps://releases.openstack.org/constraints/upper/master just redirects to https://opendev.org/openstack/requirements/raw/branch/master/upper-constraints.txt so it was the same opendev.org Anubis issue that hit checkouts. This upper constraints is hardcoded in upstream tempest, so I don\u0027t think there\u0027s anything more to do here.","commit_id":"7a7053a5dd76014998b03a864949bb058999b85a"},{"author":{"_account_id":36080,"name":"Erkin Mussurmankulov","display_name":"Eric","email":"erkin.mussurmankulov@ps.kz","username":"mongoose404","status":"PS Cloud services employee"},"change_message_id":"a191e643ad7ba14ff8c9efbc8a0e96268395ffca","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"0d5bd246_225ea4f9","in_reply_to":"9923ab2a_38145567","updated":"2026-08-11 09:58:58.000000000","message":"Ok, then it LGTM.\nWe\u0027re ready for a merge then, after the Zuul check is complete.\n\nThis MR displaces [this](https://review.opendev.org/c/openstack/trove/+/992028) MR.\n\nAlso, it unblocks a new possibility for us to disable syncing the trove code from the host on guest startup.\nIn fact, we may disable DEV mode on test instances, which assumes not running `pip install` as well. Only the `guest-log-collector.service` will remain.\nThis will increase the test execution time a bit.","commit_id":"7a7053a5dd76014998b03a864949bb058999b85a"},{"author":{"_account_id":36080,"name":"Erkin Mussurmankulov","display_name":"Eric","email":"erkin.mussurmankulov@ps.kz","username":"mongoose404","status":"PS Cloud services employee"},"change_message_id":"a191e643ad7ba14ff8c9efbc8a0e96268395ffca","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"24be9294_ef3d740d","in_reply_to":"be43b0f4_89adea74","updated":"2026-08-11 09:58:58.000000000","message":"Ok, let\u0027s do it then, only for 2026.1.","commit_id":"7a7053a5dd76014998b03a864949bb058999b85a"},{"author":{"_account_id":36080,"name":"Erkin Mussurmankulov","display_name":"Eric","email":"erkin.mussurmankulov@ps.kz","username":"mongoose404","status":"PS Cloud services employee"},"change_message_id":"831df3a06f4b84069280e9909b5a77c8c4034fda","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"95623eee_b2e6f5ce","updated":"2026-08-11 13:11:58.000000000","message":"LGTM. Nice work, thank you for the contribution!\n\n(for the history)\nThis displaces the MR: https://review.opendev.org/c/openstack/trove/+/992028","commit_id":"867b6b2623844dc21c2cda209e96f12188868ae8"}]}
