)]}'
{"releasenotes/notes/bug-1700359-b123208d51d73ae3.yaml":[{"author":{"_account_id":11564,"name":"Chris Dent","email":"cdent@anticdent.org","username":"chdent"},"change_message_id":"2a943619734869f711999bab7591435f5489def2","unresolved":false,"context_lines":[{"line_number":5,"context_line":"    ``PUT /os-services/force-down`` APIs to enable, disable, or force-down a"},{"line_number":6,"context_line":"    service will now only work with *nova-compute* services. If you are using"},{"line_number":7,"context_line":"    those APIs to try and disable a non-compute service, like nova-scheduler or"},{"line_number":8,"context_line":"    nova-conductor, those APIs will result in a 404 response."},{"line_number":9,"context_line":""},{"line_number":10,"context_line":"    There really never was a good reason to disable or enable non-compute"},{"line_number":11,"context_line":"    services anyway since that would not do anything. The nova-scheduler and"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"df3967d1_ca776c3a","line":8,"updated":"2017-08-01 21:27:43.000000000","message":"It\u0027s too late, and I wasn\u0027t there to comment, so I snoozed and lost. But the correct response would have been 400. 404 only applies to the actual URL, not something mentioned in the body.\n\nThis is, however, not at all a big deal.","commit_id":"598314a4bb53ffff2a8872bd6b290ef8d1d6c532"},{"author":{"_account_id":7166,"name":"Sylvain Bauza","email":"sbauza@redhat.com","username":"sbauza"},"change_message_id":"14f7d27e724fec0e054fae107e0bd42671a59228","unresolved":false,"context_lines":[{"line_number":5,"context_line":"    ``PUT /os-services/force-down`` APIs to enable, disable, or force-down a"},{"line_number":6,"context_line":"    service will now only work with *nova-compute* services. If you are using"},{"line_number":7,"context_line":"    those APIs to try and disable a non-compute service, like nova-scheduler or"},{"line_number":8,"context_line":"    nova-conductor, those APIs will result in a 404 response."},{"line_number":9,"context_line":""},{"line_number":10,"context_line":"    There really never was a good reason to disable or enable non-compute"},{"line_number":11,"context_line":"    services anyway since that would not do anything. The nova-scheduler and"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"df3967d1_b377f7d4","line":8,"in_reply_to":"df3967d1_0577a7d6","updated":"2017-08-02 09:51:23.000000000","message":"Well, given the involved change didn\u0027t require a microversion, it\u0027s free to just amend the API *and* that relnote in the same patch as a follow-up.\nFor the sake of consistency, since the API returns 404, let\u0027s the relnote be correct.","commit_id":"598314a4bb53ffff2a8872bd6b290ef8d1d6c532"},{"author":{"_account_id":6167,"name":"Ken\u0027ichi Ohmichi","email":"ken1ohmichi@gmail.com","username":"oomichi"},"change_message_id":"c12ae99e110a1196ad66e6c04ebed6a37b6a93a6","unresolved":false,"context_lines":[{"line_number":5,"context_line":"    ``PUT /os-services/force-down`` APIs to enable, disable, or force-down a"},{"line_number":6,"context_line":"    service will now only work with *nova-compute* services. If you are using"},{"line_number":7,"context_line":"    those APIs to try and disable a non-compute service, like nova-scheduler or"},{"line_number":8,"context_line":"    nova-conductor, those APIs will result in a 404 response."},{"line_number":9,"context_line":""},{"line_number":10,"context_line":"    There really never was a good reason to disable or enable non-compute"},{"line_number":11,"context_line":"    services anyway since that would not do anything. The nova-scheduler and"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"df3967d1_0577a7d6","line":8,"in_reply_to":"df3967d1_ca776c3a","updated":"2017-08-01 22:22:06.000000000","message":"Yeah Chris is right here. This is a common mistake related to 400/404. It would be fine to keep current reno as current behavior of API.\n\nhttps://github.com/openstack/api-wg/blob/master/guidelines/http.rst#failure-code-clarifications\n\n If a request contains a reference to a nonexistent resource\n in the body (not URI), the code should be 400 Bad Request.\n Do not use 404 NotFound","commit_id":"598314a4bb53ffff2a8872bd6b290ef8d1d6c532"}]}
