)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":7166,"name":"Sylvain Bauza","email":"sbauza@redhat.com","username":"sbauza"},"change_message_id":"e52ccb515c55e5b4f5bf77bf5dd714c348952aaa","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"0285e017_c4abb2c1","updated":"2022-02-23 12:11:46.000000000","message":"Don\u0027t see any issues so far.","commit_id":"19cd4bf6707b71dfc482a2fe5dcaa5b09b2fd84d"},{"author":{"_account_id":9708,"name":"Balazs Gibizer","display_name":"gibi","email":"gibizer@gmail.com","username":"gibi"},"change_message_id":"c961ae2ba9d93b36a1cd6ee1be0e009fb7217dfe","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"ba06a0fc_3d9a0000","updated":"2022-02-24 16:52:20.000000000","message":"I reread the series this morning and it looks good (the assisted snapshot is now fixed too). So I upgrading my vote to +2. Most of the changes are disable by default and the testing well covers the behavior of the APIs. So I believe it is safe to land this.","commit_id":"19cd4bf6707b71dfc482a2fe5dcaa5b09b2fd84d"}],"nova/tests/unit/policies/test_servers.py":[{"author":{"_account_id":4393,"name":"Dan Smith","email":"dms@danplanet.com","username":"danms"},"change_message_id":"0e692650f94298d1c35ed55d3be3fddd7c9ad569","unresolved":true,"context_lines":[{"line_number":834,"context_line":"    @mock.patch(\u0027nova.compute.api.API.rebuild\u0027)"},{"line_number":835,"context_line":"    def test_server_rebuild_with_extended_attr_policy(self, mock_rebuild,"},{"line_number":836,"context_line":"        mock_get, mock_bdm):"},{"line_number":837,"context_line":"        mock_rebuild.return_value \u003d self.instance"},{"line_number":838,"context_line":"        rule \u003d policies.SERVERS % \u0027rebuild\u0027"},{"line_number":839,"context_line":"        # server \u0027rebuild\u0027 policy is checked before extended attributes"},{"line_number":840,"context_line":"        # policy so we have to allow it for everyone otherwise it will fail"}],"source_content_type":"text/x-python","patch_set":1,"id":"3458d045_787e4b1b","line":837,"updated":"2022-02-16 14:37:48.000000000","message":"I\u0027m sure there\u0027s a reason, but why did you need to add these return values?","commit_id":"de65684c449bba03a779575a78e3b4dbcdfa155c"},{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"eda491308e1dead36538db91cffb06ec30527dff","unresolved":true,"context_lines":[{"line_number":834,"context_line":"    @mock.patch(\u0027nova.compute.api.API.rebuild\u0027)"},{"line_number":835,"context_line":"    def test_server_rebuild_with_extended_attr_policy(self, mock_rebuild,"},{"line_number":836,"context_line":"        mock_get, mock_bdm):"},{"line_number":837,"context_line":"        mock_rebuild.return_value \u003d self.instance"},{"line_number":838,"context_line":"        rule \u003d policies.SERVERS % \u0027rebuild\u0027"},{"line_number":839,"context_line":"        # server \u0027rebuild\u0027 policy is checked before extended attributes"},{"line_number":840,"context_line":"        # policy so we have to allow it for everyone otherwise it will fail"}],"source_content_type":"text/x-python","patch_set":1,"id":"b64c77ce_837d43af","line":837,"in_reply_to":"3458d045_787e4b1b","updated":"2022-02-17 05:00:03.000000000","message":"I over updated it. for update_instance we need to add return value as it return the same instance from this call from API. but for rebuild that is not case and only return value on get_instance work fine. \n\nDone.","commit_id":"de65684c449bba03a779575a78e3b4dbcdfa155c"}]}