)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":17405,"name":"Jacob Gregor","email":"jgregor@us.ibm.com","username":"jgregor"},"change_message_id":"cf081f45471ffbccfab9e18c97c95fb198110147","unresolved":false,"context_lines":[{"line_number":5,"context_line":"CommitDate: 2015-11-11 22:12:32 +0000"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"EMC VMAX - last volume in VMAX3 storage group"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"This patch fixes deleting the last volume in a storage group"},{"line_number":10,"context_line":"for the VMAX3."},{"line_number":11,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"da85f559_068bfe1f","line":8,"updated":"2015-11-12 15:15:52.000000000","message":"It would be helpful if you described the bug that is being fixed here.","commit_id":"4f3333663f699dd5840abbb57bb60fdf7277dc0f"},{"author":{"_account_id":1736,"name":"Ivan Kolodyazhny","email":"e0ne@e0ne.info","username":"e0ne"},"change_message_id":"52e04cc151aec1ee4ed140b6bd47ce17539f11dc","unresolved":false,"context_lines":[{"line_number":5,"context_line":"CommitDate: 2015-11-11 22:12:32 +0000"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"EMC VMAX - last volume in VMAX3 storage group"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"This patch fixes deleting the last volume in a storage group"},{"line_number":10,"context_line":"for the VMAX3."},{"line_number":11,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"da85f559_b86b2bf0","line":8,"in_reply_to":"da85f559_068bfe1f","updated":"2015-11-12 16:42:19.000000000","message":"+1","commit_id":"4f3333663f699dd5840abbb57bb60fdf7277dc0f"},{"author":{"_account_id":12670,"name":"Helen Walsh","email":"helen.walsh@emc.com","username":"walshh2"},"change_message_id":"3368d91d53b664f98631d68257f0695268f151d3","unresolved":false,"context_lines":[{"line_number":5,"context_line":"CommitDate: 2015-11-11 22:12:32 +0000"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"EMC VMAX - last volume in VMAX3 storage group"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"This patch fixes deleting the last volume in a storage group"},{"line_number":10,"context_line":"for the VMAX3."},{"line_number":11,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"da85f559_612f2bf7","line":8,"in_reply_to":"da85f559_068bfe1f","updated":"2015-11-16 22:05:39.000000000","message":"Sure.  I will","commit_id":"4f3333663f699dd5840abbb57bb60fdf7277dc0f"},{"author":{"_account_id":1736,"name":"Ivan Kolodyazhny","email":"e0ne@e0ne.info","username":"e0ne"},"change_message_id":"52e04cc151aec1ee4ed140b6bd47ce17539f11dc","unresolved":false,"context_lines":[{"line_number":9,"context_line":"This patch fixes deleting the last volume in a storage group"},{"line_number":10,"context_line":"for the VMAX3."},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"DocImpact"},{"line_number":13,"context_line":"Closes-Bug: #1512796"},{"line_number":14,"context_line":""},{"line_number":15,"context_line":"Change-Id: Ideb2aaffe4fbc1ae2137d569d7b137ac0edef573"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"da85f559_d845cf6e","line":12,"updated":"2015-11-12 16:42:19.000000000","message":"Do we really need DocImpact here?","commit_id":"4f3333663f699dd5840abbb57bb60fdf7277dc0f"},{"author":{"_account_id":12670,"name":"Helen Walsh","email":"helen.walsh@emc.com","username":"walshh2"},"change_message_id":"3368d91d53b664f98631d68257f0695268f151d3","unresolved":false,"context_lines":[{"line_number":9,"context_line":"This patch fixes deleting the last volume in a storage group"},{"line_number":10,"context_line":"for the VMAX3."},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"DocImpact"},{"line_number":13,"context_line":"Closes-Bug: #1512796"},{"line_number":14,"context_line":""},{"line_number":15,"context_line":"Change-Id: Ideb2aaffe4fbc1ae2137d569d7b137ac0edef573"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"da85f559_a1ee73b2","line":12,"in_reply_to":"da85f559_d845cf6e","updated":"2015-11-16 22:05:39.000000000","message":"Don\u0027t think so, I will remove it","commit_id":"4f3333663f699dd5840abbb57bb60fdf7277dc0f"},{"author":{"_account_id":7198,"name":"Jay Bryant","email":"jungleboyj@electronicjungle.net","username":"jsbryant"},"change_message_id":"4ad9075734267e730bca301a150553e7e187068f","unresolved":false,"context_lines":[{"line_number":8,"context_line":""},{"line_number":9,"context_line":"This patch fixes deleting the last volume in a storage group"},{"line_number":10,"context_line":"for the VMAX3.  If a masking view did not exist we tried to"},{"line_number":11,"context_line":"get an instance of it which threw and exception.  Now we "},{"line_number":12,"context_line":"handle both scenarios."},{"line_number":13,"context_line":""},{"line_number":14,"context_line":"Closes-Bug: #1512796"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"da85f559_ef19ab23","line":11,"updated":"2015-11-13 17:12:01.000000000","message":"White space at eol and typo and -\u003e an","commit_id":"8d33fbe20ff3b2fbfaa882e915432cd424049ac8"},{"author":{"_account_id":12670,"name":"Helen Walsh","email":"helen.walsh@emc.com","username":"walshh2"},"change_message_id":"52959ca10c05a4cf3d147ccb98e113c9692b7a9b","unresolved":false,"context_lines":[{"line_number":8,"context_line":""},{"line_number":9,"context_line":"This patch fixes deleting the last volume in a storage group"},{"line_number":10,"context_line":"for the VMAX3.  If a masking view did not exist we tried to"},{"line_number":11,"context_line":"get an instance of it which threw and exception.  Now we "},{"line_number":12,"context_line":"handle both scenarios."},{"line_number":13,"context_line":""},{"line_number":14,"context_line":"Closes-Bug: #1512796"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"da85f559_b530d608","line":11,"in_reply_to":"da85f559_ef19ab23","updated":"2015-11-13 17:50:59.000000000","message":"Sure.  Will fix it.  Thanks","commit_id":"8d33fbe20ff3b2fbfaa882e915432cd424049ac8"},{"author":{"_account_id":1736,"name":"Ivan Kolodyazhny","email":"e0ne@e0ne.info","username":"e0ne"},"change_message_id":"da5dbc692e1ba4ff488483fbbdb742e6fc4ba57c","unresolved":false,"context_lines":[{"line_number":4,"context_line":"Commit:     Helen Walsh \u003chelen.walsh@emc.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2015-11-13 17:52:00 +0000"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"EMC VMAX - last volume in VMAX3 storage group"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"This patch fixes deleting the last volume in a storage group"},{"line_number":10,"context_line":"for the VMAX3.  If a masking view did not exist we tried to"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":4,"id":"ba8a016a_cd72dead","line":7,"updated":"2015-11-16 12:23:22.000000000","message":"Commit message title should describe changes in a more clear way","commit_id":"b7bda9f5cb3c8bf630338ef7621bda9946a62873"},{"author":{"_account_id":1736,"name":"Ivan Kolodyazhny","email":"e0ne@e0ne.info","username":"e0ne"},"change_message_id":"abf22c3fd6a983cfa36fc8be091b04d38f352e91","unresolved":false,"context_lines":[{"line_number":7,"context_line":"EMC VMAX - Fix for last volume in VMAX3 storage group"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"This patch fixes bug in the deletion the last volume in a"},{"line_number":10,"context_line":"storage group for the VMAX3.  If a masking view did not exist"},{"line_number":11,"context_line":"we tried to get an instance of it which threw an exception."},{"line_number":12,"context_line":"Now we handle both scenarios correctly."},{"line_number":13,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":9,"id":"7a740942_4aac3f35","line":10,"updated":"2015-12-10 19:31:05.000000000","message":"Two spaces instead of one. Please, fix it if you\u0027ll do one more patch. I won\u0027t -1 on it","commit_id":"66baa99dfef5f36d67ffbd0f7354403d77c4a30a"},{"author":{"_account_id":12670,"name":"Helen Walsh","email":"helen.walsh@emc.com","username":"walshh2"},"change_message_id":"cd866e2910f157b6ed072a1d6ba09fc872d742b7","unresolved":false,"context_lines":[{"line_number":7,"context_line":"EMC VMAX - Fix for last volume in VMAX3 storage group"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"This patch fixes bug in the deletion the last volume in a"},{"line_number":10,"context_line":"storage group for the VMAX3.  If a masking view did not exist"},{"line_number":11,"context_line":"we tried to get an instance of it which threw an exception."},{"line_number":12,"context_line":"Now we handle both scenarios correctly."},{"line_number":13,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":9,"id":"7a740942_3f3b2fd3","line":10,"in_reply_to":"7a740942_4aac3f35","updated":"2015-12-10 21:59:18.000000000","message":"No problem, I will fix it","commit_id":"66baa99dfef5f36d67ffbd0f7354403d77c4a30a"}],"cinder/tests/unit/test_emc_vmax.py":[{"author":{"_account_id":17405,"name":"Jacob Gregor","email":"jgregor@us.ibm.com","username":"jgregor"},"change_message_id":"cf081f45471ffbccfab9e18c97c95fb198110147","unresolved":false,"context_lines":[{"line_number":5484,"context_line":"                          self.driver.common.masking._last_vol_in_SG,"},{"line_number":5485,"context_line":"                          conn, controllerConfigService,"},{"line_number":5486,"context_line":"                          storagegroup, storagegroup[\u0027ElementName\u0027], vol,"},{"line_number":5487,"context_line":"                          vol[\u0027name\u0027], extraSpecs)"},{"line_number":5488,"context_line":""},{"line_number":5489,"context_line":"    @mock.patch.object("},{"line_number":5490,"context_line":"        emc_vmax_utils.EMCVMAXUtils,"}],"source_content_type":"text/x-python","patch_set":1,"id":"da85f559_230b94d7","line":5487,"updated":"2015-11-12 15:15:52.000000000","message":"We should split up these tests into their own test cases.","commit_id":"4f3333663f699dd5840abbb57bb60fdf7277dc0f"},{"author":{"_account_id":1736,"name":"Ivan Kolodyazhny","email":"e0ne@e0ne.info","username":"e0ne"},"change_message_id":"52e04cc151aec1ee4ed140b6bd47ce17539f11dc","unresolved":false,"context_lines":[{"line_number":5484,"context_line":"                          self.driver.common.masking._last_vol_in_SG,"},{"line_number":5485,"context_line":"                          conn, controllerConfigService,"},{"line_number":5486,"context_line":"                          storagegroup, storagegroup[\u0027ElementName\u0027], vol,"},{"line_number":5487,"context_line":"                          vol[\u0027name\u0027], extraSpecs)"},{"line_number":5488,"context_line":""},{"line_number":5489,"context_line":"    @mock.patch.object("},{"line_number":5490,"context_line":"        emc_vmax_utils.EMCVMAXUtils,"}],"source_content_type":"text/x-python","patch_set":1,"id":"da85f559_58523f36","line":5487,"in_reply_to":"da85f559_230b94d7","updated":"2015-11-12 16:42:19.000000000","message":"+1","commit_id":"4f3333663f699dd5840abbb57bb60fdf7277dc0f"},{"author":{"_account_id":12670,"name":"Helen Walsh","email":"helen.walsh@emc.com","username":"walshh2"},"change_message_id":"e5772bee7d51d7cfbda35b68dd9e7d2eec49a541","unresolved":false,"context_lines":[{"line_number":5484,"context_line":"                          self.driver.common.masking._last_vol_in_SG,"},{"line_number":5485,"context_line":"                          conn, controllerConfigService,"},{"line_number":5486,"context_line":"                          storagegroup, storagegroup[\u0027ElementName\u0027], vol,"},{"line_number":5487,"context_line":"                          vol[\u0027name\u0027], extraSpecs)"},{"line_number":5488,"context_line":""},{"line_number":5489,"context_line":"    @mock.patch.object("},{"line_number":5490,"context_line":"        emc_vmax_utils.EMCVMAXUtils,"}],"source_content_type":"text/x-python","patch_set":1,"id":"da85f559_5a4a2929","line":5487,"in_reply_to":"da85f559_230b94d7","updated":"2015-11-12 20:28:25.000000000","message":"Thanks Jacob, I will do that.","commit_id":"4f3333663f699dd5840abbb57bb60fdf7277dc0f"},{"author":{"_account_id":1736,"name":"Ivan Kolodyazhny","email":"e0ne@e0ne.info","username":"e0ne"},"change_message_id":"da5dbc692e1ba4ff488483fbbdb742e6fc4ba57c","unresolved":false,"context_lines":[{"line_number":5454,"context_line":"        return storagegroup"},{"line_number":5455,"context_line":""},{"line_number":5456,"context_line":"    def test_last_vol_in_SG_with_MV(self):"},{"line_number":5457,"context_line":"        # Test 1. With a masking view - detach"},{"line_number":5458,"context_line":"        conn \u003d self.fake_ecom_connection()"},{"line_number":5459,"context_line":"        controllerConfigService \u003d ("},{"line_number":5460,"context_line":"            self.driver.common.utils.find_controller_configuration_service("}],"source_content_type":"text/x-python","patch_set":4,"id":"ba8a016a_8dacd6ee","line":5457,"updated":"2015-11-16 12:23:22.000000000","message":"Please, remove \u0027Test #\u0027 comments. In a concurrency mode, we don\u0027t know tests execution order","commit_id":"b7bda9f5cb3c8bf630338ef7621bda9946a62873"},{"author":{"_account_id":12670,"name":"Helen Walsh","email":"helen.walsh@emc.com","username":"walshh2"},"change_message_id":"856b4f8e1e7b906269bbb0d40952e25770a0338e","unresolved":false,"context_lines":[{"line_number":5454,"context_line":"        return storagegroup"},{"line_number":5455,"context_line":""},{"line_number":5456,"context_line":"    def test_last_vol_in_SG_with_MV(self):"},{"line_number":5457,"context_line":"        # Test 1. With a masking view - detach"},{"line_number":5458,"context_line":"        conn \u003d self.fake_ecom_connection()"},{"line_number":5459,"context_line":"        controllerConfigService \u003d ("},{"line_number":5460,"context_line":"            self.driver.common.utils.find_controller_configuration_service("}],"source_content_type":"text/x-python","patch_set":4,"id":"ba8a016a_0b5d27e9","line":5457,"in_reply_to":"ba8a016a_8dacd6ee","updated":"2015-11-16 15:10:24.000000000","message":"Will remove.  Thanks","commit_id":"b7bda9f5cb3c8bf630338ef7621bda9946a62873"}],"cinder/volume/drivers/emc/emc_vmax_fc.py":[{"author":{"_account_id":1736,"name":"Ivan Kolodyazhny","email":"e0ne@e0ne.info","username":"e0ne"},"change_message_id":"da5dbc692e1ba4ff488483fbbdb742e6fc4ba57c","unresolved":false,"context_lines":[{"line_number":45,"context_line":"        2.3 Bug 3 - Last volume in SG fix"},{"line_number":46,"context_line":"    \"\"\""},{"line_number":47,"context_line":""},{"line_number":48,"context_line":"    VERSION \u003d \"2.3 Bug 3\""},{"line_number":49,"context_line":""},{"line_number":50,"context_line":"    def __init__(self, *args, **kwargs):"},{"line_number":51,"context_line":""}],"source_content_type":"text/x-python","patch_set":4,"id":"ba8a016a_8d9376a3","line":48,"updated":"2015-11-16 12:23:22.000000000","message":"Looks like an incorrect version","commit_id":"b7bda9f5cb3c8bf630338ef7621bda9946a62873"},{"author":{"_account_id":12670,"name":"Helen Walsh","email":"helen.walsh@emc.com","username":"walshh2"},"change_message_id":"856b4f8e1e7b906269bbb0d40952e25770a0338e","unresolved":false,"context_lines":[{"line_number":45,"context_line":"        2.3 Bug 3 - Last volume in SG fix"},{"line_number":46,"context_line":"    \"\"\""},{"line_number":47,"context_line":""},{"line_number":48,"context_line":"    VERSION \u003d \"2.3 Bug 3\""},{"line_number":49,"context_line":""},{"line_number":50,"context_line":"    def __init__(self, *args, **kwargs):"},{"line_number":51,"context_line":""}],"source_content_type":"text/x-python","patch_set":4,"id":"ba8a016a_6bb9532e","line":48,"in_reply_to":"ba8a016a_8d9376a3","updated":"2015-11-16 15:10:24.000000000","message":"Yes, we are currently hashing out how to do versioning.  Will fix ASAP","commit_id":"b7bda9f5cb3c8bf630338ef7621bda9946a62873"},{"author":{"_account_id":6491,"name":"xing-yang","email":"xingyang105@gmail.com","username":"xing-yang"},"change_message_id":"ee87446c2d31e6bd109a2d9abdf52ad4a93c04ed","unresolved":false,"context_lines":[{"line_number":42,"context_line":"        2.2.2 - Update Consistency Group"},{"line_number":43,"context_line":"        2.2.3 - Pool aware scheduler(multi-pool) support"},{"line_number":44,"context_line":"        2.2.4 - Create CG from CG snapshot"},{"line_number":45,"context_line":"        2.3.3 - Last volume in SG fix"},{"line_number":46,"context_line":"    \"\"\""},{"line_number":47,"context_line":""},{"line_number":48,"context_line":"    VERSION \u003d \"2.3.3\""}],"source_content_type":"text/x-python","patch_set":6,"id":"ba8a016a_cdccf337","line":45,"updated":"2015-11-17 04:34:24.000000000","message":"This should be 2.3.1 because there\u0027s no dependencies between the different patches.  There is no guarantee that 2.3.3 will be merged after 2.3.1.  After the first one is merged, you\u0027ll have to rebase and then change all versions to 2.3.2, and so on.\n\nThe only way to guarantee the ordering is to make one patch dependent on the other. Then you\u0027ll have to update all patches if you make changes in one.","commit_id":"3446cfcc6babcf50e723811e1fbb2141ccaf429c"},{"author":{"_account_id":12670,"name":"Helen Walsh","email":"helen.walsh@emc.com","username":"walshh2"},"change_message_id":"d834d1512dd0c96b11fea8e22aafae2ca076bd4b","unresolved":false,"context_lines":[{"line_number":42,"context_line":"        2.2.2 - Update Consistency Group"},{"line_number":43,"context_line":"        2.2.3 - Pool aware scheduler(multi-pool) support"},{"line_number":44,"context_line":"        2.2.4 - Create CG from CG snapshot"},{"line_number":45,"context_line":"        2.3.3 - Last volume in SG fix"},{"line_number":46,"context_line":"    \"\"\""},{"line_number":47,"context_line":""},{"line_number":48,"context_line":"    VERSION \u003d \"2.3.3\""}],"source_content_type":"text/x-python","patch_set":6,"id":"ba8a016a_09596f5a","line":45,"in_reply_to":"ba8a016a_cdccf337","updated":"2015-11-17 16:11:50.000000000","message":"I think just having \n2.3 - Bug - Last volume in SG fix \nis probably the best.  I have 6 submissions, as you say there is no guarantee what will get submitted and in what order","commit_id":"3446cfcc6babcf50e723811e1fbb2141ccaf429c"}],"cinder/volume/drivers/emc/emc_vmax_iscsi.py":[{"author":{"_account_id":6491,"name":"xing-yang","email":"xingyang105@gmail.com","username":"xing-yang"},"change_message_id":"ee87446c2d31e6bd109a2d9abdf52ad4a93c04ed","unresolved":false,"context_lines":[{"line_number":50,"context_line":"        2.2.2 - Update Consistency Group"},{"line_number":51,"context_line":"        2.2.3 - Pool aware scheduler(multi-pool) support"},{"line_number":52,"context_line":"        2.2.4 - Create CG from CG snapshot"},{"line_number":53,"context_line":"        2.3.3 - Last volume in SG fix"},{"line_number":54,"context_line":"    \"\"\""},{"line_number":55,"context_line":""},{"line_number":56,"context_line":"    VERSION \u003d \"2.3.3\""}],"source_content_type":"text/x-python","patch_set":6,"id":"ba8a016a_4dd9e3f2","line":53,"updated":"2015-11-17 04:34:24.000000000","message":"Same here.","commit_id":"3446cfcc6babcf50e723811e1fbb2141ccaf429c"},{"author":{"_account_id":12670,"name":"Helen Walsh","email":"helen.walsh@emc.com","username":"walshh2"},"change_message_id":"d834d1512dd0c96b11fea8e22aafae2ca076bd4b","unresolved":false,"context_lines":[{"line_number":50,"context_line":"        2.2.2 - Update Consistency Group"},{"line_number":51,"context_line":"        2.2.3 - Pool aware scheduler(multi-pool) support"},{"line_number":52,"context_line":"        2.2.4 - Create CG from CG snapshot"},{"line_number":53,"context_line":"        2.3.3 - Last volume in SG fix"},{"line_number":54,"context_line":"    \"\"\""},{"line_number":55,"context_line":""},{"line_number":56,"context_line":"    VERSION \u003d \"2.3.3\""}],"source_content_type":"text/x-python","patch_set":6,"id":"ba8a016a_89123f61","line":53,"in_reply_to":"ba8a016a_4dd9e3f2","updated":"2015-11-17 16:11:50.000000000","message":"See emc_vmax_iscsi.py","commit_id":"3446cfcc6babcf50e723811e1fbb2141ccaf429c"}],"cinder/volume/drivers/emc/emc_vmax_masking.py":[{"author":{"_account_id":14274,"name":"XinXiaohui","email":"xiaohui.xin@intel.com","username":"XinXiaohui"},"change_message_id":"911151af7c303dc7e5a798de7def59c21ad17538","unresolved":false,"context_lines":[{"line_number":1804,"context_line":"        2. If it is delete the MV, remove the volume from the SG, and"},{"line_number":1805,"context_line":"           delete the SG."},{"line_number":1806,"context_line":"        3. If it is not in a MV, remove the volume from the SG and"},{"line_number":1807,"context_line":"           delete the SG."},{"line_number":1808,"context_line":""},{"line_number":1809,"context_line":"        :param conn: the ecom connection"},{"line_number":1810,"context_line":"        :param controllerConfigService: storage system instance name"}],"source_content_type":"text/x-python","patch_set":1,"id":"da85f559_c60c003e","line":1807,"updated":"2015-11-12 22:01:00.000000000","message":"The comments confuse me a while. Will it is like this:\n\n1. Check whether the volume is in the masking view.\n\n2. If it is, delete the masking view, remove the volume.....\n\n3. If it is not, remove the volume....","commit_id":"4f3333663f699dd5840abbb57bb60fdf7277dc0f"},{"author":{"_account_id":12670,"name":"Helen Walsh","email":"helen.walsh@emc.com","username":"walshh2"},"change_message_id":"3368d91d53b664f98631d68257f0695268f151d3","unresolved":false,"context_lines":[{"line_number":1804,"context_line":"        2. If it is delete the MV, remove the volume from the SG, and"},{"line_number":1805,"context_line":"           delete the SG."},{"line_number":1806,"context_line":"        3. If it is not in a MV, remove the volume from the SG and"},{"line_number":1807,"context_line":"           delete the SG."},{"line_number":1808,"context_line":""},{"line_number":1809,"context_line":"        :param conn: the ecom connection"},{"line_number":1810,"context_line":"        :param controllerConfigService: storage system instance name"}],"source_content_type":"text/x-python","patch_set":1,"id":"ba8a016a_f3a0c392","line":1807,"in_reply_to":"da85f559_c60c003e","updated":"2015-11-16 22:05:39.000000000","message":"I will clarify","commit_id":"4f3333663f699dd5840abbb57bb60fdf7277dc0f"},{"author":{"_account_id":7198,"name":"Jay Bryant","email":"jungleboyj@electronicjungle.net","username":"jsbryant"},"change_message_id":"4ad9075734267e730bca301a150553e7e187068f","unresolved":false,"context_lines":[{"line_number":1869,"context_line":"        :param storageGroupName: the Storage group name (String)"},{"line_number":1870,"context_line":"        :param volumeInstance: the volume instance"},{"line_number":1871,"context_line":"        :param volumeName: the volume name"},{"line_number":1872,"context_line":"        :param numVolsInSG: the number of volumes in the SG"},{"line_number":1873,"context_line":"        :param extraSpecs: the extra specifications"},{"line_number":1874,"context_line":"        \"\"\""},{"line_number":1875,"context_line":"        storageSystemInstanceName \u003d self.utils.find_storage_system("}],"source_content_type":"text/x-python","patch_set":3,"id":"da85f559_cf1627a7","line":1872,"updated":"2015-11-13 17:12:01.000000000","message":"Missing fastPolicyName","commit_id":"8d33fbe20ff3b2fbfaa882e915432cd424049ac8"},{"author":{"_account_id":12670,"name":"Helen Walsh","email":"helen.walsh@emc.com","username":"walshh2"},"change_message_id":"52959ca10c05a4cf3d147ccb98e113c9692b7a9b","unresolved":false,"context_lines":[{"line_number":1869,"context_line":"        :param storageGroupName: the Storage group name (String)"},{"line_number":1870,"context_line":"        :param volumeInstance: the volume instance"},{"line_number":1871,"context_line":"        :param volumeName: the volume name"},{"line_number":1872,"context_line":"        :param numVolsInSG: the number of volumes in the SG"},{"line_number":1873,"context_line":"        :param extraSpecs: the extra specifications"},{"line_number":1874,"context_line":"        \"\"\""},{"line_number":1875,"context_line":"        storageSystemInstanceName \u003d self.utils.find_storage_system("}],"source_content_type":"text/x-python","patch_set":3,"id":"da85f559_9024c0af","line":1872,"in_reply_to":"da85f559_cf1627a7","updated":"2015-11-13 17:50:59.000000000","message":"Will add this. thanks","commit_id":"8d33fbe20ff3b2fbfaa882e915432cd424049ac8"}]}