)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":28647,"name":"David","display_name":"morenod","email":"dsanzmor@redhat.com","username":"morenod"},"change_message_id":"2c09937c33f9e3986fb8b66c2e2f006bcd8bfa70","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"b42efae6_c1597219","updated":"2026-06-29 10:40:16.000000000","message":"recheck","commit_id":"81ac9487ded7994dfd41976afee4aa283ac245e4"},{"author":{"_account_id":28647,"name":"David","display_name":"morenod","email":"dsanzmor@redhat.com","username":"morenod"},"change_message_id":"7f28bce1d9c6e4dde565a3f73b84b37087590e28","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"6fee4d31_c8673a4e","updated":"2026-06-29 15:05:39.000000000","message":"we rarely SSH instances on watcher tests, I like the idea of not waiting for SSH, so I will +1 this, but probably we should add this feature as default on create_server function","commit_id":"81ac9487ded7994dfd41976afee4aa283ac245e4"}],"watcher_tempest_plugin/tests/scenario/test_execute_zone_migration.py":[{"author":{"_account_id":28006,"name":"teim-ci","display_name":"teim-ci","email":"ci@seanmooney.info","username":"ci-sean-mooney","status":"this is a third-party ci account run by sean-k-mooney on irc\nhosted at zuul.teim.app"},"tag":"autogenerated:zuul:automatic-ci","change_message_id":"4ee86ba27fca969230063c9de96942008b4771fe","unresolved":false,"context_lines":[{"line_number":227,"context_line":"        )"},{"line_number":228,"context_line":""},{"line_number":229,"context_line":"        # create a volume and attach it to an instance"},{"line_number":230,"context_line":"        instance \u003d self.create_server("},{"line_number":231,"context_line":"            image_id\u003dCONF.compute.image_ref,"},{"line_number":232,"context_line":"            wait_until\u003d\u0027ACTIVE\u0027,"},{"line_number":233,"context_line":"            validatable\u003dFalse,"}],"source_content_type":"text/x-python","patch_set":1,"id":"7626f5b4_a4144e19","line":230,"updated":"2026-06-29 10:50:24.000000000","message":"The three create_server blocks are now byte-for-byte identical (image_id, wait_until, validatable, clients). This triplicates a small but complete pattern that is a candidate for a shared helper.\n\n**Severity**: SUGGESTION | **Confidence**: 0.8\n\n**Benefit**: A small helper such as _create_test_instance() in the VolumeBase class would centralize the server-creation kwargs and keep future changes (e.g. flavor, clients) consistent across all volume tests.\n\n**Recommendation**:\nConsider adding a private helper on TestExecuteZoneMigrationStrategyVolumeBase that wraps self.create_server(image_id\u003dCONF.compute.image_ref, wait_until\u003d\u0027ACTIVE\u0027, validatable\u003dFalse, clients\u003dself.os_primary) and call it from all three sites. Not blocking; the duplication is largely pre-existing.","commit_id":"81ac9487ded7994dfd41976afee4aa283ac245e4"}]}