)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"cb7d3572c78806b1a31b9082ee4f82850dcc7894","unresolved":false,"context_lines":[{"line_number":4,"context_line":"Commit:     Rodolfo Alonso Hernandez \u003crodolfo.alonso.hernandez@intel.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2017-06-01 10:44:02 +0000"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"\"--type\" should be required option in \"network qos rule create\""},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"when we create a network qos rule we need specify"},{"line_number":10,"context_line":"the type so that we can call the corresponding"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":5,"id":"df140735_117d92e6","line":7,"range":{"start_line":7,"start_character":0,"end_line":7,"end_character":63},"updated":"2017-06-01 10:47:06.000000000","message":"nit: \"type\" is already a required option. Should be:\n\n\u0027Enforce \"type\" parameter in \"qos rule create\" using parser\"","commit_id":"cdc54dae25662beff187a79eb3e5bc49bfa52841"},{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"cb7d3572c78806b1a31b9082ee4f82850dcc7894","unresolved":false,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"\"--type\" should be required option in \"network qos rule create\""},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"when we create a network qos rule we need specify"},{"line_number":10,"context_line":"the type so that we can call the corresponding"},{"line_number":11,"context_line":"API, so type is required and we need to check it"},{"line_number":12,"context_line":"when get the args"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":5,"id":"df140735_51442ab7","line":9,"updated":"2017-06-01 10:47:06.000000000","message":"nit: use impersonal senteces\n\n\"When a network qos rule is created...\"","commit_id":"cdc54dae25662beff187a79eb3e5bc49bfa52841"}],"openstackclient/network/v2/network_qos_rule.py":[{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"84a561754bf0ca0e334861d3c52d8c3f949eb604","unresolved":false,"context_lines":[{"line_number":78,"context_line":"                   {\u0027rule_id\u0027: parsed_args.id})"},{"line_number":79,"context_line":"            raise exceptions.CommandError(msg)"},{"line_number":80,"context_line":"    else:"},{"line_number":81,"context_line":"        if not parsed_args.type:"},{"line_number":82,"context_line":"            msg \u003d _(\u0027\"Create\" rule command requires argument \"type\"\u0027)"},{"line_number":83,"context_line":"            raise exceptions.CommandError(msg)"},{"line_number":84,"context_line":"        rule_type \u003d parsed_args.type"},{"line_number":85,"context_line":"    if parsed_args.max_kbps is not None:"},{"line_number":86,"context_line":"        attrs[\u0027max_kbps\u0027] \u003d parsed_args.max_kbps"}],"source_content_type":"text/x-python","patch_set":1,"id":"bff0334d_858deb32","line":83,"range":{"start_line":81,"start_character":0,"end_line":83,"end_character":46},"updated":"2017-04-07 09:38:27.000000000","message":"You can now remove this section","commit_id":"b3d56394797a842bfafbe562055fc613de099253"},{"author":{"_account_id":21514,"name":"Huanxuan Ao","email":"aohuanxuan@163.com","username":"aohuanxuan"},"change_message_id":"036d7038669dadbbaaf4fa15517a8ee8b53d574b","unresolved":false,"context_lines":[{"line_number":78,"context_line":"                   {\u0027rule_id\u0027: parsed_args.id})"},{"line_number":79,"context_line":"            raise exceptions.CommandError(msg)"},{"line_number":80,"context_line":"    else:"},{"line_number":81,"context_line":"        if not parsed_args.type:"},{"line_number":82,"context_line":"            msg \u003d _(\u0027\"Create\" rule command requires argument \"type\"\u0027)"},{"line_number":83,"context_line":"            raise exceptions.CommandError(msg)"},{"line_number":84,"context_line":"        rule_type \u003d parsed_args.type"},{"line_number":85,"context_line":"    if parsed_args.max_kbps is not None:"},{"line_number":86,"context_line":"        attrs[\u0027max_kbps\u0027] \u003d parsed_args.max_kbps"}],"source_content_type":"text/x-python","patch_set":1,"id":"bff0334d_a4af7c7b","line":83,"range":{"start_line":81,"start_character":0,"end_line":83,"end_character":46},"in_reply_to":"bff0334d_858deb32","updated":"2017-04-11 04:53:21.000000000","message":"Done, thanks","commit_id":"b3d56394797a842bfafbe562055fc613de099253"}]}
