)]}'
{"cinder/api/v2/types.py":[{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"03701a40c27bc9c1ba9f9c51d0573f49bc65478b","unresolved":true,"context_lines":[{"line_number":64,"context_line":"        context.authorize(type_policy.GET_POLICY, target_obj\u003dvol_type)"},{"line_number":65,"context_line":"        return self._view_builder.show(req, vol_type)"},{"line_number":66,"context_line":""},{"line_number":67,"context_line":"    @common.process_general_filtering(\u0027volume_type\u0027)"},{"line_number":68,"context_line":"    def _process_volume_type_filtering(self, context\u003dNone, filters\u003dNone,"},{"line_number":69,"context_line":"                                       req_version\u003dNone):"},{"line_number":70,"context_line":"        api_utils.remove_invalid_filter_options("}],"source_content_type":"text/x-python","patch_set":2,"id":"a1179f21_7a0b06ce","line":67,"updated":"2021-03-24 04:41:40.000000000","message":"This wrapper changes the behavior of this function when it\u0027s called with a mv \u003e\u003d 3.52 ... it won\u0027t execute lines 70-73, and so it won\u0027t actually call the function you\u0027ve modified at lines 114-115.","commit_id":"70295d3b9774b1464909eba745addba3809db2d8"},{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"03701a40c27bc9c1ba9f9c51d0573f49bc65478b","unresolved":true,"context_lines":[{"line_number":80,"context_line":"        filters \u003d params"},{"line_number":81,"context_line":"        context \u003d req.environ[\u0027cinder.context\u0027]"},{"line_number":82,"context_line":"        req_version \u003d req.api_version_request"},{"line_number":83,"context_line":"        if req_version.matches(mv.SUPPORT_VOLUME_TYPE_FILTER):"},{"line_number":84,"context_line":"            self._process_volume_type_filtering(context\u003dcontext,"},{"line_number":85,"context_line":"                                                filters\u003dfilters,"},{"line_number":86,"context_line":"                                                req_version\u003dreq_version)"}],"source_content_type":"text/x-python","patch_set":2,"id":"9262c1aa_8f9f3863","line":83,"range":{"start_line":83,"start_character":12,"end_line":83,"end_character":62},"updated":"2021-03-24 04:41:40.000000000","message":"Although this class \"lives\" in the v2 directory, it\u0027s also used by the v3 API.  Filtering the list is only available for microversion 3.52+.\n\nAlso, this is a bit different from the regular resource filtering introduced in mv 3.31, because the az is stored in the volume type extra specs.  You can see what I mean if you look at the tests/unit/api/v3/test_types.py","commit_id":"70295d3b9774b1464909eba745addba3809db2d8"}],"cinder/tests/unit/api/v2/test_types.py":[{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"03701a40c27bc9c1ba9f9c51d0573f49bc65478b","unresolved":true,"context_lines":[{"line_number":544,"context_line":"                                 output[\u0027volume_types\u0027][i])"},{"line_number":545,"context_line":""},{"line_number":546,"context_line":"    @mock.patch(\u0027cinder.api.common.get_enabled_resource_filters\u0027)"},{"line_number":547,"context_line":"    def test_get_vol_type_filter_options(self, mock_get_filters):"},{"line_number":548,"context_line":"        self.controller._get_vol_type_filter_options()"},{"line_number":549,"context_line":"        mock_get_filters.assert_called_once_with("},{"line_number":550,"context_line":"            \u0027volume_type\u0027)"}],"source_content_type":"text/x-python","patch_set":2,"id":"558c6e7e_0a1512fb","line":547,"updated":"2021-03-24 04:41:40.000000000","message":"Resource filtering is only available in v3, so this test should fail.\n\nThe passing test should be in cinder/tests/unit/api/v3/test_types.py","commit_id":"70295d3b9774b1464909eba745addba3809db2d8"}]}