)]}'
{"tempest/lib/common/rest_client.py":[{"author":{"_account_id":6167,"name":"Ken\u0027ichi Ohmichi","email":"ken1ohmichi@gmail.com","username":"oomichi"},"change_message_id":"4cf12783561dc92b31608df88ec21a86ccd04630","unresolved":false,"context_lines":[{"line_number":316,"context_line":"        \"\"\""},{"line_number":317,"context_line":"        return self.request(\u0027PATCH\u0027, url, extra_headers, headers, body)"},{"line_number":318,"context_line":""},{"line_number":319,"context_line":"    def put(self, url, body\u003dNone, headers\u003dNone, extra_headers\u003dFalse,"},{"line_number":320,"context_line":"            chunked\u003dFalse):"},{"line_number":321,"context_line":"        \"\"\"Send a HTTP PUT request using keystone service catalog and auth"},{"line_number":322,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"9f436f4f_30cdbd84","line":319,"range":{"start_line":319,"start_character":23,"end_line":319,"end_character":32},"updated":"2017-08-16 18:32:24.000000000","message":"PUT needs to contain a body(\"request message payload\") from https://tools.ietf.org/html/rfc7231#section-4.3.4\n\n 4.3.4.  PUT\n\n   The PUT method requests that the state of the target resource be\n   created or replaced with the state defined by the representation\n   enclosed in the request message payload.\n\nThat is a reason why this method\u0027s body is not optional.\nBut I can understand neutron team wants to have such API from API design viewpoint.\nNova also has API design issues, so I am fine to make the body optional for testing.\n\nJust one thing from me is that this patch changes the stable library interface which we define. So please add a release note to explain this change for users.\nMasayuki\u0027s point also is good for me.","commit_id":"1b6867a04e983f3de2c3583d38ae75c8c42a28ec"},{"author":{"_account_id":8655,"name":"Jakub Libosvar","email":"libosvar@redhat.com","username":"jlibosva"},"change_message_id":"bb2010f1cbd5c8aff58a8082d3c50e468310e34d","unresolved":false,"context_lines":[{"line_number":316,"context_line":"        \"\"\""},{"line_number":317,"context_line":"        return self.request(\u0027PATCH\u0027, url, extra_headers, headers, body)"},{"line_number":318,"context_line":""},{"line_number":319,"context_line":"    def put(self, url, body\u003dNone, headers\u003dNone, extra_headers\u003dFalse,"},{"line_number":320,"context_line":"            chunked\u003dFalse):"},{"line_number":321,"context_line":"        \"\"\"Send a HTTP PUT request using keystone service catalog and auth"},{"line_number":322,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"9f436f4f_900c6994","line":319,"range":{"start_line":319,"start_character":23,"end_line":319,"end_character":32},"in_reply_to":"9f436f4f_30cdbd84","updated":"2017-08-16 18:41:20.000000000","message":"Aha, thanks for explanation. The feature requesting this patch is still not in-tree and will slip to the Queens - which mean we can re-define the API to correctly reflect mentioned rfc.","commit_id":"1b6867a04e983f3de2c3583d38ae75c8c42a28ec"}]}