)]}'
{"cinder/tests/unit/test_emc_vnxdirect.py":[{"author":{"_account_id":14428,"name":"Bharat Kumar Kobagana","email":"bharath.kobagana@gmail.com","username":"bharatkk"},"change_message_id":"e7bdf3faec580460ea56af9aae7db7dee889c145","unresolved":false,"context_lines":[{"line_number":26,"context_line":"from cinder.volume.drivers.emc import emc_cli_fc"},{"line_number":27,"context_line":"from cinder.volume.drivers.emc import emc_cli_iscsi"},{"line_number":28,"context_line":"from cinder.volume.drivers.emc import emc_vnx_cli"},{"line_number":29,"context_line":"from cinder.volume.drivers.emc.emc_vnx_cli import VNXError"},{"line_number":30,"context_line":"from cinder.zonemanager import fc_san_lookup_service as fc_service"},{"line_number":31,"context_line":""},{"line_number":32,"context_line":""}],"source_content_type":"text/x-python","patch_set":2,"id":"3afb71cf_87ed7df1","line":29,"updated":"2015-06-08 07:29:16.000000000","message":"Please import modules not classes/objects.\n\nhttp://docs.openstack.org/developer/hacking/#imports","commit_id":"e1b62e20d5872ab72705096dc9e310dbb08a7904"},{"author":{"_account_id":13206,"name":"Cedric Zhuang","email":"cedric.zhuang@emc.com","username":"CedricZhuang"},"change_message_id":"08f38f7722861fd0bc2eee49bcca42cbf6dd5275","unresolved":false,"context_lines":[{"line_number":26,"context_line":"from cinder.volume.drivers.emc import emc_cli_fc"},{"line_number":27,"context_line":"from cinder.volume.drivers.emc import emc_cli_iscsi"},{"line_number":28,"context_line":"from cinder.volume.drivers.emc import emc_vnx_cli"},{"line_number":29,"context_line":"from cinder.volume.drivers.emc.emc_vnx_cli import VNXError"},{"line_number":30,"context_line":"from cinder.zonemanager import fc_san_lookup_service as fc_service"},{"line_number":31,"context_line":""},{"line_number":32,"context_line":""}],"source_content_type":"text/x-python","patch_set":2,"id":"3afb71cf_c2f48388","line":29,"in_reply_to":"3afb71cf_87ed7df1","updated":"2015-06-08 08:01:32.000000000","message":"Updated.  Thanks.","commit_id":"e1b62e20d5872ab72705096dc9e310dbb08a7904"},{"author":{"_account_id":14428,"name":"Bharat Kumar Kobagana","email":"bharath.kobagana@gmail.com","username":"bharatkk"},"change_message_id":"e7bdf3faec580460ea56af9aae7db7dee889c145","unresolved":false,"context_lines":[{"line_number":4343,"context_line":"        self.assertTrue(has_error)"},{"line_number":4344,"context_line":""},{"line_number":4345,"context_line":"    def test_has_error_multi_line(self):"},{"line_number":4346,"context_line":"        output \u003d \"\\nCould not retrieve the specified (pool lun). \\nThe (pool lun) may not exist\""},{"line_number":4347,"context_line":"        has_error \u003d VNXError.has_error(output, VNXError.GENERAL_NOT_FOUND)"},{"line_number":4348,"context_line":"        self.assertTrue(has_error)"},{"line_number":4349,"context_line":""}],"source_content_type":"text/x-python","patch_set":2,"id":"3afb71cf_a7fd193c","line":4346,"updated":"2015-06-08 07:29:16.000000000","message":"Too long...","commit_id":"e1b62e20d5872ab72705096dc9e310dbb08a7904"},{"author":{"_account_id":13206,"name":"Cedric Zhuang","email":"cedric.zhuang@emc.com","username":"CedricZhuang"},"change_message_id":"08f38f7722861fd0bc2eee49bcca42cbf6dd5275","unresolved":false,"context_lines":[{"line_number":4343,"context_line":"        self.assertTrue(has_error)"},{"line_number":4344,"context_line":""},{"line_number":4345,"context_line":"    def test_has_error_multi_line(self):"},{"line_number":4346,"context_line":"        output \u003d \"\\nCould not retrieve the specified (pool lun). \\nThe (pool lun) may not exist\""},{"line_number":4347,"context_line":"        has_error \u003d VNXError.has_error(output, VNXError.GENERAL_NOT_FOUND)"},{"line_number":4348,"context_line":"        self.assertTrue(has_error)"},{"line_number":4349,"context_line":""}],"source_content_type":"text/x-python","patch_set":2,"id":"3afb71cf_c2692327","line":4346,"in_reply_to":"3afb71cf_a7fd193c","updated":"2015-06-08 08:01:32.000000000","message":"Done","commit_id":"e1b62e20d5872ab72705096dc9e310dbb08a7904"},{"author":{"_account_id":14428,"name":"Bharat Kumar Kobagana","email":"bharath.kobagana@gmail.com","username":"bharatkk"},"change_message_id":"e7bdf3faec580460ea56af9aae7db7dee889c145","unresolved":false,"context_lines":[{"line_number":4364,"context_line":"                                       VNXError.LUN_ALREADY_EXPANDED,"},{"line_number":4365,"context_line":"                                       VNXError.LUN_NOT_MIGRATING)"},{"line_number":4366,"context_line":"        self.assertFalse(has_error)"},{"line_number":4367,"context_line":""}],"source_content_type":"text/x-python","patch_set":2,"id":"3afb71cf_07d50db8","line":4367,"updated":"2015-06-08 07:29:16.000000000","message":"Please remove this blank line at the end.","commit_id":"e1b62e20d5872ab72705096dc9e310dbb08a7904"},{"author":{"_account_id":13206,"name":"Cedric Zhuang","email":"cedric.zhuang@emc.com","username":"CedricZhuang"},"change_message_id":"08f38f7722861fd0bc2eee49bcca42cbf6dd5275","unresolved":false,"context_lines":[{"line_number":4364,"context_line":"                                       VNXError.LUN_ALREADY_EXPANDED,"},{"line_number":4365,"context_line":"                                       VNXError.LUN_NOT_MIGRATING)"},{"line_number":4366,"context_line":"        self.assertFalse(has_error)"},{"line_number":4367,"context_line":""}],"source_content_type":"text/x-python","patch_set":2,"id":"3afb71cf_62614f39","line":4367,"in_reply_to":"3afb71cf_07d50db8","updated":"2015-06-08 08:01:32.000000000","message":"PEP 8 says a new line is required at the end of the file.  Are you sure this empty line should be removed?","commit_id":"e1b62e20d5872ab72705096dc9e310dbb08a7904"}],"cinder/volume/drivers/emc/emc_vnx_cli.py":[{"author":{"_account_id":14428,"name":"Bharat Kumar Kobagana","email":"bharath.kobagana@gmail.com","username":"bharatkk"},"change_message_id":"e7bdf3faec580460ea56af9aae7db7dee889c145","unresolved":false,"context_lines":[{"line_number":181,"context_line":"    LUN_IS_PREPARING \u003d 0x712d8e0e"},{"line_number":182,"context_line":"    LUN_IN_SG \u003d \u0027contained in a Storage Group|LUN mapping still exists\u0027"},{"line_number":183,"context_line":"    LUN_NOT_MIGRATING \u003d (\u0027The specified source LUN is \u0027"},{"line_number":184,"context_line":"                        \u0027not currently migrating\u0027)"},{"line_number":185,"context_line":""},{"line_number":186,"context_line":"    CG_IS_DELETING \u003d 0x712d8801"},{"line_number":187,"context_line":"    CG_EXISTED \u003d 0x716d8021"}],"source_content_type":"text/x-python","patch_set":2,"id":"3afb71cf_e7555150","line":184,"updated":"2015-06-08 07:29:16.000000000","message":"Add one space before.","commit_id":"e1b62e20d5872ab72705096dc9e310dbb08a7904"},{"author":{"_account_id":13206,"name":"Cedric Zhuang","email":"cedric.zhuang@emc.com","username":"CedricZhuang"},"change_message_id":"08f38f7722861fd0bc2eee49bcca42cbf6dd5275","unresolved":false,"context_lines":[{"line_number":181,"context_line":"    LUN_IS_PREPARING \u003d 0x712d8e0e"},{"line_number":182,"context_line":"    LUN_IN_SG \u003d \u0027contained in a Storage Group|LUN mapping still exists\u0027"},{"line_number":183,"context_line":"    LUN_NOT_MIGRATING \u003d (\u0027The specified source LUN is \u0027"},{"line_number":184,"context_line":"                        \u0027not currently migrating\u0027)"},{"line_number":185,"context_line":""},{"line_number":186,"context_line":"    CG_IS_DELETING \u003d 0x712d8801"},{"line_number":187,"context_line":"    CG_EXISTED \u003d 0x716d8021"}],"source_content_type":"text/x-python","patch_set":2,"id":"3afb71cf_e2dc1fe6","line":184,"in_reply_to":"3afb71cf_e7555150","updated":"2015-06-08 08:01:32.000000000","message":"Done","commit_id":"e1b62e20d5872ab72705096dc9e310dbb08a7904"},{"author":{"_account_id":14428,"name":"Bharat Kumar Kobagana","email":"bharath.kobagana@gmail.com","username":"bharatkk"},"change_message_id":"e7bdf3faec580460ea56af9aae7db7dee889c145","unresolved":false,"context_lines":[{"line_number":191,"context_line":"    SNAP_NAME_IN_USE \u003d 0x716d8003"},{"line_number":192,"context_line":"    SNAP_ALREADY_MOUNTED \u003d 0x716d8055"},{"line_number":193,"context_line":"    SNAP_NOT_ATTACHED \u003d (\u0027The specified Snapshot mount point \u0027"},{"line_number":194,"context_line":"                        \u0027is not currently attached.\u0027)"},{"line_number":195,"context_line":""},{"line_number":196,"context_line":"    @classmethod"},{"line_number":197,"context_line":"    def get_all(cls):"}],"source_content_type":"text/x-python","patch_set":2,"id":"3afb71cf_c7501561","line":194,"updated":"2015-06-08 07:29:16.000000000","message":"Same as above.","commit_id":"e1b62e20d5872ab72705096dc9e310dbb08a7904"},{"author":{"_account_id":13206,"name":"Cedric Zhuang","email":"cedric.zhuang@emc.com","username":"CedricZhuang"},"change_message_id":"08f38f7722861fd0bc2eee49bcca42cbf6dd5275","unresolved":false,"context_lines":[{"line_number":191,"context_line":"    SNAP_NAME_IN_USE \u003d 0x716d8003"},{"line_number":192,"context_line":"    SNAP_ALREADY_MOUNTED \u003d 0x716d8055"},{"line_number":193,"context_line":"    SNAP_NOT_ATTACHED \u003d (\u0027The specified Snapshot mount point \u0027"},{"line_number":194,"context_line":"                        \u0027is not currently attached.\u0027)"},{"line_number":195,"context_line":""},{"line_number":196,"context_line":"    @classmethod"},{"line_number":197,"context_line":"    def get_all(cls):"}],"source_content_type":"text/x-python","patch_set":2,"id":"3afb71cf_c2d7e3bd","line":194,"in_reply_to":"3afb71cf_c7501561","updated":"2015-06-08 08:01:32.000000000","message":"Done","commit_id":"e1b62e20d5872ab72705096dc9e310dbb08a7904"},{"author":{"_account_id":14428,"name":"Bharat Kumar Kobagana","email":"bharath.kobagana@gmail.com","username":"bharatkk"},"change_message_id":"e7bdf3faec580460ea56af9aae7db7dee889c145","unresolved":false,"context_lines":[{"line_number":203,"context_line":"        return (isinstance(name, str)"},{"line_number":204,"context_line":"               and hasattr(cls, name)"},{"line_number":205,"context_line":"               and not callable(name)"},{"line_number":206,"context_line":"               and name.isupper())"},{"line_number":207,"context_line":""},{"line_number":208,"context_line":"    @staticmethod"},{"line_number":209,"context_line":"    def _match(output, error_code):"}],"source_content_type":"text/x-python","patch_set":2,"id":"3afb71cf_47642548","line":206,"updated":"2015-06-08 07:29:16.000000000","message":"For above three lines:\n\nPlease give one more space to match the indendation.","commit_id":"e1b62e20d5872ab72705096dc9e310dbb08a7904"},{"author":{"_account_id":13206,"name":"Cedric Zhuang","email":"cedric.zhuang@emc.com","username":"CedricZhuang"},"change_message_id":"08f38f7722861fd0bc2eee49bcca42cbf6dd5275","unresolved":false,"context_lines":[{"line_number":203,"context_line":"        return (isinstance(name, str)"},{"line_number":204,"context_line":"               and hasattr(cls, name)"},{"line_number":205,"context_line":"               and not callable(name)"},{"line_number":206,"context_line":"               and name.isupper())"},{"line_number":207,"context_line":""},{"line_number":208,"context_line":"    @staticmethod"},{"line_number":209,"context_line":"    def _match(output, error_code):"}],"source_content_type":"text/x-python","patch_set":2,"id":"3afb71cf_22e91781","line":206,"in_reply_to":"3afb71cf_47642548","updated":"2015-06-08 08:01:32.000000000","message":"Done","commit_id":"e1b62e20d5872ab72705096dc9e310dbb08a7904"},{"author":{"_account_id":14428,"name":"Bharat Kumar Kobagana","email":"bharath.kobagana@gmail.com","username":"bharatkk"},"change_message_id":"e7bdf3faec580460ea56af9aae7db7dee889c145","unresolved":false,"context_lines":[{"line_number":210,"context_line":"        is_match \u003d False"},{"line_number":211,"context_line":"        if VNXError._is_enum(error_code):"},{"line_number":212,"context_line":"            error_code \u003d getattr(VNXError, error_code)"},{"line_number":213,"context_line":"        if isinstance(error_code, int):"},{"line_number":214,"context_line":"            error_code \u003d hex(error_code)"},{"line_number":215,"context_line":"        if isinstance(error_code, str):"},{"line_number":216,"context_line":"            error_code \u003d error_code.strip()"}],"source_content_type":"text/x-python","patch_set":2,"id":"3afb71cf_0702ad6f","line":213,"updated":"2015-06-08 07:29:16.000000000","message":"I think here \"elif\" required. Please check.","commit_id":"e1b62e20d5872ab72705096dc9e310dbb08a7904"},{"author":{"_account_id":13206,"name":"Cedric Zhuang","email":"cedric.zhuang@emc.com","username":"CedricZhuang"},"change_message_id":"08f38f7722861fd0bc2eee49bcca42cbf6dd5275","unresolved":false,"context_lines":[{"line_number":210,"context_line":"        is_match \u003d False"},{"line_number":211,"context_line":"        if VNXError._is_enum(error_code):"},{"line_number":212,"context_line":"            error_code \u003d getattr(VNXError, error_code)"},{"line_number":213,"context_line":"        if isinstance(error_code, int):"},{"line_number":214,"context_line":"            error_code \u003d hex(error_code)"},{"line_number":215,"context_line":"        if isinstance(error_code, str):"},{"line_number":216,"context_line":"            error_code \u003d error_code.strip()"}],"source_content_type":"text/x-python","patch_set":2,"id":"3afb71cf_e28adfbe","line":213,"in_reply_to":"3afb71cf_0702ad6f","updated":"2015-06-08 08:01:32.000000000","message":"\"if\" is on purpose.  I have added an extra empty line to make it clearer.","commit_id":"e1b62e20d5872ab72705096dc9e310dbb08a7904"},{"author":{"_account_id":14428,"name":"Bharat Kumar Kobagana","email":"bharath.kobagana@gmail.com","username":"bharatkk"},"change_message_id":"e7bdf3faec580460ea56af9aae7db7dee889c145","unresolved":false,"context_lines":[{"line_number":212,"context_line":"            error_code \u003d getattr(VNXError, error_code)"},{"line_number":213,"context_line":"        if isinstance(error_code, int):"},{"line_number":214,"context_line":"            error_code \u003d hex(error_code)"},{"line_number":215,"context_line":"        if isinstance(error_code, str):"},{"line_number":216,"context_line":"            error_code \u003d error_code.strip()"},{"line_number":217,"context_line":"            found \u003d re.findall(error_code, output,"},{"line_number":218,"context_line":"                               flags\u003dre.IGNORECASE)"}],"source_content_type":"text/x-python","patch_set":2,"id":"3afb71cf_c7f49588","line":215,"updated":"2015-06-08 07:29:16.000000000","message":"Same as above.","commit_id":"e1b62e20d5872ab72705096dc9e310dbb08a7904"},{"author":{"_account_id":13206,"name":"Cedric Zhuang","email":"cedric.zhuang@emc.com","username":"CedricZhuang"},"change_message_id":"08f38f7722861fd0bc2eee49bcca42cbf6dd5275","unresolved":false,"context_lines":[{"line_number":212,"context_line":"            error_code \u003d getattr(VNXError, error_code)"},{"line_number":213,"context_line":"        if isinstance(error_code, int):"},{"line_number":214,"context_line":"            error_code \u003d hex(error_code)"},{"line_number":215,"context_line":"        if isinstance(error_code, str):"},{"line_number":216,"context_line":"            error_code \u003d error_code.strip()"},{"line_number":217,"context_line":"            found \u003d re.findall(error_code, output,"},{"line_number":218,"context_line":"                               flags\u003dre.IGNORECASE)"}],"source_content_type":"text/x-python","patch_set":2,"id":"3afb71cf_c285a3ad","line":215,"in_reply_to":"3afb71cf_c7f49588","updated":"2015-06-08 08:01:32.000000000","message":"\"if\" is on purpose.  I have added an extra empty line to make it clearer.","commit_id":"e1b62e20d5872ab72705096dc9e310dbb08a7904"},{"author":{"_account_id":14428,"name":"Bharat Kumar Kobagana","email":"bharath.kobagana@gmail.com","username":"bharatkk"},"change_message_id":"e7bdf3faec580460ea56af9aae7db7dee889c145","unresolved":false,"context_lines":[{"line_number":223,"context_line":"    def has_error(cls, output, *error_codes):"},{"line_number":224,"context_line":"        if error_codes is None or len(error_codes) \u003d\u003d 0:"},{"line_number":225,"context_line":"            error_codes \u003d VNXError.get_all()"},{"line_number":226,"context_line":"        return any([cls._match(output, error_code) for error_code in error_codes])"},{"line_number":227,"context_line":""},{"line_number":228,"context_line":""},{"line_number":229,"context_line":"@decorate_all_methods(log_enter_exit)"}],"source_content_type":"text/x-python","patch_set":2,"id":"3afb71cf_677a81f3","line":226,"updated":"2015-06-08 07:29:16.000000000","message":"This line is too long. (\u003e79)","commit_id":"e1b62e20d5872ab72705096dc9e310dbb08a7904"},{"author":{"_account_id":13206,"name":"Cedric Zhuang","email":"cedric.zhuang@emc.com","username":"CedricZhuang"},"change_message_id":"08f38f7722861fd0bc2eee49bcca42cbf6dd5275","unresolved":false,"context_lines":[{"line_number":223,"context_line":"    def has_error(cls, output, *error_codes):"},{"line_number":224,"context_line":"        if error_codes is None or len(error_codes) \u003d\u003d 0:"},{"line_number":225,"context_line":"            error_codes \u003d VNXError.get_all()"},{"line_number":226,"context_line":"        return any([cls._match(output, error_code) for error_code in error_codes])"},{"line_number":227,"context_line":""},{"line_number":228,"context_line":""},{"line_number":229,"context_line":"@decorate_all_methods(log_enter_exit)"}],"source_content_type":"text/x-python","patch_set":2,"id":"3afb71cf_424e5397","line":226,"in_reply_to":"3afb71cf_677a81f3","updated":"2015-06-08 08:01:32.000000000","message":"Done","commit_id":"e1b62e20d5872ab72705096dc9e310dbb08a7904"},{"author":{"_account_id":2861,"name":"Vincent Hou","email":"shou@us.ibm.com","username":"houshengbo"},"change_message_id":"cf7ced2ba351a4e612a5f5e913d0929716a5b99d","unresolved":false,"context_lines":[{"line_number":213,"context_line":""},{"line_number":214,"context_line":"        if isinstance(error_code, int):"},{"line_number":215,"context_line":"            error_code \u003d hex(error_code)"},{"line_number":216,"context_line":"            "},{"line_number":217,"context_line":"        if isinstance(error_code, str):"},{"line_number":218,"context_line":"            error_code \u003d error_code.strip()"},{"line_number":219,"context_line":"            found \u003d re.findall(error_code, output,"}],"source_content_type":"text/x-python","patch_set":3,"id":"3afb71cf_c1a59e04","line":216,"updated":"2015-06-08 09:25:17.000000000","message":"Identation issue.","commit_id":"ce612135c66199eaba6e4e0539ee8928ce66837e"},{"author":{"_account_id":13206,"name":"Cedric Zhuang","email":"cedric.zhuang@emc.com","username":"CedricZhuang"},"change_message_id":"ea9f3699af9fea9cfdfa089beae84c0c59deecb7","unresolved":false,"context_lines":[{"line_number":213,"context_line":""},{"line_number":214,"context_line":"        if isinstance(error_code, int):"},{"line_number":215,"context_line":"            error_code \u003d hex(error_code)"},{"line_number":216,"context_line":"            "},{"line_number":217,"context_line":"        if isinstance(error_code, str):"},{"line_number":218,"context_line":"            error_code \u003d error_code.strip()"},{"line_number":219,"context_line":"            found \u003d re.findall(error_code, output,"}],"source_content_type":"text/x-python","patch_set":3,"id":"3afb71cf_21a9127c","line":216,"in_reply_to":"3afb71cf_c1a59e04","updated":"2015-06-08 09:58:04.000000000","message":"Done","commit_id":"ce612135c66199eaba6e4e0539ee8928ce66837e"},{"author":{"_account_id":7198,"name":"Jay Bryant","email":"jungleboyj@electronicjungle.net","username":"jsbryant"},"change_message_id":"14b75743dff0559c5dc71b52fbc2df773ff817d3","unresolved":false,"context_lines":[{"line_number":172,"context_line":""},{"line_number":173,"context_line":"class VNXError(object):"},{"line_number":174,"context_line":""},{"line_number":175,"context_line":"    GENERAL_NOT_FOUND \u003d \u0027cannot find|not exist\u0027"},{"line_number":176,"context_line":""},{"line_number":177,"context_line":"    SG_NAME_IN_USE \u003d \u0027Storage Group name already in use\u0027"},{"line_number":178,"context_line":""}],"source_content_type":"text/x-python","patch_set":5,"id":"3afb71cf_04ee12ec","line":175,"updated":"2015-06-08 16:29:33.000000000","message":"Suggest \u0027cannot find|does not exist\u0027","commit_id":"8c217bb0fdd8a7e317ae787415b4628267941a87"},{"author":{"_account_id":13206,"name":"Cedric Zhuang","email":"cedric.zhuang@emc.com","username":"CedricZhuang"},"change_message_id":"bc66087c46fd803098a1530cd3953b2f1b0df306","unresolved":false,"context_lines":[{"line_number":172,"context_line":""},{"line_number":173,"context_line":"class VNXError(object):"},{"line_number":174,"context_line":""},{"line_number":175,"context_line":"    GENERAL_NOT_FOUND \u003d \u0027cannot find|not exist\u0027"},{"line_number":176,"context_line":""},{"line_number":177,"context_line":"    SG_NAME_IN_USE \u003d \u0027Storage Group name already in use\u0027"},{"line_number":178,"context_line":""}],"source_content_type":"text/x-python","patch_set":5,"id":"3afb71cf_c23575da","line":175,"in_reply_to":"3afb71cf_04ee12ec","updated":"2015-06-09 03:30:19.000000000","message":"I hope I could but I need to match \u0027may not exist\u0027 as well as \u0027does not exist\u0027.","commit_id":"8c217bb0fdd8a7e317ae787415b4628267941a87"},{"author":{"_account_id":6491,"name":"xing-yang","email":"xingyang105@gmail.com","username":"xing-yang"},"change_message_id":"259d36f85685a9b0340f76acc4139bddd4e86ca0","unresolved":false,"context_lines":[{"line_number":172,"context_line":""},{"line_number":173,"context_line":"class VNXError(object):"},{"line_number":174,"context_line":""},{"line_number":175,"context_line":"    GENERAL_NOT_FOUND \u003d \u0027cannot find|not exist\u0027"},{"line_number":176,"context_line":""},{"line_number":177,"context_line":"    SG_NAME_IN_USE \u003d \u0027Storage Group name already in use\u0027"},{"line_number":178,"context_line":""}],"source_content_type":"text/x-python","patch_set":5,"id":"fa32b979_d77f29dd","line":175,"in_reply_to":"3afb71cf_c23575da","updated":"2015-06-24 14:17:15.000000000","message":"I think this one does look confusing. Can you add a comment like this:  \"# Match \u0027may not exist\u0027 or \u0027does not exist\u0027 in the CLI error message.\"?","commit_id":"8c217bb0fdd8a7e317ae787415b4628267941a87"},{"author":{"_account_id":13206,"name":"Cedric Zhuang","email":"cedric.zhuang@emc.com","username":"CedricZhuang"},"change_message_id":"4126c3fd0317026201232c18947c5aaa27d4564a","unresolved":false,"context_lines":[{"line_number":172,"context_line":""},{"line_number":173,"context_line":"class VNXError(object):"},{"line_number":174,"context_line":""},{"line_number":175,"context_line":"    GENERAL_NOT_FOUND \u003d \u0027cannot find|not exist\u0027"},{"line_number":176,"context_line":""},{"line_number":177,"context_line":"    SG_NAME_IN_USE \u003d \u0027Storage Group name already in use\u0027"},{"line_number":178,"context_line":""}],"source_content_type":"text/x-python","patch_set":5,"id":"fa32b979_ebfd483c","line":175,"in_reply_to":"fa32b979_d77f29dd","updated":"2015-06-26 10:03:54.000000000","message":"Thanks for the comment.  I have updated it to \u0027cannot find|may not exist|does not exist\u0027.  Compared to a comment, I would prefer to change the code.","commit_id":"8c217bb0fdd8a7e317ae787415b4628267941a87"}]}