)]}'
{"cinder/volume/drivers/dell_emc/vmax/common.py":[{"author":{"_account_id":6491,"name":"xing-yang","email":"xingyang105@gmail.com","username":"xing-yang"},"change_message_id":"f25559bd0b9a0e33452770fe930512528aee615f","unresolved":false,"context_lines":[{"line_number":2022,"context_line":"                storage_system, volume, connector)"},{"line_number":2023,"context_line":"        except Exception:"},{"line_number":2024,"context_line":"            exception_message \u003d (_("},{"line_number":2025,"context_line":"                \"Unable to get fc targets.\"))"},{"line_number":2026,"context_line":"            raise exception.VolumeBackendAPIException("},{"line_number":2027,"context_line":"                data\u003dexception_message)"},{"line_number":2028,"context_line":""}],"source_content_type":"text/x-python","patch_set":2,"id":"7ffa3b31_d09293f6","line":2025,"updated":"2017-04-18 16:26:41.000000000","message":"This doesn\u0027t need to go to line 2025.  Please move it up and remove the outer ().","commit_id":"80ad8da6c71237f4d8b77d0bac6c4f3fb33fea6e"},{"author":{"_account_id":6491,"name":"xing-yang","email":"xingyang105@gmail.com","username":"xing-yang"},"change_message_id":"f25559bd0b9a0e33452770fe930512528aee615f","unresolved":false,"context_lines":[{"line_number":2027,"context_line":"                data\u003dexception_message)"},{"line_number":2028,"context_line":""},{"line_number":2029,"context_line":"        LOG.debug(\"There are %(len)lu endpoints.\","},{"line_number":2030,"context_line":"                  {\u0027len\u0027: len(fc_targets)})"},{"line_number":2031,"context_line":"        for fc_target in fc_targets:"},{"line_number":2032,"context_line":"            wwn \u003d fc_target"},{"line_number":2033,"context_line":"            # Add target wwn to the list if it is not already there."}],"source_content_type":"text/x-python","patch_set":2,"id":"7ffa3b31_904c0b9f","line":2030,"updated":"2017-04-18 16:26:41.000000000","message":"This should be able to fit on line 2029.","commit_id":"80ad8da6c71237f4d8b77d0bac6c4f3fb33fea6e"}],"cinder/volume/drivers/dell_emc/vmax/iscsi.py":[{"author":{"_account_id":7198,"name":"Jay Bryant","email":"jungleboyj@electronicjungle.net","username":"jsbryant"},"change_message_id":"7e45dee8a57b75b6c00c7d0f6364e53b60caadf3","unresolved":false,"context_lines":[{"line_number":83,"context_line":"              - Support for compression on All Flash"},{"line_number":84,"context_line":"              - Volume replication 2.1 (bp add-vmax-replication)"},{"line_number":85,"context_line":"              - rename and restructure driver (bp vmax-rename-dell-emc)"},{"line_number":86,"context_line":""},{"line_number":87,"context_line":"    \"\"\""},{"line_number":88,"context_line":""},{"line_number":89,"context_line":"    VERSION \u003d \"2.5.0\""}],"source_content_type":"text/x-python","patch_set":3,"id":"7ffa3b31_b92236a3","side":"PARENT","line":86,"updated":"2017-04-19 13:08:24.000000000","message":"Why is this change also included?","commit_id":"81ece6a9f2ac9b4ff3efe304bab847006f8b0aef"},{"author":{"_account_id":12670,"name":"Helen Walsh","email":"helen.walsh@emc.com","username":"walshh2"},"change_message_id":"beba67e93a5654152a32d2014cbd5de37c5eef57","unresolved":false,"context_lines":[{"line_number":83,"context_line":"              - Support for compression on All Flash"},{"line_number":84,"context_line":"              - Volume replication 2.1 (bp add-vmax-replication)"},{"line_number":85,"context_line":"              - rename and restructure driver (bp vmax-rename-dell-emc)"},{"line_number":86,"context_line":""},{"line_number":87,"context_line":"    \"\"\""},{"line_number":88,"context_line":""},{"line_number":89,"context_line":"    VERSION \u003d \"2.5.0\""}],"source_content_type":"text/x-python","patch_set":3,"id":"7ffa3b31_a3c78764","side":"PARENT","line":86,"in_reply_to":"7ffa3b31_b92236a3","updated":"2017-04-20 07:50:47.000000000","message":"It should not be included.  I will revert this file.  Thank you","commit_id":"81ece6a9f2ac9b4ff3efe304bab847006f8b0aef"}]}