)]}'
{"openstackclient/compute/v2/server_group.py":[{"author":{"_account_id":21514,"name":"Huanxuan Ao","email":"aohuanxuan@163.com","username":"aohuanxuan"},"change_message_id":"b7f9b1306de82087c9567a1c8d99fc1e51be4bb2","unresolved":false,"context_lines":[{"line_number":55,"context_line":"        parser.add_argument("},{"line_number":56,"context_line":"            \u0027--policy\u0027,"},{"line_number":57,"context_line":"            metavar\u003d\u0027\u003cpolicy\u003e\u0027,"},{"line_number":58,"context_line":"            action\u003d\u0027append\u0027,"},{"line_number":59,"context_line":"            choices\u003d[\u0027affinity\u0027, \u0027anti-affinity\u0027],"},{"line_number":60,"context_line":"            required\u003dTrue,"},{"line_number":61,"context_line":"            help\u003d_(\"Add a policy to \u003cname\u003e \""}],"source_content_type":"text/x-python","patch_set":2,"id":"1a430d35_f48ae519","line":58,"updated":"2017-02-09 05:11:34.000000000","message":"Why not remove this if multi values is not alowed?","commit_id":"0144a77c6576a304ec20b1b877d4ae560a442ec2"},{"author":{"_account_id":6482,"name":"Steve Martinelli","email":"s.martinelli@gmail.com","username":"stevemar"},"change_message_id":"3801d427e4db7dd830985715ca57d6efd52dff5e","unresolved":false,"context_lines":[{"line_number":57,"context_line":"            metavar\u003d\u0027\u003cpolicy\u003e\u0027,"},{"line_number":58,"context_line":"            action\u003d\u0027append\u0027,"},{"line_number":59,"context_line":"            choices\u003d[\u0027affinity\u0027, \u0027anti-affinity\u0027],"},{"line_number":60,"context_line":"            required\u003dTrue,"},{"line_number":61,"context_line":"            help\u003d_(\"Add a policy to \u003cname\u003e \""},{"line_number":62,"context_line":"                   \"(\u0027affinity\u0027 or \u0027anti-affinity\u0027) (required)\")"},{"line_number":63,"context_line":"        )"}],"source_content_type":"text/x-python","patch_set":2,"id":"1a430d35_fc111553","line":60,"range":{"start_line":60,"start_character":12,"end_line":60,"end_character":26},"updated":"2017-02-08 11:57:41.000000000","message":"now that we\u0027re using \"choices\", do we want to give the option a default value and remove the \u0027required\u003dTrue\u0027 portion?","commit_id":"0144a77c6576a304ec20b1b877d4ae560a442ec2"},{"author":{"_account_id":21514,"name":"Huanxuan Ao","email":"aohuanxuan@163.com","username":"aohuanxuan"},"change_message_id":"b7f9b1306de82087c9567a1c8d99fc1e51be4bb2","unresolved":false,"context_lines":[{"line_number":57,"context_line":"            metavar\u003d\u0027\u003cpolicy\u003e\u0027,"},{"line_number":58,"context_line":"            action\u003d\u0027append\u0027,"},{"line_number":59,"context_line":"            choices\u003d[\u0027affinity\u0027, \u0027anti-affinity\u0027],"},{"line_number":60,"context_line":"            required\u003dTrue,"},{"line_number":61,"context_line":"            help\u003d_(\"Add a policy to \u003cname\u003e \""},{"line_number":62,"context_line":"                   \"(\u0027affinity\u0027 or \u0027anti-affinity\u0027) (required)\")"},{"line_number":63,"context_line":"        )"}],"source_content_type":"text/x-python","patch_set":2,"id":"1a430d35_7498b578","line":60,"range":{"start_line":60,"start_character":12,"end_line":60,"end_character":26},"in_reply_to":"1a430d35_fc111553","updated":"2017-02-09 05:11:34.000000000","message":"++ I think this is a good idea if this is required and have choices","commit_id":"0144a77c6576a304ec20b1b877d4ae560a442ec2"},{"author":{"_account_id":6482,"name":"Steve Martinelli","email":"s.martinelli@gmail.com","username":"stevemar"},"change_message_id":"75bbdbf366c86a145cd1ea63e0ce8ff972b18a2e","unresolved":false,"context_lines":[{"line_number":70,"context_line":"        policy_list.append(parsed_args.policy)"},{"line_number":71,"context_line":"        server_group \u003d compute_client.server_groups.create("},{"line_number":72,"context_line":"            name\u003dparsed_args.name,"},{"line_number":73,"context_line":"            policies\u003dpolicy_list)"},{"line_number":74,"context_line":"        info.update(server_group._info)"},{"line_number":75,"context_line":""},{"line_number":76,"context_line":"        columns \u003d _get_columns(info)"}],"source_content_type":"text/x-python","patch_set":3,"id":"1a430d35_85e1569d","line":73,"range":{"start_line":73,"start_character":12,"end_line":73,"end_character":32},"updated":"2017-02-09 16:25:55.000000000","message":"you can just pass in:\n\n  policies\u003d[parsed_args.policy]\n\nhere","commit_id":"f9e5942a2d46d57aaf497cc942cc175f65411521"},{"author":{"_account_id":24081,"name":"zhiyong.dai","email":"zhiyong.dai@easystack.cn","username":"zhiyong.dai"},"change_message_id":"98a9ca2d23743b30c1423f80f8ebe934832f1c75","unresolved":false,"context_lines":[{"line_number":70,"context_line":"        policy_list.append(parsed_args.policy)"},{"line_number":71,"context_line":"        server_group \u003d compute_client.server_groups.create("},{"line_number":72,"context_line":"            name\u003dparsed_args.name,"},{"line_number":73,"context_line":"            policies\u003dpolicy_list)"},{"line_number":74,"context_line":"        info.update(server_group._info)"},{"line_number":75,"context_line":""},{"line_number":76,"context_line":"        columns \u003d _get_columns(info)"}],"source_content_type":"text/x-python","patch_set":3,"id":"fa31d9ce_d4d78121","line":73,"range":{"start_line":73,"start_character":12,"end_line":73,"end_character":32},"in_reply_to":"1a430d35_85e1569d","updated":"2017-02-10 01:55:01.000000000","message":"Done, thanks","commit_id":"f9e5942a2d46d57aaf497cc942cc175f65411521"}]}
