)]}'
{"/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":"e154e9482a29da4a7015a6194aedcda1805e0e59","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"f8ab552d_68665105","updated":"2026-09-04 08:51:03.000000000","message":"It seems that something is wrong with the current shutdown logic. A high shutdown timeout was added to the test execution; it\u0027s unexpected behavior. The graceful shutdown process of the idle database container should be very quick.\n\nWe should figure out what\u0027s wrong and fix it in the first place.","commit_id":"de3fb5739fe13341adc7ae0446422172ac82b7ed"},{"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":"0e022a9753c9d5eb2a77fffe47bf0c1f17c4874d","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"70e59184_3b901b72","updated":"2026-09-17 09:23:26.000000000","message":"Trivial change, we need this in 2026.2.","commit_id":"de3fb5739fe13341adc7ae0446422172ac82b7ed"},{"author":{"_account_id":39300,"name":"Olessya Khussainova","display_name":"Olessya Khussainova","email":"olessya.khussainova@ps.kz","username":"Cellofun"},"change_message_id":"43548c507ce75eec9c8043f95cf680a909755ce1","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"57867e47_d8f371ed","in_reply_to":"f8ab552d_68665105","updated":"2026-09-07 07:48:05.000000000","message":"You were right! After a resize, the container can already be `exited`, while Docker still returns its last health status as `healthy`. Trove checked health first, so it thought that the database was still running.\n\nHere\u0027s [a separate fix](https://review.opendev.org/c/openstack/trove/+/1004378) for this.","commit_id":"de3fb5739fe13341adc7ae0446422172ac82b7ed"}],"releasenotes/notes/use-high-timeout-for-shutdown-9596140bb6ec49ce.yaml":[{"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":"c72bc67cc3e7433212d1c3e89d09b25ba1a64128","unresolved":true,"context_lines":[{"line_number":2,"context_line":"fixes:"},{"line_number":3,"context_line":"  - |"},{"line_number":4,"context_line":"    Database shutdown requests now use ``agent_call_high_timeout`` instead of"},{"line_number":5,"context_line":"    ``agent_call_low_timeout``. This prevents flavor resize from failing when"},{"line_number":6,"context_line":"    stopping a database under load takes longer than the short request"},{"line_number":7,"context_line":"    timeout."}],"source_content_type":"text/x-yaml","patch_set":1,"id":"bf321a76_4dc409c3","line":5,"updated":"2026-09-03 11:45:50.000000000","message":"Not only does flavor resizing cause a database shutdown. We have a separate call for shutdown, and other operations do this too. So we should rephrase it somehow, I suppose.","commit_id":"bf46308507b8d4e020dcb22f33e259ea9063a6a2"},{"author":{"_account_id":39300,"name":"Olessya Khussainova","display_name":"Olessya Khussainova","email":"olessya.khussainova@ps.kz","username":"Cellofun"},"change_message_id":"e7e4bb4bdb044eefe9d79963c0cdf92a5b1b1c18","unresolved":false,"context_lines":[{"line_number":2,"context_line":"fixes:"},{"line_number":3,"context_line":"  - |"},{"line_number":4,"context_line":"    Database shutdown requests now use ``agent_call_high_timeout`` instead of"},{"line_number":5,"context_line":"    ``agent_call_low_timeout``. This prevents flavor resize from failing when"},{"line_number":6,"context_line":"    stopping a database under load takes longer than the short request"},{"line_number":7,"context_line":"    timeout."}],"source_content_type":"text/x-yaml","patch_set":1,"id":"336fc3ed_84b4e552","line":5,"in_reply_to":"bf321a76_4dc409c3","updated":"2026-09-04 05:43:03.000000000","message":"Done!","commit_id":"bf46308507b8d4e020dcb22f33e259ea9063a6a2"}],"trove/tests/unittests/guestagent/test_api.py":[{"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":"c72bc67cc3e7433212d1c3e89d09b25ba1a64128","unresolved":true,"context_lines":[{"line_number":19,"context_line":"from trove.tests.unittests import trove_testtools"},{"line_number":20,"context_line":""},{"line_number":21,"context_line":""},{"line_number":22,"context_line":"class GuestAgentAPITest(trove_testtools.TestCase):"},{"line_number":23,"context_line":""},{"line_number":24,"context_line":"    def test_stop_db_uses_high_timeout(self):"},{"line_number":25,"context_line":"        guest_api \u003d api.API.__new__(api.API)"}],"source_content_type":"text/x-python","patch_set":1,"id":"d446778d_770ac26e","line":22,"updated":"2026-09-03 11:45:50.000000000","message":"I think the unit tests are excessive here.\n1. Guest api calls are tested indirectly by integration tests already.\n2. Testing this is basically against common sense, how do you think? )","commit_id":"bf46308507b8d4e020dcb22f33e259ea9063a6a2"},{"author":{"_account_id":39300,"name":"Olessya Khussainova","display_name":"Olessya Khussainova","email":"olessya.khussainova@ps.kz","username":"Cellofun"},"change_message_id":"e7e4bb4bdb044eefe9d79963c0cdf92a5b1b1c18","unresolved":false,"context_lines":[{"line_number":19,"context_line":"from trove.tests.unittests import trove_testtools"},{"line_number":20,"context_line":""},{"line_number":21,"context_line":""},{"line_number":22,"context_line":"class GuestAgentAPITest(trove_testtools.TestCase):"},{"line_number":23,"context_line":""},{"line_number":24,"context_line":"    def test_stop_db_uses_high_timeout(self):"},{"line_number":25,"context_line":"        guest_api \u003d api.API.__new__(api.API)"}],"source_content_type":"text/x-python","patch_set":1,"id":"c4e71159_b1149b1a","line":22,"in_reply_to":"d446778d_770ac26e","updated":"2026-09-04 05:43:03.000000000","message":"You\u0027re right 🥲 That was just me trying to boost the unit test coverage, lol","commit_id":"bf46308507b8d4e020dcb22f33e259ea9063a6a2"}]}