)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":4690,"name":"melanie witt","display_name":"melwitt","email":"melwittt@gmail.com","username":"melwitt"},"change_message_id":"7656c7c22d18118b43a3495047e4acd16a445ac4","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"c843a4b4_7dbcecf2","updated":"2022-05-21 06:01:33.000000000","message":"Looks like this is working, jobs passing, looks OK to me. Thanks gmann++!","commit_id":"e5e768e248e31104b86a57a43b3ab18fe90618f5"},{"author":{"_account_id":17685,"name":"Elod Illes","email":"elod.illes@est.tech","username":"elod.illes"},"change_message_id":"c57f2a56b5b45fe33e62c4d7328d5c889ba50452","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":6,"id":"837ec99b_b0f7d073","updated":"2022-05-23 12:18:35.000000000","message":"Gate is passing nicely. Melanie already +2\u0027d the patch but commit message needed an update to make our cherry-pick check test pass, so i\u0027ve updated the commit message only, hence approving it now with single +2.","commit_id":"e21fb6793443e9eb6eb25cb3be0894ce25670829"},{"author":{"_account_id":4690,"name":"melanie witt","display_name":"melwitt","email":"melwittt@gmail.com","username":"melwitt"},"change_message_id":"888e927f96c3d3137bc0e2ff921d8774fdf3d36f","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":6,"id":"a3c0c7d2_f18a3a93","updated":"2022-05-23 17:47:46.000000000","message":"It looks like we need a more general solution because TEMPEST_VENV_UPPER_CONSTRAINTS\u003dmaster is being used on all stable branches, not only stable/train:\n\nhttps://etherpad.opendev.org/p/nova-stable-branch-ci","commit_id":"e21fb6793443e9eb6eb25cb3be0894ce25670829"},{"author":{"_account_id":17685,"name":"Elod Illes","email":"elod.illes@est.tech","username":"elod.illes"},"change_message_id":"5ace8bda87981e3fbe4faf9b743426885001253e","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":6,"id":"0ecf9398_75ebe1fe","updated":"2022-05-23 07:39:03.000000000","message":"Thanks Ghanshyam for fixing this!\n\nThis patch is stably-only and needed only in train because in newer branches we have zuulv3 grenade job [1] and thus this code does not exist there.\n\nNOTE: i\u0027ve added the \u0027stable-only\u0027 tag in commit message otherwise our cherry-pick check would fail on gate.\n\n[1] Ibbb3930a6e629e93a424b3ae048f599f11923be3","commit_id":"e21fb6793443e9eb6eb25cb3be0894ce25670829"},{"author":{"_account_id":4690,"name":"melanie witt","display_name":"melwitt","email":"melwittt@gmail.com","username":"melwitt"},"change_message_id":"309f12ad7dd4994cb5ccf307ca69985ec967c5ee","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":6,"id":"5572e2fd_91a80385","updated":"2022-05-23 17:21:30.000000000","message":"recheck\n\nnova.tests.functional.api.client.OpenStackApiException: Unexpected status code: {\"conflictingRequest\": {\"message\": \"Cannot \u0027migrate\u0027 instance 0d23053a-2c66-4523-9342-515f3577e892 while it is in vm_state building\", \"code\": 409}}","commit_id":"e21fb6793443e9eb6eb25cb3be0894ce25670829"},{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"22e1af053d6c65eaa0a0b9b2a50793e00685c217","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":6,"id":"4ac53c9a_1d98fe45","in_reply_to":"56860783_0b26175c","updated":"2022-05-23 23:34:47.000000000","message":"yeah, EM branches are the only pinned (ussuri is yet to do) with older Tempest and so does constraints","commit_id":"e21fb6793443e9eb6eb25cb3be0894ce25670829"},{"author":{"_account_id":4690,"name":"melanie witt","display_name":"melwitt","email":"melwittt@gmail.com","username":"melwitt"},"change_message_id":"31fcd47878316aa72a8645acdaf0d71e3cd9367f","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":6,"id":"56860783_0b26175c","in_reply_to":"8be130b5_780a3474","updated":"2022-05-23 17:58:14.000000000","message":"OK, my bad ... I see what you are saying now, tempest is pinned on stable/train but not elsewhere. Sorry for the noise.","commit_id":"e21fb6793443e9eb6eb25cb3be0894ce25670829"},{"author":{"_account_id":17685,"name":"Elod Illes","email":"elod.illes@est.tech","username":"elod.illes"},"change_message_id":"b23895c1c1edd22dd7d963c46e18222d451df8af","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":6,"id":"8be130b5_780a3474","in_reply_to":"a3c0c7d2_f18a3a93","updated":"2022-05-23 17:54:15.000000000","message":"for branches not in EM this is kind of expected if i\u0027m not mistaken: always use tempest from master.\n\nThough I thought this is only needed in Train because in victoria and ussuri this hook is not existing as there are zuulv3 grenade job.","commit_id":"e21fb6793443e9eb6eb25cb3be0894ce25670829"}]}
