)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":10239,"name":"Dmitry Tantsur","email":"dtantsur@protonmail.com","username":"dtantsur"},"change_message_id":"328bbf28942d272995f1f144967452eb43c2d8bb","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"6cce3085_db003536","updated":"2023-06-14 10:00:20.000000000","message":"All great, I just think the scope can be reduced a bit - see inline.","commit_id":"ec423be695dee1906f01e38e2a475c59167cf609"},{"author":{"_account_id":23851,"name":"Riccardo Pittau","email":"elfosardo@gmail.com","username":"elfosardo"},"change_message_id":"7de6ce36ae71e4a2e543b15dd7c06282f839e2c3","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"c508dd3d_cea53b96","updated":"2023-06-20 08:34:34.000000000","message":"thanks!","commit_id":"a087bc0b4bacb288884312ac9c24a0804498f9d9"}],"openstack/cloud/_baremetal.py":[{"author":{"_account_id":10239,"name":"Dmitry Tantsur","email":"dtantsur@protonmail.com","username":"dtantsur"},"change_message_id":"328bbf28942d272995f1f144967452eb43c2d8bb","unresolved":true,"context_lines":[{"line_number":653,"context_line":"        )"},{"line_number":654,"context_line":"        self.baremetal.wait_for_node_reservation(node, timeout)"},{"line_number":655,"context_line":""},{"line_number":656,"context_line":"    def get_machine_inventory(self, name_or_id):"},{"line_number":657,"context_line":"        \"\"\"Get hardware inventory by Machine name or uuid"},{"line_number":658,"context_line":""},{"line_number":659,"context_line":"        :param name_or_id: A node name or UUID whose inventory will be looked up."}],"source_content_type":"text/x-python","patch_set":1,"id":"4b83ae7d_e803f951","line":656,"updated":"2023-06-14 10:00:20.000000000","message":"I suggest we don\u0027t add this method. This file is an attempt to create a very high-level baremetal API, and node inventory is in my opinion not high-level enough.","commit_id":"ec423be695dee1906f01e38e2a475c59167cf609"},{"author":{"_account_id":35929,"name":"Mahnoor Asghar","display_name":"Mahnoor Asghar","email":"masghar@redhat.com","username":"mahnoorasghar"},"change_message_id":"e6d1ccef05bc03da1537508f0de4098af81c1d04","unresolved":true,"context_lines":[{"line_number":653,"context_line":"        )"},{"line_number":654,"context_line":"        self.baremetal.wait_for_node_reservation(node, timeout)"},{"line_number":655,"context_line":""},{"line_number":656,"context_line":"    def get_machine_inventory(self, name_or_id):"},{"line_number":657,"context_line":"        \"\"\"Get hardware inventory by Machine name or uuid"},{"line_number":658,"context_line":""},{"line_number":659,"context_line":"        :param name_or_id: A node name or UUID whose inventory will be looked up."}],"source_content_type":"text/x-python","patch_set":1,"id":"856d0772_9b69dad7","line":656,"in_reply_to":"4b83ae7d_e803f951","updated":"2023-06-14 10:18:40.000000000","message":"I see. I have removed the addition to the cloud layer.","commit_id":"ec423be695dee1906f01e38e2a475c59167cf609"}]}