)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":11655,"name":"Julia Kreger","email":"juliaashleykreger@gmail.com","username":"jkreger","status":"Flying to the moon with a Jetpack!"},"change_message_id":"589dfda5944f316eaa87f52080685bb53ffe84e0","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"77ddc3d3_5b72cb45","updated":"2026-10-08 17:50:21.000000000","message":"Overall, I think it is good, but I\u0027d just prefer a slight expansion on the unit test. Technically the add case covers it, but we should cover it in both cases just in case the base class is ever changed.","commit_id":"0fc04ce2b0ba14aa05ca1aac00405fa77bc38f05"}],"ironicclient/osc/v1/baremetal_portgroup.py":[{"author":{"_account_id":11655,"name":"Julia Kreger","email":"juliaashleykreger@gmail.com","username":"jkreger","status":"Flying to the moon with a Jetpack!"},"change_message_id":"589dfda5944f316eaa87f52080685bb53ffe84e0","unresolved":false,"context_lines":[{"line_number":591,"context_line":"        failures: list[str] \u003d []"},{"line_number":592,"context_line":"        for port in parsed_args.ports:"},{"line_number":593,"context_line":"            try:"},{"line_number":594,"context_line":"                self._update_port(client, portgroup_uuid, port)"},{"line_number":595,"context_line":"            except exc.ClientException as e:"},{"line_number":596,"context_line":"                failures.append("},{"line_number":597,"context_line":"                    _(\"Failed to update port %(port)s: %(error)s\")"}],"source_content_type":"text/x-python","patch_set":2,"id":"182bddb7_7396bed6","line":594,"updated":"2026-10-08 17:50:21.000000000","message":"Cool cool, getting called in a loop.","commit_id":"0fc04ce2b0ba14aa05ca1aac00405fa77bc38f05"}],"ironicclient/tests/unit/osc/v1/test_baremetal_portgroup.py":[{"author":{"_account_id":11655,"name":"Julia Kreger","email":"juliaashleykreger@gmail.com","username":"jkreger","status":"Flying to the moon with a Jetpack!"},"change_message_id":"589dfda5944f316eaa87f52080685bb53ffe84e0","unresolved":true,"context_lines":[{"line_number":959,"context_line":""},{"line_number":960,"context_line":"        patch \u003d [{\u0027op\u0027: \u0027add\u0027, \u0027path\u0027: \u0027/portgroup_uuid\u0027,"},{"line_number":961,"context_line":"                  \u0027value\u0027: baremetal_fakes.baremetal_portgroup_uuid}]"},{"line_number":962,"context_line":"        self.baremetal_mock.port.update.assert_has_calls("},{"line_number":963,"context_line":"            [mock.call(\u0027port-1\u0027, patch), mock.call(\u0027port-2\u0027, patch)])"},{"line_number":964,"context_line":""},{"line_number":965,"context_line":"    def test_add_no_ports(self) -\u003e None:"},{"line_number":966,"context_line":"        cmd \u003d baremetal_portgroup.AddBaremetalPortGroup(self.app, None)"}],"source_content_type":"text/x-python","patch_set":2,"id":"c09eb78a_6674e1b0","line":963,"range":{"start_line":962,"start_character":19,"end_line":963,"end_character":69},"updated":"2026-10-08 17:50:21.000000000","message":"I guess I\u0027m glad this is here, because it is proving both calls get across which matches the text description.","commit_id":"0fc04ce2b0ba14aa05ca1aac00405fa77bc38f05"},{"author":{"_account_id":11655,"name":"Julia Kreger","email":"juliaashleykreger@gmail.com","username":"jkreger","status":"Flying to the moon with a Jetpack!"},"change_message_id":"589dfda5944f316eaa87f52080685bb53ffe84e0","unresolved":true,"context_lines":[{"line_number":975,"context_line":"                       baremetal_fakes.baremetal_portgroup_uuid},"},{"line_number":976,"context_line":"                loaded\u003dTrue))"},{"line_number":977,"context_line":"        cmd \u003d baremetal_portgroup.RemoveBaremetalPortGroup(self.app, None)"},{"line_number":978,"context_line":"        self._run(cmd, \u0027port-1\u0027)"},{"line_number":979,"context_line":""},{"line_number":980,"context_line":"        self.baremetal_mock.port.update.assert_called_once_with("},{"line_number":981,"context_line":"            \u0027port-1\u0027, [{\u0027op\u0027: \u0027remove\u0027, \u0027path\u0027: \u0027/portgroup_uuid\u0027}])"}],"source_content_type":"text/x-python","patch_set":2,"id":"aad9a9c6_b45b86e0","line":978,"updated":"2026-10-08 17:50:21.000000000","message":"But here, overall, think we need to attempt to exercise the multiple entries being removed to match the description text.","commit_id":"0fc04ce2b0ba14aa05ca1aac00405fa77bc38f05"}]}