)]}'
{"patrole_tempest_plugin/tests/api/compute/test_images_rbac.py":[{"author":{"_account_id":23186,"name":"Felipe Monteiro","email":"felipe.carneiro.monteiro@gmail.com","username":"felipe.monteiro"},"change_message_id":"9e8b4e2f83912b13606948316f272f2a470108f3","unresolved":false,"context_lines":[{"line_number":52,"context_line":"        if CONF.image_feature_enabled.api_v2:"},{"line_number":53,"context_line":"            cls.glance_image_client \u003d cls.os_primary.image_client_v2"},{"line_number":54,"context_line":"        elif CONF.image_feature_enabled.api_v1:"},{"line_number":55,"context_line":"            cls.glance_image_client \u003d cls.os_primary.image_client"},{"line_number":56,"context_line":"        else:"},{"line_number":57,"context_line":"            raise lib_exc.InvalidConfiguration("},{"line_number":58,"context_line":"                \u0027Either api_v1 or api_v2 must be True in \u0027"}],"source_content_type":"text/x-python","patch_set":1,"id":"7f287b81_55310c0b","line":55,"updated":"2017-08-23 20:32:19.000000000","message":"This doesn\u0027t fix the issue; it just swaps the order. Users with only glance v1 enabled will still run into error.","commit_id":"d552e8e9a22afc031346d29453196cdffe4484a1"},{"author":{"_account_id":23186,"name":"Felipe Monteiro","email":"felipe.carneiro.monteiro@gmail.com","username":"felipe.monteiro"},"change_message_id":"9e8b4e2f83912b13606948316f272f2a470108f3","unresolved":false,"context_lines":[{"line_number":61,"context_line":"    @classmethod"},{"line_number":62,"context_line":"    def resource_setup(cls):"},{"line_number":63,"context_line":"        super(ImagesRbacTest, cls).resource_setup()"},{"line_number":64,"context_line":"        cls.image \u003d cls.glance_image_client.create_image("},{"line_number":65,"context_line":"            name\u003ddata_utils.rand_name(cls.__name__ + \u0027-image\u0027))"},{"line_number":66,"context_line":""},{"line_number":67,"context_line":"    @classmethod"},{"line_number":68,"context_line":"    def resource_cleanup(cls):"}],"source_content_type":"text/x-python","patch_set":1,"id":"7f287b81_f54a387e","line":65,"range":{"start_line":64,"start_character":0,"end_line":65,"end_character":63},"updated":"2017-08-23 20:32:19.000000000","message":"The real fix for this will be something like:\n\n        params \u003d {\n            \u0027name\u0027: data_utils.rand_name(cls.__name__ + \u0027-image\u0027)\n        }\n        if CONF.image_feature_enabled.api_v1:\n            params \u003d {\u0027headers\u0027: common_image.image_meta_to_headers(**params)}\n        cls.image \u003d cls.glance_image_client.create_image(**params)\n\nSource: https://github.com/openstack/tempest/blob/5fc2548eae225ebe84ab378a6136f1be9cb8b443/tempest/api/compute/images/test_image_metadata.py","commit_id":"d552e8e9a22afc031346d29453196cdffe4484a1"}]}