)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":2874,"name":"yong sheng gong","email":"gong.yongsheng@99cloud.net","username":"gongysh"},"change_message_id":"bc1d50225fccf36e145ac16bc92881629858e99c","unresolved":false,"context_lines":[{"line_number":4,"context_line":"Commit:     Akihiro MOTOKI \u003cmotoki@da.jp.nec.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2012-12-03 20:41:55 +0900"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Add --dns, --host-route, --enable-dhcp to subnet-create"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Fixes bug 1042982"},{"line_number":10,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"AAAAMX%2F%2F%2BRc%3D","line":7,"updated":"2012-12-04 12:42:38.000000000","message":"we are using --disable-dhcp, right?","commit_id":"f0d5cc86182c794401f1d6b8b7277e83c14b7d56"},{"author":{"_account_id":841,"name":"Akihiro Motoki","email":"amotoki@gmail.com","username":"amotoki"},"unresolved":false,"context_lines":[{"line_number":4,"context_line":"Commit:     Akihiro MOTOKI \u003cmotoki@da.jp.nec.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2012-12-03 20:41:55 +0900"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Add --dns, --host-route, --enable-dhcp to subnet-create"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Fixes bug 1042982"},{"line_number":10,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"AAAAMX%2F%2F%2BP4%3D","line":7,"in_reply_to":"AAAAMX%2F%2F%2BRc%3D","updated":"2012-12-05 07:19:16.000000000","message":"Will fix.","commit_id":"f0d5cc86182c794401f1d6b8b7277e83c14b7d56"}],"quantumclient/quantum/v2_0/subnet.py":[{"author":{"_account_id":261,"name":"Salvatore Orlando","email":"salv.orlando@gmail.com","username":"salvatore-orlando"},"unresolved":false,"context_lines":[{"line_number":108,"context_line":"            action\u003d\u0027append\u0027,"},{"line_number":109,"context_line":"            help\u003dargparse.SUPPRESS)"},{"line_number":110,"context_line":"        parser.add_argument("},{"line_number":111,"context_line":"            \u0027--host-route\u0027,"},{"line_number":112,"context_line":"            action\u003d\u0027append\u0027,"},{"line_number":113,"context_line":"            help\u003d\u0027Additional route: \u0027"},{"line_number":114,"context_line":"            \u0027destination\u003d\u003ccidr\u003e,nexthop\u003d\u003cip_address\u003e \u0027"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAAMX%2F%2F%2BUg%3D","line":111,"updated":"2012-12-04 12:15:35.000000000","message":"At the moment we are still support commands both with dashes and underscores as separators (see allocation-pool for an example).\n\nWe should do the same for host-route and disable-dhcp until the decision on removing the underscores is finalized","commit_id":"f0d5cc86182c794401f1d6b8b7277e83c14b7d56"},{"author":{"_account_id":841,"name":"Akihiro Motoki","email":"amotoki@gmail.com","username":"amotoki"},"unresolved":false,"context_lines":[{"line_number":108,"context_line":"            action\u003d\u0027append\u0027,"},{"line_number":109,"context_line":"            help\u003dargparse.SUPPRESS)"},{"line_number":110,"context_line":"        parser.add_argument("},{"line_number":111,"context_line":"            \u0027--host-route\u0027,"},{"line_number":112,"context_line":"            action\u003d\u0027append\u0027,"},{"line_number":113,"context_line":"            help\u003d\u0027Additional route: \u0027"},{"line_number":114,"context_line":"            \u0027destination\u003d\u003ccidr\u003e,nexthop\u003d\u003cip_address\u003e \u0027"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAAMX%2F%2F%2BPw%3D","line":111,"in_reply_to":"AAAAMX%2F%2F%2BSU%3D","updated":"2012-12-05 07:19:16.000000000","message":"My intention is what yong said. \u0027-\u0027 is introduced to keep consistency with other clients and \u0027_\u0027 in option is deprecated.","commit_id":"f0d5cc86182c794401f1d6b8b7277e83c14b7d56"},{"author":{"_account_id":2874,"name":"yong sheng gong","email":"gong.yongsheng@99cloud.net","username":"gongysh"},"change_message_id":"bc1d50225fccf36e145ac16bc92881629858e99c","unresolved":false,"context_lines":[{"line_number":108,"context_line":"            action\u003d\u0027append\u0027,"},{"line_number":109,"context_line":"            help\u003dargparse.SUPPRESS)"},{"line_number":110,"context_line":"        parser.add_argument("},{"line_number":111,"context_line":"            \u0027--host-route\u0027,"},{"line_number":112,"context_line":"            action\u003d\u0027append\u0027,"},{"line_number":113,"context_line":"            help\u003d\u0027Additional route: \u0027"},{"line_number":114,"context_line":"            \u0027destination\u003d\u003ccidr\u003e,nexthop\u003d\u003cip_address\u003e \u0027"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAAMX%2F%2F%2BSU%3D","line":111,"in_reply_to":"AAAAMX%2F%2F%2BUg%3D","updated":"2012-12-04 12:42:38.000000000","message":"Hi Dear Salvatore,\nI think we keep both because we have _ in options before we change _ into - in options.\nWe can use only - version for new options. No need to have _ in new options.","commit_id":"f0d5cc86182c794401f1d6b8b7277e83c14b7d56"},{"author":{"_account_id":2874,"name":"yong sheng gong","email":"gong.yongsheng@99cloud.net","username":"gongysh"},"change_message_id":"bc1d50225fccf36e145ac16bc92881629858e99c","unresolved":false,"context_lines":[{"line_number":120,"context_line":"            \u0027(This option can be repeated)\u0027)"},{"line_number":121,"context_line":"        parser.add_argument("},{"line_number":122,"context_line":"            \u0027--disable-dhcp\u0027,"},{"line_number":123,"context_line":"            action\u003d\u0027store_true\u0027,"},{"line_number":124,"context_line":"            help\u003d\u0027Disable DHCP for this subnet\u0027)"},{"line_number":125,"context_line":"        parser.add_argument("},{"line_number":126,"context_line":"            \u0027network_id\u0027, metavar\u003d\u0027network\u0027,"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAAMX%2F%2F%2BlA%3D","line":123,"updated":"2012-12-04 04:28:34.000000000","message":"I think we use use --enable-dhcp so that we will enable dhcp by default.","commit_id":"f0d5cc86182c794401f1d6b8b7277e83c14b7d56"},{"author":{"_account_id":841,"name":"Akihiro Motoki","email":"amotoki@gmail.com","username":"amotoki"},"unresolved":false,"context_lines":[{"line_number":120,"context_line":"            \u0027(This option can be repeated)\u0027)"},{"line_number":121,"context_line":"        parser.add_argument("},{"line_number":122,"context_line":"            \u0027--disable-dhcp\u0027,"},{"line_number":123,"context_line":"            action\u003d\u0027store_true\u0027,"},{"line_number":124,"context_line":"            help\u003d\u0027Disable DHCP for this subnet\u0027)"},{"line_number":125,"context_line":"        parser.add_argument("},{"line_number":126,"context_line":"            \u0027network_id\u0027, metavar\u003d\u0027network\u0027,"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAAMX%2F%2F%2BOc%3D","line":123,"in_reply_to":"AAAAMX%2F%2F%2BSA%3D","updated":"2012-12-05 07:19:16.000000000","message":"I agree keeping consistency with admin-state-down.\n\nBefore changing the patch, let me ask a question.\n\ntrue/false mapping for admin-state-down is a little odd to me.\n\nIn my understanding, parsed_args.xxxxx (for true/false) is usually interpreted as a flag, so it is reasonable parsed_args.xxxxx \u003d\u003d True means \u0027xxxxx\u0027 to me (i.e., parsed_args.disable_dhcp \u003d\u003d True means \u0027a user want to disable dhcp\u0027. It is similar to --no-gateway above.\n\nOn the other hand, parsed_args.admin_state_down is used as a symbol, and parsed_args.admin_state_down \u003d\u003d False means \u0027a user want to set admin_state_up to False\u0027.","commit_id":"f0d5cc86182c794401f1d6b8b7277e83c14b7d56"},{"author":{"_account_id":2874,"name":"yong sheng gong","email":"gong.yongsheng@99cloud.net","username":"gongysh"},"change_message_id":"bc1d50225fccf36e145ac16bc92881629858e99c","unresolved":false,"context_lines":[{"line_number":120,"context_line":"            \u0027(This option can be repeated)\u0027)"},{"line_number":121,"context_line":"        parser.add_argument("},{"line_number":122,"context_line":"            \u0027--disable-dhcp\u0027,"},{"line_number":123,"context_line":"            action\u003d\u0027store_true\u0027,"},{"line_number":124,"context_line":"            help\u003d\u0027Disable DHCP for this subnet\u0027)"},{"line_number":125,"context_line":"        parser.add_argument("},{"line_number":126,"context_line":"            \u0027network_id\u0027, metavar\u003d\u0027network\u0027,"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAAMX%2F%2F%2BRo%3D","line":123,"in_reply_to":"AAAAMX%2F%2F%2BUo%3D","updated":"2012-12-04 12:42:38.000000000","message":"One thing said in different side.","commit_id":"f0d5cc86182c794401f1d6b8b7277e83c14b7d56"},{"author":{"_account_id":261,"name":"Salvatore Orlando","email":"salv.orlando@gmail.com","username":"salvatore-orlando"},"unresolved":false,"context_lines":[{"line_number":120,"context_line":"            \u0027(This option can be repeated)\u0027)"},{"line_number":121,"context_line":"        parser.add_argument("},{"line_number":122,"context_line":"            \u0027--disable-dhcp\u0027,"},{"line_number":123,"context_line":"            action\u003d\u0027store_true\u0027,"},{"line_number":124,"context_line":"            help\u003d\u0027Disable DHCP for this subnet\u0027)"},{"line_number":125,"context_line":"        parser.add_argument("},{"line_number":126,"context_line":"            \u0027network_id\u0027, metavar\u003d\u0027network\u0027,"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAAMX%2F%2F%2BUo%3D","line":123,"in_reply_to":"AAAAMX%2F%2F%2Bio%3D","updated":"2012-12-04 12:15:35.000000000","message":"I vote for consistency with admin-state-down\nI still find very confusing that I can do both:\n--admin-state-down\nand\n--admin_state_up\u003dFalse\n\nand similarly here:\n--disable-dhcp\nand\n--enable_dhcp\u003dFalse\n\nBut this is another story","commit_id":"f0d5cc86182c794401f1d6b8b7277e83c14b7d56"},{"author":{"_account_id":2874,"name":"yong sheng gong","email":"gong.yongsheng@99cloud.net","username":"gongysh"},"change_message_id":"bc1d50225fccf36e145ac16bc92881629858e99c","unresolved":false,"context_lines":[{"line_number":120,"context_line":"            \u0027(This option can be repeated)\u0027)"},{"line_number":121,"context_line":"        parser.add_argument("},{"line_number":122,"context_line":"            \u0027--disable-dhcp\u0027,"},{"line_number":123,"context_line":"            action\u003d\u0027store_true\u0027,"},{"line_number":124,"context_line":"            help\u003d\u0027Disable DHCP for this subnet\u0027)"},{"line_number":125,"context_line":"        parser.add_argument("},{"line_number":126,"context_line":"            \u0027network_id\u0027, metavar\u003d\u0027network\u0027,"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAAMX%2F%2F%2BSA%3D","line":123,"in_reply_to":"AAAAMX%2F%2F%2Bio%3D","updated":"2012-12-04 12:42:38.000000000","message":"how about:\ndefault\u003dFalse, action\u003d\u0027store_true\u0027.\n\nIt is easy to understand. It means if we have no --disable-dhcp given, the parsed_args.disable_dhcp will be false. Otherwise it will be true.","commit_id":"f0d5cc86182c794401f1d6b8b7277e83c14b7d56"},{"author":{"_account_id":841,"name":"Akihiro Motoki","email":"amotoki@gmail.com","username":"amotoki"},"unresolved":false,"context_lines":[{"line_number":120,"context_line":"            \u0027(This option can be repeated)\u0027)"},{"line_number":121,"context_line":"        parser.add_argument("},{"line_number":122,"context_line":"            \u0027--disable-dhcp\u0027,"},{"line_number":123,"context_line":"            action\u003d\u0027store_true\u0027,"},{"line_number":124,"context_line":"            help\u003d\u0027Disable DHCP for this subnet\u0027)"},{"line_number":125,"context_line":"        parser.add_argument("},{"line_number":126,"context_line":"            \u0027network_id\u0027, metavar\u003d\u0027network\u0027,"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAAMX%2F%2F%2Bio%3D","line":123,"in_reply_to":"AAAAMX%2F%2F%2BlA%3D","updated":"2012-12-04 06:19:42.000000000","message":"Are you saying \n\nBy default \u0027enable_dhcp\u0027 is enabled in subnet-create. I prepared this option to set enable_dhcp to False.\n\nThere are similar cases In other create commands (net-create and port-create). They have boolean command line options to specify non-default value. For example, --admin_state_down option requests admin_state\u003dFalse. This is the background of --disable-dhcp.","commit_id":"f0d5cc86182c794401f1d6b8b7277e83c14b7d56"},{"author":{"_account_id":2874,"name":"yong sheng gong","email":"gong.yongsheng@99cloud.net","username":"gongysh"},"change_message_id":"bc1d50225fccf36e145ac16bc92881629858e99c","unresolved":false,"context_lines":[{"line_number":114,"context_line":"            \u0027destination\u003d\u003ccidr\u003e,nexthop\u003d\u003cip_address\u003e \u0027"},{"line_number":115,"context_line":"            \u0027(This option can be repeated)\u0027)"},{"line_number":116,"context_line":"        parser.add_argument("},{"line_number":117,"context_line":"            \u0027--dns\u0027,"},{"line_number":118,"context_line":"            action\u003d\u0027append\u0027,"},{"line_number":119,"context_line":"            help\u003d\u0027DNS name server for this subnet \u0027"},{"line_number":120,"context_line":"            \u0027(This option can be repeated)\u0027)"}],"source_content_type":"text/x-python","patch_set":2,"id":"AAAAMX%2F%2F4cQ%3D","line":117,"updated":"2012-12-11 00:28:52.000000000","message":"Can we use dns-nameservers as option name,\nand use nargs\u003d\u0027+\u0027 and remove action \u003d \u0027append\u0027?\nThis is consistent with attribute name and it is like what we are done:\nquantum subnet-create net1 10.0.1.0/24 --dns_nameservers name1 name2\n\nbut with - in known parts of the command line:\nquantum subnet-create --dns_nameservers name1 name2  net1  10.0.1.0/24\n\nsame as host routes.","commit_id":"e5252e01b86dbd15ac039ab855f060fe42e4a763"},{"author":{"_account_id":841,"name":"Akihiro Motoki","email":"amotoki@gmail.com","username":"amotoki"},"unresolved":false,"context_lines":[{"line_number":114,"context_line":"            \u0027destination\u003d\u003ccidr\u003e,nexthop\u003d\u003cip_address\u003e \u0027"},{"line_number":115,"context_line":"            \u0027(This option can be repeated)\u0027)"},{"line_number":116,"context_line":"        parser.add_argument("},{"line_number":117,"context_line":"            \u0027--dns\u0027,"},{"line_number":118,"context_line":"            action\u003d\u0027append\u0027,"},{"line_number":119,"context_line":"            help\u003d\u0027DNS name server for this subnet \u0027"},{"line_number":120,"context_line":"            \u0027(This option can be repeated)\u0027)"}],"source_content_type":"text/x-python","patch_set":2,"id":"AAAAMX%2F%2F4K8%3D","line":117,"in_reply_to":"AAAAMX%2F%2F4Mc%3D","updated":"2012-12-11 09:01:46.000000000","message":"Thanks for checking. I will use single form and rename the option name according to your suggestion.\n\nI just encountered the same thing. In our CLI known options must be placed before positional parameters and argparse cannot handle a concatenation of nargs\u003d\u0027*\u0027 and positional parameters. (argparse  nargs\u003d\u0027*\u0027 seems to do greedy match.)\n\n\"quantum subnet-create --dns-nameservers 10.56.133.1 net2 172.16.1.0/24\" does not work and\n\"quantum subnet-create --dns-nameservers 10.56.133.1 --name aaa net2 172.16.1.0/24\" works....","commit_id":"e5252e01b86dbd15ac039ab855f060fe42e4a763"},{"author":{"_account_id":2874,"name":"yong sheng gong","email":"gong.yongsheng@99cloud.net","username":"gongysh"},"change_message_id":"bc1d50225fccf36e145ac16bc92881629858e99c","unresolved":false,"context_lines":[{"line_number":114,"context_line":"            \u0027destination\u003d\u003ccidr\u003e,nexthop\u003d\u003cip_address\u003e \u0027"},{"line_number":115,"context_line":"            \u0027(This option can be repeated)\u0027)"},{"line_number":116,"context_line":"        parser.add_argument("},{"line_number":117,"context_line":"            \u0027--dns\u0027,"},{"line_number":118,"context_line":"            action\u003d\u0027append\u0027,"},{"line_number":119,"context_line":"            help\u003d\u0027DNS name server for this subnet \u0027"},{"line_number":120,"context_line":"            \u0027(This option can be repeated)\u0027)"}],"source_content_type":"text/x-python","patch_set":2,"id":"AAAAMX%2F%2F4Mc%3D","line":117,"in_reply_to":"AAAAMX%2F%2F4Og%3D","updated":"2012-12-11 08:22:38.000000000","message":"It seems we cannot use nargs\u003d\"+\", So I think we can use only single form.","commit_id":"e5252e01b86dbd15ac039ab855f060fe42e4a763"},{"author":{"_account_id":841,"name":"Akihiro Motoki","email":"amotoki@gmail.com","username":"amotoki"},"unresolved":false,"context_lines":[{"line_number":114,"context_line":"            \u0027destination\u003d\u003ccidr\u003e,nexthop\u003d\u003cip_address\u003e \u0027"},{"line_number":115,"context_line":"            \u0027(This option can be repeated)\u0027)"},{"line_number":116,"context_line":"        parser.add_argument("},{"line_number":117,"context_line":"            \u0027--dns\u0027,"},{"line_number":118,"context_line":"            action\u003d\u0027append\u0027,"},{"line_number":119,"context_line":"            help\u003d\u0027DNS name server for this subnet \u0027"},{"line_number":120,"context_line":"            \u0027(This option can be repeated)\u0027)"}],"source_content_type":"text/x-python","patch_set":2,"id":"AAAAMX%2F%2F4Og%3D","line":117,"in_reply_to":"AAAAMX%2F%2F4bE%3D","updated":"2012-12-11 08:17:20.000000000","message":"Does the following fit your plan?\n- This patch introduces plural form options (--dns-nameservers and --host-routes) (At this timing some options are plural form and others are single form)\n- Changing to plural form options is done in another patch.","commit_id":"e5252e01b86dbd15ac039ab855f060fe42e4a763"},{"author":{"_account_id":2874,"name":"yong sheng gong","email":"gong.yongsheng@99cloud.net","username":"gongysh"},"change_message_id":"bc1d50225fccf36e145ac16bc92881629858e99c","unresolved":false,"context_lines":[{"line_number":114,"context_line":"            \u0027destination\u003d\u003ccidr\u003e,nexthop\u003d\u003cip_address\u003e \u0027"},{"line_number":115,"context_line":"            \u0027(This option can be repeated)\u0027)"},{"line_number":116,"context_line":"        parser.add_argument("},{"line_number":117,"context_line":"            \u0027--dns\u0027,"},{"line_number":118,"context_line":"            action\u003d\u0027append\u0027,"},{"line_number":119,"context_line":"            help\u003d\u0027DNS name server for this subnet \u0027"},{"line_number":120,"context_line":"            \u0027(This option can be repeated)\u0027)"}],"source_content_type":"text/x-python","patch_set":2,"id":"AAAAMX%2F%2F4bE%3D","line":117,"in_reply_to":"AAAAMX%2F%2F4bM%3D","updated":"2012-12-11 00:59:42.000000000","message":"Yes, but I am worrying about the back compatible. I think we can deprecate the previous ones and add new ones.\n\nWe can do it in another patch.","commit_id":"e5252e01b86dbd15ac039ab855f060fe42e4a763"},{"author":{"_account_id":841,"name":"Akihiro Motoki","email":"amotoki@gmail.com","username":"amotoki"},"unresolved":false,"context_lines":[{"line_number":114,"context_line":"            \u0027destination\u003d\u003ccidr\u003e,nexthop\u003d\u003cip_address\u003e \u0027"},{"line_number":115,"context_line":"            \u0027(This option can be repeated)\u0027)"},{"line_number":116,"context_line":"        parser.add_argument("},{"line_number":117,"context_line":"            \u0027--dns\u0027,"},{"line_number":118,"context_line":"            action\u003d\u0027append\u0027,"},{"line_number":119,"context_line":"            help\u003d\u0027DNS name server for this subnet \u0027"},{"line_number":120,"context_line":"            \u0027(This option can be repeated)\u0027)"}],"source_content_type":"text/x-python","patch_set":2,"id":"AAAAMX%2F%2F4bM%3D","line":117,"in_reply_to":"AAAAMX%2F%2F4cQ%3D","updated":"2012-12-11 00:55:46.000000000","message":"Changing to \u0027--dns-nameservers\u0027 and \u0027--host-routes\u0027 are a good direction and easier to understand.\n\nOn the other hand, --allocation-pool for subnet-create and --fixed-ip for port-create are also can be specified multiple times. What do you think about the consistency with them? IMO, if we adopt --dns-nameservers, --allocation-pools also should be supported. It will avoid some confusion.  (nova boot --nic option is similar).","commit_id":"e5252e01b86dbd15ac039ab855f060fe42e4a763"},{"author":{"_account_id":841,"name":"Akihiro Motoki","email":"amotoki@gmail.com","username":"amotoki"},"unresolved":false,"context_lines":[{"line_number":102,"context_line":"            action\u003d\u0027append\u0027,"},{"line_number":103,"context_line":"            help\u003d\u0027Allocation pool IP addresses for this subnet: \u0027"},{"line_number":104,"context_line":"            \u0027start\u003d\u003cip_address\u003e,end\u003d\u003cip_address\u003e \u0027"},{"line_number":105,"context_line":"            \u0027(This option can be repeated)\u0027)"},{"line_number":106,"context_line":"        parser.add_argument("},{"line_number":107,"context_line":"            \u0027--allocation_pool\u0027,"},{"line_number":108,"context_line":"            action\u003d\u0027append\u0027,"}],"source_content_type":"text/x-python","patch_set":3,"id":"AAAAMX%2F%2F1hc%3D","line":105,"updated":"2012-12-13 10:25:15.000000000","message":"I applied the same change as --host-route to --allocation-pool to keep consistency in the code.","commit_id":"313b119577a07e5f6ac0a4dbc5f4c4edde14ccb1"},{"author":{"_account_id":2874,"name":"yong sheng gong","email":"gong.yongsheng@99cloud.net","username":"gongysh"},"change_message_id":"bc1d50225fccf36e145ac16bc92881629858e99c","unresolved":false,"context_lines":[{"line_number":109,"context_line":"            help\u003dargparse.SUPPRESS)"},{"line_number":110,"context_line":"        parser.add_argument("},{"line_number":111,"context_line":"            \u0027--host-route\u0027,"},{"line_number":112,"context_line":"            action\u003d\u0027append\u0027,"},{"line_number":113,"context_line":"            help\u003d\u0027Additional route: \u0027"},{"line_number":114,"context_line":"            \u0027destination\u003d\u003ccidr\u003e,nexthop\u003d\u003cip_address\u003e \u0027"},{"line_number":115,"context_line":"            \u0027(This option can be repeated)\u0027)"}],"source_content_type":"text/x-python","patch_set":3,"id":"AAAAMX%2F%2F4GE%3D","line":112,"updated":"2012-12-11 12:03:31.000000000","message":"add:\ndest\u003d\u0027host_routes\u0027\nmetavar\u003d\u0027HOST_ROUTE\u0027\ntype\u003dutils.str2dict","commit_id":"313b119577a07e5f6ac0a4dbc5f4c4edde14ccb1"},{"author":{"_account_id":841,"name":"Akihiro Motoki","email":"amotoki@gmail.com","username":"amotoki"},"unresolved":false,"context_lines":[{"line_number":109,"context_line":"            help\u003dargparse.SUPPRESS)"},{"line_number":110,"context_line":"        parser.add_argument("},{"line_number":111,"context_line":"            \u0027--host-route\u0027,"},{"line_number":112,"context_line":"            action\u003d\u0027append\u0027,"},{"line_number":113,"context_line":"            help\u003d\u0027Additional route: \u0027"},{"line_number":114,"context_line":"            \u0027destination\u003d\u003ccidr\u003e,nexthop\u003d\u003cip_address\u003e \u0027"},{"line_number":115,"context_line":"            \u0027(This option can be repeated)\u0027)"}],"source_content_type":"text/x-python","patch_set":3,"id":"AAAAMX%2F%2F1hk%3D","line":112,"in_reply_to":"AAAAMX%2F%2F4GE%3D","updated":"2012-12-13 10:25:15.000000000","message":"Done. I use metavar\u003d\u0027destination\u003dCIDR,nexthop\u003dIP_ADDR\u0027 to simplify the help message.","commit_id":"313b119577a07e5f6ac0a4dbc5f4c4edde14ccb1"},{"author":{"_account_id":2874,"name":"yong sheng gong","email":"gong.yongsheng@99cloud.net","username":"gongysh"},"change_message_id":"bc1d50225fccf36e145ac16bc92881629858e99c","unresolved":false,"context_lines":[{"line_number":115,"context_line":"            \u0027(This option can be repeated)\u0027)"},{"line_number":116,"context_line":"        parser.add_argument("},{"line_number":117,"context_line":"            \u0027--dns-nameserver\u0027,"},{"line_number":118,"context_line":"            action\u003d\u0027append\u0027,"},{"line_number":119,"context_line":"            help\u003d\u0027DNS name server for this subnet \u0027"},{"line_number":120,"context_line":"            \u0027(This option can be repeated)\u0027)"},{"line_number":121,"context_line":"        parser.add_argument("}],"source_content_type":"text/x-python","patch_set":3,"id":"AAAAMX%2F%2F4F0%3D","line":118,"updated":"2012-12-11 12:03:31.000000000","message":"add: dest\u003d\u0027dns_nameservers\u0027 and metavar\u003d\u0027DNS_NAMESERVER\u0027","commit_id":"313b119577a07e5f6ac0a4dbc5f4c4edde14ccb1"},{"author":{"_account_id":841,"name":"Akihiro Motoki","email":"amotoki@gmail.com","username":"amotoki"},"unresolved":false,"context_lines":[{"line_number":115,"context_line":"            \u0027(This option can be repeated)\u0027)"},{"line_number":116,"context_line":"        parser.add_argument("},{"line_number":117,"context_line":"            \u0027--dns-nameserver\u0027,"},{"line_number":118,"context_line":"            action\u003d\u0027append\u0027,"},{"line_number":119,"context_line":"            help\u003d\u0027DNS name server for this subnet \u0027"},{"line_number":120,"context_line":"            \u0027(This option can be repeated)\u0027)"},{"line_number":121,"context_line":"        parser.add_argument("}],"source_content_type":"text/x-python","patch_set":3,"id":"AAAAMX%2F%2F1hg%3D","line":118,"in_reply_to":"AAAAMX%2F%2F4F0%3D","updated":"2012-12-13 10:25:15.000000000","message":"Done.","commit_id":"313b119577a07e5f6ac0a4dbc5f4c4edde14ccb1"},{"author":{"_account_id":2874,"name":"yong sheng gong","email":"gong.yongsheng@99cloud.net","username":"gongysh"},"change_message_id":"bc1d50225fccf36e145ac16bc92881629858e99c","unresolved":false,"context_lines":[{"line_number":161,"context_line":"        routes \u003d []"},{"line_number":162,"context_line":"        if parsed_args.host_route:"},{"line_number":163,"context_line":"            for route in parsed_args.host_route:"},{"line_number":164,"context_line":"                routes.append(utils.str2dict(route))"},{"line_number":165,"context_line":"        if routes:"},{"line_number":166,"context_line":"            body[\u0027subnet\u0027].update({\u0027host_routes\u0027: routes})"},{"line_number":167,"context_line":""}],"source_content_type":"text/x-python","patch_set":3,"id":"AAAAMX%2F%2F4Fs%3D","line":164,"updated":"2012-12-11 12:03:31.000000000","message":"with above change, you can access parsed_args.host_routes here.","commit_id":"313b119577a07e5f6ac0a4dbc5f4c4edde14ccb1"},{"author":{"_account_id":841,"name":"Akihiro Motoki","email":"amotoki@gmail.com","username":"amotoki"},"unresolved":false,"context_lines":[{"line_number":161,"context_line":"        routes \u003d []"},{"line_number":162,"context_line":"        if parsed_args.host_route:"},{"line_number":163,"context_line":"            for route in parsed_args.host_route:"},{"line_number":164,"context_line":"                routes.append(utils.str2dict(route))"},{"line_number":165,"context_line":"        if routes:"},{"line_number":166,"context_line":"            body[\u0027subnet\u0027].update({\u0027host_routes\u0027: routes})"},{"line_number":167,"context_line":""}],"source_content_type":"text/x-python","patch_set":3,"id":"AAAAMX%2F%2F1hY%3D","line":164,"in_reply_to":"AAAAMX%2F%2F4Fs%3D","updated":"2012-12-13 10:25:15.000000000","message":"Done","commit_id":"313b119577a07e5f6ac0a4dbc5f4c4edde14ccb1"},{"author":{"_account_id":2874,"name":"yong sheng gong","email":"gong.yongsheng@99cloud.net","username":"gongysh"},"change_message_id":"bc1d50225fccf36e145ac16bc92881629858e99c","unresolved":false,"context_lines":[{"line_number":167,"context_line":""},{"line_number":168,"context_line":"        if parsed_args.dns_nameserver:"},{"line_number":169,"context_line":"            body[\u0027subnet\u0027].update("},{"line_number":170,"context_line":"                {\u0027dns_nameservers\u0027: parsed_args.dns_nameserver})"},{"line_number":171,"context_line":""},{"line_number":172,"context_line":"        return body"},{"line_number":173,"context_line":""}],"source_content_type":"text/x-python","patch_set":3,"id":"AAAAMX%2F%2F4Fo%3D","line":170,"updated":"2012-12-11 12:03:31.000000000","message":"same as host_routes","commit_id":"313b119577a07e5f6ac0a4dbc5f4c4edde14ccb1"},{"author":{"_account_id":841,"name":"Akihiro Motoki","email":"amotoki@gmail.com","username":"amotoki"},"unresolved":false,"context_lines":[{"line_number":167,"context_line":""},{"line_number":168,"context_line":"        if parsed_args.dns_nameserver:"},{"line_number":169,"context_line":"            body[\u0027subnet\u0027].update("},{"line_number":170,"context_line":"                {\u0027dns_nameservers\u0027: parsed_args.dns_nameserver})"},{"line_number":171,"context_line":""},{"line_number":172,"context_line":"        return body"},{"line_number":173,"context_line":""}],"source_content_type":"text/x-python","patch_set":3,"id":"AAAAMX%2F%2F1hU%3D","line":170,"in_reply_to":"AAAAMX%2F%2F4Fo%3D","updated":"2012-12-13 10:25:15.000000000","message":"Done","commit_id":"313b119577a07e5f6ac0a4dbc5f4c4edde14ccb1"}]}