)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":16312,"name":"Alfredo Moralejo","email":"amoralej@redhat.com","username":"amoralej"},"change_message_id":"2d3b16b6fb0dfea4fa25e84e3fec177f64fa40f8","unresolved":true,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Move function for creating instances from 1 to variable number"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Current behavior for creating multiple instances per host iterating over create_one_instance_per_host function is working fine because this block https://opendev.org/openstack/watcher-tempest-plugin/src/branch/master/watcher_tempest_plugin/tests/scenario/base.py#L263 never return anything, as it is executed as non admin and instances are on a different project."},{"line_number":10,"context_line":""},{"line_number":11,"context_line":"If any time we change the client to be admin, like on https://review.opendev.org/c/openstack/watcher-tempest-plugin/+/950389, iterating over create_one_instance_per_host will fail."},{"line_number":12,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":4,"id":"6a9ce14c_20c79fcf","line":9,"range":{"start_line":9,"start_character":146,"end_line":9,"end_character":267},"updated":"2025-06-05 17:18:50.000000000","message":"linking code lines from a branch is usually not a good idea as it will not point to the expected content as soon as there is any change in the file. The recommendation is to use a specific commit, like:\n\nhttps://opendev.org/openstack/watcher-tempest-plugin/src/commit/8aa31042302740929043f866e5540cf95af3ebf2/watcher_tempest_plugin/tests/scenario/base.py#L263\n\nAlso, commit message lines should be wrapped at 72 characters.\n\nhttps://wiki.openstack.org/wiki/GitCommitMessages","commit_id":"a8597e5c2a5f0cc00e69f9edb82f64a2a0035d55"},{"author":{"_account_id":28647,"name":"David","display_name":"morenod","email":"dsanzmor@redhat.com","username":"morenod"},"change_message_id":"ca0ce8ae3a96e251a08039e27aab622af7f6ae6a","unresolved":false,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Move function for creating instances from 1 to variable number"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Current behavior for creating multiple instances per host iterating over create_one_instance_per_host function is working fine because this block https://opendev.org/openstack/watcher-tempest-plugin/src/branch/master/watcher_tempest_plugin/tests/scenario/base.py#L263 never return anything, as it is executed as non admin and instances are on a different project."},{"line_number":10,"context_line":""},{"line_number":11,"context_line":"If any time we change the client to be admin, like on https://review.opendev.org/c/openstack/watcher-tempest-plugin/+/950389, iterating over create_one_instance_per_host will fail."},{"line_number":12,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":4,"id":"b6be5227_130df3c1","line":9,"range":{"start_line":9,"start_character":146,"end_line":9,"end_character":267},"in_reply_to":"6a9ce14c_20c79fcf","updated":"2025-06-06 07:49:00.000000000","message":"Done","commit_id":"a8597e5c2a5f0cc00e69f9edb82f64a2a0035d55"}],"/PATCHSET_LEVEL":[{"author":{"_account_id":30002,"name":"Douglas Viroel","email":"viroel@gmail.com","username":"dviroel"},"change_message_id":"2f5c7e958fbb92928138139d5387c3c2acdc9cbb","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":7,"id":"7f531e8e_211d72b5","updated":"2025-06-09 13:43:42.000000000","message":"LGTM, you just need to move out from WIP if you think that is ready now. Thanks David.","commit_id":"f83e498354d466e7c808ca5e3becf584457965a6"}]}