)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":10213,"name":"Alon Marx","email":"alonma@il.ibm.com","username":"alonmarx"},"change_message_id":"2e2337ba4f79d6a78a8b83994ce05509e2f38174","unresolved":false,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"IBM storage: switch to generic volume group"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"For Pike, IBM storage driver needs to switch to generic volume group"},{"line_number":10,"context_line":"from consistency group. Because IBM storage driver manages both Spectrum"},{"line_number":11,"context_line":"Accelerate Family and DS8000 Family, this change will only change the"},{"line_number":12,"context_line":"common code shared by these storages. Changes for these storage systems"},{"line_number":13,"context_line":"will be in separate reviews."},{"line_number":14,"context_line":""},{"line_number":15,"context_line":"DocImpact"},{"line_number":16,"context_line":"Implements: blueprint ibm-storage-vg"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"3a1ff146_72ede18a","line":13,"range":{"start_line":9,"start_character":0,"end_line":13,"end_character":28},"updated":"2017-03-15 10:46:52.000000000","message":"Limit lines to 72 chars","commit_id":"e47bf6123cdd98ce3ceebba6f2eb86b165e62b16"},{"author":{"_account_id":17564,"name":"jiamin","email":"shljia@cn.ibm.com","username":"jiamin"},"change_message_id":"f69fcaeae051e0e1ccb3d1e7b33deea0dfe15839","unresolved":false,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"IBM storage: switch to generic volume group"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"For Pike, IBM storage driver needs to switch to generic volume group"},{"line_number":10,"context_line":"from consistency group. Because IBM storage driver manages both Spectrum"},{"line_number":11,"context_line":"Accelerate Family and DS8000 Family, this change will only change the"},{"line_number":12,"context_line":"common code shared by these storages. Changes for these storage systems"},{"line_number":13,"context_line":"will be in separate reviews."},{"line_number":14,"context_line":""},{"line_number":15,"context_line":"DocImpact"},{"line_number":16,"context_line":"Implements: blueprint ibm-storage-vg"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"3a1ff146_fd0500c2","line":13,"range":{"start_line":9,"start_character":0,"end_line":13,"end_character":28},"in_reply_to":"3a1ff146_72ede18a","updated":"2017-03-16 01:48:47.000000000","message":"Thanks Alon, done","commit_id":"e47bf6123cdd98ce3ceebba6f2eb86b165e62b16"},{"author":{"_account_id":10213,"name":"Alon Marx","email":"alonma@il.ibm.com","username":"alonmarx"},"change_message_id":"2e2337ba4f79d6a78a8b83994ce05509e2f38174","unresolved":false,"context_lines":[{"line_number":12,"context_line":"common code shared by these storages. Changes for these storage systems"},{"line_number":13,"context_line":"will be in separate reviews."},{"line_number":14,"context_line":""},{"line_number":15,"context_line":"DocImpact"},{"line_number":16,"context_line":"Implements: blueprint ibm-storage-vg"},{"line_number":17,"context_line":""},{"line_number":18,"context_line":"Change-Id: Id9964f868c2dc14aaee37e4bcbba330fb4575492"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"3a1ff146_05d0e952","line":15,"range":{"start_line":15,"start_character":0,"end_line":15,"end_character":9},"updated":"2017-03-15 10:46:52.000000000","message":"remove DocImpact","commit_id":"e47bf6123cdd98ce3ceebba6f2eb86b165e62b16"},{"author":{"_account_id":17564,"name":"jiamin","email":"shljia@cn.ibm.com","username":"jiamin"},"change_message_id":"f69fcaeae051e0e1ccb3d1e7b33deea0dfe15839","unresolved":false,"context_lines":[{"line_number":12,"context_line":"common code shared by these storages. Changes for these storage systems"},{"line_number":13,"context_line":"will be in separate reviews."},{"line_number":14,"context_line":""},{"line_number":15,"context_line":"DocImpact"},{"line_number":16,"context_line":"Implements: blueprint ibm-storage-vg"},{"line_number":17,"context_line":""},{"line_number":18,"context_line":"Change-Id: Id9964f868c2dc14aaee37e4bcbba330fb4575492"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"3a1ff146_3d279865","line":15,"range":{"start_line":15,"start_character":0,"end_line":15,"end_character":9},"in_reply_to":"3a1ff146_05d0e952","updated":"2017-03-16 01:48:47.000000000","message":"Thanks Alon, I will remove it, because DS8K patch already has it.","commit_id":"e47bf6123cdd98ce3ceebba6f2eb86b165e62b16"}],"cinder/volume/drivers/ibm/ibm_storage/ibm_storage.py":[{"author":{"_account_id":10213,"name":"Alon Marx","email":"alonma@il.ibm.com","username":"alonmarx"},"change_message_id":"2e2337ba4f79d6a78a8b83994ce05509e2f38174","unresolved":false,"context_lines":[{"line_number":223,"context_line":""},{"line_number":224,"context_line":"        return self.proxy.retype(ctxt, volume, new_type, diff, host)"},{"line_number":225,"context_line":""},{"line_number":226,"context_line":"    def create_group(self, context, group):"},{"line_number":227,"context_line":"        \"\"\"Creates a group.\"\"\""},{"line_number":228,"context_line":""},{"line_number":229,"context_line":"        return self.proxy.create_group(context, group)"}],"source_content_type":"text/x-python","patch_set":2,"id":"3a1ff146_c51a5117","line":226,"range":{"start_line":226,"start_character":0,"end_line":226,"end_character":43},"updated":"2017-03-15 10:46:52.000000000","message":"The manager code still calls the CG functions, so for the time being we can\u0027t remove these functions. Need to discuss with xyang if/when is she planning to remove them from the manager.","commit_id":"e47bf6123cdd98ce3ceebba6f2eb86b165e62b16"},{"author":{"_account_id":17564,"name":"jiamin","email":"shljia@cn.ibm.com","username":"jiamin"},"change_message_id":"f69fcaeae051e0e1ccb3d1e7b33deea0dfe15839","unresolved":false,"context_lines":[{"line_number":223,"context_line":""},{"line_number":224,"context_line":"        return self.proxy.retype(ctxt, volume, new_type, diff, host)"},{"line_number":225,"context_line":""},{"line_number":226,"context_line":"    def create_group(self, context, group):"},{"line_number":227,"context_line":"        \"\"\"Creates a group.\"\"\""},{"line_number":228,"context_line":""},{"line_number":229,"context_line":"        return self.proxy.create_group(context, group)"}],"source_content_type":"text/x-python","patch_set":2,"id":"3a1ff146_9dd96c86","line":226,"range":{"start_line":226,"start_character":0,"end_line":226,"end_character":43},"in_reply_to":"3a1ff146_0e11e152","updated":"2017-03-16 01:48:47.000000000","message":"Alon, Isaac, please refer to https://review.openstack.org/446018","commit_id":"e47bf6123cdd98ce3ceebba6f2eb86b165e62b16"},{"author":{"_account_id":13047,"name":"Isaac Beckman","email":"beckmani@gmail.com","username":"isaacb"},"change_message_id":"eeace98272f81807f8e8ca9e4d74a60f6b4d5106","unresolved":false,"context_lines":[{"line_number":223,"context_line":""},{"line_number":224,"context_line":"        return self.proxy.retype(ctxt, volume, new_type, diff, host)"},{"line_number":225,"context_line":""},{"line_number":226,"context_line":"    def create_group(self, context, group):"},{"line_number":227,"context_line":"        \"\"\"Creates a group.\"\"\""},{"line_number":228,"context_line":""},{"line_number":229,"context_line":"        return self.proxy.create_group(context, group)"}],"source_content_type":"text/x-python","patch_set":2,"id":"3a1ff146_0e11e152","line":226,"range":{"start_line":226,"start_character":0,"end_line":226,"end_character":43},"in_reply_to":"3a1ff146_c51a5117","updated":"2017-03-15 14:09:02.000000000","message":"Alon,\nAccording to xing-yang, manager\u0027s \"create_consistencygroup\" will not be called in Pike","commit_id":"e47bf6123cdd98ce3ceebba6f2eb86b165e62b16"}]}