)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":23083,"name":"TommyLike","email":"tommylikehu@gmail.com","username":"TommyLike"},"change_message_id":"32fe14793fd2d44e833dc0221c9e8a06f634edd4","unresolved":false,"context_lines":[{"line_number":12,"context_line":"Server side patch is here:"},{"line_number":13,"context_line":"    https://review.openstack.org/#/c/352228/"},{"line_number":14,"context_line":""},{"line_number":15,"context_line":"Change-Id: I462c3ab8c9c3a6a1b434748f81d208359ffd2431"},{"line_number":16,"context_line":"Implements: blueprint replication-cg"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":14,"id":"ff0f0b1f_80957d35","line":15,"range":{"start_line":15,"start_character":0,"end_line":15,"end_character":52},"updated":"2017-05-23 22:38:32.000000000","message":"Add Depends-On tag here?","commit_id":"7caf70aa2ef19f9ab7a7d42f152bb3e6d5c76dd7"}],"cinderclient/tests/unit/v3/fakes.py":[{"author":{"_account_id":23602,"name":"Michael Dovgal","email":"dovgalmichael@gmail.com","username":"mdovgal"},"change_message_id":"a2730af0952a19933472d9496f5370af1c345ef9","unresolved":false,"context_lines":[{"line_number":355,"context_line":"        action \u003d list(body)[0]"},{"line_number":356,"context_line":"        if action \u003d\u003d \u0027delete\u0027:"},{"line_number":357,"context_line":"            assert \u0027delete-volumes\u0027 in body[action]"},{"line_number":358,"context_line":"        elif action \u003d\u003d \u0027enable_replication\u0027:"},{"line_number":359,"context_line":"            assert \u0027enable_replication\u0027 in body"},{"line_number":360,"context_line":"        elif action \u003d\u003d \u0027disable_replication\u0027:"},{"line_number":361,"context_line":"            assert \u0027disable_replication\u0027 in body"},{"line_number":362,"context_line":"        elif action \u003d\u003d \u0027failover_replication\u0027:"},{"line_number":363,"context_line":"            assert \u0027failover_replication\u0027 in body"},{"line_number":364,"context_line":"        elif action \u003d\u003d \u0027list_replication_targets\u0027:"},{"line_number":365,"context_line":"            assert \u0027list_replication_targets\u0027 in body"},{"line_number":366,"context_line":"        else:"},{"line_number":367,"context_line":"            raise AssertionError(\"Unexpected action: %s\" % action)"},{"line_number":368,"context_line":"        return (resp, {}, {})"},{"line_number":369,"context_line":""}],"source_content_type":"text/x-python","patch_set":9,"id":"7ffa3b31_4c022771","line":366,"range":{"start_line":358,"start_character":0,"end_line":366,"end_character":1},"updated":"2017-04-20 14:25:22.000000000","message":"I think it\u0027s better to write something like\nif action in actions:\n    assert action in body\n\nand why we have \"\u0027delete-volumes\u0027 in body[action]\" but the rest just in body?","commit_id":"4bc69573e8cf8e419eebd0b20306214d3e3ec3ab"},{"author":{"_account_id":6491,"name":"xing-yang","email":"xingyang105@gmail.com","username":"xing-yang"},"change_message_id":"0df5465253611745543e40cfa644fe1189bda107","unresolved":false,"context_lines":[{"line_number":355,"context_line":"        action \u003d list(body)[0]"},{"line_number":356,"context_line":"        if action \u003d\u003d \u0027delete\u0027:"},{"line_number":357,"context_line":"            assert \u0027delete-volumes\u0027 in body[action]"},{"line_number":358,"context_line":"        elif action \u003d\u003d \u0027enable_replication\u0027:"},{"line_number":359,"context_line":"            assert \u0027enable_replication\u0027 in body"},{"line_number":360,"context_line":"        elif action \u003d\u003d \u0027disable_replication\u0027:"},{"line_number":361,"context_line":"            assert \u0027disable_replication\u0027 in body"},{"line_number":362,"context_line":"        elif action \u003d\u003d \u0027failover_replication\u0027:"},{"line_number":363,"context_line":"            assert \u0027failover_replication\u0027 in body"},{"line_number":364,"context_line":"        elif action \u003d\u003d \u0027list_replication_targets\u0027:"},{"line_number":365,"context_line":"            assert \u0027list_replication_targets\u0027 in body"},{"line_number":366,"context_line":"        else:"},{"line_number":367,"context_line":"            raise AssertionError(\"Unexpected action: %s\" % action)"},{"line_number":368,"context_line":"        return (resp, {}, {})"},{"line_number":369,"context_line":""}],"source_content_type":"text/x-python","patch_set":9,"id":"7ffa3b31_f74f5468","line":366,"range":{"start_line":358,"start_character":0,"end_line":366,"end_character":1},"in_reply_to":"7ffa3b31_4c022771","updated":"2017-04-21 17:33:23.000000000","message":"This is because all the replication actions have an empty body.  {\u0027xxxx_replication\u0027: {}}.  So we don\u0027t have anything to assert in body[\u0027xxxx_replication\u0027].\n\nFor the delete action, we can also do assert \u0027delete\u0027 in body and then assert \u0027delete-volumes\u0027 in body[\u0027delete\u0027].  But since this is a patch on replication, I won\u0027t want to touch \u0027delete\u0027 now.","commit_id":"4bc69573e8cf8e419eebd0b20306214d3e3ec3ab"}],"cinderclient/tests/unit/v3/test_shell.py":[{"author":{"_account_id":23083,"name":"TommyLike","email":"tommylikehu@gmail.com","username":"TommyLike"},"change_message_id":"16b10f86b38fe481b9a93e936153754398dcb679","unresolved":false,"context_lines":[{"line_number":717,"context_line":"        self.assert_called(\u0027POST\u0027, \u0027/groups/1234/action\u0027, expected)"},{"line_number":718,"context_line":""},{"line_number":719,"context_line":"    def test_group_failover_replication(self):"},{"line_number":720,"context_line":"        cmd \u003d \u0027--os-volume-api-version 3.35 group-failover-replication 1234\u0027"},{"line_number":721,"context_line":"        self.run_command(cmd)"},{"line_number":722,"context_line":"        expected \u003d {\u0027failover_replication\u0027:"},{"line_number":723,"context_line":"                    {\u0027allow_attached_volume\u0027: False,"}],"source_content_type":"text/x-python","patch_set":16,"id":"df140735_d23a8632","line":720,"range":{"start_line":720,"start_character":7,"end_line":720,"end_character":76},"updated":"2017-06-01 02:44:46.000000000","message":"Add some testcases that the two options are specified :)","commit_id":"ec4d1a39deeb0d46bd49816d711b948fa5741740"},{"author":{"_account_id":6491,"name":"xing-yang","email":"xingyang105@gmail.com","username":"xing-yang"},"change_message_id":"602f70de83c3f14d5716cb0aad8ae7adf1d5b044","unresolved":false,"context_lines":[{"line_number":717,"context_line":"        self.assert_called(\u0027POST\u0027, \u0027/groups/1234/action\u0027, expected)"},{"line_number":718,"context_line":""},{"line_number":719,"context_line":"    def test_group_failover_replication(self):"},{"line_number":720,"context_line":"        cmd \u003d \u0027--os-volume-api-version 3.35 group-failover-replication 1234\u0027"},{"line_number":721,"context_line":"        self.run_command(cmd)"},{"line_number":722,"context_line":"        expected \u003d {\u0027failover_replication\u0027:"},{"line_number":723,"context_line":"                    {\u0027allow_attached_volume\u0027: False,"}],"source_content_type":"text/x-python","patch_set":16,"id":"df140735_e27453d1","line":720,"range":{"start_line":720,"start_character":7,"end_line":720,"end_character":76},"in_reply_to":"df140735_17c9e724","updated":"2017-06-02 20:44:46.000000000","message":"Done","commit_id":"ec4d1a39deeb0d46bd49816d711b948fa5741740"},{"author":{"_account_id":6491,"name":"xing-yang","email":"xingyang105@gmail.com","username":"xing-yang"},"change_message_id":"00a09d2d6393622ea2f888d9f18d78ad6dd996f7","unresolved":false,"context_lines":[{"line_number":717,"context_line":"        self.assert_called(\u0027POST\u0027, \u0027/groups/1234/action\u0027, expected)"},{"line_number":718,"context_line":""},{"line_number":719,"context_line":"    def test_group_failover_replication(self):"},{"line_number":720,"context_line":"        cmd \u003d \u0027--os-volume-api-version 3.35 group-failover-replication 1234\u0027"},{"line_number":721,"context_line":"        self.run_command(cmd)"},{"line_number":722,"context_line":"        expected \u003d {\u0027failover_replication\u0027:"},{"line_number":723,"context_line":"                    {\u0027allow_attached_volume\u0027: False,"}],"source_content_type":"text/x-python","patch_set":16,"id":"df140735_17c9e724","line":720,"range":{"start_line":720,"start_character":7,"end_line":720,"end_character":76},"in_reply_to":"df140735_d23a8632","updated":"2017-06-02 04:14:26.000000000","message":"Will add soon.","commit_id":"ec4d1a39deeb0d46bd49816d711b948fa5741740"},{"author":{"_account_id":4523,"name":"Eric Harney","email":"eharney@redhat.com","username":"eharney"},"change_message_id":"60332b14890f7477d730af972360133b48dc3d79","unresolved":false,"context_lines":[{"line_number":721,"context_line":"        expected \u003d {\u0027disable_replication\u0027: {}}"},{"line_number":722,"context_line":"        self.assert_called(\u0027POST\u0027, \u0027/groups/1234/action\u0027, body\u003dexpected)"},{"line_number":723,"context_line":""},{"line_number":724,"context_line":"    @ddt.data((False, False), (True, False),"},{"line_number":725,"context_line":"              (False, \"backend1\"), (True, \"backend1\"),"},{"line_number":726,"context_line":"              (False, \"default\"), (True, \"default\"))"},{"line_number":727,"context_line":"    @ddt.unpack"}],"source_content_type":"text/x-python","patch_set":21,"id":"5f201791_35a8478a","line":724,"range":{"start_line":724,"start_character":22,"end_line":724,"end_character":27},"updated":"2017-06-26 20:30:51.000000000","message":"It would be less confusing to use None or \"\" here.","commit_id":"dfda8cd8c081443555bf8f43bd6d8d44cfbb453c"},{"author":{"_account_id":6491,"name":"xing-yang","email":"xingyang105@gmail.com","username":"xing-yang"},"change_message_id":"9d76114aea6b589704737174d7f3280b0d6dbadc","unresolved":false,"context_lines":[{"line_number":721,"context_line":"        expected \u003d {\u0027disable_replication\u0027: {}}"},{"line_number":722,"context_line":"        self.assert_called(\u0027POST\u0027, \u0027/groups/1234/action\u0027, body\u003dexpected)"},{"line_number":723,"context_line":""},{"line_number":724,"context_line":"    @ddt.data((False, False), (True, False),"},{"line_number":725,"context_line":"              (False, \"backend1\"), (True, \"backend1\"),"},{"line_number":726,"context_line":"              (False, \"default\"), (True, \"default\"))"},{"line_number":727,"context_line":"    @ddt.unpack"}],"source_content_type":"text/x-python","patch_set":21,"id":"3f1d235d_e17a03b8","line":724,"range":{"start_line":724,"start_character":22,"end_line":724,"end_character":27},"in_reply_to":"5f201791_35a8478a","updated":"2017-07-05 20:02:48.000000000","message":"Done","commit_id":"dfda8cd8c081443555bf8f43bd6d8d44cfbb453c"}],"cinderclient/v3/groups.py":[{"author":{"_account_id":23083,"name":"TommyLike","email":"tommylikehu@gmail.com","username":"TommyLike"},"change_message_id":"16b10f86b38fe481b9a93e936153754398dcb679","unresolved":false,"context_lines":[{"line_number":232,"context_line":"    def list_replication_targets(self, group):"},{"line_number":233,"context_line":"        \"\"\"List replication targets for a group."},{"line_number":234,"context_line":""},{"line_number":235,"context_line":"        :param group: the :class:`Group` to enable replication."},{"line_number":236,"context_line":"        \"\"\""},{"line_number":237,"context_line":"        body \u003d {\u0027list_replication_targets\u0027: {}}"},{"line_number":238,"context_line":"        self.run_hooks(\u0027modify_body_for_action\u0027, body, \u0027group\u0027)"}],"source_content_type":"text/x-python","patch_set":16,"id":"df140735_12eb5eb0","line":235,"range":{"start_line":235,"start_character":43,"end_line":235,"end_character":62},"updated":"2017-06-01 02:44:46.000000000","message":"to list replication targets?","commit_id":"ec4d1a39deeb0d46bd49816d711b948fa5741740"},{"author":{"_account_id":6491,"name":"xing-yang","email":"xingyang105@gmail.com","username":"xing-yang"},"change_message_id":"00a09d2d6393622ea2f888d9f18d78ad6dd996f7","unresolved":false,"context_lines":[{"line_number":232,"context_line":"    def list_replication_targets(self, group):"},{"line_number":233,"context_line":"        \"\"\"List replication targets for a group."},{"line_number":234,"context_line":""},{"line_number":235,"context_line":"        :param group: the :class:`Group` to enable replication."},{"line_number":236,"context_line":"        \"\"\""},{"line_number":237,"context_line":"        body \u003d {\u0027list_replication_targets\u0027: {}}"},{"line_number":238,"context_line":"        self.run_hooks(\u0027modify_body_for_action\u0027, body, \u0027group\u0027)"}],"source_content_type":"text/x-python","patch_set":16,"id":"df140735_9796174c","line":235,"range":{"start_line":235,"start_character":43,"end_line":235,"end_character":62},"in_reply_to":"df140735_12eb5eb0","updated":"2017-06-02 04:14:26.000000000","message":"Done","commit_id":"ec4d1a39deeb0d46bd49816d711b948fa5741740"}],"cinderclient/v3/shell.py":[{"author":{"_account_id":23083,"name":"TommyLike","email":"tommylikehu@gmail.com","username":"TommyLike"},"change_message_id":"16b10f86b38fe481b9a93e936153754398dcb679","unresolved":false,"context_lines":[{"line_number":1185,"context_line":"           metavar\u003d\u0027\u003csecondary_backend_id\u003e\u0027,"},{"line_number":1186,"context_line":"           help\u003d\u0027Secondary backend id. Default\u003dNone.\u0027)"},{"line_number":1187,"context_line":"def do_group_failover_replication(cs, args):"},{"line_number":1188,"context_line":"    \"\"\"Fails over replication for group.\"\"\""},{"line_number":1189,"context_line":""},{"line_number":1190,"context_line":"    shell_utils.find_group(cs, args.group).failover_replication("},{"line_number":1191,"context_line":"        args.allow_attached_volume,"}],"source_content_type":"text/x-python","patch_set":16,"id":"df140735_77bf2c61","line":1188,"range":{"start_line":1188,"start_character":7,"end_line":1188,"end_character":17},"updated":"2017-06-01 02:44:46.000000000","message":"I thought it was failover, which one is correct?","commit_id":"ec4d1a39deeb0d46bd49816d711b948fa5741740"},{"author":{"_account_id":6491,"name":"xing-yang","email":"xingyang105@gmail.com","username":"xing-yang"},"change_message_id":"00a09d2d6393622ea2f888d9f18d78ad6dd996f7","unresolved":false,"context_lines":[{"line_number":1185,"context_line":"           metavar\u003d\u0027\u003csecondary_backend_id\u003e\u0027,"},{"line_number":1186,"context_line":"           help\u003d\u0027Secondary backend id. Default\u003dNone.\u0027)"},{"line_number":1187,"context_line":"def do_group_failover_replication(cs, args):"},{"line_number":1188,"context_line":"    \"\"\"Fails over replication for group.\"\"\""},{"line_number":1189,"context_line":""},{"line_number":1190,"context_line":"    shell_utils.find_group(cs, args.group).failover_replication("},{"line_number":1191,"context_line":"        args.allow_attached_volume,"}],"source_content_type":"text/x-python","patch_set":16,"id":"df140735_37a74baa","line":1188,"range":{"start_line":1188,"start_character":7,"end_line":1188,"end_character":17},"in_reply_to":"df140735_77bf2c61","updated":"2017-06-02 04:14:26.000000000","message":"\u0027failover\u0027 is a noun.  I\u0027m using a verb here.  We can\u0027t say \u0027failovers\u0027 or \u0027failovered\u0027.  We can say \u0027fails over\u0027 or \u0027failed over\u0027.","commit_id":"ec4d1a39deeb0d46bd49816d711b948fa5741740"},{"author":{"_account_id":23083,"name":"TommyLike","email":"tommylikehu@gmail.com","username":"TommyLike"},"change_message_id":"16b10f86b38fe481b9a93e936153754398dcb679","unresolved":false,"context_lines":[{"line_number":1187,"context_line":"def do_group_failover_replication(cs, args):"},{"line_number":1188,"context_line":"    \"\"\"Fails over replication for group.\"\"\""},{"line_number":1189,"context_line":""},{"line_number":1190,"context_line":"    shell_utils.find_group(cs, args.group).failover_replication("},{"line_number":1191,"context_line":"        args.allow_attached_volume,"},{"line_number":1192,"context_line":"        args.secondary_backend_id)"},{"line_number":1193,"context_line":""},{"line_number":1194,"context_line":""},{"line_number":1195,"context_line":"@api_versions.wraps(\u00273.35\u0027)"}],"source_content_type":"text/x-python","patch_set":16,"id":"df140735_d29e0653","line":1192,"range":{"start_line":1190,"start_character":4,"end_line":1192,"end_character":34},"updated":"2017-06-01 02:44:46.000000000","message":"```\nshell_utils.find_group(cs, args.group).failover_replication(\n        allow_attached_volume\u003dargs.allow_attached_volume,\n        secondary_backend_id\u003dargs.secondary_backend_id)\n```","commit_id":"ec4d1a39deeb0d46bd49816d711b948fa5741740"},{"author":{"_account_id":6491,"name":"xing-yang","email":"xingyang105@gmail.com","username":"xing-yang"},"change_message_id":"00a09d2d6393622ea2f888d9f18d78ad6dd996f7","unresolved":false,"context_lines":[{"line_number":1187,"context_line":"def do_group_failover_replication(cs, args):"},{"line_number":1188,"context_line":"    \"\"\"Fails over replication for group.\"\"\""},{"line_number":1189,"context_line":""},{"line_number":1190,"context_line":"    shell_utils.find_group(cs, args.group).failover_replication("},{"line_number":1191,"context_line":"        args.allow_attached_volume,"},{"line_number":1192,"context_line":"        args.secondary_backend_id)"},{"line_number":1193,"context_line":""},{"line_number":1194,"context_line":""},{"line_number":1195,"context_line":"@api_versions.wraps(\u00273.35\u0027)"}],"source_content_type":"text/x-python","patch_set":16,"id":"df140735_17dfa70d","line":1192,"range":{"start_line":1190,"start_character":4,"end_line":1192,"end_character":34},"in_reply_to":"df140735_d29e0653","updated":"2017-06-02 04:14:26.000000000","message":"Done","commit_id":"ec4d1a39deeb0d46bd49816d711b948fa5741740"},{"author":{"_account_id":23083,"name":"TommyLike","email":"tommylikehu@gmail.com","username":"TommyLike"},"change_message_id":"16b10f86b38fe481b9a93e936153754398dcb679","unresolved":false,"context_lines":[{"line_number":1221,"context_line":"    columns \u003d []"},{"line_number":1222,"context_line":"    rep_targets \u003d replication_targets.get(\u0027replication_targets\u0027)"},{"line_number":1223,"context_line":"    if rep_targets and len(rep_targets) \u003e 0:"},{"line_number":1224,"context_line":"        keys \u003d rep_targets[0].keys()"},{"line_number":1225,"context_line":"        for key in keys:"},{"line_number":1226,"context_line":"            columns.append(key)"},{"line_number":1227,"context_line":"        utils.print_list(rep_targets, columns)"},{"line_number":1228,"context_line":""},{"line_number":1229,"context_line":"@utils.arg(\u0027--all-tenants\u0027,"},{"line_number":1230,"context_line":"           dest\u003d\u0027all_tenants\u0027,"}],"source_content_type":"text/x-python","patch_set":16,"id":"df140735_32854209","line":1227,"range":{"start_line":1224,"start_character":6,"end_line":1227,"end_character":46},"updated":"2017-06-01 02:44:46.000000000","message":"nit.\n```\nutils.print_list(rep_targets, [key for key in rep_targets[0].keys])\n```","commit_id":"ec4d1a39deeb0d46bd49816d711b948fa5741740"},{"author":{"_account_id":6491,"name":"xing-yang","email":"xingyang105@gmail.com","username":"xing-yang"},"change_message_id":"00a09d2d6393622ea2f888d9f18d78ad6dd996f7","unresolved":false,"context_lines":[{"line_number":1221,"context_line":"    columns \u003d []"},{"line_number":1222,"context_line":"    rep_targets \u003d replication_targets.get(\u0027replication_targets\u0027)"},{"line_number":1223,"context_line":"    if rep_targets and len(rep_targets) \u003e 0:"},{"line_number":1224,"context_line":"        keys \u003d rep_targets[0].keys()"},{"line_number":1225,"context_line":"        for key in keys:"},{"line_number":1226,"context_line":"            columns.append(key)"},{"line_number":1227,"context_line":"        utils.print_list(rep_targets, columns)"},{"line_number":1228,"context_line":""},{"line_number":1229,"context_line":"@utils.arg(\u0027--all-tenants\u0027,"},{"line_number":1230,"context_line":"           dest\u003d\u0027all_tenants\u0027,"}],"source_content_type":"text/x-python","patch_set":16,"id":"df140735_57463fc2","line":1227,"range":{"start_line":1224,"start_character":6,"end_line":1227,"end_character":46},"in_reply_to":"df140735_32854209","updated":"2017-06-02 04:14:26.000000000","message":"Done","commit_id":"ec4d1a39deeb0d46bd49816d711b948fa5741740"},{"author":{"_account_id":4523,"name":"Eric Harney","email":"eharney@redhat.com","username":"eharney"},"change_message_id":"60332b14890f7477d730af972360133b48dc3d79","unresolved":false,"context_lines":[{"line_number":1189,"context_line":"@utils.arg(\u0027group\u0027,"},{"line_number":1190,"context_line":"           metavar\u003d\u0027\u003cgroup\u003e\u0027,"},{"line_number":1191,"context_line":"           help\u003d\u0027Name or ID of the group.\u0027)"},{"line_number":1192,"context_line":"@utils.arg(\u0027--allow-attached-volume\u0027,"},{"line_number":1193,"context_line":"           action\u003d\u0027store_true\u0027,"},{"line_number":1194,"context_line":"           default\u003dFalse,"},{"line_number":1195,"context_line":"           help\u003d\u0027Allows or disallows group with \u0027"}],"source_content_type":"text/x-python","patch_set":21,"id":"5f201791_95715b47","line":1192,"range":{"start_line":1192,"start_character":12,"end_line":1192,"end_character":35},"updated":"2017-06-26 20:30:51.000000000","message":"Wouldn\u0027t it be more appropriate to call this \"allow-attached-volumes\" rather than singular volume?","commit_id":"dfda8cd8c081443555bf8f43bd6d8d44cfbb453c"},{"author":{"_account_id":6491,"name":"xing-yang","email":"xingyang105@gmail.com","username":"xing-yang"},"change_message_id":"9d76114aea6b589704737174d7f3280b0d6dbadc","unresolved":false,"context_lines":[{"line_number":1189,"context_line":"@utils.arg(\u0027group\u0027,"},{"line_number":1190,"context_line":"           metavar\u003d\u0027\u003cgroup\u003e\u0027,"},{"line_number":1191,"context_line":"           help\u003d\u0027Name or ID of the group.\u0027)"},{"line_number":1192,"context_line":"@utils.arg(\u0027--allow-attached-volume\u0027,"},{"line_number":1193,"context_line":"           action\u003d\u0027store_true\u0027,"},{"line_number":1194,"context_line":"           default\u003dFalse,"},{"line_number":1195,"context_line":"           help\u003d\u0027Allows or disallows group with \u0027"}],"source_content_type":"text/x-python","patch_set":21,"id":"3f1d235d_8f381bee","line":1192,"range":{"start_line":1192,"start_character":12,"end_line":1192,"end_character":35},"in_reply_to":"5f201791_95715b47","updated":"2017-07-05 20:02:48.000000000","message":"That does sound better, but \"allow-attached-volume\" also matches the param \"allow_attached_volume\" in the API which is already merged.  Maybe better to keep it consistent?","commit_id":"dfda8cd8c081443555bf8f43bd6d8d44cfbb453c"},{"author":{"_account_id":7198,"name":"Jay Bryant","email":"jungleboyj@electronicjungle.net","username":"jsbryant"},"change_message_id":"fbb6094da07ab07b4b1eed0626a3b0dc152118b9","unresolved":false,"context_lines":[{"line_number":1218,"context_line":"           help\u003d\u0027Name or ID of the group.\u0027)"},{"line_number":1219,"context_line":"def do_group_list_replication_targets(cs, args):"},{"line_number":1220,"context_line":"    \"\"\"Lists replication targets for group."},{"line_number":1221,"context_line":""},{"line_number":1222,"context_line":"    Example value for replication_targets:"},{"line_number":1223,"context_line":"        {"},{"line_number":1224,"context_line":"            \u0027replication_targets\u0027: ["}],"source_content_type":"text/x-python","patch_set":22,"id":"3f1d235d_ef0ac8cf","line":1221,"updated":"2017-07-06 15:53:41.000000000","message":"This will fail the docs build.  Need:\n.. code-block: json\nthen indent everything under that please.","commit_id":"78a1a3bf8bdeb27f3cd13b0a5e18705782e32353"},{"author":{"_account_id":6491,"name":"xing-yang","email":"xingyang105@gmail.com","username":"xing-yang"},"change_message_id":"17b759b2ee08330886c6860b18a6b250907967ef","unresolved":false,"context_lines":[{"line_number":1218,"context_line":"           help\u003d\u0027Name or ID of the group.\u0027)"},{"line_number":1219,"context_line":"def do_group_list_replication_targets(cs, args):"},{"line_number":1220,"context_line":"    \"\"\"Lists replication targets for group."},{"line_number":1221,"context_line":""},{"line_number":1222,"context_line":"    Example value for replication_targets:"},{"line_number":1223,"context_line":"        {"},{"line_number":1224,"context_line":"            \u0027replication_targets\u0027: ["}],"source_content_type":"text/x-python","patch_set":22,"id":"3f1d235d_f3e98ee2","line":1221,"in_reply_to":"3f1d235d_ef0ac8cf","updated":"2017-07-06 17:17:54.000000000","message":"Will fix it.  Thanks!","commit_id":"78a1a3bf8bdeb27f3cd13b0a5e18705782e32353"}],"releasenotes/notes/replication-group-v3-api-022900ce6bf8feba.yaml":[{"author":{"_account_id":23083,"name":"TommyLike","email":"tommylikehu@gmail.com","username":"TommyLike"},"change_message_id":"16b10f86b38fe481b9a93e936153754398dcb679","unresolved":false,"context_lines":[{"line_number":1,"context_line":"---"},{"line_number":2,"context_line":"features:"},{"line_number":3,"context_line":"  - |"},{"line_number":4,"context_line":"    Add support for replication group APIs enable_replication,"},{"line_number":5,"context_line":"    disable_replication, failover_replication and"},{"line_number":6,"context_line":"    list_replication_targets."}],"source_content_type":"text/x-yaml","patch_set":16,"id":"df140735_d7c71845","line":4,"range":{"start_line":4,"start_character":4,"end_line":4,"end_character":8},"updated":"2017-06-01 02:44:46.000000000","message":"nit: Added","commit_id":"ec4d1a39deeb0d46bd49816d711b948fa5741740"},{"author":{"_account_id":23083,"name":"TommyLike","email":"tommylikehu@gmail.com","username":"TommyLike"},"change_message_id":"16b10f86b38fe481b9a93e936153754398dcb679","unresolved":false,"context_lines":[{"line_number":1,"context_line":"---"},{"line_number":2,"context_line":"features:"},{"line_number":3,"context_line":"  - |"},{"line_number":4,"context_line":"    Add support for replication group APIs enable_replication,"},{"line_number":5,"context_line":"    disable_replication, failover_replication and"},{"line_number":6,"context_line":"    list_replication_targets."}],"source_content_type":"text/x-yaml","patch_set":16,"id":"df140735_77de0c12","line":4,"range":{"start_line":4,"start_character":42,"end_line":4,"end_character":61},"updated":"2017-06-01 02:44:46.000000000","message":"nit: ``enable_replication`` and the rests","commit_id":"ec4d1a39deeb0d46bd49816d711b948fa5741740"},{"author":{"_account_id":6491,"name":"xing-yang","email":"xingyang105@gmail.com","username":"xing-yang"},"change_message_id":"00a09d2d6393622ea2f888d9f18d78ad6dd996f7","unresolved":false,"context_lines":[{"line_number":1,"context_line":"---"},{"line_number":2,"context_line":"features:"},{"line_number":3,"context_line":"  - |"},{"line_number":4,"context_line":"    Add support for replication group APIs enable_replication,"},{"line_number":5,"context_line":"    disable_replication, failover_replication and"},{"line_number":6,"context_line":"    list_replication_targets."}],"source_content_type":"text/x-yaml","patch_set":16,"id":"df140735_b7859bd1","line":4,"range":{"start_line":4,"start_character":4,"end_line":4,"end_character":8},"in_reply_to":"df140735_d7c71845","updated":"2017-06-02 04:14:26.000000000","message":"Done","commit_id":"ec4d1a39deeb0d46bd49816d711b948fa5741740"}]}