)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"1a3463733ee0cc21b8ec6a9a670785f30577369b","unresolved":true,"context_lines":[{"line_number":12,"context_line":"This behavior is unexpected according to the documentation of the"},{"line_number":13,"context_line":"strategy."},{"line_number":14,"context_line":""},{"line_number":15,"context_line":"This patch is fixing that behavior and making sure that not duplicated"},{"line_number":16,"context_line":"actions are added to the solution, to handle the case where instances"},{"line_number":17,"context_line":"migration actions are created when analyzing the volumes if the"},{"line_number":18,"context_line":"`with_attached_volume` parameter is enabled. The patch is also removing"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":9,"id":"b2106e10_4f764f49","line":15,"range":{"start_line":15,"start_character":56,"end_line":15,"end_character":59},"updated":"2025-09-16 17:59:09.000000000","message":"nit: no","commit_id":"296856101f7cafe1b687119d9a88101e7c9e1af6"}],"/PATCHSET_LEVEL":[{"author":{"_account_id":34452,"name":"Joan Gilabert","display_name":"jgilaber","email":"jgilaber@redhat.com","username":"jgilaber"},"change_message_id":"1537a4242c49b63a0836f71e6491b2c4544484c2","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"ca8858ba_b11c09ce","updated":"2025-06-18 10:41:19.000000000","message":"lgtm, thanks for the fix Alfredo!","commit_id":"2092e2b09258396312bb5612e4ad0d170032f0a9"},{"author":{"_account_id":12393,"name":"chandan kumar","display_name":"Chandan Kumar","email":"chkumar@redhat.com","username":"chkumar246"},"change_message_id":"dd15141400caea197edd87e28e94e4fb251b53db","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"c0a02f0a_645dbc36","updated":"2025-06-24 04:09:36.000000000","message":"lgtm!","commit_id":"f1a33854d5c32d41797039d06fcf7d08c405fdb2"},{"author":{"_account_id":34452,"name":"Joan Gilabert","display_name":"jgilaber","email":"jgilaber@redhat.com","username":"jgilaber"},"change_message_id":"eec196c41cc89d25145a053c80900fffd8660f3c","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"c38c7162_431c0e78","updated":"2025-08-19 06:19:47.000000000","message":"@amoralej@redhat.com I had to sign off this patch to upload one that I had rebased on top of it, feel free to override it","commit_id":"023f6123dea1569079c8b6a5be2aa6f53ac95e49"},{"author":{"_account_id":16312,"name":"Alfredo Moralejo","email":"amoralej@redhat.com","username":"amoralej"},"change_message_id":"49e6202ca56e23f3f73e06da01185a430200690e","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"52582967_0e8bac1b","in_reply_to":"c38c7162_431c0e78","updated":"2025-08-20 11:57:19.000000000","message":"no problem, thanks for the rebase.","commit_id":"023f6123dea1569079c8b6a5be2aa6f53ac95e49"},{"author":{"_account_id":30002,"name":"Douglas Viroel","email":"viroel@gmail.com","username":"dviroel"},"change_message_id":"b68f915d5340c78bbfb4caeeea3d396a327b9580","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":6,"id":"d895302c_a7017869","updated":"2025-09-02 12:32:54.000000000","message":"lgtm","commit_id":"c429b88362eb09b9b38d689e1c73a8f06afb3912"},{"author":{"_account_id":30002,"name":"Douglas Viroel","email":"viroel@gmail.com","username":"dviroel"},"change_message_id":"4fdcac896d7fcc5ace810b2431334c85b1e20abc","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":9,"id":"3d3efc36_884ef5e1","updated":"2025-09-16 13:17:46.000000000","message":"adding vote again, patch was rebased","commit_id":"296856101f7cafe1b687119d9a88101e7c9e1af6"}],"watcher/decision_engine/strategy/strategies/zone_migration.py":[{"author":{"_account_id":34452,"name":"Joan Gilabert","display_name":"jgilaber","email":"jgilaber@redhat.com","username":"jgilaber"},"change_message_id":"83ff671ffa13891bdfed578a6993ded66d3dbfb3","unresolved":true,"context_lines":[{"line_number":416,"context_line":""},{"line_number":417,"context_line":"        for instance in instances:"},{"line_number":418,"context_line":"            if self._instance_migration_exists(instance.id):"},{"line_number":419,"context_line":"                LOG.debug(\"A migration action for instance %s already exist\","},{"line_number":420,"context_line":"                          instance.id)"},{"line_number":421,"context_line":"            src_node \u003d getattr(instance, \u0027OS-EXT-SRV-ATTR:host\u0027)"},{"line_number":422,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"882599f3_781688bf","line":419,"updated":"2025-06-11 14:20:17.000000000","message":"shouldn\u0027t we add a `continue` here to avoid adding the migration again?","commit_id":"856a3d5116272274688a7e660ac83f5e8017fa94"},{"author":{"_account_id":16312,"name":"Alfredo Moralejo","email":"amoralej@redhat.com","username":"amoralej"},"change_message_id":"22438faab305cb8d9bc2a88ff36b7f1bfc11d7f5","unresolved":false,"context_lines":[{"line_number":416,"context_line":""},{"line_number":417,"context_line":"        for instance in instances:"},{"line_number":418,"context_line":"            if self._instance_migration_exists(instance.id):"},{"line_number":419,"context_line":"                LOG.debug(\"A migration action for instance %s already exist\","},{"line_number":420,"context_line":"                          instance.id)"},{"line_number":421,"context_line":"            src_node \u003d getattr(instance, \u0027OS-EXT-SRV-ATTR:host\u0027)"},{"line_number":422,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"56bfe16a_305e9769","line":419,"in_reply_to":"882599f3_781688bf","updated":"2025-06-12 06:28:46.000000000","message":"yes, thanks!","commit_id":"856a3d5116272274688a7e660ac83f5e8017fa94"},{"author":{"_account_id":30002,"name":"Douglas Viroel","email":"viroel@gmail.com","username":"dviroel"},"change_message_id":"b68f915d5340c78bbfb4caeeea3d396a327b9580","unresolved":true,"context_lines":[{"line_number":295,"context_line":"            if k \u003d\u003d VOLUME:"},{"line_number":296,"context_line":"                self.volumes_migration(targets, action_counter)"},{"line_number":297,"context_line":"            elif k \u003d\u003d INSTANCE:"},{"line_number":298,"context_line":"                if self.volume_count \u003d\u003d 0 and self.volume_update_count \u003d\u003d 0:"},{"line_number":299,"context_line":"                    # if with_attached_volume is true,"},{"line_number":300,"context_line":"                    # instance having attached volumes already migrated,"},{"line_number":301,"context_line":"                    # migrate instances which does not have attached volumes"}],"source_content_type":"text/x-python","patch_set":6,"id":"c4f4048b_f8dfbbdb","side":"PARENT","line":298,"range":{"start_line":298,"start_character":16,"end_line":298,"end_character":76},"updated":"2025-09-02 12:32:54.000000000","message":"have you figure it out why this check exist? I still didn\u0027t understand why it was here.","commit_id":"cb08e4d7201657e110858534af7c5c48f8e60f7b"},{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"1a3463733ee0cc21b8ec6a9a670785f30577369b","unresolved":false,"context_lines":[{"line_number":295,"context_line":"            if k \u003d\u003d VOLUME:"},{"line_number":296,"context_line":"                self.volumes_migration(targets, action_counter)"},{"line_number":297,"context_line":"            elif k \u003d\u003d INSTANCE:"},{"line_number":298,"context_line":"                if self.volume_count \u003d\u003d 0 and self.volume_update_count \u003d\u003d 0:"},{"line_number":299,"context_line":"                    # if with_attached_volume is true,"},{"line_number":300,"context_line":"                    # instance having attached volumes already migrated,"},{"line_number":301,"context_line":"                    # migrate instances which does not have attached volumes"}],"source_content_type":"text/x-python","patch_set":6,"id":"4024758f_eedb3eee","side":"PARENT","line":298,"range":{"start_line":298,"start_character":16,"end_line":298,"end_character":76},"in_reply_to":"c4f4048b_f8dfbbdb","updated":"2025-09-16 17:59:09.000000000","message":"given it nolonger exists in v9 im going to assume it was remvoe dbecause it hasd no useful benift after the refactor and it has been removed.","commit_id":"cb08e4d7201657e110858534af7c5c48f8e60f7b"}]}