)]}'
{"tempest/common/cred_provider.py":[{"author":{"_account_id":6167,"name":"Ken\u0027ichi Ohmichi","email":"ken1ohmichi@gmail.com","username":"oomichi"},"change_message_id":"94535c5e1ba3ecd0edd6e5a8db096e98a64118d9","unresolved":false,"context_lines":[{"line_number":108,"context_line":"        :param network_resources: Network resources required for the"},{"line_number":109,"context_line":"                                  credentials"},{"line_number":110,"context_line":"        :param credentials_domain: Domain credentials belong to"},{"line_number":111,"context_line":"        :param admin_role name of the role of the admin account"},{"line_number":112,"context_line":"        \"\"\""},{"line_number":113,"context_line":"        self.identity_version \u003d identity_version"},{"line_number":114,"context_line":"        self.name \u003d name or \"test_creds\""}],"source_content_type":"text/x-python","patch_set":6,"id":"5a2ca52d_2cca5aa2","line":111,"updated":"2015-10-14 04:04:06.000000000","message":"need to add \":\" and Capital N like\n\n :param admin_role: Name of the role of the admin account","commit_id":"65d8a6247e15fd4c0223914c805d3f562f49b4cc"},{"author":{"_account_id":1921,"name":"Andrea Frittoli","email":"andrea.frittoli@gmail.com","username":"andrea-frittoli"},"change_message_id":"f41f9ca132b748a6b861ba842a04e8c4a82132fc","unresolved":false,"context_lines":[{"line_number":108,"context_line":"        :param network_resources: Network resources required for the"},{"line_number":109,"context_line":"                                  credentials"},{"line_number":110,"context_line":"        :param credentials_domain: Domain credentials belong to"},{"line_number":111,"context_line":"        :param admin_role name of the role of the admin account"},{"line_number":112,"context_line":"        \"\"\""},{"line_number":113,"context_line":"        self.identity_version \u003d identity_version"},{"line_number":114,"context_line":"        self.name \u003d name or \"test_creds\""}],"source_content_type":"text/x-python","patch_set":6,"id":"5a2ca52d_3a24be0b","line":111,"in_reply_to":"5a2ca52d_2cca5aa2","updated":"2015-10-14 11:50:01.000000000","message":"Done","commit_id":"65d8a6247e15fd4c0223914c805d3f562f49b4cc"},{"author":{"_account_id":7350,"name":"Jordan Pittier","email":"jordan.pittier@gmail.com","username":"JordanP"},"change_message_id":"afc357f02ca9118132b1bf6cbdc98dc07a3bffee","unresolved":false,"context_lines":[{"line_number":107,"context_line":"        :param network_resources: Network resources required for the"},{"line_number":108,"context_line":"                                  credentials"},{"line_number":109,"context_line":"        :param credentials_domain: Domain credentials belong to"},{"line_number":110,"context_line":"        :param admin_role: Name of the role of the admin account"},{"line_number":111,"context_line":"        \"\"\""},{"line_number":112,"context_line":"        self.identity_version \u003d identity_version"},{"line_number":113,"context_line":"        self.name \u003d name or \"test_creds\""}],"source_content_type":"text/x-python","patch_set":10,"id":"1a26ad4f_25f9de4d","line":110,"range":{"start_line":110,"start_character":45,"end_line":110,"end_character":58},"updated":"2015-10-30 01:45:28.000000000","message":"I would go with \"Name of the role required to administrate Keystone.\" (as stated in tempest/config.py). Because there could be no \"admin\" account (it could be name \"foobar\" and have the role \u0027admin\u0027). And it\u0027s better to decouple the notion of role and account.\n\nI know, I am splitting hair in two here :)","commit_id":"29491a780ae722e03fca072bf7ef00835f13f889"}]}