)]}'
{"api-ref/source/v2/qos.inc":[{"author":{"_account_id":9531,"name":"liuyulong","display_name":"LIU Yulong","email":"i@liuyulong.me","username":"LIU-Yulong"},"change_message_id":"f065300c1f866bc6230450e4d111c37fc89566b2","unresolved":true,"context_lines":[{"line_number":921,"context_line":""},{"line_number":922,"context_line":"Normal response codes: 202"},{"line_number":923,"context_line":""},{"line_number":924,"context_line":"Error response codes: 400, 401, 404, 501"},{"line_number":925,"context_line":""},{"line_number":926,"context_line":"Request"},{"line_number":927,"context_line":"-------"}],"source_content_type":"text/x-c++src","patch_set":1,"id":"fbaf99b4_afc2383f","line":924,"range":{"start_line":924,"start_character":37,"end_line":924,"end_character":40},"updated":"2021-06-01 02:34:38.000000000","message":"I have two questions:\n1. is there some ERROR log trace in neutron-server for this response code? If so, that is not a good thing for neutron. We should fix that in neutron.\n2. 50X means \"Server error xxx\" typically, so I don\u0027t think a 50X to user is a proper way. It\u0027s better to return \"40X\", something like this:\"Bad request, this action is not supported, and it is rejected.\"","commit_id":"c3f6029f987a22e4ed5f2ac46618b6bf435a6f9d"},{"author":{"_account_id":11975,"name":"Slawek Kaplonski","email":"skaplons@redhat.com","username":"slaweq"},"change_message_id":"d0eb5546085adbd87f90d013b1043ff15e7969f2","unresolved":true,"context_lines":[{"line_number":921,"context_line":""},{"line_number":922,"context_line":"Normal response codes: 202"},{"line_number":923,"context_line":""},{"line_number":924,"context_line":"Error response codes: 400, 401, 404, 501"},{"line_number":925,"context_line":""},{"line_number":926,"context_line":"Request"},{"line_number":927,"context_line":"-------"}],"source_content_type":"text/x-c++src","patch_set":1,"id":"d8903b47_5a088fba","line":924,"range":{"start_line":924,"start_character":37,"end_line":924,"end_character":40},"in_reply_to":"b50d2a07_82aa530e","updated":"2021-06-01 07:15:44.000000000","message":"It\u0027s exactly as Lajos said. Discussion about it is in the other review. That patch is just updating our docs to reflect what we currently really have.","commit_id":"c3f6029f987a22e4ed5f2ac46618b6bf435a6f9d"},{"author":{"_account_id":8313,"name":"Lajos Katona","display_name":"lajoskatona","email":"katonalala@gmail.com","username":"elajkat","status":"Ericsson Software Technology"},"change_message_id":"63a08d40201c2e1e3b88464802e46cec072e143d","unresolved":true,"context_lines":[{"line_number":921,"context_line":""},{"line_number":922,"context_line":"Normal response codes: 202"},{"line_number":923,"context_line":""},{"line_number":924,"context_line":"Error response codes: 400, 401, 404, 501"},{"line_number":925,"context_line":""},{"line_number":926,"context_line":"Request"},{"line_number":927,"context_line":"-------"}],"source_content_type":"text/x-c++src","patch_set":1,"id":"b50d2a07_82aa530e","line":924,"range":{"start_line":924,"start_character":37,"end_line":924,"end_character":40},"in_reply_to":"fbaf99b4_afc2383f","updated":"2021-06-01 04:20:44.000000000","message":"The related discussion from the minimum guaranteed pps spec:\nhttps://review.opendev.org/c/openstack/neutron-specs/+/785236/7/specs/xena/qos-minimum-guaranteed-packet-rate.rst#187","commit_id":"c3f6029f987a22e4ed5f2ac46618b6bf435a6f9d"}]}