)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":15905,"name":"Cao Xuan Hoang","email":"xuanhoangqb@gmail.com","username":"hoangcx"},"change_message_id":"37396d29ea518d6543106893f5c1f383432df6f2","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"fa6399be_1f1c8476","line":17,"updated":"2016-10-12 09:19:12.000000000","message":"Can you add the following to help tracking the work: Partially Implements: blueprint network-commands-options\nThanks!","commit_id":"91e00909b296d8d0ac30063fb49321b4028b70a4"},{"author":{"_account_id":17776,"name":"Reedip","email":"reedip.banerjee@gmail.com","username":"Reedip"},"change_message_id":"b57e1998ba48e897f22d7d6113fbfe3786657e4e","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"fa6399be_fdffd8a7","line":17,"in_reply_to":"fa6399be_1f1c8476","updated":"2016-10-12 09:50:51.000000000","message":"Done","commit_id":"91e00909b296d8d0ac30063fb49321b4028b70a4"}],"doc/source/command-objects/router.rst":[{"author":{"_account_id":8410,"name":"Richard Theis","email":"rtheis@us.ibm.com","username":"rtheis"},"change_message_id":"5feadd9df5fd79a0c405903a01d19319d18107bf","unresolved":false,"context_lines":[{"line_number":237,"context_line":""},{"line_number":238,"context_line":".. option:: --ha"},{"line_number":239,"context_line":""},{"line_number":240,"context_line":"    Set the router as highly available"},{"line_number":241,"context_line":""},{"line_number":242,"context_line":".. option:: --no-ha"},{"line_number":243,"context_line":""}],"source_content_type":"text/x-rst","patch_set":3,"id":"fa6399be_843780af","line":240,"updated":"2016-10-11 17:31:54.000000000","message":"The --distributed and --centralized options include (disabled router only) in the help text.  I think the same applies here and should be documented.","commit_id":"10fd3b424a733943061f376879a631d82baad405"},{"author":{"_account_id":17776,"name":"Reedip","email":"reedip.banerjee@gmail.com","username":"Reedip"},"change_message_id":"10a631fa33de50e8754f2b566280f44d73053d33","unresolved":false,"context_lines":[{"line_number":237,"context_line":""},{"line_number":238,"context_line":".. option:: --ha"},{"line_number":239,"context_line":""},{"line_number":240,"context_line":"    Set the router as highly available"},{"line_number":241,"context_line":""},{"line_number":242,"context_line":".. option:: --no-ha"},{"line_number":243,"context_line":""}],"source_content_type":"text/x-rst","patch_set":3,"id":"fa6399be_48b80be6","line":240,"in_reply_to":"fa6399be_843780af","updated":"2016-10-12 04:43:41.000000000","message":"Oh ok, got it :)","commit_id":"10fd3b424a733943061f376879a631d82baad405"},{"author":{"_account_id":8410,"name":"Richard Theis","email":"rtheis@us.ibm.com","username":"rtheis"},"change_message_id":"95150c037de383d9ee357a49143f2767104466bf","unresolved":false,"context_lines":[{"line_number":237,"context_line":""},{"line_number":238,"context_line":".. option:: --ha"},{"line_number":239,"context_line":""},{"line_number":240,"context_line":"    Set the router as highly available  (disabled router only)"},{"line_number":241,"context_line":""},{"line_number":242,"context_line":".. option:: --no-ha"},{"line_number":243,"context_line":""}],"source_content_type":"text/x-rst","patch_set":5,"id":"fa6399be_bf98ed1c","line":240,"range":{"start_line":240,"start_character":40,"end_line":240,"end_character":62},"updated":"2016-10-12 13:11:01.000000000","message":"Please update the help to match.  Also there\u0027s an extra space before this.","commit_id":"acb4519f735f6e97c345816cafc44782a59d7ff1"},{"author":{"_account_id":8410,"name":"Richard Theis","email":"rtheis@us.ibm.com","username":"rtheis"},"change_message_id":"95150c037de383d9ee357a49143f2767104466bf","unresolved":false,"context_lines":[{"line_number":241,"context_line":""},{"line_number":242,"context_line":".. option:: --no-ha"},{"line_number":243,"context_line":""},{"line_number":244,"context_line":"    Clear high availablability attribute of the router  (disabled router only)"},{"line_number":245,"context_line":""},{"line_number":246,"context_line":".. _router_set-router:"},{"line_number":247,"context_line":".. describe:: \u003crouter\u003e"}],"source_content_type":"text/x-rst","patch_set":5,"id":"fa6399be_bfbf0d5f","line":244,"updated":"2016-10-12 13:11:01.000000000","message":"Same as above.","commit_id":"acb4519f735f6e97c345816cafc44782a59d7ff1"}],"openstackclient/network/v2/router.py":[{"author":{"_account_id":21514,"name":"Huanxuan Ao","email":"aohuanxuan@163.com","username":"aohuanxuan"},"change_message_id":"54f322290bf02e8f8f18d56065f1ccdfa56a0cb6","unresolved":false,"context_lines":[{"line_number":436,"context_line":"        parser.add_argument("},{"line_number":437,"context_line":"            \u0027--ha\u0027,"},{"line_number":438,"context_line":"            action\u003d\u0027store_true\u0027,"},{"line_number":439,"context_line":"            help\u003d_(\"Set the router as highly available\")"},{"line_number":440,"context_line":"        )"},{"line_number":441,"context_line":"        # TODO(tangchen): Support setting \u0027external_gateway_info\u0027 property in"},{"line_number":442,"context_line":"        # \u0027router set\u0027 command."}],"source_content_type":"text/x-python","patch_set":1,"id":"1a95cdbc_4c342bfb","line":439,"updated":"2016-10-08 11:20:18.000000000","message":"Can we set the router to be not highly available by a inverse option (such as \"--no-ha\")?","commit_id":"dfcb62ed42938ddc707a20b8449f12ef9ccdf018"},{"author":{"_account_id":8410,"name":"Richard Theis","email":"rtheis@us.ibm.com","username":"rtheis"},"change_message_id":"1aee0162da91e8a0b8856f07f3cc14b75a601cb8","unresolved":false,"context_lines":[{"line_number":436,"context_line":"        parser.add_argument("},{"line_number":437,"context_line":"            \u0027--ha\u0027,"},{"line_number":438,"context_line":"            action\u003d\u0027store_true\u0027,"},{"line_number":439,"context_line":"            help\u003d_(\"Set the router as highly available\")"},{"line_number":440,"context_line":"        )"},{"line_number":441,"context_line":"        # TODO(tangchen): Support setting \u0027external_gateway_info\u0027 property in"},{"line_number":442,"context_line":"        # \u0027router set\u0027 command."}],"source_content_type":"text/x-python","patch_set":1,"id":"1a95cdbc_bd6f97fa","line":439,"in_reply_to":"1a95cdbc_4c342bfb","updated":"2016-10-08 17:42:34.000000000","message":"Also, what are the certain conditions in which the --ha option or --no-ha option are supported?  We may want to document these conditions.","commit_id":"dfcb62ed42938ddc707a20b8449f12ef9ccdf018"},{"author":{"_account_id":17776,"name":"Reedip","email":"reedip.banerjee@gmail.com","username":"Reedip"},"change_message_id":"079839d05a8d1f14e6ed4b1fa5b3cc2a84eb0479","unresolved":false,"context_lines":[{"line_number":436,"context_line":"        parser.add_argument("},{"line_number":437,"context_line":"            \u0027--ha\u0027,"},{"line_number":438,"context_line":"            action\u003d\u0027store_true\u0027,"},{"line_number":439,"context_line":"            help\u003d_(\"Set the router as highly available\")"},{"line_number":440,"context_line":"        )"},{"line_number":441,"context_line":"        # TODO(tangchen): Support setting \u0027external_gateway_info\u0027 property in"},{"line_number":442,"context_line":"        # \u0027router set\u0027 command."}],"source_content_type":"text/x-python","patch_set":1,"id":"1a95cdbc_844efa38","line":439,"in_reply_to":"1a95cdbc_4c342bfb","updated":"2016-10-10 04:46:35.000000000","message":"Done","commit_id":"dfcb62ed42938ddc707a20b8449f12ef9ccdf018"},{"author":{"_account_id":17776,"name":"Reedip","email":"reedip.banerjee@gmail.com","username":"Reedip"},"change_message_id":"6defcd0f22fc560ee68b92f3a00cff42d6028f3b","unresolved":false,"context_lines":[{"line_number":436,"context_line":"        parser.add_argument("},{"line_number":437,"context_line":"            \u0027--ha\u0027,"},{"line_number":438,"context_line":"            action\u003d\u0027store_true\u0027,"},{"line_number":439,"context_line":"            help\u003d_(\"Set the router as highly available\")"},{"line_number":440,"context_line":"        )"},{"line_number":441,"context_line":"        # TODO(tangchen): Support setting \u0027external_gateway_info\u0027 property in"},{"line_number":442,"context_line":"        # \u0027router set\u0027 command."}],"source_content_type":"text/x-python","patch_set":1,"id":"1a95cdbc_04ecaa28","line":439,"in_reply_to":"1a95cdbc_bd6f97fa","updated":"2016-10-10 04:41:09.000000000","message":"Hi Richard,\nThe specification for setting a router as HA depends on the neutron server.\nSome of the basic requirements which are checked to make a router HA are:\n- That the router is not administered ( admin-state is False /disabled )\n- That the router has 2 L3 agents are required.\n\nThese are checked in the neutron server itself","commit_id":"dfcb62ed42938ddc707a20b8449f12ef9ccdf018"},{"author":{"_account_id":21514,"name":"Huanxuan Ao","email":"aohuanxuan@163.com","username":"aohuanxuan"},"change_message_id":"355fad74cc378be1fbbe1562e766458826b4cebd","unresolved":false,"context_lines":[{"line_number":458,"context_line":""},{"line_number":459,"context_line":"        # Get the route attributes."},{"line_number":460,"context_line":"        if parsed_args.ha:"},{"line_number":461,"context_line":"            attrs[\u0027ha\u0027] \u003d parsed_args.ha"},{"line_number":462,"context_line":"        elif parsed_args.no_ha:"},{"line_number":463,"context_line":"            attrs[\u0027ha\u0027] \u003d False"},{"line_number":464,"context_line":"        if parsed_args.no_route:"}],"source_content_type":"text/x-python","patch_set":2,"id":"1a95cdbc_e74fb668","line":461,"updated":"2016-10-10 10:53:06.000000000","message":"You can feel free to change this line to\n    attrs[\u0027ha\u0027] \u003d True\nto match the code below","commit_id":"d2ede453baad97b12a3a1131342fc74a26f1be2a"},{"author":{"_account_id":17776,"name":"Reedip","email":"reedip.banerjee@gmail.com","username":"Reedip"},"change_message_id":"52f017f41d85742aecf3f09fa1e26d99ccab43e9","unresolved":false,"context_lines":[{"line_number":458,"context_line":""},{"line_number":459,"context_line":"        # Get the route attributes."},{"line_number":460,"context_line":"        if parsed_args.ha:"},{"line_number":461,"context_line":"            attrs[\u0027ha\u0027] \u003d parsed_args.ha"},{"line_number":462,"context_line":"        elif parsed_args.no_ha:"},{"line_number":463,"context_line":"            attrs[\u0027ha\u0027] \u003d False"},{"line_number":464,"context_line":"        if parsed_args.no_route:"}],"source_content_type":"text/x-python","patch_set":2,"id":"1a95cdbc_0a1827d2","line":461,"in_reply_to":"1a95cdbc_e74fb668","updated":"2016-10-12 04:43:47.000000000","message":"Done , but I think the previous iteration was also ok","commit_id":"d2ede453baad97b12a3a1131342fc74a26f1be2a"}],"releasenotes/notes/add-ha-to-router-update-6a38a73cc112b2fc.yaml":[{"author":{"_account_id":21514,"name":"Huanxuan Ao","email":"aohuanxuan@163.com","username":"aohuanxuan"},"change_message_id":"54f322290bf02e8f8f18d56065f1ccdfa56a0cb6","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 to update high-availability property of "},{"line_number":5,"context_line":"    a router by adding ``--ha`` option to ``router set``"},{"line_number":6,"context_line":"    CLI."}],"source_content_type":"text/x-yaml","patch_set":1,"id":"1a95cdbc_0c586355","line":4,"range":{"start_line":4,"start_character":55,"end_line":4,"end_character":56},"updated":"2016-10-08 11:20:18.000000000","message":"Redundant blank space","commit_id":"dfcb62ed42938ddc707a20b8449f12ef9ccdf018"},{"author":{"_account_id":17776,"name":"Reedip","email":"reedip.banerjee@gmail.com","username":"Reedip"},"change_message_id":"079839d05a8d1f14e6ed4b1fa5b3cc2a84eb0479","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 to update high-availability property of "},{"line_number":5,"context_line":"    a router by adding ``--ha`` option to ``router set``"},{"line_number":6,"context_line":"    CLI."}],"source_content_type":"text/x-yaml","patch_set":1,"id":"1a95cdbc_a4493633","line":4,"range":{"start_line":4,"start_character":55,"end_line":4,"end_character":56},"in_reply_to":"1a95cdbc_0c586355","updated":"2016-10-10 04:46:35.000000000","message":"Done","commit_id":"dfcb62ed42938ddc707a20b8449f12ef9ccdf018"},{"author":{"_account_id":21514,"name":"Huanxuan Ao","email":"aohuanxuan@163.com","username":"aohuanxuan"},"change_message_id":"355fad74cc378be1fbbe1562e766458826b4cebd","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"1a95cdbc_676ba6b7","line":7,"updated":"2016-10-10 10:53:06.000000000","message":"You will need to add the bug like here.","commit_id":"d2ede453baad97b12a3a1131342fc74a26f1be2a"}]}