)]}'
{"cinder/tests/test_netapp_nfs.py":[{"author":{"_account_id":4355,"name":"Avishay Traeger","email":"avishay@stratoscale.com","username":"avishay-il"},"change_message_id":"8929f7696e47513ed7e5e20b502c3b2efb2e2b00","unresolved":false,"context_lines":[{"line_number":72,"context_line":""},{"line_number":73,"context_line":""},{"line_number":74,"context_line":"class NetappDirectCmodeNfsDriverTestCase(test.TestCase):"},{"line_number":75,"context_line":"    \"\"\"Test direct NetApp C Mode driver\"\"\""},{"line_number":76,"context_line":"    def setUp(self):"},{"line_number":77,"context_line":"        super(NetappDirectCmodeNfsDriverTestCase, self).setUp()"},{"line_number":78,"context_line":"        self._custom_setup()"}],"source_content_type":"text/x-python","patch_set":4,"id":"AAAAPn%2F%2Fz8Q%3D","line":75,"updated":"2013-06-20 16:46:24.000000000","message":"Docstring should end with a period.","commit_id":"d1318caab8b7fd256cc2108a690c7715714fee1e"},{"author":{"_account_id":4355,"name":"Avishay Traeger","email":"avishay@stratoscale.com","username":"avishay-il"},"change_message_id":"8929f7696e47513ed7e5e20b502c3b2efb2e2b00","unresolved":false,"context_lines":[{"line_number":78,"context_line":"        self._custom_setup()"},{"line_number":79,"context_line":""},{"line_number":80,"context_line":"    def test_create_snapshot(self):"},{"line_number":81,"context_line":"        \"\"\"Test snapshot can be created and deleted\"\"\""},{"line_number":82,"context_line":"        mox \u003d self.mox"},{"line_number":83,"context_line":"        drv \u003d self._driver"},{"line_number":84,"context_line":""}],"source_content_type":"text/x-python","patch_set":4,"id":"AAAAPn%2F%2Fz8M%3D","line":81,"updated":"2013-06-20 16:46:24.000000000","message":"Docstring should end with a period.","commit_id":"d1318caab8b7fd256cc2108a690c7715714fee1e"}],"cinder/volume/drivers/netapp/common.py":[{"author":{"_account_id":2243,"name":"John Griffith","email":"john.griffith8@gmail.com","username":"john-griffith"},"change_message_id":"3db11e99435bb5bcd810abfd39859e709741ea46","unresolved":false,"context_lines":[{"line_number":105,"context_line":"            storage_family, storage_protocol, *args, **kwargs):"},{"line_number":106,"context_line":"        \"\"\"\"Creates an appropriate driver based on family and protocol.\"\"\""},{"line_number":107,"context_line":"        LOG.info(_(\u0027Requested unified config: %(storage_family)s and \u0027"},{"line_number":108,"context_line":"                   \u0027%(storage_protocol)s\u0027) % locals())"},{"line_number":109,"context_line":"        storage_family \u003d storage_family.lower()"},{"line_number":110,"context_line":"        family_meta \u003d netapp_unified_plugin_registry.get(storage_family)"},{"line_number":111,"context_line":"        if not family_meta:"}],"source_content_type":"text/x-python","patch_set":5,"id":"AAAAPn%2F%2Fz0M%3D","line":108,"updated":"2013-06-20 17:51:00.000000000","message":"Should remove locals here as well..","commit_id":"c2062632d15f98a58cee99fe1e0eded46059cbaf"},{"author":{"_account_id":2243,"name":"John Griffith","email":"john.griffith8@gmail.com","username":"john-griffith"},"change_message_id":"3db11e99435bb5bcd810abfd39859e709741ea46","unresolved":false,"context_lines":[{"line_number":118,"context_line":"            raise exception.InvalidInput("},{"line_number":119,"context_line":"                reason\u003d_(\u0027No default storage protocol found\u0027"},{"line_number":120,"context_line":"                         \u0027 for storage family %(storage_family)s\u0027)"},{"line_number":121,"context_line":"                % locals())"},{"line_number":122,"context_line":"        storage_protocol \u003d storage_protocol.lower()"},{"line_number":123,"context_line":"        driver_loc \u003d family_meta.get(storage_protocol)"},{"line_number":124,"context_line":"        if not driver_loc:"}],"source_content_type":"text/x-python","patch_set":5,"id":"AAAAPn%2F%2Fz0I%3D","line":121,"updated":"2013-06-20 17:51:00.000000000","message":"locals","commit_id":"c2062632d15f98a58cee99fe1e0eded46059cbaf"},{"author":{"_account_id":2243,"name":"John Griffith","email":"john.griffith8@gmail.com","username":"john-griffith"},"change_message_id":"3db11e99435bb5bcd810abfd39859e709741ea46","unresolved":false,"context_lines":[{"line_number":125,"context_line":"            raise exception.InvalidInput("},{"line_number":126,"context_line":"                reason\u003d_(\u0027Protocol %(storage_protocol)s is not supported\u0027"},{"line_number":127,"context_line":"                         \u0027 for storage family %(storage_family)s\u0027)"},{"line_number":128,"context_line":"                % locals())"},{"line_number":129,"context_line":"        NetAppDriverFactory.check_netapp_driver(driver_loc)"},{"line_number":130,"context_line":"        kwargs \u003d kwargs or {}"},{"line_number":131,"context_line":"        kwargs[\u0027netapp_mode\u0027] \u003d \u0027proxy\u0027"}],"source_content_type":"text/x-python","patch_set":5,"id":"AAAAPn%2F%2Fz0E%3D","line":128,"updated":"2013-06-20 17:51:00.000000000","message":"locals","commit_id":"c2062632d15f98a58cee99fe1e0eded46059cbaf"},{"author":{"_account_id":2243,"name":"John Griffith","email":"john.griffith8@gmail.com","username":"john-griffith"},"change_message_id":"3db11e99435bb5bcd810abfd39859e709741ea46","unresolved":false,"context_lines":[{"line_number":131,"context_line":"        kwargs[\u0027netapp_mode\u0027] \u003d \u0027proxy\u0027"},{"line_number":132,"context_line":"        driver \u003d importutils.import_object(driver_loc, *args, **kwargs)"},{"line_number":133,"context_line":"        LOG.info(_(\u0027NetApp driver of family %(storage_family)s and protocol\u0027"},{"line_number":134,"context_line":"                   \u0027 %(storage_protocol)s loaded\u0027) % locals())"},{"line_number":135,"context_line":"        return driver"},{"line_number":136,"context_line":""},{"line_number":137,"context_line":"    @staticmethod"}],"source_content_type":"text/x-python","patch_set":5,"id":"AAAAPn%2F%2Fz0A%3D","line":134,"updated":"2013-06-20 17:51:00.000000000","message":"locals","commit_id":"c2062632d15f98a58cee99fe1e0eded46059cbaf"}],"cinder/volume/drivers/netapp/iscsi.py":[{"author":{"_account_id":4355,"name":"Avishay Traeger","email":"avishay@stratoscale.com","username":"avishay-il"},"change_message_id":"8929f7696e47513ed7e5e20b502c3b2efb2e2b00","unresolved":false,"context_lines":[{"line_number":214,"context_line":"        name \u003d volume[\u0027name\u0027]"},{"line_number":215,"context_line":"        lun_id \u003d self._map_lun(name, initiator_name, \u0027iscsi\u0027, None)"},{"line_number":216,"context_line":"        msg \u003d _(\"Mapped LUN %(name)s to the initiator %(initiator_name)s\")"},{"line_number":217,"context_line":"        LOG.debug(msg % locals())"},{"line_number":218,"context_line":"        iqn \u003d self._get_iscsi_service_details()"},{"line_number":219,"context_line":"        target_details_list \u003d self._get_target_details()"},{"line_number":220,"context_line":"        msg \u003d _(\"Succesfully fetched target details for LUN %(name)s and \""}],"source_content_type":"text/x-python","patch_set":4,"id":"AAAAPn%2F%2Fz8o%3D","line":217,"updated":"2013-06-20 16:46:24.000000000","message":"Please do not use locals()","commit_id":"d1318caab8b7fd256cc2108a690c7715714fee1e"},{"author":{"_account_id":4355,"name":"Avishay Traeger","email":"avishay@stratoscale.com","username":"avishay-il"},"change_message_id":"8929f7696e47513ed7e5e20b502c3b2efb2e2b00","unresolved":false,"context_lines":[{"line_number":1077,"context_line":"                if clone_ops_info.get_child_content(\u0027clone-state\u0027)\\"},{"line_number":1078,"context_line":"                        \u003d\u003d \u0027completed\u0027:"},{"line_number":1079,"context_line":"                    LOG.debug(_(\"Clone operation with src %(name)s\""},{"line_number":1080,"context_line":"                                \" and dest %(new_name)s completed\") % locals())"},{"line_number":1081,"context_line":"                else:"},{"line_number":1082,"context_line":"                    LOG.debug(_(\"Clone operation with src %(name)s\""},{"line_number":1083,"context_line":"                                \" and dest %(new_name)s failed\") % locals())"}],"source_content_type":"text/x-python","patch_set":4,"id":"AAAAPn%2F%2Fz8g%3D","line":1080,"updated":"2013-06-20 16:46:24.000000000","message":"Please do not use locals()","commit_id":"d1318caab8b7fd256cc2108a690c7715714fee1e"},{"author":{"_account_id":4355,"name":"Avishay Traeger","email":"avishay@stratoscale.com","username":"avishay-il"},"change_message_id":"8929f7696e47513ed7e5e20b502c3b2efb2e2b00","unresolved":false,"context_lines":[{"line_number":1080,"context_line":"                                \" and dest %(new_name)s completed\") % locals())"},{"line_number":1081,"context_line":"                else:"},{"line_number":1082,"context_line":"                    LOG.debug(_(\"Clone operation with src %(name)s\""},{"line_number":1083,"context_line":"                                \" and dest %(new_name)s failed\") % locals())"},{"line_number":1084,"context_line":"                    raise NaApiError("},{"line_number":1085,"context_line":"                        clone_ops_info.get_child_content(\u0027error\u0027),"},{"line_number":1086,"context_line":"                        clone_ops_info.get_child_content(\u0027reason\u0027))"}],"source_content_type":"text/x-python","patch_set":4,"id":"AAAAPn%2F%2Fz8c%3D","line":1083,"updated":"2013-06-20 16:46:24.000000000","message":"Please do not use locals()","commit_id":"d1318caab8b7fd256cc2108a690c7715714fee1e"}],"cinder/volume/drivers/netapp/nfs.py":[{"author":{"_account_id":1207,"name":"Duncan Thomas","email":"duncan.thomas@gmail.com","username":"duncan-thomas"},"change_message_id":"6a3544175da13194559372a10bb0d37d959c94a4","unresolved":false,"context_lines":[{"line_number":67,"context_line":""},{"line_number":68,"context_line":"    def check_for_setup_error(self):"},{"line_number":69,"context_line":"        \"\"\"Returns an error if prerequisites aren\u0027t met.\"\"\""},{"line_number":70,"context_line":"        return NotImplementedError()"},{"line_number":71,"context_line":""},{"line_number":72,"context_line":"    def create_volume_from_snapshot(self, volume, snapshot):"},{"line_number":73,"context_line":"        \"\"\"Creates a volume from a snapshot.\"\"\""}],"source_content_type":"text/x-python","patch_set":3,"id":"AAAAPn%2F%2F2zg%3D","line":70,"updated":"2013-06-19 17:55:47.000000000","message":"Raise not return?","commit_id":"2c653e1513c30a5dd04e95fe88222686b4ec7ff6"},{"author":{"_account_id":6094,"name":"Navneet Singh","email":"navsinghnitd@gmail.com","username":"singn"},"change_message_id":"9a7baeb24737eaac2dd3145cde0a6e7b9a45fc3c","unresolved":false,"context_lines":[{"line_number":67,"context_line":""},{"line_number":68,"context_line":"    def check_for_setup_error(self):"},{"line_number":69,"context_line":"        \"\"\"Returns an error if prerequisites aren\u0027t met.\"\"\""},{"line_number":70,"context_line":"        return NotImplementedError()"},{"line_number":71,"context_line":""},{"line_number":72,"context_line":"    def create_volume_from_snapshot(self, volume, snapshot):"},{"line_number":73,"context_line":"        \"\"\"Creates a volume from a snapshot.\"\"\""}],"source_content_type":"text/x-python","patch_set":3,"id":"AAAAPn%2F%2F054%3D","line":70,"in_reply_to":"AAAAPn%2F%2F2zg%3D","updated":"2013-06-20 10:04:42.000000000","message":"will change it.","commit_id":"2c653e1513c30a5dd04e95fe88222686b4ec7ff6"},{"author":{"_account_id":1207,"name":"Duncan Thomas","email":"duncan.thomas@gmail.com","username":"duncan-thomas"},"change_message_id":"6a3544175da13194559372a10bb0d37d959c94a4","unresolved":false,"context_lines":[{"line_number":365,"context_line":"        (major, minor) \u003d self._get_ontapi_version()"},{"line_number":366,"context_line":"        client.set_api_version(major, minor)"},{"line_number":367,"context_line":""},{"line_number":368,"context_line":"    def _invoke_successfully(self, na_element, vfiler\u003dNone):"},{"line_number":369,"context_line":"        \"\"\"Invoke the api for successful result."},{"line_number":370,"context_line":""},{"line_number":371,"context_line":"        If vfiler is present then invokes vfiler api"}],"source_content_type":"text/x-python","patch_set":3,"id":"AAAAPn%2F%2F2zI%3D","line":368,"updated":"2013-06-19 17:55:47.000000000","message":"This is identical to the above declaration... many move it to utils too?","commit_id":"2c653e1513c30a5dd04e95fe88222686b4ec7ff6"},{"author":{"_account_id":6094,"name":"Navneet Singh","email":"navsinghnitd@gmail.com","username":"singn"},"change_message_id":"9a7baeb24737eaac2dd3145cde0a6e7b9a45fc3c","unresolved":false,"context_lines":[{"line_number":365,"context_line":"        (major, minor) \u003d self._get_ontapi_version()"},{"line_number":366,"context_line":"        client.set_api_version(major, minor)"},{"line_number":367,"context_line":""},{"line_number":368,"context_line":"    def _invoke_successfully(self, na_element, vfiler\u003dNone):"},{"line_number":369,"context_line":"        \"\"\"Invoke the api for successful result."},{"line_number":370,"context_line":""},{"line_number":371,"context_line":"        If vfiler is present then invokes vfiler api"}],"source_content_type":"text/x-python","patch_set":3,"id":"AAAAPn%2F%2F05o%3D","line":368,"in_reply_to":"AAAAPn%2F%2F2zI%3D","updated":"2013-06-20 10:04:42.000000000","message":"Even though it looks same, its actually different. vfiler and vserver are very different entities and their values are used in a completely different way by api. These are dependent on two different families of netapp storage systems. Its not possible to combine these two methods without changing signature. Its beneficial to keep them in the same file as they wont be used anywhere else.","commit_id":"2c653e1513c30a5dd04e95fe88222686b4ec7ff6"},{"author":{"_account_id":4355,"name":"Avishay Traeger","email":"avishay@stratoscale.com","username":"avishay-il"},"change_message_id":"8929f7696e47513ed7e5e20b502c3b2efb2e2b00","unresolved":false,"context_lines":[{"line_number":77,"context_line":"        if vol_size !\u003d snap_size:"},{"line_number":78,"context_line":"            msg \u003d _(\u0027Cannot create volume of size %(vol_size)s from \u0027"},{"line_number":79,"context_line":"                    \u0027snapshot of size %(snap_size)s\u0027)"},{"line_number":80,"context_line":"            raise exception.CinderException(msg % locals())"},{"line_number":81,"context_line":""},{"line_number":82,"context_line":"        self._clone_volume(snapshot.name, volume.name, snapshot.volume_id)"},{"line_number":83,"context_line":"        share \u003d self._get_volume_location(snapshot.volume_id)"}],"source_content_type":"text/x-python","patch_set":4,"id":"AAAAPn%2F%2Fz9c%3D","line":80,"updated":"2013-06-20 16:46:24.000000000","message":"Please do not use locals()","commit_id":"d1318caab8b7fd256cc2108a690c7715714fee1e"},{"author":{"_account_id":4355,"name":"Avishay Traeger","email":"avishay@stratoscale.com","username":"avishay-il"},"change_message_id":"8929f7696e47513ed7e5e20b502c3b2efb2e2b00","unresolved":false,"context_lines":[{"line_number":111,"context_line":"        return (nfs_server_ip + \u0027:\u0027 + export_path)"},{"line_number":112,"context_line":""},{"line_number":113,"context_line":"    def _clone_volume(self, volume_name, clone_name, volume_id):"},{"line_number":114,"context_line":"        \"\"\"Clones mounted volume with OnCommand proxy API\"\"\""},{"line_number":115,"context_line":"        raise NotImplementedError()"},{"line_number":116,"context_line":""},{"line_number":117,"context_line":"    def _get_provider_location(self, volume_id):"}],"source_content_type":"text/x-python","patch_set":4,"id":"AAAAPn%2F%2Fz9Q%3D","line":114,"updated":"2013-06-20 16:46:24.000000000","message":"Docstring should end with a period.","commit_id":"d1318caab8b7fd256cc2108a690c7715714fee1e"},{"author":{"_account_id":4355,"name":"Avishay Traeger","email":"avishay@stratoscale.com","username":"avishay-il"},"change_message_id":"8929f7696e47513ed7e5e20b502c3b2efb2e2b00","unresolved":false,"context_lines":[{"line_number":116,"context_line":""},{"line_number":117,"context_line":"    def _get_provider_location(self, volume_id):"},{"line_number":118,"context_line":"        \"\"\""},{"line_number":119,"context_line":"        Returns provider location for given volume"},{"line_number":120,"context_line":"        :param volume_id:"},{"line_number":121,"context_line":"        \"\"\""},{"line_number":122,"context_line":"        volume \u003d self.db.volume_get(self._context, volume_id)"}],"source_content_type":"text/x-python","patch_set":4,"id":"AAAAPn%2F%2Fz88%3D","line":119,"updated":"2013-06-20 16:46:24.000000000","message":"Docstring should end with a period.","commit_id":"d1318caab8b7fd256cc2108a690c7715714fee1e"},{"author":{"_account_id":4355,"name":"Avishay Traeger","email":"avishay@stratoscale.com","username":"avishay-il"},"change_message_id":"8929f7696e47513ed7e5e20b502c3b2efb2e2b00","unresolved":false,"context_lines":[{"line_number":117,"context_line":"    def _get_provider_location(self, volume_id):"},{"line_number":118,"context_line":"        \"\"\""},{"line_number":119,"context_line":"        Returns provider location for given volume"},{"line_number":120,"context_line":"        :param volume_id:"},{"line_number":121,"context_line":"        \"\"\""},{"line_number":122,"context_line":"        volume \u003d self.db.volume_get(self._context, volume_id)"},{"line_number":123,"context_line":"        return volume.provider_location"}],"source_content_type":"text/x-python","patch_set":4,"id":"AAAAPn%2F%2Fz9A%3D","line":120,"updated":"2013-06-20 16:46:24.000000000","message":"Needed?","commit_id":"d1318caab8b7fd256cc2108a690c7715714fee1e"},{"author":{"_account_id":4355,"name":"Avishay Traeger","email":"avishay@stratoscale.com","username":"avishay-il"},"change_message_id":"8929f7696e47513ed7e5e20b502c3b2efb2e2b00","unresolved":false,"context_lines":[{"line_number":123,"context_line":"        return volume.provider_location"},{"line_number":124,"context_line":""},{"line_number":125,"context_line":"    def _get_host_ip(self, volume_id):"},{"line_number":126,"context_line":"        \"\"\"Returns IP address for the given volume\"\"\""},{"line_number":127,"context_line":"        return self._get_provider_location(volume_id).split(\u0027:\u0027)[0]"},{"line_number":128,"context_line":""},{"line_number":129,"context_line":"    def _get_export_path(self, volume_id):"}],"source_content_type":"text/x-python","patch_set":4,"id":"AAAAPn%2F%2Fz9M%3D","line":126,"updated":"2013-06-20 16:46:24.000000000","message":"Docstring should end with a period.","commit_id":"d1318caab8b7fd256cc2108a690c7715714fee1e"},{"author":{"_account_id":4355,"name":"Avishay Traeger","email":"avishay@stratoscale.com","username":"avishay-il"},"change_message_id":"8929f7696e47513ed7e5e20b502c3b2efb2e2b00","unresolved":false,"context_lines":[{"line_number":127,"context_line":"        return self._get_provider_location(volume_id).split(\u0027:\u0027)[0]"},{"line_number":128,"context_line":""},{"line_number":129,"context_line":"    def _get_export_path(self, volume_id):"},{"line_number":130,"context_line":"        \"\"\"Returns NFS export path for the given volume\"\"\""},{"line_number":131,"context_line":"        return self._get_provider_location(volume_id).split(\u0027:\u0027)[1]"},{"line_number":132,"context_line":""},{"line_number":133,"context_line":"    def _volume_not_present(self, nfs_mount, volume_name):"}],"source_content_type":"text/x-python","patch_set":4,"id":"AAAAPn%2F%2Fz84%3D","line":130,"updated":"2013-06-20 16:46:24.000000000","message":"Docstring should end with a period.","commit_id":"d1318caab8b7fd256cc2108a690c7715714fee1e"},{"author":{"_account_id":4355,"name":"Avishay Traeger","email":"avishay@stratoscale.com","username":"avishay-il"},"change_message_id":"8929f7696e47513ed7e5e20b502c3b2efb2e2b00","unresolved":false,"context_lines":[{"line_number":132,"context_line":""},{"line_number":133,"context_line":"    def _volume_not_present(self, nfs_mount, volume_name):"},{"line_number":134,"context_line":"        \"\"\""},{"line_number":135,"context_line":"        Check if volume exists"},{"line_number":136,"context_line":"        \"\"\""},{"line_number":137,"context_line":"        try:"},{"line_number":138,"context_line":"            self._try_execute(\u0027ls\u0027, self._get_volume_path(nfs_mount,"}],"source_content_type":"text/x-python","patch_set":4,"id":"AAAAPn%2F%2Fz80%3D","line":135,"updated":"2013-06-20 16:46:24.000000000","message":"Docstring should end with a period.","commit_id":"d1318caab8b7fd256cc2108a690c7715714fee1e"},{"author":{"_account_id":4355,"name":"Avishay Traeger","email":"avishay@stratoscale.com","username":"avishay-il"},"change_message_id":"8929f7696e47513ed7e5e20b502c3b2efb2e2b00","unresolved":false,"context_lines":[{"line_number":331,"context_line":"            return vol_id.get_child_content(\u0027name\u0027)"},{"line_number":332,"context_line":"        raise exception.NotFound(_(\"\"\"No volume on cluster with vserver"},{"line_number":333,"context_line":"                                   %(vserver)s and junction path %(junction)s"},{"line_number":334,"context_line":"                                   \"\"\") % locals())"},{"line_number":335,"context_line":""},{"line_number":336,"context_line":"    def _clone_file(self, volume, src_path, dest_path, vserver\u003dNone):"},{"line_number":337,"context_line":"        \"\"\"Clones file on vserver.\"\"\""}],"source_content_type":"text/x-python","patch_set":4,"id":"AAAAPn%2F%2Fz8w%3D","line":334,"updated":"2013-06-20 16:46:24.000000000","message":"Please do not use locals().","commit_id":"d1318caab8b7fd256cc2108a690c7715714fee1e"},{"author":{"_account_id":4355,"name":"Avishay Traeger","email":"avishay@stratoscale.com","username":"avishay-il"},"change_message_id":"8929f7696e47513ed7e5e20b502c3b2efb2e2b00","unresolved":false,"context_lines":[{"line_number":417,"context_line":"        :returns: clone-id"},{"line_number":418,"context_line":"        \"\"\""},{"line_number":419,"context_line":"        LOG.debug(_(\"\"\"Cloning with src %(src_path)s, dest %(dest_path)s\"\"\")"},{"line_number":420,"context_line":"                  % locals())"},{"line_number":421,"context_line":"        clone_start \u003d NaElement.create_node_with_children("},{"line_number":422,"context_line":"            \u0027clone-start\u0027,"},{"line_number":423,"context_line":"            **{\u0027source-path\u0027: src_path,"}],"source_content_type":"text/x-python","patch_set":4,"id":"AAAAPn%2F%2Fz8s%3D","line":420,"updated":"2013-06-20 16:46:24.000000000","message":"Please do not use locals()","commit_id":"d1318caab8b7fd256cc2108a690c7715714fee1e"}],"cinder/volume/manager.py":[{"author":{"_account_id":1207,"name":"Duncan Thomas","email":"duncan.thomas@gmail.com","username":"duncan-thomas"},"change_message_id":"6a3544175da13194559372a10bb0d37d959c94a4","unresolved":false,"context_lines":[{"line_number":82,"context_line":"    \u0027cinder.volume.san.SolarisISCSIDriver\u0027:"},{"line_number":83,"context_line":"    \u0027cinder.volume.drivers.san.solaris.SolarisISCSIDriver\u0027,"},{"line_number":84,"context_line":"    \u0027cinder.volume.san.HpSanISCSIDriver\u0027:"},{"line_number":85,"context_line":"    \u0027cinder.volume.drivers.san.hp_lefthand.HpSanISCSIDriver\u0027,"},{"line_number":86,"context_line":"    \u0027cinder.volume.nfs.NfsDriver\u0027:"},{"line_number":87,"context_line":"    \u0027cinder.volume.drivers.nfs.NfsDriver\u0027,"},{"line_number":88,"context_line":"    \u0027cinder.volume.solidfire.SolidFire\u0027:"}],"source_content_type":"text/x-python","patch_set":3,"id":"AAAAPn%2F%2F3aI%3D","line":85,"updated":"2013-06-19 17:55:47.000000000","message":"Is taking these mappings out going to break existing config files that assume they are there? \n\nDo you care?","commit_id":"2c653e1513c30a5dd04e95fe88222686b4ec7ff6"},{"author":{"_account_id":6094,"name":"Navneet Singh","email":"navsinghnitd@gmail.com","username":"singn"},"change_message_id":"9a7baeb24737eaac2dd3145cde0a6e7b9a45fc3c","unresolved":false,"context_lines":[{"line_number":82,"context_line":"    \u0027cinder.volume.san.SolarisISCSIDriver\u0027:"},{"line_number":83,"context_line":"    \u0027cinder.volume.drivers.san.solaris.SolarisISCSIDriver\u0027,"},{"line_number":84,"context_line":"    \u0027cinder.volume.san.HpSanISCSIDriver\u0027:"},{"line_number":85,"context_line":"    \u0027cinder.volume.drivers.san.hp_lefthand.HpSanISCSIDriver\u0027,"},{"line_number":86,"context_line":"    \u0027cinder.volume.nfs.NfsDriver\u0027:"},{"line_number":87,"context_line":"    \u0027cinder.volume.drivers.nfs.NfsDriver\u0027,"},{"line_number":88,"context_line":"    \u0027cinder.volume.solidfire.SolidFire\u0027:"}],"source_content_type":"text/x-python","patch_set":3,"id":"AAAAPn%2F%2F06M%3D","line":85,"in_reply_to":"AAAAPn%2F%2F3aI%3D","updated":"2013-06-20 10:04:42.000000000","message":"The mappings are for a particular driver i.e. mapping old version of same drv to new version of it. We are deprecating these drivers. Hence the mapping do not make sense here as driver itself wont be present. We are ok to tell customers to change configuration.","commit_id":"2c653e1513c30a5dd04e95fe88222686b4ec7ff6"}]}