)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":4393,"name":"Dan Smith","email":"dms@danplanet.com","username":"danms"},"change_message_id":"44fe6ecaecfcbffb1cffe32e05f95a92ffbfb407","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"c9eb9245_d79a9407","updated":"2022-06-28 13:57:15.000000000","message":"Normally I wouldn\u0027t think this would be a super important change. Even though the justification from the recently-changed PUT operation applies here, the case for doing this is much weaker for DELETE, IMHO. The 200 PUT was telling the user something untrue - that the cache operation was done, and converting to 202 was important signaling. For DELETE, 200-\u003e204 doesn\u0027t signal anything other than \"I\u0027m not returning the object to you\" which is not nearly as important.\n\nHowever, here are some points in favor of doing this:\n\n- The spec says 204\n- The API-REF says 204\n- All the other delete operations in glance are 204\n- It\u0027s admin-only so not widely-used\n- It\u0027s less than a cycle old\n\nSo I\u0027m kinda meh about it, and wish we had microversions to cleanly signal this. However, we did bump the image service version for the PUT change. Are we saying this is covered under that as well because we only need one bump per release or what?\n\nIf others are okay with this then I am too, but let\u0027s make sure.","commit_id":"36c753fcca98c587a235bddff44d76eb4c2984cc"},{"author":{"_account_id":4393,"name":"Dan Smith","email":"dms@danplanet.com","username":"danms"},"change_message_id":"e6ebc8877b1ee7cd7a2f6e1da9fc139f8f717212","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"278c8963_c0a4c9af","updated":"2022-06-28 17:36:49.000000000","message":"Discussion on IRC resulted in a decision to proceed with this while we have the opportunity.","commit_id":"a71f7ba44e57a7945addb82afd2a19962d5c04cc"},{"author":{"_account_id":5202,"name":"Erno Kuvaja","email":"jokke@usr.fi","username":"jokke"},"change_message_id":"d444dd1f91acb272837a8ad92b00c37fee7673ea","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"609aaf70_91ad283a","updated":"2022-06-29 10:07:51.000000000","message":"LGTM","commit_id":"a71f7ba44e57a7945addb82afd2a19962d5c04cc"},{"author":{"_account_id":4393,"name":"Dan Smith","email":"dms@danplanet.com","username":"danms"},"change_message_id":"bb6b22dff5fdb2c2a6751fff10d7b6874b68f31e","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"d49d1e3e_2afdfd87","updated":"2022-06-29 14:13:32.000000000","message":"Sorry, I set RP instead of CR :D","commit_id":"a71f7ba44e57a7945addb82afd2a19962d5c04cc"},{"author":{"_account_id":9303,"name":"Abhishek Kekane","email":"akekane@redhat.com","username":"abhishekkekane"},"change_message_id":"d88cf05865ef6d237f12b2df0cc498a4d2a6f5a3","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"edc8e509_f1f22d2c","updated":"2022-06-29 17:00:57.000000000","message":"recheck cinder failure","commit_id":"a71f7ba44e57a7945addb82afd2a19962d5c04cc"},{"author":{"_account_id":9303,"name":"Abhishek Kekane","email":"akekane@redhat.com","username":"abhishekkekane"},"change_message_id":"f3b6b255e08083196fba8203f4055ca9e0ae90bf","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"f63e0eb6_7d7020d3","updated":"2022-06-29 15:40:50.000000000","message":"recheck post failure","commit_id":"a71f7ba44e57a7945addb82afd2a19962d5c04cc"}]}