)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":6816,"name":"Jesse Pretorius","email":"jesse@odyssey4.me","username":"jesse-pretorius"},"change_message_id":"6bb055f81e1258eefb13b98dd508ba961290ab3d","unresolved":false,"context_lines":[{"line_number":7,"context_line":"Add support for Identity Providers"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Change-Id: I7f0ea852fd03ecca64e0387e5019335a4664aba6"},{"line_number":10,"context_line":"Requires: https://review.opendev.org/714120"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"df33271e_db88f820","line":10,"range":{"start_line":10,"start_character":0,"end_line":10,"end_character":8},"updated":"2020-03-23 14:22:20.000000000","message":"I think this may need to be \u0027Depends-On\u0027","commit_id":"39758682040b678ddb5eaa38b5e590d197fb1fdd"}],"openstack/identity/v3/identity_provider.py":[{"author":{"_account_id":8482,"name":"Colleen Murphy","email":"colleen@gazlene.net","username":"krinkle"},"change_message_id":"50a7691cda8280bcc70e1895ee3da39b7fa2961c","unresolved":false,"context_lines":[{"line_number":25,"context_line":"    allow_delete \u003d True"},{"line_number":26,"context_line":"    allow_list \u003d True"},{"line_number":27,"context_line":"    create_method \u003d \u0027PUT\u0027"},{"line_number":28,"context_line":"    create_exclude_id_from_body \u003d True"},{"line_number":29,"context_line":"    commit_method \u003d \u0027PATCH\u0027"},{"line_number":30,"context_line":""},{"line_number":31,"context_line":"    _query_mapping \u003d resource.QueryParameters("}],"source_content_type":"text/x-python","patch_set":3,"id":"df33271e_c2a3e54e","line":28,"range":{"start_line":28,"start_character":4,"end_line":28,"end_character":38},"updated":"2020-03-23 18:30:42.000000000","message":"not sure how widespread this is but maybe not including the ID in the body should be the general case for when the create method is PUT, or maybe always? it\u0027s kind of a weird quirk of this API that it allows you to set the ID explicitly","commit_id":"8f7ec9221af954748f9b0d5af3cf18475eac0493"},{"author":{"_account_id":14369,"name":"Mark Chappell","email":"mchappel@redhat.com","username":"tremble"},"change_message_id":"bc56409906a6f8981b2f98061a988e27639a0c03","unresolved":false,"context_lines":[{"line_number":25,"context_line":"    allow_delete \u003d True"},{"line_number":26,"context_line":"    allow_list \u003d True"},{"line_number":27,"context_line":"    create_method \u003d \u0027PUT\u0027"},{"line_number":28,"context_line":"    create_exclude_id_from_body \u003d True"},{"line_number":29,"context_line":"    commit_method \u003d \u0027PATCH\u0027"},{"line_number":30,"context_line":""},{"line_number":31,"context_line":"    _query_mapping \u003d resource.QueryParameters("}],"source_content_type":"text/x-python","patch_set":3,"id":"df33271e_fda96bdf","line":28,"range":{"start_line":28,"start_character":4,"end_line":28,"end_character":38},"in_reply_to":"df33271e_c2a3e54e","updated":"2020-03-23 19:48:04.000000000","message":"What I found interesting was that the API docs explicitly talk about not sending ID when create_method \u003d \u0027POST\u0027 (giving it as an example that throws a 400).\n\nI didn\u0027t want to switch the default behaviour since it\u0027s explicitly tested for and neither mordred nor I could see why it would be that way, which left us worried about some weird side effect...","commit_id":"8f7ec9221af954748f9b0d5af3cf18475eac0493"}]}