)]}'
{".zuul.yaml":[{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"b9b61c461803e133509aee91c8a5542c78700a73","unresolved":true,"context_lines":[{"line_number":11,"context_line":"        - watcher-tempest-functional"},{"line_number":12,"context_line":"        - watcher-tempest-actuator"},{"line_number":13,"context_line":"        - watcher-tempest-strategies"},{"line_number":14,"context_line":"        - watcher-prometheus-integration"},{"line_number":15,"context_line":"        - watcher-tempest-functional-2024-2"},{"line_number":16,"context_line":"        - watcher-tempest-functional-2024-1"},{"line_number":17,"context_line":"        - watcher-tempest-functional-2023-2"}],"source_content_type":"text/x-yaml","patch_set":8,"id":"a1c10200_3b33095b","line":14,"updated":"2025-04-04 13:39:03.000000000","message":"+1","commit_id":"8753a8d9f6ea20a6fd2b719bf0f6fc4081f6efdb"}],"/PATCHSET_LEVEL":[{"author":{"_account_id":12393,"name":"chandan kumar","display_name":"Chandan Kumar","email":"chkumar@redhat.com","username":"chkumar246"},"change_message_id":"2aec3563ffa270720301d2b2956ff1210e75f704","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"d0f9afde_60f32216","updated":"2025-04-01 04:07:36.000000000","message":"Overall looks good, Needs one improvement!","commit_id":"6bd06e8e1ec7c07cd3a7967b95956fb8a17b17ef"},{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"b9b61c461803e133509aee91c8a5542c78700a73","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":8,"id":"88a0b8bd_51cb13dd","updated":"2025-04-04 13:39:03.000000000","message":"if you fix up the comments this is good to go.\n\nmore verbose then i orgianlly hoped but more flexible too.","commit_id":"8753a8d9f6ea20a6fd2b719bf0f6fc4081f6efdb"},{"author":{"_account_id":12393,"name":"chandan kumar","display_name":"Chandan Kumar","email":"chkumar@redhat.com","username":"chkumar246"},"change_message_id":"bacb7377e921b44bb3504303768ccf4359b3c3a7","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":8,"id":"253ead70_16470f98","updated":"2025-04-02 04:36:38.000000000","message":"much better now!\ntest results: https://review.opendev.org/c/openstack/watcher/+/945627/7#message-c5c0934e8f16dc8201fa7ae0a4d507b054120ca7","commit_id":"8753a8d9f6ea20a6fd2b719bf0f6fc4081f6efdb"},{"author":{"_account_id":16312,"name":"Alfredo Moralejo","email":"amoralej@redhat.com","username":"amoralej"},"change_message_id":"a2fd3e3d505da9c7e8e799b4738612803a1cf17a","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":8,"id":"cc8e6034_79a20324","updated":"2025-04-01 13:20:00.000000000","message":"recheck","commit_id":"8753a8d9f6ea20a6fd2b719bf0f6fc4081f6efdb"},{"author":{"_account_id":16312,"name":"Alfredo Moralejo","email":"amoralej@redhat.com","username":"amoralej"},"change_message_id":"79f78dcb0e6b326cfbbf9d974f9f5a1f06abcdfb","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":9,"id":"ee84eeed_5203873a","updated":"2025-04-09 13:31:01.000000000","message":"check-rdo","commit_id":"8492ed11c3f43cd78455dae44b0af5b9b2c7ca9a"},{"author":{"_account_id":30002,"name":"Douglas Viroel","email":"viroel@gmail.com","username":"dviroel"},"change_message_id":"51c79e3ff1283c23470c214eeae036fe7e25a6f5","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":11,"id":"7d5964e6_6efc3dfd","updated":"2025-04-23 17:54:02.000000000","message":"LGTM, \nChecking test outputs, we can see that test_execute_host_maintenance_strategy is not injecting instance metrics, while test_execute_vm_workload_consolidation_strategy is [1]\n\n[1] https://322149cf6bb5d1d418ae-1e96e5dddd851f74afd1442b0a5110b5.ssl.cf1.rackcdn.com/openstack/a1221012e84b45ac9caf4ab7d7eb26e7/controller/logs/tempest_log.txt","commit_id":"4bf088e619334e2ad1a78f01574454b7afe2e20c"},{"author":{"_account_id":12393,"name":"chandan kumar","display_name":"Chandan Kumar","email":"chkumar@redhat.com","username":"chkumar246"},"change_message_id":"ca8d477feb6c9ebf632f4eeaf5e3ae4ee784c062","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":11,"id":"a989d7a5_03e9bde4","updated":"2025-04-21 11:23:40.000000000","message":"readding my votes, zuul.yaml got changed only due to rebase.","commit_id":"4bf088e619334e2ad1a78f01574454b7afe2e20c"}],"watcher_tempest_plugin/tests/scenario/base.py":[{"author":{"_account_id":30002,"name":"Douglas Viroel","email":"viroel@gmail.com","username":"dviroel"},"change_message_id":"51c79e3ff1283c23470c214eeae036fe7e25a6f5","unresolved":true,"context_lines":[{"line_number":235,"context_line":"        :param metrics: The metrics add to resource when using Gnocchi"},{"line_number":236,"context_line":"        :param run_command: the command you want to run in the new instances"},{"line_number":237,"context_line":"        :param inject: If set to True, inject metrics for created instances"},{"line_number":238,"context_line":"        :returns: A list of instance UUIDs."},{"line_number":239,"context_line":"        \"\"\""},{"line_number":240,"context_line":"        compute_nodes \u003d self.get_compute_nodes_setup()"},{"line_number":241,"context_line":"        instances \u003d self.mgr.servers_client.list_servers("}],"source_content_type":"text/x-python","patch_set":11,"id":"024c58a6_f166f484","line":238,"range":{"start_line":238,"start_character":0,"end_line":238,"end_character":43},"updated":"2025-04-23 17:54:02.000000000","message":"+1","commit_id":"4bf088e619334e2ad1a78f01574454b7afe2e20c"}],"watcher_tempest_plugin/tests/scenario/test_execute_basic_optim.py":[{"author":{"_account_id":12393,"name":"chandan kumar","display_name":"Chandan Kumar","email":"chkumar@redhat.com","username":"chkumar246"},"change_message_id":"2aec3563ffa270720301d2b2956ff1210e75f704","unresolved":true,"context_lines":[{"line_number":73,"context_line":"        # wait for compute model updates"},{"line_number":74,"context_line":"        self.wait_for_instances_in_model(instances)"},{"line_number":75,"context_line":"        self.make_host_statistic()"},{"line_number":76,"context_line":"        self._create_one_instance_per_host_with_statistic("},{"line_number":77,"context_line":"            inject\u003dINJECT_METRICS)"},{"line_number":78,"context_line":""},{"line_number":79,"context_line":"        _, goal \u003d self.client.show_goal(self.GOAL_NAME)"}],"source_content_type":"text/x-python","patch_set":5,"id":"2e62d8a1_3d883d35","line":76,"updated":"2025-04-01 04:07:36.000000000","message":"I might be wrong here. At Line 71, \n```\ninstances \u003d self._create_one_instance_per_host_with_statistic()\n```\nand we are adding the same method at Line 76.\n```\nself._create_one_instance_per_host_with_statistic(inject\u003dINJECT_METRICS)\n```\nI think we can drop Line 76 and modify line 71 like this.\n```\ninstances \u003d self._create_one_instance_per_host_with_statistic(inject\u003dINJECT_METRICS)\n```","commit_id":"6bd06e8e1ec7c07cd3a7967b95956fb8a17b17ef"},{"author":{"_account_id":16312,"name":"Alfredo Moralejo","email":"amoralej@redhat.com","username":"amoralej"},"change_message_id":"23d7dfde016d3e6fdc8d7515c94fb9fc9852b9fc","unresolved":false,"context_lines":[{"line_number":73,"context_line":"        # wait for compute model updates"},{"line_number":74,"context_line":"        self.wait_for_instances_in_model(instances)"},{"line_number":75,"context_line":"        self.make_host_statistic()"},{"line_number":76,"context_line":"        self._create_one_instance_per_host_with_statistic("},{"line_number":77,"context_line":"            inject\u003dINJECT_METRICS)"},{"line_number":78,"context_line":""},{"line_number":79,"context_line":"        _, goal \u003d self.client.show_goal(self.GOAL_NAME)"}],"source_content_type":"text/x-python","patch_set":5,"id":"450d7295_6d90a342","line":76,"in_reply_to":"2e62d8a1_3d883d35","updated":"2025-04-01 06:28:36.000000000","message":"Thanks!","commit_id":"6bd06e8e1ec7c07cd3a7967b95956fb8a17b17ef"}],"watcher_tempest_plugin/tests/scenario/test_execute_strategies.py":[{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"b9b61c461803e133509aee91c8a5542c78700a73","unresolved":true,"context_lines":[{"line_number":48,"context_line":"                \"skipping multinode tests.\")"},{"line_number":49,"context_line":""},{"line_number":50,"context_line":"    def test_execute_basic_strategy(self):"},{"line_number":51,"context_line":"        # This test requires metrics injection"},{"line_number":52,"context_line":"        INJECT_METRICS \u003d False"},{"line_number":53,"context_line":""},{"line_number":54,"context_line":"        self.addCleanup(self.rollback_compute_nodes_status)"},{"line_number":55,"context_line":"        self.addCleanup(self.wait_delete_instances_from_model)"}],"source_content_type":"text/x-python","patch_set":8,"id":"f491a913_22aee770","line":52,"range":{"start_line":51,"start_character":7,"end_line":52,"end_character":30},"updated":"2025-04-04 13:39:03.000000000","message":"you did not update the commend when you set it to false here and below","commit_id":"8753a8d9f6ea20a6fd2b719bf0f6fc4081f6efdb"},{"author":{"_account_id":16312,"name":"Alfredo Moralejo","email":"amoralej@redhat.com","username":"amoralej"},"change_message_id":"a3a13f2f350af3a082d574ec2eaea10b2c09d746","unresolved":true,"context_lines":[{"line_number":48,"context_line":"                \"skipping multinode tests.\")"},{"line_number":49,"context_line":""},{"line_number":50,"context_line":"    def test_execute_basic_strategy(self):"},{"line_number":51,"context_line":"        # This test requires metrics injection"},{"line_number":52,"context_line":"        INJECT_METRICS \u003d False"},{"line_number":53,"context_line":""},{"line_number":54,"context_line":"        self.addCleanup(self.rollback_compute_nodes_status)"},{"line_number":55,"context_line":"        self.addCleanup(self.wait_delete_instances_from_model)"}],"source_content_type":"text/x-python","patch_set":8,"id":"5fd67bde_0f379e87","line":52,"range":{"start_line":51,"start_character":7,"end_line":52,"end_character":30},"in_reply_to":"f491a913_22aee770","updated":"2025-04-08 10:25:55.000000000","message":"Thanks! fixed","commit_id":"8753a8d9f6ea20a6fd2b719bf0f6fc4081f6efdb"}]}