)]}'
{"cinder/volume/drivers/emc/emc_vmax_common.py":[{"author":{"_account_id":16917,"name":"Alex O\u0027Rourke","email":"alex.orourke@hpe.com","username":"aorourke"},"change_message_id":"1208c513e0c14c88773c7c6057c5511b81f2b1c1","unresolved":false,"context_lines":[{"line_number":2362,"context_line":"        LOG.info(_LI(\"Create Consistency Group: %(group)s.\"),"},{"line_number":2363,"context_line":"                 {\u0027group\u0027: group[\u0027id\u0027]})"},{"line_number":2364,"context_line":""},{"line_number":2365,"context_line":"        modelUpdate \u003d {\u0027status\u0027: \u0027available\u0027}"},{"line_number":2366,"context_line":"        volumeTypeId \u003d group[\u0027volume_type_id\u0027].replace(\",\", \"\")"},{"line_number":2367,"context_line":""},{"line_number":2368,"context_line":"        cgName \u003d self.utils.truncate_string(group[\u0027id\u0027], 8)"}],"source_content_type":"text/x-python","patch_set":1,"id":"da6ed579_a0c8da55","line":2365,"range":{"start_line":2365,"start_character":33,"end_line":2365,"end_character":44},"updated":"2016-01-15 16:54:11.000000000","message":"I don\u0027t think this should be changed back. There was a patch that swithched everything over to using the fields.ConsistencyGroupStatus.*   https://github.com/openstack/cinder/commit/14ef9330b16cb11a5707213ec56f5585ee9d55e3","commit_id":"55c4e89a56972a862872b9cc02bf500d28a76c31"},{"author":{"_account_id":6491,"name":"xing-yang","email":"xingyang105@gmail.com","username":"xing-yang"},"change_message_id":"b6874a172c1af4cc5eb44bec9f72e1f141d5a541","unresolved":false,"context_lines":[{"line_number":2362,"context_line":"        LOG.info(_LI(\"Create Consistency Group: %(group)s.\"),"},{"line_number":2363,"context_line":"                 {\u0027group\u0027: group[\u0027id\u0027]})"},{"line_number":2364,"context_line":""},{"line_number":2365,"context_line":"        modelUpdate \u003d {\u0027status\u0027: \u0027available\u0027}"},{"line_number":2366,"context_line":"        volumeTypeId \u003d group[\u0027volume_type_id\u0027].replace(\",\", \"\")"},{"line_number":2367,"context_line":""},{"line_number":2368,"context_line":"        cgName \u003d self.utils.truncate_string(group[\u0027id\u0027], 8)"}],"source_content_type":"text/x-python","patch_set":1,"id":"da6ed579_d22fdaa5","line":2365,"range":{"start_line":2365,"start_character":33,"end_line":2365,"end_character":44},"in_reply_to":"da6ed579_a0c8da55","updated":"2016-01-15 20:56:23.000000000","message":"Alex is right.  Please rebase and pick up the latest changes.","commit_id":"55c4e89a56972a862872b9cc02bf500d28a76c31"},{"author":{"_account_id":16917,"name":"Alex O\u0027Rourke","email":"alex.orourke@hpe.com","username":"aorourke"},"change_message_id":"1208c513e0c14c88773c7c6057c5511b81f2b1c1","unresolved":false,"context_lines":[{"line_number":4123,"context_line":"                     \"This adds and/or removes volumes from a CG.\"),"},{"line_number":4124,"context_line":"                 {\u0027group\u0027: group[\u0027id\u0027]})"},{"line_number":4125,"context_line":""},{"line_number":4126,"context_line":"        modelUpdate \u003d {\u0027status\u0027: \u0027available\u0027}"},{"line_number":4127,"context_line":"        volumeTypeId \u003d group[\u0027volume_type_id\u0027].replace(\",\", \"\")"},{"line_number":4128,"context_line":""},{"line_number":4129,"context_line":"        cg_name \u003d self.utils.truncate_string(group[\u0027id\u0027], 8)"}],"source_content_type":"text/x-python","patch_set":1,"id":"da6ed579_20d94a57","line":4126,"range":{"start_line":4126,"start_character":33,"end_line":4126,"end_character":44},"updated":"2016-01-15 16:54:11.000000000","message":"Same","commit_id":"55c4e89a56972a862872b9cc02bf500d28a76c31"},{"author":{"_account_id":16917,"name":"Alex O\u0027Rourke","email":"alex.orourke@hpe.com","username":"aorourke"},"change_message_id":"1208c513e0c14c88773c7c6057c5511b81f2b1c1","unresolved":false,"context_lines":[{"line_number":4223,"context_line":"            raise exception.VolumeBackendAPIException("},{"line_number":4224,"context_line":"                data\u003dexceptionMessage)"},{"line_number":4225,"context_line":""},{"line_number":4226,"context_line":"        modelUpdate \u003d {\u0027status\u0027: \u0027available\u0027}"},{"line_number":4227,"context_line":""},{"line_number":4228,"context_line":"        _poolInstanceName, storageSystem \u003d ("},{"line_number":4229,"context_line":"            self._get_pool_and_storage_system(extraSpecs))"}],"source_content_type":"text/x-python","patch_set":1,"id":"da6ed579_406b1e85","line":4226,"range":{"start_line":4226,"start_character":33,"end_line":4226,"end_character":44},"updated":"2016-01-15 16:54:11.000000000","message":"Same","commit_id":"55c4e89a56972a862872b9cc02bf500d28a76c31"},{"author":{"_account_id":14274,"name":"XinXiaohui","email":"xiaohui.xin@intel.com","username":"XinXiaohui"},"change_message_id":"614d008b11ef9f7fc5859e5f145619b9d764b9d3","unresolved":false,"context_lines":[{"line_number":2639,"context_line":"        :param context: the context"},{"line_number":2640,"context_line":"        :param cgsnapshot: the consistency group snapshot to be created"},{"line_number":2641,"context_line":"        :param snapshots: snapshots"},{"line_number":2642,"context_line":"        :returns: dict -- modelUpdate"},{"line_number":2643,"context_line":"        :returns: list -- list of snapshots"},{"line_number":2644,"context_line":"        :raises: VolumeBackendAPIException"},{"line_number":2645,"context_line":"        \"\"\""}],"source_content_type":"text/x-python","patch_set":6,"id":"dae33548_351d4810","line":2642,"range":{"start_line":2642,"start_character":26,"end_line":2642,"end_character":37},"updated":"2016-02-14 06:59:28.000000000","message":"As you have changed the return name, please also change it here accordingly. -- model_update","commit_id":"de3c9db66a50f648148e366c60e82196698ccd7d"},{"author":{"_account_id":12670,"name":"Helen Walsh","email":"helen.walsh@emc.com","username":"walshh2"},"change_message_id":"99c8c5c6af1c0703f76c8451e5773645b5db951f","unresolved":false,"context_lines":[{"line_number":2639,"context_line":"        :param context: the context"},{"line_number":2640,"context_line":"        :param cgsnapshot: the consistency group snapshot to be created"},{"line_number":2641,"context_line":"        :param snapshots: snapshots"},{"line_number":2642,"context_line":"        :returns: dict -- modelUpdate"},{"line_number":2643,"context_line":"        :returns: list -- list of snapshots"},{"line_number":2644,"context_line":"        :raises: VolumeBackendAPIException"},{"line_number":2645,"context_line":"        \"\"\""}],"source_content_type":"text/x-python","patch_set":6,"id":"dae33548_36976d49","line":2642,"range":{"start_line":2642,"start_character":26,"end_line":2642,"end_character":37},"in_reply_to":"dae33548_351d4810","updated":"2016-02-17 16:03:57.000000000","message":"Sure, I will change them","commit_id":"de3c9db66a50f648148e366c60e82196698ccd7d"},{"author":{"_account_id":14274,"name":"XinXiaohui","email":"xiaohui.xin@intel.com","username":"XinXiaohui"},"change_message_id":"614d008b11ef9f7fc5859e5f145619b9d764b9d3","unresolved":false,"context_lines":[{"line_number":2640,"context_line":"        :param cgsnapshot: the consistency group snapshot to be created"},{"line_number":2641,"context_line":"        :param snapshots: snapshots"},{"line_number":2642,"context_line":"        :returns: dict -- modelUpdate"},{"line_number":2643,"context_line":"        :returns: list -- list of snapshots"},{"line_number":2644,"context_line":"        :raises: VolumeBackendAPIException"},{"line_number":2645,"context_line":"        \"\"\""},{"line_number":2646,"context_line":"        consistencyGroup \u003d cgsnapshot.get(\u0027consistencygroup\u0027)"}],"source_content_type":"text/x-python","patch_set":6,"id":"dae33548_b5d79822","line":2643,"range":{"start_line":2643,"start_character":18,"end_line":2643,"end_character":22},"updated":"2016-02-14 06:59:28.000000000","message":"ditto","commit_id":"de3c9db66a50f648148e366c60e82196698ccd7d"}],"cinder/volume/drivers/emc/emc_vmax_utils.py":[{"author":{"_account_id":8912,"name":"Ryan McNair","email":"rdmcnair@us.ibm.com","username":"rdmcnair"},"change_message_id":"f69a63729c801e55a9fddd66bd9f68eeceebf6b8","unresolved":false,"context_lines":[{"line_number":2272,"context_line":"            self, context, volumes, cgId, status\u003d\u0027available\u0027):"},{"line_number":2273,"context_line":"        \"\"\"Update the volume model\u0027s status and return it."},{"line_number":2274,"context_line":""},{"line_number":2275,"context_line":"        :param context: the context"},{"line_number":2276,"context_line":"        :param volumes: volumes object api"},{"line_number":2277,"context_line":"        :param cgId: cg id"},{"line_number":2278,"context_line":"        :param status: string value reflects the status of the member volume"}],"source_content_type":"text/x-python","patch_set":5,"id":"5a5ae5dd_37afc995","line":2275,"updated":"2016-02-04 23:43:51.000000000","message":"This is just a list of volumes objects, not the volumes API, correct?","commit_id":"2282c7bd681f89cc6b12627a6fe09bfdec910a63"},{"author":{"_account_id":12924,"name":"Patrick East","email":"east.patrick@gmail.com","username":"patrick.east"},"change_message_id":"170495335bfc747ab23b5b400ed0fcbf2be0453e","unresolved":false,"context_lines":[{"line_number":2272,"context_line":"            self, context, volumes, cgId, status\u003d\u0027available\u0027):"},{"line_number":2273,"context_line":"        \"\"\"Update the volume model\u0027s status and return it."},{"line_number":2274,"context_line":""},{"line_number":2275,"context_line":"        :param context: the context"},{"line_number":2276,"context_line":"        :param volumes: volumes object api"},{"line_number":2277,"context_line":"        :param cgId: cg id"},{"line_number":2278,"context_line":"        :param status: string value reflects the status of the member volume"}],"source_content_type":"text/x-python","patch_set":5,"id":"5a5ae5dd_69655d1c","line":2275,"in_reply_to":"5a5ae5dd_37afc995","updated":"2016-02-05 04:00:48.000000000","message":"Yea, thats kind of a confusing description since it is just the objects.","commit_id":"2282c7bd681f89cc6b12627a6fe09bfdec910a63"},{"author":{"_account_id":6491,"name":"xing-yang","email":"xingyang105@gmail.com","username":"xing-yang"},"change_message_id":"2de96ed7ec8bfa7b59194d0811479336f264cb3c","unresolved":false,"context_lines":[{"line_number":2273,"context_line":"        \"\"\"Update the volume model\u0027s status and return it."},{"line_number":2274,"context_line":""},{"line_number":2275,"context_line":"        :param context: the context"},{"line_number":2276,"context_line":"        :param volumes: volumes object api"},{"line_number":2277,"context_line":"        :param cgId: cg id"},{"line_number":2278,"context_line":"        :param status: string value reflects the status of the member volume"},{"line_number":2279,"context_line":"        :returns: volume_model_updates - updated volumes"}],"source_content_type":"text/x-python","patch_set":5,"id":"5a5ae5dd_d653a069","line":2276,"updated":"2016-02-11 01:09:28.000000000","message":"Yes, should change the description to \"a list of volume objects\".","commit_id":"2282c7bd681f89cc6b12627a6fe09bfdec910a63"},{"author":{"_account_id":12670,"name":"Helen Walsh","email":"helen.walsh@emc.com","username":"walshh2"},"change_message_id":"c3ac95780276acb5aea3915dcb766dde8eea5fc9","unresolved":false,"context_lines":[{"line_number":2273,"context_line":"        \"\"\"Update the volume model\u0027s status and return it."},{"line_number":2274,"context_line":""},{"line_number":2275,"context_line":"        :param context: the context"},{"line_number":2276,"context_line":"        :param volumes: volumes object api"},{"line_number":2277,"context_line":"        :param cgId: cg id"},{"line_number":2278,"context_line":"        :param status: string value reflects the status of the member volume"},{"line_number":2279,"context_line":"        :returns: volume_model_updates - updated volumes"}],"source_content_type":"text/x-python","patch_set":5,"id":"3a57f1b5_97d1ef59","line":2276,"in_reply_to":"5a5ae5dd_d653a069","updated":"2016-02-11 10:17:41.000000000","message":"( will change this","commit_id":"2282c7bd681f89cc6b12627a6fe09bfdec910a63"},{"author":{"_account_id":14274,"name":"XinXiaohui","email":"xiaohui.xin@intel.com","username":"XinXiaohui"},"change_message_id":"614d008b11ef9f7fc5859e5f145619b9d764b9d3","unresolved":false,"context_lines":[{"line_number":2273,"context_line":"        \"\"\"Update the volume model\u0027s status and return it."},{"line_number":2274,"context_line":""},{"line_number":2275,"context_line":"        :param context: the context"},{"line_number":2276,"context_line":"        :param volumes: volumes object api"},{"line_number":2277,"context_line":"        :param cgId: cg id"},{"line_number":2278,"context_line":"        :param status: string value reflects the status of the member volume"},{"line_number":2279,"context_line":"        :returns: volume_model_updates - updated volumes"}],"source_content_type":"text/x-python","patch_set":6,"id":"dae33548_55260c17","line":2276,"range":{"start_line":2276,"start_character":24,"end_line":2276,"end_character":31},"updated":"2016-02-14 06:59:28.000000000","message":"The explanation is interesting -- not sure what is \"volumes object api\". :-)","commit_id":"de3c9db66a50f648148e366c60e82196698ccd7d"},{"author":{"_account_id":12670,"name":"Helen Walsh","email":"helen.walsh@emc.com","username":"walshh2"},"change_message_id":"99c8c5c6af1c0703f76c8451e5773645b5db951f","unresolved":false,"context_lines":[{"line_number":2273,"context_line":"        \"\"\"Update the volume model\u0027s status and return it."},{"line_number":2274,"context_line":""},{"line_number":2275,"context_line":"        :param context: the context"},{"line_number":2276,"context_line":"        :param volumes: volumes object api"},{"line_number":2277,"context_line":"        :param cgId: cg id"},{"line_number":2278,"context_line":"        :param status: string value reflects the status of the member volume"},{"line_number":2279,"context_line":"        :returns: volume_model_updates - updated volumes"}],"source_content_type":"text/x-python","patch_set":6,"id":"dae33548_d600c168","line":2276,"range":{"start_line":2276,"start_character":24,"end_line":2276,"end_character":31},"in_reply_to":"dae33548_55260c17","updated":"2016-02-17 16:03:57.000000000","message":"I guess it should be a \"list of volume objects\"","commit_id":"de3c9db66a50f648148e366c60e82196698ccd7d"}]}