)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":17711,"name":"Margaret Frances","email":"margaret_frances@cable.comcast.com","username":"margaret-frances"},"change_message_id":"aa0cebf6f16f7ae9f7f7f6e09b91fa127977aeca","unresolved":false,"context_lines":[{"line_number":4,"context_line":"Commit:     reedip \u003creedip.banerjee@nectechnologies.in\u003e"},{"line_number":5,"context_line":"CommitDate: 2016-03-08 17:52:43 +0900"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Improve help messages for NeutronClient"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"This patch improves some grammatical issues in"},{"line_number":10,"context_line":"the help messages of NeutronClient."}],"source_content_type":"text/x-gerrit-commit-message","patch_set":14,"id":"5aef4532_234e616f","line":7,"range":{"start_line":7,"start_character":26,"end_line":7,"end_character":39},"updated":"2016-03-08 13:12:18.000000000","message":"Per http://docs.openstack.org/contributor-guide/writing-style/openstack-components.html I believe this (and below) should be neutronclient","commit_id":"ba2fa2fa9e8af2a71c747a36cc7d2074f875d95f"},{"author":{"_account_id":17776,"name":"Reedip","email":"reedip.banerjee@gmail.com","username":"Reedip"},"change_message_id":"f291ebcecaa56d2190c9e827341953acffcc1bcb","unresolved":false,"context_lines":[{"line_number":4,"context_line":"Commit:     reedip \u003creedip.banerjee@nectechnologies.in\u003e"},{"line_number":5,"context_line":"CommitDate: 2016-03-08 17:52:43 +0900"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Improve help messages for NeutronClient"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"This patch improves some grammatical issues in"},{"line_number":10,"context_line":"the help messages of NeutronClient."}],"source_content_type":"text/x-gerrit-commit-message","patch_set":14,"id":"5aef4532_3a4e566f","line":7,"range":{"start_line":7,"start_character":26,"end_line":7,"end_character":39},"in_reply_to":"5aef4532_234e616f","updated":"2016-03-09 08:47:15.000000000","message":"Done","commit_id":"ba2fa2fa9e8af2a71c747a36cc7d2074f875d95f"}],"neutronclient/common/utils.py":[{"author":{"_account_id":17776,"name":"Reedip","email":"reedip.banerjee@gmail.com","username":"Reedip"},"change_message_id":"33d41d36bb852708cb1eab3930ce6361874e47e6","unresolved":false,"context_lines":[{"line_number":198,"context_line":"        cmd +\u003d help_option + \u0027 of the \u0027 + help_str + \u0027 to be created.\u0027"},{"line_number":199,"context_line":"    else:"},{"line_number":200,"context_line":"        cmd +\u003d \u0027Updated \u0027 + help_option + \u0027 of the \u0027 + help_str + \u0027.\u0027"},{"line_number":201,"context_line":"    return cmd"}],"source_content_type":"text/x-python","patch_set":7,"id":"7a740942_e58e1878","line":201,"updated":"2015-12-14 02:36:46.000000000","message":"@Akihiro :\nFor name and description , I have tried to create a common function which can be used.\nI have removed the translation _() though, after observing the H701 rule .\nFrom https://bugs.launchpad.net/horizon/+bug/1351196/comments/1, I got to understand that if the value inside _() is never empty, we do not need it.\nPlease correct my understanding if I am wrong.","commit_id":"bc7670dc8f30225f57d77b727a19421bf9306169"},{"author":{"_account_id":8119,"name":"Eric Brown","email":"eric_wade_brown@yahoo.com","username":"ericwb"},"change_message_id":"9d41b201e64f7c08e79b61468869ba7d2a2b5355","unresolved":false,"context_lines":[{"line_number":196,"context_line":"        return False"},{"line_number":197,"context_line":""},{"line_number":198,"context_line":""},{"line_number":199,"context_line":"def gethelp(help_option, help_str, is_create):"},{"line_number":200,"context_line":"    cmd \u003d \u0027\u0027"},{"line_number":201,"context_line":"    if is_create:"},{"line_number":202,"context_line":"        cmd +\u003d help_option + \u0027 of the \u0027 + help_str + \u0027 to be created.\u0027"}],"source_content_type":"text/x-python","patch_set":12,"id":"9a68dd71_d276bf05","line":199,"range":{"start_line":199,"start_character":4,"end_line":199,"end_character":11},"updated":"2016-01-27 05:40:13.000000000","message":"I think better as get_help","commit_id":"71b4f2ebf16264dc281c5bab1522df5d26c15fee"},{"author":{"_account_id":841,"name":"Akihiro Motoki","email":"amotoki@gmail.com","username":"amotoki"},"change_message_id":"92f97d9adc4adb2b5f6efd7f294e68e0562e3680","unresolved":false,"context_lines":[{"line_number":199,"context_line":"def gethelp(help_option, help_str, is_create):"},{"line_number":200,"context_line":"    cmd \u003d \u0027\u0027"},{"line_number":201,"context_line":"    if is_create:"},{"line_number":202,"context_line":"        cmd +\u003d help_option + \u0027 of the \u0027 + help_str + \u0027 to be created.\u0027"},{"line_number":203,"context_line":"    else:"},{"line_number":204,"context_line":"        cmd +\u003d \u0027Updated \u0027 + help_option + \u0027 of the \u0027 + help_str + \u0027.\u0027"},{"line_number":205,"context_line":"    return cmd"}],"source_content_type":"text/x-python","patch_set":12,"id":"7a5de9d1_6470b440","line":202,"updated":"2016-02-02 02:40:24.000000000","message":"You need to consider translation support.\nIn translation support, you should allow translators to control their word order, i.e., you need to use string substitution like \"%(help_option)s of the %(str)s to be created\".\n\nHowever, as I commented in listener.py, the above is still not very good.\n\nAs a conclusion, I oppose  this method.","commit_id":"71b4f2ebf16264dc281c5bab1522df5d26c15fee"}],"neutronclient/neutron/v2_0/address_scope.py":[{"author":{"_account_id":18573,"name":"Petri Kemppainen","email":"petri.kemppainen@nokia.com","username":"petrkemp"},"change_message_id":"f03a0503e4e174f48eac18b17bbd5428918364d4","unresolved":false,"context_lines":[{"line_number":67,"context_line":"    resource \u003d \u0027address_scope\u0027"},{"line_number":68,"context_line":""},{"line_number":69,"context_line":"    def add_known_arguments(self, parser):"},{"line_number":70,"context_line":"        parser.add_argument(\u0027--name\u0027,"},{"line_number":71,"context_line":"                            help\u003d_(\u0027Updated Name of the address scope.\u0027))"},{"line_number":72,"context_line":""},{"line_number":73,"context_line":"    def args2body(self, parsed_args):"}],"source_content_type":"text/x-python","patch_set":5,"id":"7a740942_d50eaee4","line":70,"updated":"2015-12-10 12:59:15.000000000","message":"Name --\u003e name","commit_id":"3893a9a1a2d8ae0a05043fbb0dcac6b82acba57f"}],"neutronclient/neutron/v2_0/flavor/flavor.py":[{"author":{"_account_id":17711,"name":"Margaret Frances","email":"margaret_frances@cable.comcast.com","username":"margaret-frances"},"change_message_id":"aa0cebf6f16f7ae9f7f7f6e09b91fa127977aeca","unresolved":false,"context_lines":[{"line_number":143,"context_line":"        parser.add_argument("},{"line_number":144,"context_line":"            \u0027flavor\u0027,"},{"line_number":145,"context_line":"            metavar\u003d\u0027FLAVOR\u0027,"},{"line_number":146,"context_line":"            help\u003d_(\u0027ID or name of the flavor which is to be disassociated.\u0027))"},{"line_number":147,"context_line":"        parser.add_argument("},{"line_number":148,"context_line":"            \u0027flavor_profile\u0027,"},{"line_number":149,"context_line":"            metavar\u003d\u0027FLAVOR_PROFILE\u0027,"}],"source_content_type":"text/x-python","patch_set":14,"id":"5aef4532_a3b0b144","line":146,"range":{"start_line":146,"start_character":45,"end_line":146,"end_character":50},"updated":"2016-03-08 13:12:18.000000000","message":"s/which/that :)\n\nor, imo better: \"...of the flavor to be disassociated.\"","commit_id":"ba2fa2fa9e8af2a71c747a36cc7d2074f875d95f"},{"author":{"_account_id":17776,"name":"Reedip","email":"reedip.banerjee@gmail.com","username":"Reedip"},"change_message_id":"f291ebcecaa56d2190c9e827341953acffcc1bcb","unresolved":false,"context_lines":[{"line_number":143,"context_line":"        parser.add_argument("},{"line_number":144,"context_line":"            \u0027flavor\u0027,"},{"line_number":145,"context_line":"            metavar\u003d\u0027FLAVOR\u0027,"},{"line_number":146,"context_line":"            help\u003d_(\u0027ID or name of the flavor which is to be disassociated.\u0027))"},{"line_number":147,"context_line":"        parser.add_argument("},{"line_number":148,"context_line":"            \u0027flavor_profile\u0027,"},{"line_number":149,"context_line":"            metavar\u003d\u0027FLAVOR_PROFILE\u0027,"}],"source_content_type":"text/x-python","patch_set":14,"id":"5aef4532_e402950a","line":146,"range":{"start_line":146,"start_character":45,"end_line":146,"end_character":50},"in_reply_to":"5aef4532_a3b0b144","updated":"2016-03-09 08:47:15.000000000","message":"I think \" which is to be disassociated\" also works.\n\"which\" can be safely used here","commit_id":"ba2fa2fa9e8af2a71c747a36cc7d2074f875d95f"},{"author":{"_account_id":14605,"name":"vikram.choudhary","email":"vikschw@gmail.com","username":"vikram.choudhary"},"change_message_id":"2be13bf0287225065481ef261f88f1e1889d382d","unresolved":false,"context_lines":[{"line_number":143,"context_line":"        parser.add_argument("},{"line_number":144,"context_line":"            \u0027flavor\u0027,"},{"line_number":145,"context_line":"            metavar\u003d\u0027FLAVOR\u0027,"},{"line_number":146,"context_line":"            help\u003d_(\u0027ID or name of the flavor which is to be disassociated.\u0027))"},{"line_number":147,"context_line":"        parser.add_argument("},{"line_number":148,"context_line":"            \u0027flavor_profile\u0027,"},{"line_number":149,"context_line":"            metavar\u003d\u0027FLAVOR_PROFILE\u0027,"}],"source_content_type":"text/x-python","patch_set":23,"id":"3aaa91ec_032a1d90","line":146,"updated":"2016-06-29 14:24:25.000000000","message":"How about \"ID or name of the flavor to be disassociated\"?","commit_id":"8fddbba75a92c5e64b66c823d3870bd96baf3f3b"},{"author":{"_account_id":22314,"name":"Ryan Selden","email":"seldenr@icloud.com","username":"seldenr"},"change_message_id":"1196923eff2a7e44370f5a4d9175f13bd1358fc1","unresolved":false,"context_lines":[{"line_number":143,"context_line":"        parser.add_argument("},{"line_number":144,"context_line":"            \u0027flavor\u0027,"},{"line_number":145,"context_line":"            metavar\u003d\u0027FLAVOR\u0027,"},{"line_number":146,"context_line":"            help\u003d_(\u0027ID or name of the flavor which is to be disassociated.\u0027))"},{"line_number":147,"context_line":"        parser.add_argument("},{"line_number":148,"context_line":"            \u0027flavor_profile\u0027,"},{"line_number":149,"context_line":"            metavar\u003d\u0027FLAVOR_PROFILE\u0027,"}],"source_content_type":"text/x-python","patch_set":23,"id":"3aaa91ec_4dc1a4e3","line":146,"in_reply_to":"3aaa91ec_032a1d90","updated":"2016-06-30 20:09:29.000000000","message":"Agreed","commit_id":"8fddbba75a92c5e64b66c823d3870bd96baf3f3b"}],"neutronclient/neutron/v2_0/floatingip.py":[{"author":{"_account_id":18573,"name":"Petri Kemppainen","email":"petri.kemppainen@nokia.com","username":"petrkemp"},"change_message_id":"f03a0503e4e174f48eac18b17bbd5428918364d4","unresolved":false,"context_lines":[{"line_number":47,"context_line":"    def add_known_arguments(self, parser):"},{"line_number":48,"context_line":"        parser.add_argument("},{"line_number":49,"context_line":"            \u0027floating_network_id\u0027, metavar\u003d\u0027FLOATING_NETWORK\u0027,"},{"line_number":50,"context_line":"            help\u003d_(\u0027ID or name of the network from which\u0027"},{"line_number":51,"context_line":"                   \u0027 floating IP is allocated.\u0027))"},{"line_number":52,"context_line":"        parser.add_argument("},{"line_number":53,"context_line":"            \u0027--port-id\u0027,"}],"source_content_type":"text/x-python","patch_set":5,"id":"7a740942_b5fd223d","line":50,"updated":"2015-12-10 12:59:15.000000000","message":"Remove space to the end of the previous line","commit_id":"3893a9a1a2d8ae0a05043fbb0dcac6b82acba57f"},{"author":{"_account_id":17711,"name":"Margaret Frances","email":"margaret_frances@cable.comcast.com","username":"margaret-frances"},"change_message_id":"aa0cebf6f16f7ae9f7f7f6e09b91fa127977aeca","unresolved":false,"context_lines":[{"line_number":49,"context_line":"        parser.add_argument("},{"line_number":50,"context_line":"            \u0027floating_network_id\u0027, metavar\u003d\u0027FLOATING_NETWORK\u0027,"},{"line_number":51,"context_line":"            help\u003d_(\u0027ID or name of the network from which \u0027"},{"line_number":52,"context_line":"                   \u0027floating IP is allocated.\u0027))"},{"line_number":53,"context_line":"        parser.add_argument("},{"line_number":54,"context_line":"            \u0027--port-id\u0027,"},{"line_number":55,"context_line":"            help\u003d_(\u0027ID of the port to be associated with the floating IP.\u0027))"}],"source_content_type":"text/x-python","patch_set":14,"id":"5aef4532_637d096e","line":52,"range":{"start_line":52,"start_character":20,"end_line":52,"end_character":28},"updated":"2016-03-08 13:12:18.000000000","message":"\"the floating IP\" (?)","commit_id":"ba2fa2fa9e8af2a71c747a36cc7d2074f875d95f"},{"author":{"_account_id":17776,"name":"Reedip","email":"reedip.banerjee@gmail.com","username":"Reedip"},"change_message_id":"f291ebcecaa56d2190c9e827341953acffcc1bcb","unresolved":false,"context_lines":[{"line_number":49,"context_line":"        parser.add_argument("},{"line_number":50,"context_line":"            \u0027floating_network_id\u0027, metavar\u003d\u0027FLOATING_NETWORK\u0027,"},{"line_number":51,"context_line":"            help\u003d_(\u0027ID or name of the network from which \u0027"},{"line_number":52,"context_line":"                   \u0027floating IP is allocated.\u0027))"},{"line_number":53,"context_line":"        parser.add_argument("},{"line_number":54,"context_line":"            \u0027--port-id\u0027,"},{"line_number":55,"context_line":"            help\u003d_(\u0027ID of the port to be associated with the floating IP.\u0027))"}],"source_content_type":"text/x-python","patch_set":14,"id":"5aef4532_dfc0042d","line":52,"range":{"start_line":52,"start_character":20,"end_line":52,"end_character":28},"in_reply_to":"5aef4532_637d096e","updated":"2016-03-09 08:47:15.000000000","message":"Done","commit_id":"ba2fa2fa9e8af2a71c747a36cc7d2074f875d95f"}],"neutronclient/neutron/v2_0/fw/firewall.py":[{"author":{"_account_id":18573,"name":"Petri Kemppainen","email":"petri.kemppainen@nokia.com","username":"petrkemp"},"change_message_id":"f03a0503e4e174f48eac18b17bbd5428918364d4","unresolved":false,"context_lines":[{"line_number":61,"context_line":"            metavar\u003d\u0027ROUTER\u0027,"},{"line_number":62,"context_line":"            action\u003d\u0027append\u0027,"},{"line_number":63,"context_line":"            help\u003d_(\u0027ID or name of the routers associated to this firewall \u0027"},{"line_number":64,"context_line":"                   \u0027(requires FWaaS router insertion extension, this option\u0027"},{"line_number":65,"context_line":"                   \u0027 can be repeated)\u0027))"},{"line_number":66,"context_line":""},{"line_number":67,"context_line":"    def args2body(self, parsed_args):"}],"source_content_type":"text/x-python","patch_set":5,"id":"7a740942_40775a9f","line":64,"updated":"2015-12-10 12:59:15.000000000","message":"Move space to the end of the previous line","commit_id":"3893a9a1a2d8ae0a05043fbb0dcac6b82acba57f"},{"author":{"_account_id":18573,"name":"Petri Kemppainen","email":"petri.kemppainen@nokia.com","username":"petrkemp"},"change_message_id":"f03a0503e4e174f48eac18b17bbd5428918364d4","unresolved":false,"context_lines":[{"line_number":97,"context_line":"            metavar\u003d\u0027ROUTER\u0027,"},{"line_number":98,"context_line":"            action\u003d\u0027append\u0027,"},{"line_number":99,"context_line":"            help\u003d_(\u0027ID or name of the routers associated to the firewall\u0027"},{"line_number":100,"context_line":"                   \u0027(requires FWaaS router insertion extension, this option\u0027"},{"line_number":101,"context_line":"                   \u0027 can be repeated)\u0027))"},{"line_number":102,"context_line":"        router_sg.add_argument("},{"line_number":103,"context_line":"            \u0027--no-routers\u0027,"}],"source_content_type":"text/x-python","patch_set":5,"id":"7a740942_40851a57","line":100,"updated":"2015-12-10 12:59:15.000000000","message":"Move space to the end of the previous line","commit_id":"3893a9a1a2d8ae0a05043fbb0dcac6b82acba57f"},{"author":{"_account_id":17711,"name":"Margaret Frances","email":"margaret_frances@cable.comcast.com","username":"margaret-frances"},"change_message_id":"4044077192243bfff145529ac5d356a501038768","unresolved":false,"context_lines":[{"line_number":60,"context_line":"            dest\u003d\u0027routers\u0027,"},{"line_number":61,"context_line":"            metavar\u003d\u0027ROUTER\u0027,"},{"line_number":62,"context_line":"            action\u003d\u0027append\u0027,"},{"line_number":63,"context_line":"            help\u003d_(\u0027ID or name of the routers associated to this firewall \u0027"},{"line_number":64,"context_line":"                   \u0027(requires FWaaS router insertion extension, this option \u0027"},{"line_number":65,"context_line":"                   \u0027can be repeated)\u0027))"},{"line_number":66,"context_line":""}],"source_content_type":"text/x-python","patch_set":8,"id":"9a68dd71_4cb9efe8","line":63,"range":{"start_line":63,"start_character":57,"end_line":63,"end_character":59},"updated":"2016-01-20 21:40:50.000000000","message":"Suggest \"with\"","commit_id":"99b5f1e0ff0f7723d968364834ec942e5da98c6c"},{"author":{"_account_id":17711,"name":"Margaret Frances","email":"margaret_frances@cable.comcast.com","username":"margaret-frances"},"change_message_id":"4044077192243bfff145529ac5d356a501038768","unresolved":false,"context_lines":[{"line_number":61,"context_line":"            metavar\u003d\u0027ROUTER\u0027,"},{"line_number":62,"context_line":"            action\u003d\u0027append\u0027,"},{"line_number":63,"context_line":"            help\u003d_(\u0027ID or name of the routers associated to this firewall \u0027"},{"line_number":64,"context_line":"                   \u0027(requires FWaaS router insertion extension, this option \u0027"},{"line_number":65,"context_line":"                   \u0027can be repeated)\u0027))"},{"line_number":66,"context_line":""},{"line_number":67,"context_line":"    def args2body(self, parsed_args):"},{"line_number":68,"context_line":"        client \u003d self.get_client()"}],"source_content_type":"text/x-python","patch_set":8,"id":"9a68dd71_6c12b3b6","line":65,"range":{"start_line":64,"start_character":24,"end_line":65,"end_character":36},"updated":"2016-01-20 21:40:50.000000000","message":"I\u0027m  having trouble understanding what\u0027s being said here.  The comma seem and the end-parenthesis seem misplaced (not sure).  Can it be made clear what \"this option\" refers to?","commit_id":"99b5f1e0ff0f7723d968364834ec942e5da98c6c"},{"author":{"_account_id":17211,"name":"Brandon Palm","email":"bapalm@us.ibm.com","username":"bapalm"},"change_message_id":"d7e7a5f438895dc45a89aac9d3d2732d178ece8b","unresolved":false,"context_lines":[{"line_number":61,"context_line":"            metavar\u003d\u0027ROUTER\u0027,"},{"line_number":62,"context_line":"            action\u003d\u0027append\u0027,"},{"line_number":63,"context_line":"            help\u003d_(\u0027ID or name of the routers associated with the firewall \u0027"},{"line_number":64,"context_line":"                   \u0027(requires FWaaS router insertion extension to be enabled.\u0027"},{"line_number":65,"context_line":"                   \u0027This option can be repeated.)\u0027))"},{"line_number":66,"context_line":""},{"line_number":67,"context_line":"    def args2body(self, parsed_args):"}],"source_content_type":"text/x-python","patch_set":12,"id":"7a5de9d1_17dc7817","line":64,"range":{"start_line":64,"start_character":76,"end_line":64,"end_character":78},"updated":"2016-02-02 14:06:37.000000000","message":"Missing a space at the end of line 64.","commit_id":"71b4f2ebf16264dc281c5bab1522df5d26c15fee"},{"author":{"_account_id":17211,"name":"Brandon Palm","email":"bapalm@us.ibm.com","username":"bapalm"},"change_message_id":"d7e7a5f438895dc45a89aac9d3d2732d178ece8b","unresolved":false,"context_lines":[{"line_number":97,"context_line":"            metavar\u003d\u0027ROUTER\u0027,"},{"line_number":98,"context_line":"            action\u003d\u0027append\u0027,"},{"line_number":99,"context_line":"            help\u003d_(\u0027ID or name of the routers associated with the firewall \u0027"},{"line_number":100,"context_line":"                   \u0027(requires FWaaS router insertion extension to be enabled.\u0027"},{"line_number":101,"context_line":"                   \u0027This option can be repeated.)\u0027))"},{"line_number":102,"context_line":"        router_sg.add_argument("},{"line_number":103,"context_line":"            \u0027--no-routers\u0027,"}],"source_content_type":"text/x-python","patch_set":12,"id":"7a5de9d1_f7eafcab","line":100,"range":{"start_line":100,"start_character":76,"end_line":100,"end_character":78},"updated":"2016-02-02 14:06:37.000000000","message":"Missing a space at the end of line 100.","commit_id":"71b4f2ebf16264dc281c5bab1522df5d26c15fee"},{"author":{"_account_id":17711,"name":"Margaret Frances","email":"margaret_frances@cable.comcast.com","username":"margaret-frances"},"change_message_id":"aa0cebf6f16f7ae9f7f7f6e09b91fa127977aeca","unresolved":false,"context_lines":[{"line_number":31,"context_line":"        dest\u003d\u0027routers\u0027,"},{"line_number":32,"context_line":"        metavar\u003d\u0027ROUTER\u0027,"},{"line_number":33,"context_line":"        action\u003d\u0027append\u0027,"},{"line_number":34,"context_line":"        help\u003d_(\u0027ID or name of the routers associated with the firewall \u0027"},{"line_number":35,"context_line":"               \u0027(requires FWaaS router insertion extension to be enabled. \u0027"},{"line_number":36,"context_line":"               \u0027This option can be repeated).\u0027))"},{"line_number":37,"context_line":"    router.add_argument("},{"line_number":38,"context_line":"        \u0027--no-routers\u0027,"},{"line_number":39,"context_line":"        action\u003d\u0027store_true\u0027,"}],"source_content_type":"text/x-python","patch_set":14,"id":"5aef4532_6384e9d3","line":36,"range":{"start_line":34,"start_character":16,"end_line":36,"end_character":46},"updated":"2016-03-08 13:12:18.000000000","message":"s/FWaas/fwaas (?)\n\nThe \"ID or name\" piece of things is singular but then of course \"routers\" is plural.  \n\nAlso, unless I\u0027m mistaken, I don\u0027t think that the sentence in the parenthetical is directly related to the previous part of the parenthetical.   (Plus it leads to some weird syntax to straighten it all out.)\n\nHow about something like: \"ID or name of the router associated with the firewall.  The fwaas router insertion extension must be enabled in order to use this option.  This option can be repeated: --router a, b, c.\"","commit_id":"ba2fa2fa9e8af2a71c747a36cc7d2074f875d95f"},{"author":{"_account_id":17776,"name":"Reedip","email":"reedip.banerjee@gmail.com","username":"Reedip"},"change_message_id":"f291ebcecaa56d2190c9e827341953acffcc1bcb","unresolved":false,"context_lines":[{"line_number":31,"context_line":"        dest\u003d\u0027routers\u0027,"},{"line_number":32,"context_line":"        metavar\u003d\u0027ROUTER\u0027,"},{"line_number":33,"context_line":"        action\u003d\u0027append\u0027,"},{"line_number":34,"context_line":"        help\u003d_(\u0027ID or name of the routers associated with the firewall \u0027"},{"line_number":35,"context_line":"               \u0027(requires FWaaS router insertion extension to be enabled. \u0027"},{"line_number":36,"context_line":"               \u0027This option can be repeated).\u0027))"},{"line_number":37,"context_line":"    router.add_argument("},{"line_number":38,"context_line":"        \u0027--no-routers\u0027,"},{"line_number":39,"context_line":"        action\u003d\u0027store_true\u0027,"}],"source_content_type":"text/x-python","patch_set":14,"id":"5aef4532_5f451415","line":36,"range":{"start_line":34,"start_character":16,"end_line":36,"end_character":46},"in_reply_to":"5aef4532_6384e9d3","updated":"2016-03-09 08:47:15.000000000","message":"How about:\n\nID or name of the router associated with the firewall \u0027\n               \u0027(this requires fwaas router insertion extension to be enabled. )This option can be repeated","commit_id":"ba2fa2fa9e8af2a71c747a36cc7d2074f875d95f"},{"author":{"_account_id":18573,"name":"Petri Kemppainen","email":"petri.kemppainen@nokia.com","username":"petrkemp"},"change_message_id":"46ce3de19175d74d1cbbaeab0eaac29487e92a20","unresolved":false,"context_lines":[{"line_number":114,"context_line":"        utils.add_boolean_argument("},{"line_number":115,"context_line":"            parser, \u0027--admin-state-up\u0027, dest\u003d\u0027admin_state_up\u0027,"},{"line_number":116,"context_line":"            help\u003d_(\u0027Update the admin state for the firewall\u0027"},{"line_number":117,"context_line":"                   \u0027(True means UP)\u0027))"},{"line_number":118,"context_line":""},{"line_number":119,"context_line":"    def args2body(self, parsed_args):"},{"line_number":120,"context_line":"        body \u003d parse_common_args(self.get_client(), parsed_args)"}],"source_content_type":"text/x-python","patch_set":14,"id":"5aef4532_24b4b4cf","line":117,"updated":"2016-03-08 09:18:38.000000000","message":"Should you add period to the end of line?","commit_id":"ba2fa2fa9e8af2a71c747a36cc7d2074f875d95f"},{"author":{"_account_id":17776,"name":"Reedip","email":"reedip.banerjee@gmail.com","username":"Reedip"},"change_message_id":"f291ebcecaa56d2190c9e827341953acffcc1bcb","unresolved":false,"context_lines":[{"line_number":114,"context_line":"        utils.add_boolean_argument("},{"line_number":115,"context_line":"            parser, \u0027--admin-state-up\u0027, dest\u003d\u0027admin_state_up\u0027,"},{"line_number":116,"context_line":"            help\u003d_(\u0027Update the admin state for the firewall\u0027"},{"line_number":117,"context_line":"                   \u0027(True means UP)\u0027))"},{"line_number":118,"context_line":""},{"line_number":119,"context_line":"    def args2body(self, parsed_args):"},{"line_number":120,"context_line":"        body \u003d parse_common_args(self.get_client(), parsed_args)"}],"source_content_type":"text/x-python","patch_set":14,"id":"5aef4532_df72c448","line":117,"in_reply_to":"5aef4532_24b4b4cf","updated":"2016-03-09 08:47:15.000000000","message":"Done","commit_id":"ba2fa2fa9e8af2a71c747a36cc7d2074f875d95f"},{"author":{"_account_id":17711,"name":"Margaret Frances","email":"margaret_frances@cable.comcast.com","username":"margaret-frances"},"change_message_id":"5819fea3612bc721d49b79d4d8049c419217bf43","unresolved":false,"context_lines":[{"line_number":31,"context_line":"        dest\u003d\u0027routers\u0027,"},{"line_number":32,"context_line":"        metavar\u003d\u0027ROUTER\u0027,"},{"line_number":33,"context_line":"        action\u003d\u0027append\u0027,"},{"line_number":34,"context_line":"        help\u003d_(\u0027ID or name of the router associated with the firewall. \u0027"},{"line_number":35,"context_line":"               \u0027(requires FWaaS router insertion extension to be enabled).\u0027"},{"line_number":36,"context_line":"               \u0027 This option can be repeated.\u0027))"},{"line_number":37,"context_line":"    router.add_argument("},{"line_number":38,"context_line":"        \u0027--no-routers\u0027,"}],"source_content_type":"text/x-python","patch_set":15,"id":"5aef4532_fec68f98","line":35,"range":{"start_line":34,"start_character":69,"end_line":35,"end_character":18},"updated":"2016-03-09 13:40:30.000000000","message":"Suggest either removing the period before the parenthetical or--less good, I think--capitalizing \"requires\".  If \"requires\" is capitalized, then the period after the end paren needs to be moved before it.  \n\nAlso, I think the space before the next sentence needs to be brought up to the end of the previous line.","commit_id":"a1319f84a379e57e74f70d564031e24b98958112"},{"author":{"_account_id":18573,"name":"Petri Kemppainen","email":"petri.kemppainen@nokia.com","username":"petrkemp"},"change_message_id":"a676466fd004c960002683056d2efd8c7d15d7a4","unresolved":false,"context_lines":[{"line_number":113,"context_line":"                   \u0027associated to this firewall.\u0027))"},{"line_number":114,"context_line":"        utils.add_boolean_argument("},{"line_number":115,"context_line":"            parser, \u0027--admin-state-up\u0027, dest\u003d\u0027admin_state_up\u0027,"},{"line_number":116,"context_line":"            help\u003d_(\u0027Update the admin state for the firewall\u0027"},{"line_number":117,"context_line":"                   \u0027(True means UP).\u0027))"},{"line_number":118,"context_line":""},{"line_number":119,"context_line":"    def args2body(self, parsed_args):"}],"source_content_type":"text/x-python","patch_set":21,"id":"1a122d0e_54633c41","line":116,"range":{"start_line":116,"start_character":59,"end_line":116,"end_character":60},"updated":"2016-04-25 12:13:39.000000000","message":"Space is missing.","commit_id":"0a83565c05b4558617521ccc861e06c263a14302"},{"author":{"_account_id":17776,"name":"Reedip","email":"reedip.banerjee@gmail.com","username":"Reedip"},"change_message_id":"7b5632ad0e8ded60eeb4aed7d2d916558a67bcb8","unresolved":false,"context_lines":[{"line_number":113,"context_line":"                   \u0027associated to this firewall.\u0027))"},{"line_number":114,"context_line":"        utils.add_boolean_argument("},{"line_number":115,"context_line":"            parser, \u0027--admin-state-up\u0027, dest\u003d\u0027admin_state_up\u0027,"},{"line_number":116,"context_line":"            help\u003d_(\u0027Update the admin state for the firewall\u0027"},{"line_number":117,"context_line":"                   \u0027(True means UP).\u0027))"},{"line_number":118,"context_line":""},{"line_number":119,"context_line":"    def args2body(self, parsed_args):"}],"source_content_type":"text/x-python","patch_set":21,"id":"7aa08908_36ac808b","line":116,"range":{"start_line":116,"start_character":59,"end_line":116,"end_character":60},"in_reply_to":"1a122d0e_54633c41","updated":"2016-06-14 07:05:47.000000000","message":"Done","commit_id":"0a83565c05b4558617521ccc861e06c263a14302"}],"neutronclient/neutron/v2_0/fw/firewallpolicy.py":[{"author":{"_account_id":14605,"name":"vikram.choudhary","email":"vikschw@gmail.com","username":"vikram.choudhary"},"change_message_id":"1d247591339f9c82c08a25daefddb7082ea58ef8","unresolved":false,"context_lines":[{"line_number":203,"context_line":"        parser.add_argument("},{"line_number":204,"context_line":"            \u0027firewall_rule_id\u0027,"},{"line_number":205,"context_line":"            metavar\u003d\u0027FIREWALL_RULE\u0027,"},{"line_number":206,"context_line":"            help\u003d_(\u0027Firewall rule to be removed from the policy.\u0027))"},{"line_number":207,"context_line":"        self.add_known_arguments(parser)"},{"line_number":208,"context_line":"        return parser"},{"line_number":209,"context_line":""}],"source_content_type":"text/x-python","patch_set":6,"id":"7a740942_f507f51a","line":206,"updated":"2015-12-11 08:44:40.000000000","message":"Only ID? no names over here? L195 supports both Name and ID.","commit_id":"8817f540786000e8ee21550b3fe5c72322e42da9"},{"author":{"_account_id":17776,"name":"Reedip","email":"reedip.banerjee@gmail.com","username":"Reedip"},"change_message_id":"ccd0d969d5ab2202145d71e13cd746443d0595c1","unresolved":false,"context_lines":[{"line_number":203,"context_line":"        parser.add_argument("},{"line_number":204,"context_line":"            \u0027firewall_rule_id\u0027,"},{"line_number":205,"context_line":"            metavar\u003d\u0027FIREWALL_RULE\u0027,"},{"line_number":206,"context_line":"            help\u003d_(\u0027Firewall rule to be removed from the policy.\u0027))"},{"line_number":207,"context_line":"        self.add_known_arguments(parser)"},{"line_number":208,"context_line":"        return parser"},{"line_number":209,"context_line":""}],"source_content_type":"text/x-python","patch_set":6,"id":"7a740942_2a42afae","line":206,"in_reply_to":"7a740942_45bb7ae5","updated":"2015-12-14 02:11:33.000000000","message":"Done","commit_id":"8817f540786000e8ee21550b3fe5c72322e42da9"},{"author":{"_account_id":17776,"name":"Reedip","email":"reedip.banerjee@gmail.com","username":"Reedip"},"change_message_id":"ae48fcf1b51193d440d7fcc0fc80254848d8c917","unresolved":false,"context_lines":[{"line_number":203,"context_line":"        parser.add_argument("},{"line_number":204,"context_line":"            \u0027firewall_rule_id\u0027,"},{"line_number":205,"context_line":"            metavar\u003d\u0027FIREWALL_RULE\u0027,"},{"line_number":206,"context_line":"            help\u003d_(\u0027Firewall rule to be removed from the policy.\u0027))"},{"line_number":207,"context_line":"        self.add_known_arguments(parser)"},{"line_number":208,"context_line":"        return parser"},{"line_number":209,"context_line":""}],"source_content_type":"text/x-python","patch_set":6,"id":"7a740942_f877ec38","line":206,"in_reply_to":"7a740942_f507f51a","updated":"2015-12-11 08:51:41.000000000","message":"There is no mention of ID or name in the help message.\nWhat you may have concluded was that we are using ID only because the argument name is firewall_rule_id.\n\nNow, the problem here is:\n\na) I cannot change firewall_rule_id to firewall_rule, bcz you know, backward compatibility rupture.\n\nb) I cannot state to the user to pass Name/ID to the argument firewall_rule_id, bcz, it looks silly.\n\nTherefore, unless we change the attribute name, changing the Help message wont make sense....\nThats why no mention of Name or Id   :)\n\nNow the question is .... what do you think ???","commit_id":"8817f540786000e8ee21550b3fe5c72322e42da9"},{"author":{"_account_id":14605,"name":"vikram.choudhary","email":"vikschw@gmail.com","username":"vikram.choudhary"},"change_message_id":"a51902e148c50b4484de64e9f5c1e677739d80d1","unresolved":false,"context_lines":[{"line_number":203,"context_line":"        parser.add_argument("},{"line_number":204,"context_line":"            \u0027firewall_rule_id\u0027,"},{"line_number":205,"context_line":"            metavar\u003d\u0027FIREWALL_RULE\u0027,"},{"line_number":206,"context_line":"            help\u003d_(\u0027Firewall rule to be removed from the policy.\u0027))"},{"line_number":207,"context_line":"        self.add_known_arguments(parser)"},{"line_number":208,"context_line":"        return parser"},{"line_number":209,"context_line":""}],"source_content_type":"text/x-python","patch_set":6,"id":"7a740942_f8c5a18d","line":206,"in_reply_to":"7a740942_f877ec38","updated":"2015-12-11 12:26:37.000000000","message":"L195 confirms that support for both ID and name is already there. The current help is misleading. IMO, we can just modify the help and don\u0027t touch the existing param name.","commit_id":"8817f540786000e8ee21550b3fe5c72322e42da9"},{"author":{"_account_id":841,"name":"Akihiro Motoki","email":"amotoki@gmail.com","username":"amotoki"},"change_message_id":"b1da0d9aa31a2fe61f92a7b3052e79fc619730a1","unresolved":false,"context_lines":[{"line_number":203,"context_line":"        parser.add_argument("},{"line_number":204,"context_line":"            \u0027firewall_rule_id\u0027,"},{"line_number":205,"context_line":"            metavar\u003d\u0027FIREWALL_RULE\u0027,"},{"line_number":206,"context_line":"            help\u003d_(\u0027Firewall rule to be removed from the policy.\u0027))"},{"line_number":207,"context_line":"        self.add_known_arguments(parser)"},{"line_number":208,"context_line":"        return parser"},{"line_number":209,"context_line":""}],"source_content_type":"text/x-python","patch_set":6,"id":"7a740942_45bb7ae5","line":206,"in_reply_to":"7a740942_f8c5a18d","updated":"2015-12-13 00:19:12.000000000","message":"Agree with vikram.","commit_id":"8817f540786000e8ee21550b3fe5c72322e42da9"},{"author":{"_account_id":14605,"name":"vikram.choudhary","email":"vikschw@gmail.com","username":"vikram.choudhary"},"change_message_id":"c44c9a07fa16b637b031d5004c50991b5ca1a845","unresolved":false,"context_lines":[{"line_number":202,"context_line":"        parser.add_argument("},{"line_number":203,"context_line":"            \u0027firewall_rule_id\u0027,"},{"line_number":204,"context_line":"            metavar\u003d\u0027FIREWALL_RULE\u0027,"},{"line_number":205,"context_line":"            help\u003d_(\u0027ID/Name of the Firewall rule to be removed\u0027"},{"line_number":206,"context_line":"                   \u0027 from the policy.\u0027))"},{"line_number":207,"context_line":"        self.add_known_arguments(parser)"},{"line_number":208,"context_line":"        return parser"}],"source_content_type":"text/x-python","patch_set":11,"id":"9a68dd71_92fe0704","line":205,"updated":"2016-01-27 05:09:36.000000000","message":"nit: Can you change \"ID/Name\" to \"ID or name\" for consistency.","commit_id":"727ff3af4a00939834f66e5a0a2b8ca303625fb4"},{"author":{"_account_id":8119,"name":"Eric Brown","email":"eric_wade_brown@yahoo.com","username":"ericwb"},"change_message_id":"9d41b201e64f7c08e79b61468869ba7d2a2b5355","unresolved":false,"context_lines":[{"line_number":202,"context_line":"        parser.add_argument("},{"line_number":203,"context_line":"            \u0027firewall_rule_id\u0027,"},{"line_number":204,"context_line":"            metavar\u003d\u0027FIREWALL_RULE\u0027,"},{"line_number":205,"context_line":"            help\u003d_(\u0027IDor name of the Firewall rule to be removed\u0027"},{"line_number":206,"context_line":"                   \u0027 from the policy.\u0027))"},{"line_number":207,"context_line":"        self.add_known_arguments(parser)"},{"line_number":208,"context_line":"        return parser"}],"source_content_type":"text/x-python","patch_set":12,"id":"9a68dd71_123f7726","line":205,"range":{"start_line":205,"start_character":37,"end_line":205,"end_character":45},"updated":"2016-01-27 05:40:13.000000000","message":"s/Firewall/firewall","commit_id":"71b4f2ebf16264dc281c5bab1522df5d26c15fee"},{"author":{"_account_id":8119,"name":"Eric Brown","email":"eric_wade_brown@yahoo.com","username":"ericwb"},"change_message_id":"9d41b201e64f7c08e79b61468869ba7d2a2b5355","unresolved":false,"context_lines":[{"line_number":202,"context_line":"        parser.add_argument("},{"line_number":203,"context_line":"            \u0027firewall_rule_id\u0027,"},{"line_number":204,"context_line":"            metavar\u003d\u0027FIREWALL_RULE\u0027,"},{"line_number":205,"context_line":"            help\u003d_(\u0027IDor name of the Firewall rule to be removed\u0027"},{"line_number":206,"context_line":"                   \u0027 from the policy.\u0027))"},{"line_number":207,"context_line":"        self.add_known_arguments(parser)"},{"line_number":208,"context_line":"        return parser"}],"source_content_type":"text/x-python","patch_set":12,"id":"9a68dd71_52340f49","line":205,"range":{"start_line":205,"start_character":20,"end_line":205,"end_character":25},"updated":"2016-01-27 05:40:13.000000000","message":"typo","commit_id":"71b4f2ebf16264dc281c5bab1522df5d26c15fee"},{"author":{"_account_id":14605,"name":"vikram.choudhary","email":"vikschw@gmail.com","username":"vikram.choudhary"},"change_message_id":"f01dfcfcff2f73fe74c18ccd4dde04564914d24d","unresolved":false,"context_lines":[{"line_number":202,"context_line":"        parser.add_argument("},{"line_number":203,"context_line":"            \u0027firewall_rule_id\u0027,"},{"line_number":204,"context_line":"            metavar\u003d\u0027FIREWALL_RULE\u0027,"},{"line_number":205,"context_line":"            help\u003d_(\u0027IDor name of the Firewall rule to be removed\u0027"},{"line_number":206,"context_line":"                   \u0027 from the policy.\u0027))"},{"line_number":207,"context_line":"        self.add_known_arguments(parser)"},{"line_number":208,"context_line":"        return parser"}],"source_content_type":"text/x-python","patch_set":12,"id":"7a5de9d1_cc4bc35e","line":205,"range":{"start_line":205,"start_character":20,"end_line":205,"end_character":25},"in_reply_to":"9a68dd71_52340f49","updated":"2016-02-02 09:37:24.000000000","message":"+1","commit_id":"71b4f2ebf16264dc281c5bab1522df5d26c15fee"},{"author":{"_account_id":18573,"name":"Petri Kemppainen","email":"petri.kemppainen@nokia.com","username":"petrkemp"},"change_message_id":"46ce3de19175d74d1cbbaeab0eaac29487e92a20","unresolved":false,"context_lines":[{"line_number":114,"context_line":"        utils.add_boolean_argument("},{"line_number":115,"context_line":"            parser, \u0027--shared\u0027,"},{"line_number":116,"context_line":"            help\u003d_(\u0027Update the sharing status of the policy. \u0027"},{"line_number":117,"context_line":"                   \u0027(True means shared)\u0027))"},{"line_number":118,"context_line":"        utils.add_boolean_argument("},{"line_number":119,"context_line":"            parser, \u0027--audited\u0027,"},{"line_number":120,"context_line":"            help\u003d_(\u0027Update the audit status of the policy. \u0027"}],"source_content_type":"text/x-python","patch_set":14,"id":"5aef4532_04053879","line":117,"updated":"2016-03-08 09:18:38.000000000","message":"Ditto.","commit_id":"ba2fa2fa9e8af2a71c747a36cc7d2074f875d95f"},{"author":{"_account_id":17776,"name":"Reedip","email":"reedip.banerjee@gmail.com","username":"Reedip"},"change_message_id":"f291ebcecaa56d2190c9e827341953acffcc1bcb","unresolved":false,"context_lines":[{"line_number":114,"context_line":"        utils.add_boolean_argument("},{"line_number":115,"context_line":"            parser, \u0027--shared\u0027,"},{"line_number":116,"context_line":"            help\u003d_(\u0027Update the sharing status of the policy. \u0027"},{"line_number":117,"context_line":"                   \u0027(True means shared)\u0027))"},{"line_number":118,"context_line":"        utils.add_boolean_argument("},{"line_number":119,"context_line":"            parser, \u0027--audited\u0027,"},{"line_number":120,"context_line":"            help\u003d_(\u0027Update the audit status of the policy. \u0027"}],"source_content_type":"text/x-python","patch_set":14,"id":"5aef4532_3fb768d9","line":117,"in_reply_to":"5aef4532_04053879","updated":"2016-03-09 08:47:15.000000000","message":"Done","commit_id":"ba2fa2fa9e8af2a71c747a36cc7d2074f875d95f"},{"author":{"_account_id":18573,"name":"Petri Kemppainen","email":"petri.kemppainen@nokia.com","username":"petrkemp"},"change_message_id":"46ce3de19175d74d1cbbaeab0eaac29487e92a20","unresolved":false,"context_lines":[{"line_number":118,"context_line":"        utils.add_boolean_argument("},{"line_number":119,"context_line":"            parser, \u0027--audited\u0027,"},{"line_number":120,"context_line":"            help\u003d_(\u0027Update the audit status of the policy. \u0027"},{"line_number":121,"context_line":"                   \u0027(True means auditing is enabled)\u0027))"},{"line_number":122,"context_line":""},{"line_number":123,"context_line":"    def args2body(self, parsed_args):"},{"line_number":124,"context_line":"        return parse_common_args(self.get_client(), parsed_args)"}],"source_content_type":"text/x-python","patch_set":14,"id":"5aef4532_2402f46f","line":121,"updated":"2016-03-08 09:18:38.000000000","message":"Ditto.","commit_id":"ba2fa2fa9e8af2a71c747a36cc7d2074f875d95f"},{"author":{"_account_id":17776,"name":"Reedip","email":"reedip.banerjee@gmail.com","username":"Reedip"},"change_message_id":"f291ebcecaa56d2190c9e827341953acffcc1bcb","unresolved":false,"context_lines":[{"line_number":118,"context_line":"        utils.add_boolean_argument("},{"line_number":119,"context_line":"            parser, \u0027--audited\u0027,"},{"line_number":120,"context_line":"            help\u003d_(\u0027Update the audit status of the policy. \u0027"},{"line_number":121,"context_line":"                   \u0027(True means auditing is enabled)\u0027))"},{"line_number":122,"context_line":""},{"line_number":123,"context_line":"    def args2body(self, parsed_args):"},{"line_number":124,"context_line":"        return parse_common_args(self.get_client(), parsed_args)"}],"source_content_type":"text/x-python","patch_set":14,"id":"5aef4532_5feb54b0","line":121,"in_reply_to":"5aef4532_2402f46f","updated":"2016-03-09 08:47:15.000000000","message":"Done","commit_id":"ba2fa2fa9e8af2a71c747a36cc7d2074f875d95f"},{"author":{"_account_id":18573,"name":"Petri Kemppainen","email":"petri.kemppainen@nokia.com","username":"petrkemp"},"change_message_id":"46ce3de19175d74d1cbbaeab0eaac29487e92a20","unresolved":false,"context_lines":[{"line_number":214,"context_line":"            \u0027firewall_rule_id\u0027,"},{"line_number":215,"context_line":"            metavar\u003d\u0027FIREWALL_RULE\u0027,"},{"line_number":216,"context_line":"            help\u003d_(\u0027ID or name of the firewall rule to be removed\u0027"},{"line_number":217,"context_line":"                   \u0027 from the policy.\u0027))"},{"line_number":218,"context_line":"        self.add_known_arguments(parser)"},{"line_number":219,"context_line":"        return parser"},{"line_number":220,"context_line":""}],"source_content_type":"text/x-python","patch_set":14,"id":"5aef4532_e4d64c26","line":217,"range":{"start_line":217,"start_character":20,"end_line":217,"end_character":21},"updated":"2016-03-08 09:18:38.000000000","message":"Move space to the end of the previous line.","commit_id":"ba2fa2fa9e8af2a71c747a36cc7d2074f875d95f"},{"author":{"_account_id":17776,"name":"Reedip","email":"reedip.banerjee@gmail.com","username":"Reedip"},"change_message_id":"f291ebcecaa56d2190c9e827341953acffcc1bcb","unresolved":false,"context_lines":[{"line_number":214,"context_line":"            \u0027firewall_rule_id\u0027,"},{"line_number":215,"context_line":"            metavar\u003d\u0027FIREWALL_RULE\u0027,"},{"line_number":216,"context_line":"            help\u003d_(\u0027ID or name of the firewall rule to be removed\u0027"},{"line_number":217,"context_line":"                   \u0027 from the policy.\u0027))"},{"line_number":218,"context_line":"        self.add_known_arguments(parser)"},{"line_number":219,"context_line":"        return parser"},{"line_number":220,"context_line":""}],"source_content_type":"text/x-python","patch_set":14,"id":"5aef4532_5f023470","line":217,"range":{"start_line":217,"start_character":20,"end_line":217,"end_character":21},"in_reply_to":"5aef4532_e4d64c26","updated":"2016-03-09 08:47:15.000000000","message":"Done","commit_id":"ba2fa2fa9e8af2a71c747a36cc7d2074f875d95f"}],"neutronclient/neutron/v2_0/lb/v2/listener.py":[{"author":{"_account_id":18573,"name":"Petri Kemppainen","email":"petri.kemppainen@nokia.com","username":"petrkemp"},"change_message_id":"f03a0503e4e174f48eac18b17bbd5428918364d4","unresolved":false,"context_lines":[{"line_number":55,"context_line":"            help\u003d_(\u0027Set admin state up to false.\u0027))"},{"line_number":56,"context_line":"        parser.add_argument("},{"line_number":57,"context_line":"            \u0027--connection-limit\u0027,"},{"line_number":58,"context_line":"            help\u003d_(\u0027The maximum number of connections per second allowed for \u0027"},{"line_number":59,"context_line":"                   \u0027the vip. Acceptable values are positive integer or -1 for\u0027"},{"line_number":60,"context_line":"                   \u0027unlimited (default).\u0027))"},{"line_number":61,"context_line":"        parser.add_argument("}],"source_content_type":"text/x-python","patch_set":5,"id":"7a740942_a0207e4a","line":58,"updated":"2015-12-10 12:59:15.000000000","message":"Add space to the end of the line","commit_id":"3893a9a1a2d8ae0a05043fbb0dcac6b82acba57f"},{"author":{"_account_id":841,"name":"Akihiro Motoki","email":"amotoki@gmail.com","username":"amotoki"},"change_message_id":"b1da0d9aa31a2fe61f92a7b3052e79fc619730a1","unresolved":false,"context_lines":[{"line_number":56,"context_line":"        parser.add_argument("},{"line_number":57,"context_line":"            \u0027--connection-limit\u0027,"},{"line_number":58,"context_line":"            help\u003d_(\u0027The maximum number of connections per second allowed for \u0027"},{"line_number":59,"context_line":"                   \u0027the vip. Acceptable values are positive integer or -1 for \u0027"},{"line_number":60,"context_line":"                   \u0027unlimited (default).\u0027))"},{"line_number":61,"context_line":"        parser.add_argument("},{"line_number":62,"context_line":"            \u0027--description\u0027,"}],"source_content_type":"text/x-python","patch_set":6,"id":"7a740942_05b1f200","line":59,"updated":"2015-12-13 00:19:12.000000000","message":"\"Acceptable values\" should be a singular form.","commit_id":"8817f540786000e8ee21550b3fe5c72322e42da9"},{"author":{"_account_id":841,"name":"Akihiro Motoki","email":"amotoki@gmail.com","username":"amotoki"},"change_message_id":"b1da0d9aa31a2fe61f92a7b3052e79fc619730a1","unresolved":false,"context_lines":[{"line_number":63,"context_line":"            help\u003d_(\u0027Description of the listener.\u0027))"},{"line_number":64,"context_line":"        parser.add_argument("},{"line_number":65,"context_line":"            \u0027--name\u0027,"},{"line_number":66,"context_line":"            help\u003d_(\u0027Name of the listener.\u0027))"},{"line_number":67,"context_line":"        parser.add_argument("},{"line_number":68,"context_line":"            \u0027--default-tls-container-ref\u0027,"},{"line_number":69,"context_line":"            dest\u003d\u0027default_tls_container_ref\u0027,"}],"source_content_type":"text/x-python","patch_set":6,"id":"7a740942_25b66ef9","line":66,"updated":"2015-12-13 00:19:12.000000000","message":"In other places, you use \u0027... to be created\u0027. Looks inconsistent.","commit_id":"8817f540786000e8ee21550b3fe5c72322e42da9"},{"author":{"_account_id":17776,"name":"Reedip","email":"reedip.banerjee@gmail.com","username":"Reedip"},"change_message_id":"ccd0d969d5ab2202145d71e13cd746443d0595c1","unresolved":false,"context_lines":[{"line_number":63,"context_line":"            help\u003d_(\u0027Description of the listener.\u0027))"},{"line_number":64,"context_line":"        parser.add_argument("},{"line_number":65,"context_line":"            \u0027--name\u0027,"},{"line_number":66,"context_line":"            help\u003d_(\u0027Name of the listener.\u0027))"},{"line_number":67,"context_line":"        parser.add_argument("},{"line_number":68,"context_line":"            \u0027--default-tls-container-ref\u0027,"},{"line_number":69,"context_line":"            dest\u003d\u0027default_tls_container_ref\u0027,"}],"source_content_type":"text/x-python","patch_set":6,"id":"7a740942_ea0107b0","line":66,"in_reply_to":"7a740942_25b66ef9","updated":"2015-12-14 02:11:33.000000000","message":"Need to hack it as it is common for both Create Listener and Update Listener","commit_id":"8817f540786000e8ee21550b3fe5c72322e42da9"},{"author":{"_account_id":841,"name":"Akihiro Motoki","email":"amotoki@gmail.com","username":"amotoki"},"change_message_id":"92f97d9adc4adb2b5f6efd7f294e68e0562e3680","unresolved":false,"context_lines":[{"line_number":61,"context_line":"            type\u003dint)"},{"line_number":62,"context_line":"        parser.add_argument("},{"line_number":63,"context_line":"            \u0027--description\u0027,"},{"line_number":64,"context_line":"            help\u003dutils.gethelp(\u0027Description\u0027, \u0027listener\u0027, is_create))"},{"line_number":65,"context_line":"        parser.add_argument("},{"line_number":66,"context_line":"            \u0027--name\u0027,"},{"line_number":67,"context_line":"            help\u003dutils.gethelp(\u0027Name\u0027, \u0027listener\u0027, is_create))"}],"source_content_type":"text/x-python","patch_set":12,"id":"7a5de9d1_641af4c7","line":64,"updated":"2016-02-02 02:40:24.000000000","message":"This completely loses translatability from the three reasons.\nConsider using a full string instead.\n\n- gethelp() uses concatenation (as commented in utils.py)\n- \u0027Description\u0027 and \u0027listener\u0027 should be marked as translatable.\n- Too short string like one word makes it hard to translate the word based on its context.\n\nThe last point is important, so I oppose introducing utils.gethelp()\nand suggest to use a full string instead.","commit_id":"71b4f2ebf16264dc281c5bab1522df5d26c15fee"},{"author":{"_account_id":17211,"name":"Brandon Palm","email":"bapalm@us.ibm.com","username":"bapalm"},"change_message_id":"5260d991154247162f59e18c91a7dbb78ba01456","unresolved":false,"context_lines":[{"line_number":58,"context_line":""},{"line_number":59,"context_line":"    resource \u003d \u0027listener\u0027"},{"line_number":60,"context_line":""},{"line_number":61,"context_line":"    def add_known_arguments(self, parser, is_create\u003dTrue):"},{"line_number":62,"context_line":"        parser.add_argument("},{"line_number":63,"context_line":"            \u0027--admin-state-down\u0027,"},{"line_number":64,"context_line":"            dest\u003d\u0027admin_state\u0027, action\u003d\u0027store_false\u0027,"}],"source_content_type":"text/x-python","patch_set":15,"id":"5aef4532_8e910c4a","line":61,"range":{"start_line":61,"start_character":42,"end_line":61,"end_character":51},"updated":"2016-03-09 16:19:31.000000000","message":"Is \u0027is_create\u0027 used at all?","commit_id":"a1319f84a379e57e74f70d564031e24b98958112"},{"author":{"_account_id":7787,"name":"Kevin Benton","email":"kevin@benton.pub","username":"blak111"},"change_message_id":"4e0a992c4d7b63aaf20db6ad6c62455266880ae4","unresolved":false,"context_lines":[{"line_number":45,"context_line":"        \u0027--connection-limit\u0027,"},{"line_number":46,"context_line":"        help\u003d_(\u0027The maximum number of connections per second allowed for \u0027"},{"line_number":47,"context_line":"               \u0027the vip. Positive integer or -1 for unlimited (default).\u0027),"},{"line_number":48,"context_line":"        type\u003dint)"},{"line_number":49,"context_line":"    parser.add_argument("},{"line_number":50,"context_line":"        \u0027--default-pool\u0027,"},{"line_number":51,"context_line":"        help\u003d_(\u0027Default pool for the listener.\u0027))"}],"source_content_type":"text/x-python","patch_set":22,"id":"3aaa91ec_e5576ec8","line":48,"updated":"2016-06-28 06:50:27.000000000","message":"what\u0027s the point of this change?","commit_id":"4dbd4ffa31b106bffbe90926e07976161bfe2c44"}],"neutronclient/neutron/v2_0/lb/v2/loadbalancer.py":[{"author":{"_account_id":18573,"name":"Petri Kemppainen","email":"petri.kemppainen@nokia.com","username":"petrkemp"},"change_message_id":"46ce3de19175d74d1cbbaeab0eaac29487e92a20","unresolved":false,"context_lines":[{"line_number":125,"context_line":"            # | ...                | ...   |"},{"line_number":126,"context_line":"            # +--------------------+-------+"},{"line_number":127,"context_line":"            # it has two columns and the Filed column is alphabetical,"},{"line_number":128,"context_line":"            # here covert the data dict to the 1-1 vector format below:"},{"line_number":129,"context_line":"            # [(field1, field2, field3, ...), (value1, value2, value3, ...)]"},{"line_number":130,"context_line":"            return list(zip(*sorted(stats.items())))"},{"line_number":131,"context_line":""}],"source_content_type":"text/x-python","patch_set":14,"id":"5aef4532_04eff887","line":128,"range":{"start_line":128,"start_character":19,"end_line":128,"end_character":25},"updated":"2016-03-08 09:18:38.000000000","message":"convert?","commit_id":"ba2fa2fa9e8af2a71c747a36cc7d2074f875d95f"},{"author":{"_account_id":17776,"name":"Reedip","email":"reedip.banerjee@gmail.com","username":"Reedip"},"change_message_id":"f291ebcecaa56d2190c9e827341953acffcc1bcb","unresolved":false,"context_lines":[{"line_number":125,"context_line":"            # | ...                | ...   |"},{"line_number":126,"context_line":"            # +--------------------+-------+"},{"line_number":127,"context_line":"            # it has two columns and the Filed column is alphabetical,"},{"line_number":128,"context_line":"            # here covert the data dict to the 1-1 vector format below:"},{"line_number":129,"context_line":"            # [(field1, field2, field3, ...), (value1, value2, value3, ...)]"},{"line_number":130,"context_line":"            return list(zip(*sorted(stats.items())))"},{"line_number":131,"context_line":""}],"source_content_type":"text/x-python","patch_set":14,"id":"5aef4532_dfee44b3","line":128,"range":{"start_line":128,"start_character":19,"end_line":128,"end_character":25},"in_reply_to":"5aef4532_04eff887","updated":"2016-03-09 08:47:15.000000000","message":"Done","commit_id":"ba2fa2fa9e8af2a71c747a36cc7d2074f875d95f"}],"neutronclient/neutron/v2_0/lb/vip.py":[{"author":{"_account_id":18573,"name":"Petri Kemppainen","email":"petri.kemppainen@nokia.com","username":"petrkemp"},"change_message_id":"f03a0503e4e174f48eac18b17bbd5428918364d4","unresolved":false,"context_lines":[{"line_number":41,"context_line":""},{"line_number":42,"context_line":"    def add_known_arguments(self, parser):"},{"line_number":43,"context_line":"        parser.add_argument("},{"line_number":44,"context_line":"            \u0027pool_id\u0027, metavar\u003d\u0027POOL\u0027,"},{"line_number":45,"context_line":"            help\u003d_(\u0027Pool ID or name this which this vip belongs to.\u0027))"},{"line_number":46,"context_line":"        parser.add_argument("},{"line_number":47,"context_line":"            \u0027--address\u0027,"}],"source_content_type":"text/x-python","patch_set":5,"id":"7a740942_0001526a","line":44,"updated":"2015-12-10 12:59:15.000000000","message":"Remove 1st. this","commit_id":"3893a9a1a2d8ae0a05043fbb0dcac6b82acba57f"},{"author":{"_account_id":18573,"name":"Petri Kemppainen","email":"petri.kemppainen@nokia.com","username":"petrkemp"},"change_message_id":"f03a0503e4e174f48eac18b17bbd5428918364d4","unresolved":false,"context_lines":[{"line_number":53,"context_line":"        parser.add_argument("},{"line_number":54,"context_line":"            \u0027--connection-limit\u0027,"},{"line_number":55,"context_line":"            help\u003d_(\u0027The maximum number of connections per second allowed for \u0027"},{"line_number":56,"context_line":"                   \u0027the vip. Acceptable values are positive integer or -1 for\u0027"},{"line_number":57,"context_line":"                   \u0027 unlimited (default).\u0027))"},{"line_number":58,"context_line":"        parser.add_argument("},{"line_number":59,"context_line":"            \u0027--description\u0027,"}],"source_content_type":"text/x-python","patch_set":5,"id":"7a740942_c0682a24","line":56,"updated":"2015-12-10 12:59:15.000000000","message":"Move space to the end of the previous line","commit_id":"3893a9a1a2d8ae0a05043fbb0dcac6b82acba57f"},{"author":{"_account_id":841,"name":"Akihiro Motoki","email":"amotoki@gmail.com","username":"amotoki"},"change_message_id":"b1da0d9aa31a2fe61f92a7b3052e79fc619730a1","unresolved":false,"context_lines":[{"line_number":53,"context_line":"        parser.add_argument("},{"line_number":54,"context_line":"            \u0027--connection-limit\u0027,"},{"line_number":55,"context_line":"            help\u003d_(\u0027The maximum number of connections per second allowed for \u0027"},{"line_number":56,"context_line":"                   \u0027the vip. Acceptable values are positive integer or -1 for \u0027"},{"line_number":57,"context_line":"                   \u0027unlimited (default).\u0027))"},{"line_number":58,"context_line":"        parser.add_argument("},{"line_number":59,"context_line":"            \u0027--description\u0027,"}],"source_content_type":"text/x-python","patch_set":6,"id":"7a740942_e5cce68a","line":56,"updated":"2015-12-13 00:19:12.000000000","message":"\u0027Acceptable values\u0027 should be a singular form.","commit_id":"8817f540786000e8ee21550b3fe5c72322e42da9"},{"author":{"_account_id":17776,"name":"Reedip","email":"reedip.banerjee@gmail.com","username":"Reedip"},"change_message_id":"ccd0d969d5ab2202145d71e13cd746443d0595c1","unresolved":false,"context_lines":[{"line_number":53,"context_line":"        parser.add_argument("},{"line_number":54,"context_line":"            \u0027--connection-limit\u0027,"},{"line_number":55,"context_line":"            help\u003d_(\u0027The maximum number of connections per second allowed for \u0027"},{"line_number":56,"context_line":"                   \u0027the vip. Acceptable values are positive integer or -1 for \u0027"},{"line_number":57,"context_line":"                   \u0027unlimited (default).\u0027))"},{"line_number":58,"context_line":"        parser.add_argument("},{"line_number":59,"context_line":"            \u0027--description\u0027,"}],"source_content_type":"text/x-python","patch_set":6,"id":"7a740942_8aff2365","line":56,"in_reply_to":"7a740942_e5cce68a","updated":"2015-12-14 02:11:33.000000000","message":"Done.\nAlso , I need to add ¨to be created¨ to line#64 and 60.\n\nQuery: lb-vip-update is deprecated from Liberty.\nAnd lb-vip-update is missing a lot of options( like Updating Name, Description etc).\n\nShould a bug be logged for the same , considering that the CLI might not be used from N-release?","commit_id":"8817f540786000e8ee21550b3fe5c72322e42da9"},{"author":{"_account_id":8119,"name":"Eric Brown","email":"eric_wade_brown@yahoo.com","username":"ericwb"},"change_message_id":"9d41b201e64f7c08e79b61468869ba7d2a2b5355","unresolved":false,"context_lines":[{"line_number":54,"context_line":"        parser.add_argument("},{"line_number":55,"context_line":"            \u0027--connection-limit\u0027,"},{"line_number":56,"context_line":"            help\u003d_(\u0027The maximum number of connections per second allowed for \u0027"},{"line_number":57,"context_line":"                   \u0027the vip. Acceptable value is a positive integer or -1 for \u0027"},{"line_number":58,"context_line":"                   \u0027unlimited (default).\u0027))"},{"line_number":59,"context_line":"        parser.add_argument("},{"line_number":60,"context_line":"            \u0027--description\u0027,"}],"source_content_type":"text/x-python","patch_set":12,"id":"9a68dd71_12269716","line":57,"range":{"start_line":57,"start_character":29,"end_line":57,"end_character":39},"updated":"2016-01-27 05:40:13.000000000","message":"An acceptable ....","commit_id":"71b4f2ebf16264dc281c5bab1522df5d26c15fee"},{"author":{"_account_id":17211,"name":"Brandon Palm","email":"bapalm@us.ibm.com","username":"bapalm"},"change_message_id":"d7e7a5f438895dc45a89aac9d3d2732d178ece8b","unresolved":false,"context_lines":[{"line_number":58,"context_line":"                   \u0027unlimited (default).\u0027))"},{"line_number":59,"context_line":"        parser.add_argument("},{"line_number":60,"context_line":"            \u0027--description\u0027,"},{"line_number":61,"context_line":"            help\u003dutils.gethelp(\u0027Description\u0027, \u0027vip\u0027, is_create\u003dTrue))"},{"line_number":62,"context_line":"        parser.add_argument("},{"line_number":63,"context_line":"            \u0027--name\u0027,"},{"line_number":64,"context_line":"            required\u003dTrue,"}],"source_content_type":"text/x-python","patch_set":12,"id":"7a5de9d1_771cccc1","line":61,"updated":"2016-02-02 14:06:37.000000000","message":"As others have said, a full string is probably better here for translations in the future.","commit_id":"71b4f2ebf16264dc281c5bab1522df5d26c15fee"},{"author":{"_account_id":17211,"name":"Brandon Palm","email":"bapalm@us.ibm.com","username":"bapalm"},"change_message_id":"d7e7a5f438895dc45a89aac9d3d2732d178ece8b","unresolved":false,"context_lines":[{"line_number":62,"context_line":"        parser.add_argument("},{"line_number":63,"context_line":"            \u0027--name\u0027,"},{"line_number":64,"context_line":"            required\u003dTrue,"},{"line_number":65,"context_line":"            help\u003dutils.gethelp(\u0027Name\u0027, \u0027vip\u0027, is_create\u003dTrue))"},{"line_number":66,"context_line":"        parser.add_argument("},{"line_number":67,"context_line":"            \u0027--protocol-port\u0027,"},{"line_number":68,"context_line":"            required\u003dTrue,"}],"source_content_type":"text/x-python","patch_set":12,"id":"7a5de9d1_1778f8f7","line":65,"updated":"2016-02-02 14:06:37.000000000","message":"Ditto","commit_id":"71b4f2ebf16264dc281c5bab1522df5d26c15fee"},{"author":{"_account_id":17711,"name":"Margaret Frances","email":"margaret_frances@cable.comcast.com","username":"margaret-frances"},"change_message_id":"aa0cebf6f16f7ae9f7f7f6e09b91fa127977aeca","unresolved":false,"context_lines":[{"line_number":42,"context_line":"    def add_known_arguments(self, parser):"},{"line_number":43,"context_line":"        parser.add_argument("},{"line_number":44,"context_line":"            \u0027pool_id\u0027, metavar\u003d\u0027POOL\u0027,"},{"line_number":45,"context_line":"            help\u003d_(\u0027ID or name of the pool where this vip belongs.\u0027))"},{"line_number":46,"context_line":"        parser.add_argument("},{"line_number":47,"context_line":"            \u0027--address\u0027,"},{"line_number":48,"context_line":"            help\u003d_(\u0027IP address of the vip.\u0027))"}],"source_content_type":"text/x-python","patch_set":14,"id":"5aef4532_c3e17d9c","line":45,"range":{"start_line":45,"start_character":43,"end_line":45,"end_character":48},"updated":"2016-03-08 13:12:18.000000000","message":"Suggest: s/where/to which","commit_id":"ba2fa2fa9e8af2a71c747a36cc7d2074f875d95f"},{"author":{"_account_id":17711,"name":"Margaret Frances","email":"margaret_frances@cable.comcast.com","username":"margaret-frances"},"change_message_id":"aa0cebf6f16f7ae9f7f7f6e09b91fa127977aeca","unresolved":false,"context_lines":[{"line_number":53,"context_line":"        parser.add_argument("},{"line_number":54,"context_line":"            \u0027--connection-limit\u0027,"},{"line_number":55,"context_line":"            help\u003d_(\u0027The maximum number of connections per second allowed for \u0027"},{"line_number":56,"context_line":"                   \u0027the vip. An acceptable value is a positive integer or -1 \u0027"},{"line_number":57,"context_line":"                   \u0027for unlimited (default).\u0027))"},{"line_number":58,"context_line":"        parser.add_argument("},{"line_number":59,"context_line":"            \u0027--description\u0027,"},{"line_number":60,"context_line":"            help\u003d_(\u0027Description of the vip to be created.\u0027))"}],"source_content_type":"text/x-python","patch_set":14,"id":"5aef4532_e3abd91b","line":57,"range":{"start_line":56,"start_character":29,"end_line":57,"end_character":44},"updated":"2016-03-08 13:12:18.000000000","message":"Suggest something like:\n\nValid values: any  positive integer, or, for no limit (which is the default) -1.","commit_id":"ba2fa2fa9e8af2a71c747a36cc7d2074f875d95f"},{"author":{"_account_id":17776,"name":"Reedip","email":"reedip.banerjee@gmail.com","username":"Reedip"},"change_message_id":"f291ebcecaa56d2190c9e827341953acffcc1bcb","unresolved":false,"context_lines":[{"line_number":53,"context_line":"        parser.add_argument("},{"line_number":54,"context_line":"            \u0027--connection-limit\u0027,"},{"line_number":55,"context_line":"            help\u003d_(\u0027The maximum number of connections per second allowed for \u0027"},{"line_number":56,"context_line":"                   \u0027the vip. An acceptable value is a positive integer or -1 \u0027"},{"line_number":57,"context_line":"                   \u0027for unlimited (default).\u0027))"},{"line_number":58,"context_line":"        parser.add_argument("},{"line_number":59,"context_line":"            \u0027--description\u0027,"},{"line_number":60,"context_line":"            help\u003d_(\u0027Description of the vip to be created.\u0027))"}],"source_content_type":"text/x-python","patch_set":14,"id":"5aef4532_7fd290d3","line":57,"range":{"start_line":56,"start_character":29,"end_line":57,"end_character":44},"in_reply_to":"5aef4532_e3abd91b","updated":"2016-03-09 08:47:15.000000000","message":"Modified ( in a different manner)","commit_id":"ba2fa2fa9e8af2a71c747a36cc7d2074f875d95f"},{"author":{"_account_id":18573,"name":"Petri Kemppainen","email":"petri.kemppainen@nokia.com","username":"petrkemp"},"change_message_id":"a676466fd004c960002683056d2efd8c7d15d7a4","unresolved":false,"context_lines":[{"line_number":54,"context_line":"            \u0027--connection-limit\u0027,"},{"line_number":55,"context_line":"            help\u003d_(\u0027The maximum number of connections per second allowed for \u0027"},{"line_number":56,"context_line":"                   \u0027the vip. Valid values: a positive integer or -1 \u0027"},{"line_number":57,"context_line":"                   \u0027for unlimited (default).\u0027))"},{"line_number":58,"context_line":"        parser.add_argument("},{"line_number":59,"context_line":"            \u0027--description\u0027,"},{"line_number":60,"context_line":"            help\u003d_(\u0027Description of the vip to be created.\u0027))"}],"source_content_type":"text/x-python","patch_set":21,"id":"1a122d0e_3735e2d9","line":57,"updated":"2016-04-25 12:13:39.000000000","message":"Should you add type\u003dint.. here?","commit_id":"0a83565c05b4558617521ccc861e06c263a14302"},{"author":{"_account_id":17776,"name":"Reedip","email":"reedip.banerjee@gmail.com","username":"Reedip"},"change_message_id":"7b5632ad0e8ded60eeb4aed7d2d916558a67bcb8","unresolved":false,"context_lines":[{"line_number":54,"context_line":"            \u0027--connection-limit\u0027,"},{"line_number":55,"context_line":"            help\u003d_(\u0027The maximum number of connections per second allowed for \u0027"},{"line_number":56,"context_line":"                   \u0027the vip. Valid values: a positive integer or -1 \u0027"},{"line_number":57,"context_line":"                   \u0027for unlimited (default).\u0027))"},{"line_number":58,"context_line":"        parser.add_argument("},{"line_number":59,"context_line":"            \u0027--description\u0027,"},{"line_number":60,"context_line":"            help\u003d_(\u0027Description of the vip to be created.\u0027))"}],"source_content_type":"text/x-python","patch_set":21,"id":"7aa08908_b6aeb07e","line":57,"in_reply_to":"1a122d0e_3735e2d9","updated":"2016-06-14 07:05:47.000000000","message":"Would be handled by the API","commit_id":"0a83565c05b4558617521ccc861e06c263a14302"}],"neutronclient/neutron/v2_0/metering.py":[{"author":{"_account_id":18573,"name":"Petri Kemppainen","email":"petri.kemppainen@nokia.com","username":"petrkemp"},"change_message_id":"f03a0503e4e174f48eac18b17bbd5428918364d4","unresolved":false,"context_lines":[{"line_number":42,"context_line":"            \u0027name\u0027, metavar\u003d\u0027NAME\u0027,"},{"line_number":43,"context_line":"            help\u003d_(\u0027Name of the metering label to be created.\u0027))"},{"line_number":44,"context_line":"        parser.add_argument("},{"line_number":45,"context_line":"            \u0027--description\u0027,"},{"line_number":46,"context_line":"            help\u003d_(\u0027Description of the metering label to created.\u0027))"},{"line_number":47,"context_line":"        parser.add_argument("},{"line_number":48,"context_line":"            \u0027--shared\u0027,"}],"source_content_type":"text/x-python","patch_set":5,"id":"7a740942_c084aab2","line":45,"updated":"2015-12-10 12:59:15.000000000","message":"to --\u003e to be","commit_id":"3893a9a1a2d8ae0a05043fbb0dcac6b82acba57f"},{"author":{"_account_id":14605,"name":"vikram.choudhary","email":"vikschw@gmail.com","username":"vikram.choudhary"},"change_message_id":"1d247591339f9c82c08a25daefddb7082ea58ef8","unresolved":false,"context_lines":[{"line_number":86,"context_line":"    def add_known_arguments(self, parser):"},{"line_number":87,"context_line":"        parser.add_argument("},{"line_number":88,"context_line":"            \u0027label_id\u0027, metavar\u003d\u0027LABEL\u0027,"},{"line_number":89,"context_line":"            help\u003d_(\u0027Id or name of the label.\u0027))"},{"line_number":90,"context_line":"        parser.add_argument("},{"line_number":91,"context_line":"            \u0027remote_ip_prefix\u0027, metavar\u003d\u0027REMOTE_IP_PREFIX\u0027,"},{"line_number":92,"context_line":"            help\u003d_(\u0027CIDR to match on.\u0027))"}],"source_content_type":"text/x-python","patch_set":6,"id":"7a740942_589ba0c9","line":89,"updated":"2015-12-11 08:44:40.000000000","message":"nit: s/Id/ID for consistency.","commit_id":"8817f540786000e8ee21550b3fe5c72322e42da9"},{"author":{"_account_id":841,"name":"Akihiro Motoki","email":"amotoki@gmail.com","username":"amotoki"},"change_message_id":"b1da0d9aa31a2fe61f92a7b3052e79fc619730a1","unresolved":false,"context_lines":[{"line_number":86,"context_line":"    def add_known_arguments(self, parser):"},{"line_number":87,"context_line":"        parser.add_argument("},{"line_number":88,"context_line":"            \u0027label_id\u0027, metavar\u003d\u0027LABEL\u0027,"},{"line_number":89,"context_line":"            help\u003d_(\u0027Id or name of the label.\u0027))"},{"line_number":90,"context_line":"        parser.add_argument("},{"line_number":91,"context_line":"            \u0027remote_ip_prefix\u0027, metavar\u003d\u0027REMOTE_IP_PREFIX\u0027,"},{"line_number":92,"context_line":"            help\u003d_(\u0027CIDR to match on.\u0027))"}],"source_content_type":"text/x-python","patch_set":6,"id":"7a740942_a5c25e51","line":89,"in_reply_to":"7a740942_589ba0c9","updated":"2015-12-13 00:19:12.000000000","message":"Needs \u0027to be created\u0027 for the consistency you are making?","commit_id":"8817f540786000e8ee21550b3fe5c72322e42da9"},{"author":{"_account_id":17776,"name":"Reedip","email":"reedip.banerjee@gmail.com","username":"Reedip"},"change_message_id":"ccd0d969d5ab2202145d71e13cd746443d0595c1","unresolved":false,"context_lines":[{"line_number":86,"context_line":"    def add_known_arguments(self, parser):"},{"line_number":87,"context_line":"        parser.add_argument("},{"line_number":88,"context_line":"            \u0027label_id\u0027, metavar\u003d\u0027LABEL\u0027,"},{"line_number":89,"context_line":"            help\u003d_(\u0027Id or name of the label.\u0027))"},{"line_number":90,"context_line":"        parser.add_argument("},{"line_number":91,"context_line":"            \u0027remote_ip_prefix\u0027, metavar\u003d\u0027REMOTE_IP_PREFIX\u0027,"},{"line_number":92,"context_line":"            help\u003d_(\u0027CIDR to match on.\u0027))"}],"source_content_type":"text/x-python","patch_set":6,"id":"7a740942_0aa7b3a7","line":89,"in_reply_to":"7a740942_a5c25e51","updated":"2015-12-14 02:11:33.000000000","message":"@vikram: Done\n@Akihiro: This is the label_id , for which the rule would be created.\n\nThe following CLI does not have a name option\n\nusage: neutron meter-label-rule-create [-h] [-f {json,shell,table,value,yaml}]\n                                       [-c COLUMN] [--max-width \u003cinteger\u003e]\n                                       [--noindent] [--prefix PREFIX]\n                                       [--request-format {json,xml}]\n                                       [--tenant-id TENANT_ID]\n                                       [--direction {ingress,egress}]\n                                       [--excluded]\n\nIf naming for the rule needs to be added, it is a different bug.\nShould I log it?","commit_id":"8817f540786000e8ee21550b3fe5c72322e42da9"}],"neutronclient/neutron/v2_0/network.py":[{"author":{"_account_id":18573,"name":"Petri Kemppainen","email":"petri.kemppainen@nokia.com","username":"petrkemp"},"change_message_id":"f03a0503e4e174f48eac18b17bbd5428918364d4","unresolved":false,"context_lines":[{"line_number":142,"context_line":"            default\u003dargparse.SUPPRESS,"},{"line_number":143,"context_line":"            help\u003d_(\u0027Create a VLAN transparent network.\u0027))"},{"line_number":144,"context_line":"        parser.add_argument("},{"line_number":145,"context_line":"            \u0027name\u0027, metavar\u003d\u0027NAME\u0027,"},{"line_number":146,"context_line":"            help\u003d_(\u0027Name of the network to created.\u0027))"},{"line_number":147,"context_line":""},{"line_number":148,"context_line":"        self.add_arguments_qos_policy(parser)"}],"source_content_type":"text/x-python","patch_set":5,"id":"7a740942_802a228f","line":145,"updated":"2015-12-10 12:59:15.000000000","message":"to --\u003e to be","commit_id":"3893a9a1a2d8ae0a05043fbb0dcac6b82acba57f"},{"author":{"_account_id":18573,"name":"Petri Kemppainen","email":"petri.kemppainen@nokia.com","username":"petrkemp"},"change_message_id":"a676466fd004c960002683056d2efd8c7d15d7a4","unresolved":false,"context_lines":[{"line_number":170,"context_line":"            \u0027--provider:physical_network\u0027,"},{"line_number":171,"context_line":"            metavar\u003d\u0027\u003cphysical_network_name\u003e\u0027,"},{"line_number":172,"context_line":"            help\u003d_(\u0027Name of the physical network over which the virtual\u0027"},{"line_number":173,"context_line":"                   \u0027 network is implemented.\u0027))"},{"line_number":174,"context_line":"        parser.add_argument("},{"line_number":175,"context_line":"            \u0027--provider:segmentation_id\u0027,"},{"line_number":176,"context_line":"            metavar\u003d\u0027\u003csegmentation_id\u003e\u0027,"}],"source_content_type":"text/x-python","patch_set":21,"id":"1a122d0e_17b54630","line":173,"range":{"start_line":173,"start_character":20,"end_line":173,"end_character":21},"updated":"2016-04-25 12:13:39.000000000","message":"Move space to the end of the previous line.","commit_id":"0a83565c05b4558617521ccc861e06c263a14302"},{"author":{"_account_id":17776,"name":"Reedip","email":"reedip.banerjee@gmail.com","username":"Reedip"},"change_message_id":"7b5632ad0e8ded60eeb4aed7d2d916558a67bcb8","unresolved":false,"context_lines":[{"line_number":170,"context_line":"            \u0027--provider:physical_network\u0027,"},{"line_number":171,"context_line":"            metavar\u003d\u0027\u003cphysical_network_name\u003e\u0027,"},{"line_number":172,"context_line":"            help\u003d_(\u0027Name of the physical network over which the virtual\u0027"},{"line_number":173,"context_line":"                   \u0027 network is implemented.\u0027))"},{"line_number":174,"context_line":"        parser.add_argument("},{"line_number":175,"context_line":"            \u0027--provider:segmentation_id\u0027,"},{"line_number":176,"context_line":"            metavar\u003d\u0027\u003csegmentation_id\u003e\u0027,"}],"source_content_type":"text/x-python","patch_set":21,"id":"7aa08908_d6c7fc42","line":173,"range":{"start_line":173,"start_character":20,"end_line":173,"end_character":21},"in_reply_to":"1a122d0e_17b54630","updated":"2016-06-14 07:05:47.000000000","message":"Done","commit_id":"0a83565c05b4558617521ccc861e06c263a14302"},{"author":{"_account_id":18573,"name":"Petri Kemppainen","email":"petri.kemppainen@nokia.com","username":"petrkemp"},"change_message_id":"a676466fd004c960002683056d2efd8c7d15d7a4","unresolved":false,"context_lines":[{"line_number":175,"context_line":"            \u0027--provider:segmentation_id\u0027,"},{"line_number":176,"context_line":"            metavar\u003d\u0027\u003csegmentation_id\u003e\u0027,"},{"line_number":177,"context_line":"            help\u003d_(\u0027VLAN ID for VLAN networks or tunnel-id for GRE/VXLAN\u0027"},{"line_number":178,"context_line":"                   \u0027 networks.\u0027))"},{"line_number":179,"context_line":"        utils.add_boolean_argument("},{"line_number":180,"context_line":"            parser,"},{"line_number":181,"context_line":"            \u0027--vlan-transparent\u0027,"}],"source_content_type":"text/x-python","patch_set":21,"id":"1a122d0e_b7a57205","line":178,"range":{"start_line":178,"start_character":20,"end_line":178,"end_character":21},"updated":"2016-04-25 12:13:39.000000000","message":"Move space to the end of the previous line.","commit_id":"0a83565c05b4558617521ccc861e06c263a14302"},{"author":{"_account_id":17776,"name":"Reedip","email":"reedip.banerjee@gmail.com","username":"Reedip"},"change_message_id":"7b5632ad0e8ded60eeb4aed7d2d916558a67bcb8","unresolved":false,"context_lines":[{"line_number":175,"context_line":"            \u0027--provider:segmentation_id\u0027,"},{"line_number":176,"context_line":"            metavar\u003d\u0027\u003csegmentation_id\u003e\u0027,"},{"line_number":177,"context_line":"            help\u003d_(\u0027VLAN ID for VLAN networks or tunnel-id for GRE/VXLAN\u0027"},{"line_number":178,"context_line":"                   \u0027 networks.\u0027))"},{"line_number":179,"context_line":"        utils.add_boolean_argument("},{"line_number":180,"context_line":"            parser,"},{"line_number":181,"context_line":"            \u0027--vlan-transparent\u0027,"}],"source_content_type":"text/x-python","patch_set":21,"id":"7aa08908_16bea4ad","line":178,"range":{"start_line":178,"start_character":20,"end_line":178,"end_character":21},"in_reply_to":"1a122d0e_b7a57205","updated":"2016-06-14 07:05:47.000000000","message":"Done","commit_id":"0a83565c05b4558617521ccc861e06c263a14302"}],"neutronclient/neutron/v2_0/port.py":[{"author":{"_account_id":17711,"name":"Margaret Frances","email":"margaret_frances@cable.comcast.com","username":"margaret-frances"},"change_message_id":"aa0cebf6f16f7ae9f7f7f6e09b91fa127977aeca","unresolved":false,"context_lines":[{"line_number":268,"context_line":""},{"line_number":269,"context_line":"        parser.add_argument("},{"line_number":270,"context_line":"            \u0027network_id\u0027, metavar\u003d\u0027NETWORK\u0027,"},{"line_number":271,"context_line":"            help\u003d_(\u0027ID or name of the network this port belongs to.\u0027))"},{"line_number":272,"context_line":"        dns.add_dns_argument_create(parser, self.resource, \u0027name\u0027)"},{"line_number":273,"context_line":""},{"line_number":274,"context_line":"    def args2body(self, parsed_args):"}],"source_content_type":"text/x-python","patch_set":14,"id":"5aef4532_4371eddf","line":271,"range":{"start_line":271,"start_character":38,"end_line":271,"end_character":66},"updated":"2016-03-08 13:12:18.000000000","message":"\"...network to which this port belongs.\"","commit_id":"ba2fa2fa9e8af2a71c747a36cc7d2074f875d95f"},{"author":{"_account_id":17776,"name":"Reedip","email":"reedip.banerjee@gmail.com","username":"Reedip"},"change_message_id":"f291ebcecaa56d2190c9e827341953acffcc1bcb","unresolved":false,"context_lines":[{"line_number":268,"context_line":""},{"line_number":269,"context_line":"        parser.add_argument("},{"line_number":270,"context_line":"            \u0027network_id\u0027, metavar\u003d\u0027NETWORK\u0027,"},{"line_number":271,"context_line":"            help\u003d_(\u0027ID or name of the network this port belongs to.\u0027))"},{"line_number":272,"context_line":"        dns.add_dns_argument_create(parser, self.resource, \u0027name\u0027)"},{"line_number":273,"context_line":""},{"line_number":274,"context_line":"    def args2body(self, parsed_args):"}],"source_content_type":"text/x-python","patch_set":14,"id":"5aef4532_1fc0acdf","line":271,"range":{"start_line":271,"start_character":38,"end_line":271,"end_character":66},"in_reply_to":"5aef4532_4371eddf","updated":"2016-03-09 08:47:15.000000000","message":"Although not correct for Latin Grammar, prepositions are valid endpoints to English Grammar.\nI think we can continue with this statement.","commit_id":"ba2fa2fa9e8af2a71c747a36cc7d2074f875d95f"}],"neutronclient/neutron/v2_0/qos/policy.py":[{"author":{"_account_id":18573,"name":"Petri Kemppainen","email":"petri.kemppainen@nokia.com","username":"petrkemp"},"change_message_id":"f03a0503e4e174f48eac18b17bbd5428918364d4","unresolved":false,"context_lines":[{"line_number":30,"context_line":"    def add_arguments_qos_policy(self, parser):"},{"line_number":31,"context_line":"        qos_policy_args \u003d parser.add_mutually_exclusive_group()"},{"line_number":32,"context_line":"        qos_policy_args.add_argument("},{"line_number":33,"context_line":"            \u0027--qos-policy\u0027,"},{"line_number":34,"context_line":"            help\u003d_(\u0027ID or name of the  QoS policy which \u0027"},{"line_number":35,"context_line":"                   \u0027is attached to the resource.\u0027))"},{"line_number":36,"context_line":"        return qos_policy_args"}],"source_content_type":"text/x-python","patch_set":5,"id":"7a740942_dbf25f06","line":33,"updated":"2015-12-10 12:59:15.000000000","message":"remove extra space between the and QoS","commit_id":"3893a9a1a2d8ae0a05043fbb0dcac6b82acba57f"},{"author":{"_account_id":17711,"name":"Margaret Frances","email":"margaret_frances@cable.comcast.com","username":"margaret-frances"},"change_message_id":"aa0cebf6f16f7ae9f7f7f6e09b91fa127977aeca","unresolved":false,"context_lines":[{"line_number":31,"context_line":"        qos_policy_args \u003d parser.add_mutually_exclusive_group()"},{"line_number":32,"context_line":"        qos_policy_args.add_argument("},{"line_number":33,"context_line":"            \u0027--qos-policy\u0027,"},{"line_number":34,"context_line":"            help\u003d_(\u0027ID or name of the QoS policy which \u0027"},{"line_number":35,"context_line":"                   \u0027is attached to the resource.\u0027))"},{"line_number":36,"context_line":"        return qos_policy_args"},{"line_number":37,"context_line":""}],"source_content_type":"text/x-python","patch_set":14,"id":"5aef4532_234b0189","line":34,"range":{"start_line":34,"start_character":49,"end_line":34,"end_character":54},"updated":"2016-03-08 13:12:18.000000000","message":"s/which/that :)","commit_id":"ba2fa2fa9e8af2a71c747a36cc7d2074f875d95f"},{"author":{"_account_id":17776,"name":"Reedip","email":"reedip.banerjee@gmail.com","username":"Reedip"},"change_message_id":"f291ebcecaa56d2190c9e827341953acffcc1bcb","unresolved":false,"context_lines":[{"line_number":31,"context_line":"        qos_policy_args \u003d parser.add_mutually_exclusive_group()"},{"line_number":32,"context_line":"        qos_policy_args.add_argument("},{"line_number":33,"context_line":"            \u0027--qos-policy\u0027,"},{"line_number":34,"context_line":"            help\u003d_(\u0027ID or name of the QoS policy which \u0027"},{"line_number":35,"context_line":"                   \u0027is attached to the resource.\u0027))"},{"line_number":36,"context_line":"        return qos_policy_args"},{"line_number":37,"context_line":""}],"source_content_type":"text/x-python","patch_set":14,"id":"5aef4532_5fba346f","line":34,"range":{"start_line":34,"start_character":49,"end_line":34,"end_character":54},"in_reply_to":"5aef4532_234b0189","updated":"2016-03-09 08:47:15.000000000","message":"Done","commit_id":"ba2fa2fa9e8af2a71c747a36cc7d2074f875d95f"},{"author":{"_account_id":22314,"name":"Ryan Selden","email":"seldenr@icloud.com","username":"seldenr"},"change_message_id":"1196923eff2a7e44370f5a4d9175f13bd1358fc1","unresolved":false,"context_lines":[{"line_number":31,"context_line":"        qos_policy_args \u003d parser.add_mutually_exclusive_group()"},{"line_number":32,"context_line":"        qos_policy_args.add_argument("},{"line_number":33,"context_line":"            \u0027--qos-policy\u0027,"},{"line_number":34,"context_line":"            help\u003d_(\u0027ID or name of the QoS policy that \u0027"},{"line_number":35,"context_line":"                   \u0027is attached to the resource.\u0027))"},{"line_number":36,"context_line":"        return qos_policy_args"},{"line_number":37,"context_line":""}],"source_content_type":"text/x-python","patch_set":23,"id":"3aaa91ec_5052d713","line":34,"range":{"start_line":34,"start_character":20,"end_line":34,"end_character":36},"updated":"2016-06-30 20:09:29.000000000","message":"Doesn\u0027t this imply that the QoS policy is already attached? The previous version implies that it *will* be attached. Not sure these two actually mean the same thing...","commit_id":"8fddbba75a92c5e64b66c823d3870bd96baf3f3b"}],"neutronclient/neutron/v2_0/router.py":[{"author":{"_account_id":18573,"name":"Petri Kemppainen","email":"petri.kemppainen@nokia.com","username":"petrkemp"},"change_message_id":"f03a0503e4e174f48eac18b17bbd5428918364d4","unresolved":false,"context_lines":[{"line_number":94,"context_line":""},{"line_number":95,"context_line":"    def add_known_arguments(self, parser):"},{"line_number":96,"context_line":"        parser.add_argument("},{"line_number":97,"context_line":"            \u0027--name\u0027,"},{"line_number":98,"context_line":"            help\u003d_(\u0027Updated Name of the router.\u0027))"},{"line_number":99,"context_line":"        utils.add_boolean_argument("},{"line_number":100,"context_line":"            parser, \u0027--admin-state-up\u0027, dest\u003d\u0027admin_state\u0027,"}],"source_content_type":"text/x-python","patch_set":5,"id":"7a740942_fb439b11","line":97,"updated":"2015-12-10 12:59:15.000000000","message":"Name --\u003ename","commit_id":"3893a9a1a2d8ae0a05043fbb0dcac6b82acba57f"},{"author":{"_account_id":18573,"name":"Petri Kemppainen","email":"petri.kemppainen@nokia.com","username":"petrkemp"},"change_message_id":"f03a0503e4e174f48eac18b17bbd5428918364d4","unresolved":false,"context_lines":[{"line_number":98,"context_line":"            help\u003d_(\u0027Updated Name of the router.\u0027))"},{"line_number":99,"context_line":"        utils.add_boolean_argument("},{"line_number":100,"context_line":"            parser, \u0027--admin-state-up\u0027, dest\u003d\u0027admin_state\u0027,"},{"line_number":101,"context_line":"            help\u003d_(\u0027Specify the administrative state of the router\u0027"},{"line_number":102,"context_line":"                   \u0027 (True meaning \"Up\").\u0027))"},{"line_number":103,"context_line":"        utils.add_boolean_argument("},{"line_number":104,"context_line":"            parser, \u0027--admin_state_up\u0027, dest\u003d\u0027admin_state\u0027,"}],"source_content_type":"text/x-python","patch_set":5,"id":"7a740942_fb5abbf1","line":101,"updated":"2015-12-10 12:59:15.000000000","message":"Move space to the end of the previous line","commit_id":"3893a9a1a2d8ae0a05043fbb0dcac6b82acba57f"},{"author":{"_account_id":17711,"name":"Margaret Frances","email":"margaret_frances@cable.comcast.com","username":"margaret-frances"},"change_message_id":"aa0cebf6f16f7ae9f7f7f6e09b91fa127977aeca","unresolved":false,"context_lines":[{"line_number":103,"context_line":"        utils.add_boolean_argument("},{"line_number":104,"context_line":"            parser, \u0027--admin-state-up\u0027, dest\u003d\u0027admin_state\u0027,"},{"line_number":105,"context_line":"            help\u003d_(\u0027Specify the administrative state of the router \u0027"},{"line_number":106,"context_line":"                   \u0027(True meaning \"Up\").\u0027))"},{"line_number":107,"context_line":"        utils.add_boolean_argument("},{"line_number":108,"context_line":"            parser, \u0027--admin_state_up\u0027, dest\u003d\u0027admin_state\u0027,"},{"line_number":109,"context_line":"            help\u003dargparse.SUPPRESS)"}],"source_content_type":"text/x-python","patch_set":14,"id":"5aef4532_c354dd22","line":106,"range":{"start_line":106,"start_character":21,"end_line":106,"end_character":38},"updated":"2016-03-08 13:12:18.000000000","message":"Suggest \"(where True means \"up\").","commit_id":"ba2fa2fa9e8af2a71c747a36cc7d2074f875d95f"},{"author":{"_account_id":18573,"name":"Petri Kemppainen","email":"petri.kemppainen@nokia.com","username":"petrkemp"},"change_message_id":"a676466fd004c960002683056d2efd8c7d15d7a4","unresolved":false,"context_lines":[{"line_number":117,"context_line":"        utils.add_boolean_argument("},{"line_number":118,"context_line":"            parser, \u0027--distributed\u0027, dest\u003d\u0027distributed\u0027,"},{"line_number":119,"context_line":"            help\u003d_(\u0027True means this router should operate in\u0027"},{"line_number":120,"context_line":"                   \u0027 distributed mode.\u0027))"},{"line_number":121,"context_line":"        routes_group \u003d parser.add_mutually_exclusive_group()"},{"line_number":122,"context_line":"        routes_group.add_argument("},{"line_number":123,"context_line":"            \u0027--route\u0027, metavar\u003d\u0027destination\u003dCIDR,nexthop\u003dIP_ADDR\u0027,"}],"source_content_type":"text/x-python","patch_set":21,"id":"1a122d0e_57d33e5f","line":120,"range":{"start_line":120,"start_character":20,"end_line":120,"end_character":21},"updated":"2016-04-25 12:13:39.000000000","message":"Move space to the end of the previous line.","commit_id":"0a83565c05b4558617521ccc861e06c263a14302"},{"author":{"_account_id":17776,"name":"Reedip","email":"reedip.banerjee@gmail.com","username":"Reedip"},"change_message_id":"7b5632ad0e8ded60eeb4aed7d2d916558a67bcb8","unresolved":false,"context_lines":[{"line_number":117,"context_line":"        utils.add_boolean_argument("},{"line_number":118,"context_line":"            parser, \u0027--distributed\u0027, dest\u003d\u0027distributed\u0027,"},{"line_number":119,"context_line":"            help\u003d_(\u0027True means this router should operate in\u0027"},{"line_number":120,"context_line":"                   \u0027 distributed mode.\u0027))"},{"line_number":121,"context_line":"        routes_group \u003d parser.add_mutually_exclusive_group()"},{"line_number":122,"context_line":"        routes_group.add_argument("},{"line_number":123,"context_line":"            \u0027--route\u0027, metavar\u003d\u0027destination\u003dCIDR,nexthop\u003dIP_ADDR\u0027,"}],"source_content_type":"text/x-python","patch_set":21,"id":"7aa08908_56e2ecd5","line":120,"range":{"start_line":120,"start_character":20,"end_line":120,"end_character":21},"in_reply_to":"1a122d0e_57d33e5f","updated":"2016-06-14 07:05:47.000000000","message":"Done","commit_id":"0a83565c05b4558617521ccc861e06c263a14302"}],"neutronclient/neutron/v2_0/securitygroup.py":[{"author":{"_account_id":18573,"name":"Petri Kemppainen","email":"petri.kemppainen@nokia.com","username":"petrkemp"},"change_message_id":"f03a0503e4e174f48eac18b17bbd5428918364d4","unresolved":false,"context_lines":[{"line_number":339,"context_line":"            help\u003dargparse.SUPPRESS)"},{"line_number":340,"context_line":"        parser.add_argument("},{"line_number":341,"context_line":"            \u0027--remote-group-id\u0027, metavar\u003d\u0027REMOTE_GROUP\u0027,"},{"line_number":342,"context_line":"            help\u003d_(\u0027ID or name of the remote security group\u0027"},{"line_number":343,"context_line":"                   \u0027 to which the rule is applied.\u0027))"},{"line_number":344,"context_line":"        parser.add_argument("},{"line_number":345,"context_line":"            \u0027--remote_group_id\u0027,"}],"source_content_type":"text/x-python","patch_set":5,"id":"7a740942_fb287b35","line":342,"updated":"2015-12-10 12:59:15.000000000","message":"Move space to the end of the previous line","commit_id":"3893a9a1a2d8ae0a05043fbb0dcac6b82acba57f"},{"author":{"_account_id":841,"name":"Akihiro Motoki","email":"amotoki@gmail.com","username":"amotoki"},"change_message_id":"b1da0d9aa31a2fe61f92a7b3052e79fc619730a1","unresolved":false,"context_lines":[{"line_number":123,"context_line":"    def add_known_arguments(self, parser):"},{"line_number":124,"context_line":"        parser.add_argument("},{"line_number":125,"context_line":"            \u0027name\u0027, metavar\u003d\u0027NAME\u0027,"},{"line_number":126,"context_line":"            help\u003d_(\u0027Name of the security group.\u0027))"},{"line_number":127,"context_line":"        parser.add_argument("},{"line_number":128,"context_line":"            \u0027--description\u0027,"},{"line_number":129,"context_line":"            help\u003d_(\u0027Description of the security group.\u0027))"}],"source_content_type":"text/x-python","patch_set":6,"id":"7a740942_05a8128b","line":126,"updated":"2015-12-13 00:19:12.000000000","message":"isn\u0027t \"to be created\" required for consistency?","commit_id":"8817f540786000e8ee21550b3fe5c72322e42da9"},{"author":{"_account_id":17776,"name":"Reedip","email":"reedip.banerjee@gmail.com","username":"Reedip"},"change_message_id":"ccd0d969d5ab2202145d71e13cd746443d0595c1","unresolved":false,"context_lines":[{"line_number":123,"context_line":"    def add_known_arguments(self, parser):"},{"line_number":124,"context_line":"        parser.add_argument("},{"line_number":125,"context_line":"            \u0027name\u0027, metavar\u003d\u0027NAME\u0027,"},{"line_number":126,"context_line":"            help\u003d_(\u0027Name of the security group.\u0027))"},{"line_number":127,"context_line":"        parser.add_argument("},{"line_number":128,"context_line":"            \u0027--description\u0027,"},{"line_number":129,"context_line":"            help\u003d_(\u0027Description of the security group.\u0027))"}],"source_content_type":"text/x-python","patch_set":6,"id":"7a740942_ea47c7b9","line":126,"in_reply_to":"7a740942_05a8128b","updated":"2015-12-14 02:11:33.000000000","message":"Done","commit_id":"8817f540786000e8ee21550b3fe5c72322e42da9"},{"author":{"_account_id":18573,"name":"Petri Kemppainen","email":"petri.kemppainen@nokia.com","username":"petrkemp"},"change_message_id":"46ce3de19175d74d1cbbaeab0eaac29487e92a20","unresolved":false,"context_lines":[{"line_number":320,"context_line":"            \u0027--protocol\u0027,"},{"line_number":321,"context_line":"            help\u003d_(\u0027Protocol of packet. Allowed values are \u0027"},{"line_number":322,"context_line":"                   \u0027[icmp, icmpv6, tcp, udp] and \u0027"},{"line_number":323,"context_line":"                   \u0027integer representations [0-255]\u0027))"},{"line_number":324,"context_line":"        parser.add_argument("},{"line_number":325,"context_line":"            \u0027--port-range-min\u0027,"},{"line_number":326,"context_line":"            help\u003d_(\u0027Starting port range. For ICMP it is type.\u0027))"}],"source_content_type":"text/x-python","patch_set":14,"id":"5aef4532_2f5bed2f","line":323,"updated":"2016-03-08 09:18:38.000000000","message":"Add period to the end of the line","commit_id":"ba2fa2fa9e8af2a71c747a36cc7d2074f875d95f"},{"author":{"_account_id":17776,"name":"Reedip","email":"reedip.banerjee@gmail.com","username":"Reedip"},"change_message_id":"f291ebcecaa56d2190c9e827341953acffcc1bcb","unresolved":false,"context_lines":[{"line_number":320,"context_line":"            \u0027--protocol\u0027,"},{"line_number":321,"context_line":"            help\u003d_(\u0027Protocol of packet. Allowed values are \u0027"},{"line_number":322,"context_line":"                   \u0027[icmp, icmpv6, tcp, udp] and \u0027"},{"line_number":323,"context_line":"                   \u0027integer representations [0-255]\u0027))"},{"line_number":324,"context_line":"        parser.add_argument("},{"line_number":325,"context_line":"            \u0027--port-range-min\u0027,"},{"line_number":326,"context_line":"            help\u003d_(\u0027Starting port range. For ICMP it is type.\u0027))"}],"source_content_type":"text/x-python","patch_set":14,"id":"5aef4532_9fcafc6c","line":323,"in_reply_to":"5aef4532_2f5bed2f","updated":"2016-03-09 08:47:15.000000000","message":"Done","commit_id":"ba2fa2fa9e8af2a71c747a36cc7d2074f875d95f"}]}