)]}'
{"tempest/services/identity/v3/json/inherited_roles_client.py":[{"author":{"_account_id":6167,"name":"Ken\u0027ichi Ohmichi","email":"ken1ohmichi@gmail.com","username":"oomichi"},"change_message_id":"eadd63aa0fef4edb4d508ce8a56e28b7090bbb38","unresolved":false,"context_lines":[{"line_number":20,"context_line":"class InheritedRolesClient(rest_client.RestClient):"},{"line_number":21,"context_line":"    api_version \u003d \"v3\""},{"line_number":22,"context_line":""},{"line_number":23,"context_line":"    def create_inherited_role_on_domains_user("},{"line_number":24,"context_line":"            self, domain_id, user_id, role_id):"},{"line_number":25,"context_line":"        \"\"\"Assigns a role to a user on projects owned by a domain.\"\"\""},{"line_number":26,"context_line":"        resp, body \u003d self.put("}],"source_content_type":"text/x-python","patch_set":4,"id":"9a89bdaa_1cf17601","line":23,"range":{"start_line":23,"start_character":8,"end_line":23,"end_character":15},"updated":"2016-09-07 18:02:41.000000000","message":"This is a PUT method, not a POST method.\nIn general, we are defining \"update_\" method for PUT methods and this is different from the other methods\u0027 naming.\nAt this time, I don\u0027t have so strong opinion about that because this is just assigning a role as the docstring.\nbut before merging, I\u0027d like to confirm this naming is intentional or not.","commit_id":"e20d67740478cdb422d1996077d025e5e2ffb12c"},{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"0d7a70d01ccb87c3ed815134e0c00b9464e72cc5","unresolved":false,"context_lines":[{"line_number":20,"context_line":"class InheritedRolesClient(rest_client.RestClient):"},{"line_number":21,"context_line":"    api_version \u003d \"v3\""},{"line_number":22,"context_line":""},{"line_number":23,"context_line":"    def create_inherited_role_on_domains_user("},{"line_number":24,"context_line":"            self, domain_id, user_id, role_id):"},{"line_number":25,"context_line":"        \"\"\"Assigns a role to a user on projects owned by a domain.\"\"\""},{"line_number":26,"context_line":"        resp, body \u003d self.put("}],"source_content_type":"text/x-python","patch_set":4,"id":"9a89bdaa_d0b285e4","line":23,"range":{"start_line":23,"start_character":8,"end_line":23,"end_character":15},"in_reply_to":"9a89bdaa_1cf17601","updated":"2016-09-08 00:53:11.000000000","message":"yea, that\u0027s right. Actually update_ is not much suitable here as it create/assign new role that\u0027s why i choose create_ in all such cases.","commit_id":"e20d67740478cdb422d1996077d025e5e2ffb12c"}]}