)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":31664,"name":"Omer Schwartz","email":"oschwart@redhat.com","username":"oschwart"},"change_message_id":"44f9fdb8ca2fdb426fdf597828217a31002d0e43","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"0e6a550f_e6b92f3f","updated":"2026-02-23 16:19:31.000000000","message":"I am not really happy with the current solution, ideally it should be infoblox_client library that have a stable exception constructor","commit_id":"25c198968298823a468e382e728410aa27e0677c"},{"author":{"_account_id":13252,"name":"Dr. Jens Harbott","display_name":"Jens Harbott (frickler)","email":"frickler@offenerstapel.de","username":"jrosenboom"},"change_message_id":"b402ae2513d4f3bcebc065fa185420c46a9babac","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"9428e729_22b95575","updated":"2026-02-23 18:11:03.000000000","message":"I don\u0027t mind going ahead with this either, though","commit_id":"25c198968298823a468e382e728410aa27e0677c"},{"author":{"_account_id":22623,"name":"Erik Olof Gunnar Andersson","email":"eandersson@blizzard.com","username":"eoandersson"},"change_message_id":"6cd273576e52562dc7122f8b119850b39621bcba","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"6e111d87_0a3648ee","updated":"2026-02-23 20:24:20.000000000","message":"Something like this looks enough for it to work with both the old and new version\n\ndiff --git a/designate/tests/unit/backend/test_infoblox.py b/designate/tests/unit/backend/test_infoblox.py\nindex 2a76f56c..3ae15fd8 100644\n--- a/designate/tests/unit/backend/test_infoblox.py\n+++ b/designate/tests/unit/backend/test_infoblox.py\n@@ -293,7 +293,7 @@ class BasicInfobloxBackendTestCase(InfobloxBackendTestCase):\n\n     def test_create_zone_handle_error(self):\n         self.backend.infoblox.create_dns_zone.side_effect \u003d (\n-            infoblox_exceptions.InfobloxTimeoutError(\u0027error\u0027)\n+            infoblox_exceptions.InfobloxTimeoutError(response\u003d\u0027\u0027, reason\u003d\u0027\u0027)\n         )\n\n         self.assertRaisesRegex(\n@@ -362,7 +362,7 @@ class BasicInfobloxBackendTestCase(InfobloxBackendTestCase):\n\n     def test_delete_zone_handle_error(self):\n         self.backend.infoblox.delete_dns_zone.side_effect \u003d (\n-            infoblox_exceptions.InfobloxTimeoutError(\u0027error\u0027)\n+            infoblox_exceptions.InfobloxTimeoutError(response\u003d\u0027\u0027, reason\u003d\u0027\u0027)\n         )\n\n         self.assertRaisesRegex(","commit_id":"25c198968298823a468e382e728410aa27e0677c"},{"author":{"_account_id":13252,"name":"Dr. Jens Harbott","display_name":"Jens Harbott (frickler)","email":"frickler@offenerstapel.de","username":"jrosenboom"},"change_message_id":"a7a136b654aa5e0701202fd674f5f284c2c48655","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":2,"id":"046cfedb_48e25bfd","in_reply_to":"0e6a550f_e6b92f3f","updated":"2026-02-23 16:40:39.000000000","message":"maybe just drop these tests? I don\u0027t think they bring much benefit, they would go better as real functional tests, but those we can\u0027t have with a non-free backend. so IMO we can live with some restricted test coverage rather than having to do such complicated constructions","commit_id":"25c198968298823a468e382e728410aa27e0677c"},{"author":{"_account_id":31664,"name":"Omer Schwartz","email":"oschwart@redhat.com","username":"oschwart"},"change_message_id":"a43f3317caf0659e24989bdc4c9fe0eb339a2cc2","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"ccb6f4a8_1c891c9b","updated":"2026-02-24 16:08:39.000000000","message":"Thank you for the feedback folks, I am adding a CR+2 and W+1 as well as I wouldn\u0027t like this patch to delay the requirements patch any further, I already resolved its previous conflicts https://review.opendev.org/c/openstack/requirements/+/938500","commit_id":"40abe1b52582ed98c8656321cfba0f8fe86d8d0e"},{"author":{"_account_id":31664,"name":"Omer Schwartz","email":"oschwart@redhat.com","username":"oschwart"},"change_message_id":"2fa39ea5e241c27e9fa5826819c5bf81bf8d8783","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"47ed404c_29dc5d1f","updated":"2026-02-24 11:48:08.000000000","message":"recheck failure is not related to the patch","commit_id":"40abe1b52582ed98c8656321cfba0f8fe86d8d0e"}]}
