)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"1f43e31a9bb4692bc130af9b56503c89d58edcd5","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"cf0c0733_4ce59bd4","updated":"2023-06-09 16:38:34.000000000","message":"Sorry 🙈 No serious concerns with the change but I don\u0027t want to start importing objects.","commit_id":"2db47df2ccac63e1ab1c4a4ef968fa69ed192ae8"},{"author":{"_account_id":27900,"name":"Artem Goncharov","email":"artem.goncharov@gmail.com","username":"gtema"},"change_message_id":"cfa8413878b81d1c604a0ae65362ab952b23c058","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"7abaf549_d1127726","in_reply_to":"cf0c0733_4ce59bd4","updated":"2023-06-18 10:35:46.000000000","message":"No problem.","commit_id":"2db47df2ccac63e1ab1c4a4ef968fa69ed192ae8"}],"openstackclient/tests/unit/compute/v2/test_aggregate.py":[{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"1f43e31a9bb4692bc130af9b56503c89d58edcd5","unresolved":true,"context_lines":[{"line_number":16,"context_line":"from unittest import mock"},{"line_number":17,"context_line":"from unittest.mock import call"},{"line_number":18,"context_line":""},{"line_number":19,"context_line":"from openstack.compute.v2.aggregate import Aggregate"},{"line_number":20,"context_line":"from openstack import exceptions as sdk_exceptions"},{"line_number":21,"context_line":"from openstack.test import fakes as sdk_fakes"},{"line_number":22,"context_line":"from openstack import utils as sdk_utils"}],"source_content_type":"text/x-python","patch_set":1,"id":"e4c25bdb_19ce6958","line":19,"updated":"2023-06-09 16:38:34.000000000","message":"Do not import objects, only modules [1]\n\n[1] https://docs.openstack.org/hacking/latest/user/hacking.html#imports","commit_id":"2db47df2ccac63e1ab1c4a4ef968fa69ed192ae8"},{"author":{"_account_id":27900,"name":"Artem Goncharov","email":"artem.goncharov@gmail.com","username":"gtema"},"change_message_id":"cfa8413878b81d1c604a0ae65362ab952b23c058","unresolved":false,"context_lines":[{"line_number":16,"context_line":"from unittest import mock"},{"line_number":17,"context_line":"from unittest.mock import call"},{"line_number":18,"context_line":""},{"line_number":19,"context_line":"from openstack.compute.v2.aggregate import Aggregate"},{"line_number":20,"context_line":"from openstack import exceptions as sdk_exceptions"},{"line_number":21,"context_line":"from openstack.test import fakes as sdk_fakes"},{"line_number":22,"context_line":"from openstack import utils as sdk_utils"}],"source_content_type":"text/x-python","patch_set":1,"id":"8e2b0924_e0ef4a59","line":19,"in_reply_to":"e4c25bdb_19ce6958","updated":"2023-06-18 10:35:46.000000000","message":"Done","commit_id":"2db47df2ccac63e1ab1c4a4ef968fa69ed192ae8"},{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"1f43e31a9bb4692bc130af9b56503c89d58edcd5","unresolved":true,"context_lines":[{"line_number":31,"context_line":"class TestAggregate(compute_fakes.TestComputev2):"},{"line_number":32,"context_line":"    fake_ag \u003d sdk_fakes.generate_fake_resource("},{"line_number":33,"context_line":"        Aggregate, metadata\u003d{\"availability_zone\": \"ag_zone\", \"key1\": \"value1\"}"},{"line_number":34,"context_line":"    )"},{"line_number":35,"context_line":""},{"line_number":36,"context_line":"    columns \u003d ("},{"line_number":37,"context_line":"        \u0027availability_zone\u0027,"}],"source_content_type":"text/x-python","patch_set":1,"id":"90232ced_8f923a88","line":34,"updated":"2023-06-09 16:38:34.000000000","message":"nit: We don\u0027t need to do this here, but this would be better defined in setUp to avoid cross-contamination between test runs.","commit_id":"2db47df2ccac63e1ab1c4a4ef968fa69ed192ae8"},{"author":{"_account_id":27900,"name":"Artem Goncharov","email":"artem.goncharov@gmail.com","username":"gtema"},"change_message_id":"cfa8413878b81d1c604a0ae65362ab952b23c058","unresolved":true,"context_lines":[{"line_number":31,"context_line":"class TestAggregate(compute_fakes.TestComputev2):"},{"line_number":32,"context_line":"    fake_ag \u003d sdk_fakes.generate_fake_resource("},{"line_number":33,"context_line":"        Aggregate, metadata\u003d{\"availability_zone\": \"ag_zone\", \"key1\": \"value1\"}"},{"line_number":34,"context_line":"    )"},{"line_number":35,"context_line":""},{"line_number":36,"context_line":"    columns \u003d ("},{"line_number":37,"context_line":"        \u0027availability_zone\u0027,"}],"source_content_type":"text/x-python","patch_set":1,"id":"01f0dbd0_ac18b35f","line":34,"in_reply_to":"90232ced_8f923a88","updated":"2023-06-18 10:35:46.000000000","message":"I didn\u0027t want to change anything except of switching to new fakes, but ok, addressed.","commit_id":"2db47df2ccac63e1ab1c4a4ef968fa69ed192ae8"}]}