)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"01b7b16e9453d54f9e2cc4a161a39a34ea48fbba","unresolved":false,"context_lines":[{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Glance returns 200 the success status code on \"Delete all tag"},{"line_number":10,"context_line":"definitions\" API today, but the api-ref shows 204 as the code."},{"line_number":11,"context_line":"Tempest also checks 200 as the success status code since the"},{"line_number":12,"context_line":"commit I92b2c58d11828eafd9f876a46581279bb9b26e3d"},{"line_number":13,"context_line":"This patch fixes the api-ref."},{"line_number":14,"context_line":""},{"line_number":15,"context_line":"Change-Id: Ie42ab08f61f880a13ca4f758243babc205f2629a"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"3a461143_5fa0cd9f","line":12,"range":{"start_line":11,"start_character":0,"end_line":12,"end_character":48},"updated":"2017-01-26 21:49:14.000000000","message":"I have to say, this is pretty disingenuous.\n\nThe tempest patch was merged on January 19, and as you well know, there\u0027s been a patch up for Glance since January 13 that we\u0027ve held open so that we could have a civilized discussion about the correct way to fix this bug.  So it was pretty underhanded for your team to go ahead and merge that change, and now claim that because there\u0027s a tempest test, this patch should be approved.","commit_id":"27f156c44944290ac7de9a5108931cca2a4aec8b"},{"author":{"_account_id":6167,"name":"Ken\u0027ichi Ohmichi","email":"ken1ohmichi@gmail.com","username":"oomichi"},"change_message_id":"358a739f9f8e06f44367ee63fbd3f835b4ebb109","unresolved":false,"context_lines":[{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Glance returns 200 the success status code on \"Delete all tag"},{"line_number":10,"context_line":"definitions\" API today, but the api-ref shows 204 as the code."},{"line_number":11,"context_line":"Tempest also checks 200 as the success status code since the"},{"line_number":12,"context_line":"commit I92b2c58d11828eafd9f876a46581279bb9b26e3d"},{"line_number":13,"context_line":"This patch fixes the api-ref."},{"line_number":14,"context_line":""},{"line_number":15,"context_line":"Change-Id: Ie42ab08f61f880a13ca4f758243babc205f2629a"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"3a461143_40b0e943","line":12,"range":{"start_line":11,"start_character":0,"end_line":12,"end_character":48},"in_reply_to":"3a461143_5fa0cd9f","updated":"2017-01-27 00:45:08.000000000","message":"We must keep the API stability as the first mission, and your direction is different from our understanding and guideline.\nIn addition, the Tempest patch itself was proposed last November, that mean we continue verifying the API stability 2 months before the discussion. I didn\u0027t have any objections about merging that.\n\nWe really need to block the patch which changes success status code now before making a consensus we accept the change, even if we are considered as disingenuous.","commit_id":"27f156c44944290ac7de9a5108931cca2a4aec8b"}],"api-ref/source/v2/metadefs-namespaces-tags.inc":[{"author":{"_account_id":782,"name":"John Garbutt","email":"john@johngarbutt.com","username":"johngarbutt"},"change_message_id":"67742fa911c2ff79233e0dc6f9023cae9533a625","unresolved":false,"context_lines":[{"line_number":283,"context_line":"When you successfully delete the tags from a namespace, the"},{"line_number":284,"context_line":"response is empty and the response code is ``200``."},{"line_number":285,"context_line":""},{"line_number":286,"context_line":"Normal response codes: 200"},{"line_number":287,"context_line":""},{"line_number":288,"context_line":"Error response codes: 403, 404"},{"line_number":289,"context_line":""}],"source_content_type":"text/x-c++src","patch_set":1,"id":"fa31d9ce_f8004f03","line":286,"updated":"2017-02-14 18:42:18.000000000","message":"FWIW, this is what nova decided a few years back:\nhttp://docs.openstack.org/developer/nova/policies.html#rest-apis\n\nIt was mostly based from this thinking:\nhttps://governance.openstack.org/tc/reference/principles.html#openstack-primarily-produces-software\n\nI think the docs should correctly say what the code does, given thats what the API is right now, and the current state for sure.\n\nNow the debate on changing this status code, for me is more about agreeing why this change makes sense. To sync the code with docs doesn\u0027t make any sense to me. However the aim of a better API and improving consistency, is a great reason to start a debate of if this change is worth the possible pain for breaking existing API users. But thats the other patch, and I added comments on that patch about my thoughts on that.","commit_id":"27f156c44944290ac7de9a5108931cca2a4aec8b"}]}