)]}'
{"cinder/volume/drivers/dell_emc/unity/adapter.py":[{"author":{"_account_id":6491,"name":"xing-yang","email":"xingyang105@gmail.com","username":"xing-yang"},"change_message_id":"e63f0aea25d84cb6022b67f22edc7f41e9c5f2ba","unresolved":false,"context_lines":[{"line_number":168,"context_line":""},{"line_number":169,"context_line":"        group_name \u003d (self.config.config_group if self.config.config_group"},{"line_number":170,"context_line":"                      else \u0027DEFAULT\u0027)"},{"line_number":171,"context_line":"        folder_name \u003d \u0027{}.{}\u0027.format(group_name, self.client.system.info.name)"},{"line_number":172,"context_line":"        persist_path \u003d os.path.join(cfg.CONF.state_path, \u0027unity\u0027, folder_name)"},{"line_number":173,"context_line":"        storops.TCHelper.set_up(persist_path)"},{"line_number":174,"context_line":""}],"source_content_type":"text/x-python","patch_set":4,"id":"7f231b9d_535e2976","line":171,"range":{"start_line":171,"start_character":22,"end_line":171,"end_character":78},"updated":"2017-06-16 04:27:26.000000000","message":"Please use %s format style to be consistent.","commit_id":"e145a96f70a16bd4ef9ef23edbfe99d624827af0"},{"author":{"_account_id":6491,"name":"xing-yang","email":"xingyang105@gmail.com","username":"xing-yang"},"change_message_id":"e63f0aea25d84cb6022b67f22edc7f41e9c5f2ba","unresolved":false,"context_lines":[{"line_number":564,"context_line":"                io_limit_policy\u003dvol_params.io_limit_policy,"},{"line_number":565,"context_line":"                new_size_gb\u003dvol_params.size)"},{"line_number":566,"context_line":"        except storops_ex.UnityThinCloneLimitExceededError:"},{"line_number":567,"context_line":"            LOG.info((\u0027Number of thin clones of base LUN exceeds system \u0027"},{"line_number":568,"context_line":"                      \u0027limit, dd-copy a new one and thin clone from it.\u0027))"},{"line_number":569,"context_line":"            # Copy via dd if thin clone meets the system limit"},{"line_number":570,"context_line":"            hidden \u003d copy.copy(vol_params)"},{"line_number":571,"context_line":"            hidden.name \u003d \u0027hidden-{}\u0027.format(vol_params.name)"}],"source_content_type":"text/x-python","patch_set":4,"id":"7f231b9d_b3c8dd21","line":568,"range":{"start_line":567,"start_character":20,"end_line":568,"end_character":74},"updated":"2017-06-16 04:27:26.000000000","message":"No need for double ()","commit_id":"e145a96f70a16bd4ef9ef23edbfe99d624827af0"},{"author":{"_account_id":6491,"name":"xing-yang","email":"xingyang105@gmail.com","username":"xing-yang"},"change_message_id":"e63f0aea25d84cb6022b67f22edc7f41e9c5f2ba","unresolved":false,"context_lines":[{"line_number":568,"context_line":"                      \u0027limit, dd-copy a new one and thin clone from it.\u0027))"},{"line_number":569,"context_line":"            # Copy via dd if thin clone meets the system limit"},{"line_number":570,"context_line":"            hidden \u003d copy.copy(vol_params)"},{"line_number":571,"context_line":"            hidden.name \u003d \u0027hidden-{}\u0027.format(vol_params.name)"},{"line_number":572,"context_line":"            hidden.description \u003d \u0027hidden-{}\u0027.format(vol_params.description)"},{"line_number":573,"context_line":"            copied_lun \u003d self._dd_copy(hidden, src_snap, src_lun\u003dsrc_lun)"},{"line_number":574,"context_line":"            LOG.debug(\u0027Notify storops the dd action of lun: %(src_name)s. And \u0027"},{"line_number":575,"context_line":"                      \u0027the newly copied lun is: %(copied)s.\u0027,"}],"source_content_type":"text/x-python","patch_set":4,"id":"7f231b9d_53d3c9d5","line":572,"range":{"start_line":571,"start_character":12,"end_line":572,"end_character":74},"updated":"2017-06-16 04:27:26.000000000","message":"Please use %s format style.","commit_id":"e145a96f70a16bd4ef9ef23edbfe99d624827af0"},{"author":{"_account_id":7198,"name":"Jay Bryant","email":"jungleboyj@electronicjungle.net","username":"jsbryant"},"change_message_id":"917a33a3c1a7498de416cf36d42d8800084ad062","unresolved":false,"context_lines":[{"line_number":29,"context_line":"    from storops import exception as storops_ex"},{"line_number":30,"context_line":"else:"},{"line_number":31,"context_line":"    # Set storops_ex to be None for unit test"},{"line_number":32,"context_line":"    storops_ex \u003d None"},{"line_number":33,"context_line":""},{"line_number":34,"context_line":"from cinder import exception"},{"line_number":35,"context_line":"from cinder.i18n import _"}],"source_content_type":"text/x-python","patch_set":5,"id":"5f201791_8f0fdb86","line":32,"updated":"2017-06-27 20:26:00.000000000","message":"Seems like there should be a LOG.info or LOG.warning to go with this.","commit_id":"18c6a55b567fa89b02f32f8c407e7679dcf24534"},{"author":{"_account_id":18742,"name":"Ryan Liang","email":"ryan.liang@emc.com","username":"RyanLiang"},"change_message_id":"128b22746920e363343888159877cfa4c37c9bb7","unresolved":false,"context_lines":[{"line_number":29,"context_line":"    from storops import exception as storops_ex"},{"line_number":30,"context_line":"else:"},{"line_number":31,"context_line":"    # Set storops_ex to be None for unit test"},{"line_number":32,"context_line":"    storops_ex \u003d None"},{"line_number":33,"context_line":""},{"line_number":34,"context_line":"from cinder import exception"},{"line_number":35,"context_line":"from cinder.i18n import _"}],"source_content_type":"text/x-python","patch_set":5,"id":"5f201791_2e16804a","line":32,"in_reply_to":"5f201791_8f0fdb86","updated":"2017-06-28 05:17:02.000000000","message":"In the file client.py(line 37 ~ 40), we\u0027ll raise VolumeBackendAPIException if the storops is not installed.\nSo we put the check there.","commit_id":"18c6a55b567fa89b02f32f8c407e7679dcf24534"}],"releasenotes/notes/unity-fast-clone-02ae88ba8fdef145.yaml":[{"author":{"_account_id":6491,"name":"xing-yang","email":"xingyang105@gmail.com","username":"xing-yang"},"change_message_id":"e63f0aea25d84cb6022b67f22edc7f41e9c5f2ba","unresolved":false,"context_lines":[{"line_number":1,"context_line":"---"},{"line_number":2,"context_line":"features:"},{"line_number":3,"context_line":"  - Add thin clone support. Unity storage supports the thin clone of a LUN from"},{"line_number":4,"context_line":"    OE version 4.2.0. It is more efficient than the dd solution."}],"source_content_type":"text/x-yaml","patch_set":4,"id":"7f231b9d_738005c0","line":3,"range":{"start_line":3,"start_character":4,"end_line":3,"end_character":27},"updated":"2017-06-16 04:27:26.000000000","message":"Add thin clone support in the Unity driver.","commit_id":"e145a96f70a16bd4ef9ef23edbfe99d624827af0"},{"author":{"_account_id":7198,"name":"Jay Bryant","email":"jungleboyj@electronicjungle.net","username":"jsbryant"},"change_message_id":"917a33a3c1a7498de416cf36d42d8800084ad062","unresolved":false,"context_lines":[{"line_number":2,"context_line":"features:"},{"line_number":3,"context_line":"  - Add thin clone support in the Unity driver. Unity storage supports the thin"},{"line_number":4,"context_line":"    clone of a LUN from OE version 4.2.0. It is more efficient than the dd"},{"line_number":5,"context_line":"    solution."}],"source_content_type":"text/x-yaml","patch_set":5,"id":"5f201791_6f6fdf2a","line":5,"range":{"start_line":5,"start_character":3,"end_line":5,"end_character":13},"updated":"2017-06-27 20:26:00.000000000","message":"Do you want to also explain what happens here when the thin clone maximum is reached?","commit_id":"18c6a55b567fa89b02f32f8c407e7679dcf24534"},{"author":{"_account_id":18742,"name":"Ryan Liang","email":"ryan.liang@emc.com","username":"RyanLiang"},"change_message_id":"7a58d470720760a4a19f79928981d8e516034fc8","unresolved":false,"context_lines":[{"line_number":2,"context_line":"features:"},{"line_number":3,"context_line":"  - Add thin clone support in the Unity driver. Unity storage supports the thin"},{"line_number":4,"context_line":"    clone of a LUN from OE version 4.2.0. It is more efficient than the dd"},{"line_number":5,"context_line":"    solution."}],"source_content_type":"text/x-yaml","patch_set":5,"id":"5f201791_d14e9095","line":5,"range":{"start_line":5,"start_character":3,"end_line":5,"end_character":13},"in_reply_to":"5f201791_6f6fdf2a","updated":"2017-06-28 05:17:49.000000000","message":"Done","commit_id":"18c6a55b567fa89b02f32f8c407e7679dcf24534"}]}