)]}'
{"doc/source/command-objects/consistency-group.rst":[{"author":{"_account_id":6482,"name":"Steve Martinelli","email":"s.martinelli@gmail.com","username":"stevemar"},"change_message_id":"f1b1fe63606b68a6f1202a9af1dfd21069915f9b","unresolved":false,"context_lines":[{"line_number":13,"context_line":".. code:: bash"},{"line_number":14,"context_line":""},{"line_number":15,"context_line":"    os consistency group create"},{"line_number":16,"context_line":"        --volume-type \u003cvolume-type\u003e | --source-consistency-group \u003cconsistency-group\u003e"},{"line_number":17,"context_line":"        [--description \u003cdescription\u003e]"},{"line_number":18,"context_line":"        [--availability-zone \u003cavailability-zone\u003e]"},{"line_number":19,"context_line":"        [\u003cname\u003e]"}],"source_content_type":"text/x-rst","patch_set":2,"id":"9a629dbe_6cc37138","line":16,"range":{"start_line":16,"start_character":40,"end_line":16,"end_character":64},"updated":"2016-11-12 01:02:35.000000000","message":"i think \"consistency-group-source\" sounds better, but lets see what Mike and Dean say.","commit_id":"b72e5d6e3a7199f1a3179c7039cc5528ff228a1b"},{"author":{"_account_id":170,"name":"Mike Perez","email":"thingee@gmail.com","username":"thingee"},"change_message_id":"97d0f718b0956015d11a1905ca0b8247b5135d45","unresolved":false,"context_lines":[{"line_number":13,"context_line":".. code:: bash"},{"line_number":14,"context_line":""},{"line_number":15,"context_line":"    os consistency group create"},{"line_number":16,"context_line":"        --volume-type \u003cvolume-type\u003e | --source-consistency-group \u003cconsistency-group\u003e"},{"line_number":17,"context_line":"        [--description \u003cdescription\u003e]"},{"line_number":18,"context_line":"        [--availability-zone \u003cavailability-zone\u003e]"},{"line_number":19,"context_line":"        [\u003cname\u003e]"}],"source_content_type":"text/x-rst","patch_set":2,"id":"9a629dbe_37b2a24c","line":16,"range":{"start_line":16,"start_character":40,"end_line":16,"end_character":64},"in_reply_to":"9a629dbe_6cc37138","updated":"2016-11-12 01:38:21.000000000","message":"+1","commit_id":"b72e5d6e3a7199f1a3179c7039cc5528ff228a1b"},{"author":{"_account_id":21514,"name":"Huanxuan Ao","email":"aohuanxuan@163.com","username":"aohuanxuan"},"change_message_id":"ff3d48d3259cfdbbb66a325f645cb12031225a40","unresolved":false,"context_lines":[{"line_number":13,"context_line":".. code:: bash"},{"line_number":14,"context_line":""},{"line_number":15,"context_line":"    os consistency group create"},{"line_number":16,"context_line":"        --volume-type \u003cvolume-type\u003e | --source-consistency-group \u003cconsistency-group\u003e"},{"line_number":17,"context_line":"        [--description \u003cdescription\u003e]"},{"line_number":18,"context_line":"        [--availability-zone \u003cavailability-zone\u003e]"},{"line_number":19,"context_line":"        [\u003cname\u003e]"}],"source_content_type":"text/x-rst","patch_set":2,"id":"9a629dbe_c679b994","line":16,"range":{"start_line":16,"start_character":40,"end_line":16,"end_character":64},"in_reply_to":"9a629dbe_6cc37138","updated":"2016-11-12 04:19:02.000000000","message":"I will modify it, thanks","commit_id":"b72e5d6e3a7199f1a3179c7039cc5528ff228a1b"},{"author":{"_account_id":6482,"name":"Steve Martinelli","email":"s.martinelli@gmail.com","username":"stevemar"},"change_message_id":"f1b1fe63606b68a6f1202a9af1dfd21069915f9b","unresolved":false,"context_lines":[{"line_number":12,"context_line":".. program:: consistency group create"},{"line_number":13,"context_line":".. code:: bash"},{"line_number":14,"context_line":""},{"line_number":15,"context_line":"    os consistency group create"},{"line_number":16,"context_line":"        --volume-type \u003cvolume-type\u003e | --source-consistency-group \u003cconsistency-group\u003e"},{"line_number":17,"context_line":"        [--description \u003cdescription\u003e]"},{"line_number":18,"context_line":"        [--availability-zone \u003cavailability-zone\u003e]"},{"line_number":19,"context_line":"        [\u003cname\u003e]"},{"line_number":20,"context_line":""},{"line_number":21,"context_line":".. option:: --volume-type \u003cvolume-type\u003e"},{"line_number":22,"context_line":""}],"source_content_type":"text/x-rst","patch_set":2,"id":"9a629dbe_8ca7e5a5","line":19,"range":{"start_line":15,"start_character":0,"end_line":19,"end_character":16},"updated":"2016-11-12 01:02:35.000000000","message":"should we add options for user and project here too? the docstring from cinderclient says \"User id derived from context\", same for project ID, so maybe it\u0027s not necessary for a first pass.","commit_id":"b72e5d6e3a7199f1a3179c7039cc5528ff228a1b"},{"author":{"_account_id":21514,"name":"Huanxuan Ao","email":"aohuanxuan@163.com","username":"aohuanxuan"},"change_message_id":"ff3d48d3259cfdbbb66a325f645cb12031225a40","unresolved":false,"context_lines":[{"line_number":12,"context_line":".. program:: consistency group create"},{"line_number":13,"context_line":".. code:: bash"},{"line_number":14,"context_line":""},{"line_number":15,"context_line":"    os consistency group create"},{"line_number":16,"context_line":"        --volume-type \u003cvolume-type\u003e | --source-consistency-group \u003cconsistency-group\u003e"},{"line_number":17,"context_line":"        [--description \u003cdescription\u003e]"},{"line_number":18,"context_line":"        [--availability-zone \u003cavailability-zone\u003e]"},{"line_number":19,"context_line":"        [\u003cname\u003e]"},{"line_number":20,"context_line":""},{"line_number":21,"context_line":".. option:: --volume-type \u003cvolume-type\u003e"},{"line_number":22,"context_line":""}],"source_content_type":"text/x-rst","patch_set":2,"id":"9a629dbe_865c211a","line":19,"range":{"start_line":15,"start_character":0,"end_line":19,"end_character":16},"in_reply_to":"9a629dbe_8ca7e5a5","updated":"2016-11-12 04:19:02.000000000","message":"Yes, I planned to do it, will in next patch set.","commit_id":"b72e5d6e3a7199f1a3179c7039cc5528ff228a1b"},{"author":{"_account_id":6482,"name":"Steve Martinelli","email":"s.martinelli@gmail.com","username":"stevemar"},"change_message_id":"f1b1fe63606b68a6f1202a9af1dfd21069915f9b","unresolved":false,"context_lines":[{"line_number":20,"context_line":""},{"line_number":21,"context_line":".. option:: --volume-type \u003cvolume-type\u003e"},{"line_number":22,"context_line":""},{"line_number":23,"context_line":"    Volume type of this consistency group (name or ID)"},{"line_number":24,"context_line":""},{"line_number":25,"context_line":".. option:: --source-consistency-group \u003cconsistency-group\u003e"},{"line_number":26,"context_line":""}],"source_content_type":"text/x-rst","patch_set":2,"id":"9a629dbe_ece5c1db","line":23,"range":{"start_line":23,"start_character":4,"end_line":23,"end_character":54},"updated":"2016-11-12 01:02:35.000000000","message":"i know we have \"volume types\", but the docstring in cinderclient says \"Types of volume\" -- that sounds weird to me. https://github.com/openstack/python-cinderclient/blob/master/cinderclient/v3/consistencygroups.py#L48","commit_id":"b72e5d6e3a7199f1a3179c7039cc5528ff228a1b"},{"author":{"_account_id":170,"name":"Mike Perez","email":"thingee@gmail.com","username":"thingee"},"change_message_id":"97d0f718b0956015d11a1905ca0b8247b5135d45","unresolved":false,"context_lines":[{"line_number":20,"context_line":""},{"line_number":21,"context_line":".. option:: --volume-type \u003cvolume-type\u003e"},{"line_number":22,"context_line":""},{"line_number":23,"context_line":"    Volume type of this consistency group (name or ID)"},{"line_number":24,"context_line":""},{"line_number":25,"context_line":".. option:: --source-consistency-group \u003cconsistency-group\u003e"},{"line_number":26,"context_line":""}],"source_content_type":"text/x-rst","patch_set":2,"id":"9a629dbe_f77aea75","line":23,"range":{"start_line":23,"start_character":4,"end_line":23,"end_character":54},"in_reply_to":"9a629dbe_ece5c1db","updated":"2016-11-12 01:38:21.000000000","message":"I think cinderclient should be updated.","commit_id":"b72e5d6e3a7199f1a3179c7039cc5528ff228a1b"},{"author":{"_account_id":21514,"name":"Huanxuan Ao","email":"aohuanxuan@163.com","username":"aohuanxuan"},"change_message_id":"ff3d48d3259cfdbbb66a325f645cb12031225a40","unresolved":false,"context_lines":[{"line_number":20,"context_line":""},{"line_number":21,"context_line":".. option:: --volume-type \u003cvolume-type\u003e"},{"line_number":22,"context_line":""},{"line_number":23,"context_line":"    Volume type of this consistency group (name or ID)"},{"line_number":24,"context_line":""},{"line_number":25,"context_line":".. option:: --source-consistency-group \u003cconsistency-group\u003e"},{"line_number":26,"context_line":""}],"source_content_type":"text/x-rst","patch_set":2,"id":"9a629dbe_e624fd60","line":23,"range":{"start_line":23,"start_character":4,"end_line":23,"end_character":54},"in_reply_to":"9a629dbe_f77aea75","updated":"2016-11-12 04:19:02.000000000","message":"Agree with Mike ：）","commit_id":"b72e5d6e3a7199f1a3179c7039cc5528ff228a1b"}],"openstackclient/tests/unit/volume/v2/test_consistency_group.py":[{"author":{"_account_id":8276,"name":"kiwik","email":"chenrui.momo@gmail.com","username":"kiwik"},"change_message_id":"9b74703e46b73768c7a6f5c5f9751ef92c35bfff","unresolved":false,"context_lines":[{"line_number":48,"context_line":""},{"line_number":49,"context_line":"    def setUp(self):"},{"line_number":50,"context_line":"        super(TestConsistencyGroupCreate, self).setUp()"},{"line_number":51,"context_line":"        self.new_consistency_group \u003d ("},{"line_number":52,"context_line":"            volume_fakes.FakeConsistencyGroup.create_one_consistency_group())"},{"line_number":53,"context_line":"        self.consistencygroups_mock.create.return_value \u003d ("},{"line_number":54,"context_line":"            self.new_consistency_group)"},{"line_number":55,"context_line":"        self.consistencygroups_mock.create_from_src.return_value \u003d ("}],"source_content_type":"text/x-python","patch_set":4,"id":"7a77a97e_78b1198f","line":52,"range":{"start_line":51,"start_character":0,"end_line":52,"end_character":77},"updated":"2016-11-16 02:43:34.000000000","message":"we can move these code out from setUp() to avoid building fake consistency group when each test case run, no value to -1, just a suggestion, if you push a new patch, please fix these by the way.","commit_id":"f7abf17cd86606fbf4d2153cf808ba12d9b4390c"},{"author":{"_account_id":21514,"name":"Huanxuan Ao","email":"aohuanxuan@163.com","username":"aohuanxuan"},"change_message_id":"ec84ab2814b861df73b7ee9f59ed745ddacd62a8","unresolved":false,"context_lines":[{"line_number":48,"context_line":""},{"line_number":49,"context_line":"    def setUp(self):"},{"line_number":50,"context_line":"        super(TestConsistencyGroupCreate, self).setUp()"},{"line_number":51,"context_line":"        self.new_consistency_group \u003d ("},{"line_number":52,"context_line":"            volume_fakes.FakeConsistencyGroup.create_one_consistency_group())"},{"line_number":53,"context_line":"        self.consistencygroups_mock.create.return_value \u003d ("},{"line_number":54,"context_line":"            self.new_consistency_group)"},{"line_number":55,"context_line":"        self.consistencygroups_mock.create_from_src.return_value \u003d ("}],"source_content_type":"text/x-python","patch_set":4,"id":"7a77a97e_93f5f6cc","line":52,"range":{"start_line":51,"start_character":0,"end_line":52,"end_character":77},"in_reply_to":"7a77a97e_78b1198f","updated":"2016-11-16 03:08:37.000000000","message":"Yes, you are right, we usually did like this because sometimes the \"links\" attribute will be removed in each test case so that we need to create new and full fake resource for each test cases, but I notice consistency group has no \"links\" attribute, so we need not do it like this, I will modify it, thank you!","commit_id":"f7abf17cd86606fbf4d2153cf808ba12d9b4390c"},{"author":{"_account_id":8276,"name":"kiwik","email":"chenrui.momo@gmail.com","username":"kiwik"},"change_message_id":"9b74703e46b73768c7a6f5c5f9751ef92c35bfff","unresolved":false,"context_lines":[{"line_number":58,"context_line":"            self.new_consistency_group)"},{"line_number":59,"context_line":"        self.types_mock.get.return_value \u003d self.volume_type"},{"line_number":60,"context_line":""},{"line_number":61,"context_line":"        self.data \u003d ("},{"line_number":62,"context_line":"            self.new_consistency_group.availability_zone,"},{"line_number":63,"context_line":"            self.new_consistency_group.created_at,"},{"line_number":64,"context_line":"            self.new_consistency_group.description,"},{"line_number":65,"context_line":"            self.new_consistency_group.id,"},{"line_number":66,"context_line":"            self.new_consistency_group.name,"},{"line_number":67,"context_line":"            self.new_consistency_group.status,"},{"line_number":68,"context_line":"            self.new_consistency_group.volume_types,"},{"line_number":69,"context_line":"        )"},{"line_number":70,"context_line":""},{"line_number":71,"context_line":"        # Get the command object to test"},{"line_number":72,"context_line":"        self.cmd \u003d consistency_group.CreateConsistencyGroup(self.app, None)"}],"source_content_type":"text/x-python","patch_set":4,"id":"7a77a97e_b8abb11c","line":69,"range":{"start_line":61,"start_character":0,"end_line":69,"end_character":9},"updated":"2016-11-16 02:43:34.000000000","message":"ditto","commit_id":"f7abf17cd86606fbf4d2153cf808ba12d9b4390c"},{"author":{"_account_id":21514,"name":"Huanxuan Ao","email":"aohuanxuan@163.com","username":"aohuanxuan"},"change_message_id":"ec84ab2814b861df73b7ee9f59ed745ddacd62a8","unresolved":false,"context_lines":[{"line_number":58,"context_line":"            self.new_consistency_group)"},{"line_number":59,"context_line":"        self.types_mock.get.return_value \u003d self.volume_type"},{"line_number":60,"context_line":""},{"line_number":61,"context_line":"        self.data \u003d ("},{"line_number":62,"context_line":"            self.new_consistency_group.availability_zone,"},{"line_number":63,"context_line":"            self.new_consistency_group.created_at,"},{"line_number":64,"context_line":"            self.new_consistency_group.description,"},{"line_number":65,"context_line":"            self.new_consistency_group.id,"},{"line_number":66,"context_line":"            self.new_consistency_group.name,"},{"line_number":67,"context_line":"            self.new_consistency_group.status,"},{"line_number":68,"context_line":"            self.new_consistency_group.volume_types,"},{"line_number":69,"context_line":"        )"},{"line_number":70,"context_line":""},{"line_number":71,"context_line":"        # Get the command object to test"},{"line_number":72,"context_line":"        self.cmd \u003d consistency_group.CreateConsistencyGroup(self.app, None)"}],"source_content_type":"text/x-python","patch_set":4,"id":"7a77a97e_d3ff6ea8","line":69,"range":{"start_line":61,"start_character":0,"end_line":69,"end_character":9},"in_reply_to":"7a77a97e_b8abb11c","updated":"2016-11-16 03:08:37.000000000","message":"Done","commit_id":"f7abf17cd86606fbf4d2153cf808ba12d9b4390c"}],"openstackclient/volume/v2/consistency_group.py":[{"author":{"_account_id":6482,"name":"Steve Martinelli","email":"s.martinelli@gmail.com","username":"stevemar"},"change_message_id":"f1b1fe63606b68a6f1202a9af1dfd21069915f9b","unresolved":false,"context_lines":[{"line_number":37,"context_line":"            nargs\u003d\"?\","},{"line_number":38,"context_line":"            help\u003d_(\"Name of new consistency group (default to None)\")"},{"line_number":39,"context_line":"        )"},{"line_number":40,"context_line":"        exclusive_group \u003d parser.add_mutually_exclusive_group(required\u003dTrue)"},{"line_number":41,"context_line":"        exclusive_group.add_argument("},{"line_number":42,"context_line":"            \"--volume-type\","},{"line_number":43,"context_line":"            metavar\u003d\"\u003cvolume-type\u003e\","}],"source_content_type":"text/x-python","patch_set":2,"id":"9a629dbe_ac2d89ff","line":40,"range":{"start_line":40,"start_character":62,"end_line":40,"end_character":75},"updated":"2016-11-12 01:02:35.000000000","message":"nice! i didn\u0027t know this was a thing","commit_id":"b72e5d6e3a7199f1a3179c7039cc5528ff228a1b"},{"author":{"_account_id":6482,"name":"Steve Martinelli","email":"s.martinelli@gmail.com","username":"stevemar"},"change_message_id":"f1b1fe63606b68a6f1202a9af1dfd21069915f9b","unresolved":false,"context_lines":[{"line_number":72,"context_line":"                description\u003dparsed_args.description,"},{"line_number":73,"context_line":"                availability_zone\u003dparsed_args.availability_zone"},{"line_number":74,"context_line":"            )"},{"line_number":75,"context_line":"        if parsed_args.source_consistency_group:"},{"line_number":76,"context_line":"            if parsed_args.availability_zone:"},{"line_number":77,"context_line":"                msg \u003d _(\"\u0027--availability-zone\u0027 option will not work \""},{"line_number":78,"context_line":"                        \"if create consistency group from source\")"}],"source_content_type":"text/x-python","patch_set":2,"id":"9a629dbe_0c37554d","line":75,"range":{"start_line":75,"start_character":8,"end_line":75,"end_character":10},"updated":"2016-11-12 01:02:35.000000000","message":"elif, but meh","commit_id":"b72e5d6e3a7199f1a3179c7039cc5528ff228a1b"},{"author":{"_account_id":21514,"name":"Huanxuan Ao","email":"aohuanxuan@163.com","username":"aohuanxuan"},"change_message_id":"ff3d48d3259cfdbbb66a325f645cb12031225a40","unresolved":false,"context_lines":[{"line_number":72,"context_line":"                description\u003dparsed_args.description,"},{"line_number":73,"context_line":"                availability_zone\u003dparsed_args.availability_zone"},{"line_number":74,"context_line":"            )"},{"line_number":75,"context_line":"        if parsed_args.source_consistency_group:"},{"line_number":76,"context_line":"            if parsed_args.availability_zone:"},{"line_number":77,"context_line":"                msg \u003d _(\"\u0027--availability-zone\u0027 option will not work \""},{"line_number":78,"context_line":"                        \"if create consistency group from source\")"}],"source_content_type":"text/x-python","patch_set":2,"id":"9a629dbe_a6d2055a","line":75,"range":{"start_line":75,"start_character":8,"end_line":75,"end_character":10},"in_reply_to":"9a629dbe_0c37554d","updated":"2016-11-12 04:19:02.000000000","message":"OK will modify it.","commit_id":"b72e5d6e3a7199f1a3179c7039cc5528ff228a1b"},{"author":{"_account_id":6482,"name":"Steve Martinelli","email":"s.martinelli@gmail.com","username":"stevemar"},"change_message_id":"f1b1fe63606b68a6f1202a9af1dfd21069915f9b","unresolved":false,"context_lines":[{"line_number":75,"context_line":"        if parsed_args.source_consistency_group:"},{"line_number":76,"context_line":"            if parsed_args.availability_zone:"},{"line_number":77,"context_line":"                msg \u003d _(\"\u0027--availability-zone\u0027 option will not work \""},{"line_number":78,"context_line":"                        \"if create consistency group from source\")"},{"line_number":79,"context_line":"                LOG.warning(msg)"},{"line_number":80,"context_line":"            consistency_group_id \u003d utils.find_resource("},{"line_number":81,"context_line":"                volume_client.consistencygroups,"}],"source_content_type":"text/x-python","patch_set":2,"id":"9a629dbe_ac066983","line":78,"range":{"start_line":78,"start_character":28,"end_line":78,"end_character":34},"updated":"2016-11-12 01:02:35.000000000","message":"creating*","commit_id":"b72e5d6e3a7199f1a3179c7039cc5528ff228a1b"},{"author":{"_account_id":21514,"name":"Huanxuan Ao","email":"aohuanxuan@163.com","username":"aohuanxuan"},"change_message_id":"ff3d48d3259cfdbbb66a325f645cb12031225a40","unresolved":false,"context_lines":[{"line_number":75,"context_line":"        if parsed_args.source_consistency_group:"},{"line_number":76,"context_line":"            if parsed_args.availability_zone:"},{"line_number":77,"context_line":"                msg \u003d _(\"\u0027--availability-zone\u0027 option will not work \""},{"line_number":78,"context_line":"                        \"if create consistency group from source\")"},{"line_number":79,"context_line":"                LOG.warning(msg)"},{"line_number":80,"context_line":"            consistency_group_id \u003d utils.find_resource("},{"line_number":81,"context_line":"                volume_client.consistencygroups,"}],"source_content_type":"text/x-python","patch_set":2,"id":"9a629dbe_668f0d75","line":78,"range":{"start_line":78,"start_character":28,"end_line":78,"end_character":34},"in_reply_to":"9a629dbe_ac066983","updated":"2016-11-12 04:19:02.000000000","message":"Thank you :)","commit_id":"b72e5d6e3a7199f1a3179c7039cc5528ff228a1b"},{"author":{"_account_id":6482,"name":"Steve Martinelli","email":"s.martinelli@gmail.com","username":"stevemar"},"change_message_id":"f1b1fe63606b68a6f1202a9af1dfd21069915f9b","unresolved":false,"context_lines":[{"line_number":74,"context_line":"            )"},{"line_number":75,"context_line":"        if parsed_args.source_consistency_group:"},{"line_number":76,"context_line":"            if parsed_args.availability_zone:"},{"line_number":77,"context_line":"                msg \u003d _(\"\u0027--availability-zone\u0027 option will not work \""},{"line_number":78,"context_line":"                        \"if create consistency group from source\")"},{"line_number":79,"context_line":"                LOG.warning(msg)"},{"line_number":80,"context_line":"            consistency_group_id \u003d utils.find_resource("},{"line_number":81,"context_line":"                volume_client.consistencygroups,"},{"line_number":82,"context_line":"                parsed_args.source_consistency_group).id"}],"source_content_type":"text/x-python","patch_set":2,"id":"9a629dbe_ecfce173","line":79,"range":{"start_line":77,"start_character":16,"end_line":79,"end_character":32},"updated":"2016-11-12 01:02:35.000000000","message":"should we exit / raise an exception at this point?","commit_id":"b72e5d6e3a7199f1a3179c7039cc5528ff228a1b"},{"author":{"_account_id":21514,"name":"Huanxuan Ao","email":"aohuanxuan@163.com","username":"aohuanxuan"},"change_message_id":"ff3d48d3259cfdbbb66a325f645cb12031225a40","unresolved":false,"context_lines":[{"line_number":74,"context_line":"            )"},{"line_number":75,"context_line":"        if parsed_args.source_consistency_group:"},{"line_number":76,"context_line":"            if parsed_args.availability_zone:"},{"line_number":77,"context_line":"                msg \u003d _(\"\u0027--availability-zone\u0027 option will not work \""},{"line_number":78,"context_line":"                        \"if create consistency group from source\")"},{"line_number":79,"context_line":"                LOG.warning(msg)"},{"line_number":80,"context_line":"            consistency_group_id \u003d utils.find_resource("},{"line_number":81,"context_line":"                volume_client.consistencygroups,"},{"line_number":82,"context_line":"                parsed_args.source_consistency_group).id"}],"source_content_type":"text/x-python","patch_set":2,"id":"9a629dbe_e656bdcc","line":79,"range":{"start_line":77,"start_character":16,"end_line":79,"end_character":32},"in_reply_to":"9a629dbe_ecfce173","updated":"2016-11-12 04:19:02.000000000","message":"I do not sure about this, I do not think it is a serious error that should make creating exited. But everyway I think we will need add message like \"（not available if creating form source）\" in the help message of \"--availability-zone\" option.","commit_id":"b72e5d6e3a7199f1a3179c7039cc5528ff228a1b"},{"author":{"_account_id":6482,"name":"Steve Martinelli","email":"s.martinelli@gmail.com","username":"stevemar"},"change_message_id":"f1b1fe63606b68a6f1202a9af1dfd21069915f9b","unresolved":false,"context_lines":[{"line_number":82,"context_line":"                parsed_args.source_consistency_group).id"},{"line_number":83,"context_line":"            consistency_group \u003d ("},{"line_number":84,"context_line":"                volume_client.consistencygroups.create_from_src("},{"line_number":85,"context_line":"                    None,"},{"line_number":86,"context_line":"                    consistency_group_id,"},{"line_number":87,"context_line":"                    name\u003dparsed_args.name,"},{"line_number":88,"context_line":"                    description\u003dparsed_args.description"}],"source_content_type":"text/x-python","patch_set":2,"id":"9a629dbe_cc171da1","line":85,"range":{"start_line":85,"start_character":20,"end_line":85,"end_character":24},"updated":"2016-11-12 01:02:35.000000000","message":"maybe create a variable before this call so we know it\u0027s a TODO.\n\n  # TODO: Add logic to create from consistency group snapshot\n  _cgsnapshot_id \u003d None\n\nand pass in _cgsnapshot_id instead of None?","commit_id":"b72e5d6e3a7199f1a3179c7039cc5528ff228a1b"},{"author":{"_account_id":21514,"name":"Huanxuan Ao","email":"aohuanxuan@163.com","username":"aohuanxuan"},"change_message_id":"ff3d48d3259cfdbbb66a325f645cb12031225a40","unresolved":false,"context_lines":[{"line_number":82,"context_line":"                parsed_args.source_consistency_group).id"},{"line_number":83,"context_line":"            consistency_group \u003d ("},{"line_number":84,"context_line":"                volume_client.consistencygroups.create_from_src("},{"line_number":85,"context_line":"                    None,"},{"line_number":86,"context_line":"                    consistency_group_id,"},{"line_number":87,"context_line":"                    name\u003dparsed_args.name,"},{"line_number":88,"context_line":"                    description\u003dparsed_args.description"}],"source_content_type":"text/x-python","patch_set":2,"id":"9a629dbe_66e62d45","line":85,"range":{"start_line":85,"start_character":20,"end_line":85,"end_character":24},"in_reply_to":"9a629dbe_cc171da1","updated":"2016-11-12 04:19:02.000000000","message":"OK, I will leave a TODO here, and plan to add \"--consistency-group-snapshot\" option after \"consistency_group_snapshot\" resource being added in OSC.","commit_id":"b72e5d6e3a7199f1a3179c7039cc5528ff228a1b"},{"author":{"_account_id":8276,"name":"kiwik","email":"chenrui.momo@gmail.com","username":"kiwik"},"change_message_id":"9b74703e46b73768c7a6f5c5f9751ef92c35bfff","unresolved":false,"context_lines":[{"line_number":85,"context_line":"            consistency_group_snapshot \u003d None"},{"line_number":86,"context_line":"            # TODO(Huanxuan Ao): Support for creating from consistency group"},{"line_number":87,"context_line":"            # snapshot after adding \"consistency_group_snapshot\" resource"},{"line_number":88,"context_line":""},{"line_number":89,"context_line":"            consistency_group \u003d ("},{"line_number":90,"context_line":"                volume_client.consistencygroups.create_from_src("},{"line_number":91,"context_line":"                    consistency_group_snapshot,"}],"source_content_type":"text/x-python","patch_set":4,"id":"7a77a97e_388401d3","line":88,"updated":"2016-11-16 02:43:34.000000000","message":"remove the blank line.","commit_id":"f7abf17cd86606fbf4d2153cf808ba12d9b4390c"}]}