)]}'
{"tempest/stress/driver.py":[{"author":{"_account_id":6167,"name":"Ken\u0027ichi Ohmichi","email":"ken1ohmichi@gmail.com","username":"oomichi"},"change_message_id":"b5b4c7d9386ff74e979a5f37796d13705f9d8eb9","unresolved":false,"context_lines":[{"line_number":154,"context_line":"                    projects_client \u003d None"},{"line_number":155,"context_line":"                    roles_client \u003d None"},{"line_number":156,"context_line":"                    users_client \u003d None"},{"line_number":157,"context_line":"                try:"},{"line_number":158,"context_line":"                    domain \u003d (identity_client.auth_provider.credentials."},{"line_number":159,"context_line":"                              project_domain_name)"},{"line_number":160,"context_line":"                except AttributeError:"},{"line_number":161,"context_line":"                    domain \u003d \u0027Default\u0027"},{"line_number":162,"context_line":"                credentials_client \u003d cred_client.get_creds_client("},{"line_number":163,"context_line":"                    identity_client, projects_client, roles_client,"},{"line_number":164,"context_line":"                    users_client, project_domain_name\u003ddomain)"}],"source_content_type":"text/x-python","patch_set":2,"id":"7a5de9d1_bb88d123","line":161,"range":{"start_line":157,"start_character":0,"end_line":161,"end_character":38},"updated":"2016-01-28 05:24:40.000000000","message":"How about\n\n domain \u003d identity_client.auth_provider.credentials.get(\u0027project_domain_name\u0027, \u0027Default\u0027)\n\n?\n\nI prefer to avoid this kind of try-except if that is common case.","commit_id":"ebc7e55243c14fbd8c3c36c813961ca948f6b7de"},{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"b2647fd549fae8536cc6f0ec2e14f069eb985e7a","unresolved":false,"context_lines":[{"line_number":154,"context_line":"                    projects_client \u003d None"},{"line_number":155,"context_line":"                    roles_client \u003d None"},{"line_number":156,"context_line":"                    users_client \u003d None"},{"line_number":157,"context_line":"                try:"},{"line_number":158,"context_line":"                    domain \u003d (identity_client.auth_provider.credentials."},{"line_number":159,"context_line":"                              project_domain_name)"},{"line_number":160,"context_line":"                except AttributeError:"},{"line_number":161,"context_line":"                    domain \u003d \u0027Default\u0027"},{"line_number":162,"context_line":"                credentials_client \u003d cred_client.get_creds_client("},{"line_number":163,"context_line":"                    identity_client, projects_client, roles_client,"},{"line_number":164,"context_line":"                    users_client, project_domain_name\u003ddomain)"}],"source_content_type":"text/x-python","patch_set":2,"id":"7a5de9d1_5bcf5d56","line":161,"range":{"start_line":157,"start_character":0,"end_line":161,"end_character":38},"in_reply_to":"7a5de9d1_bb88d123","updated":"2016-01-28 06:30:32.000000000","message":"Nice idea. cred and others are always present here.","commit_id":"ebc7e55243c14fbd8c3c36c813961ca948f6b7de"},{"author":{"_account_id":7428,"name":"Yaroslav Lobankov","email":"lobankovyv@gmail.com","username":"ylobankov"},"change_message_id":"cc9681c1fec07e15810004a480d96e771006cd6e","unresolved":false,"context_lines":[{"line_number":155,"context_line":"                    roles_client \u003d None"},{"line_number":156,"context_line":"                    users_client \u003d None"},{"line_number":157,"context_line":"                domain \u003d (identity_client.auth_provider.credentials."},{"line_number":158,"context_line":"                          get(\u0027project_domain_name\u0027, \u0027Default\u0027))"},{"line_number":159,"context_line":"                credentials_client \u003d cred_client.get_creds_client("},{"line_number":160,"context_line":"                    identity_client, projects_client, roles_client,"},{"line_number":161,"context_line":"                    users_client, project_domain_name\u003ddomain)"}],"source_content_type":"text/x-python","patch_set":3,"id":"7a5de9d1_ec068d1f","line":158,"updated":"2016-01-28 07:54:09.000000000","message":"What about using the config option for domain instead of hardcoded \u0027Default\u0027 here?","commit_id":"627debfeb8139b249c48a1e4af4bb492a5d87211"},{"author":{"_account_id":7350,"name":"Jordan Pittier","email":"jordan.pittier@gmail.com","username":"JordanP"},"change_message_id":"3011daefd24e9f8be3e0f08014345a0f577f8e44","unresolved":false,"context_lines":[{"line_number":155,"context_line":"                    roles_client \u003d None"},{"line_number":156,"context_line":"                    users_client \u003d None"},{"line_number":157,"context_line":"                domain \u003d (identity_client.auth_provider.credentials."},{"line_number":158,"context_line":"                          get(\u0027project_domain_name\u0027, \u0027Default\u0027))"},{"line_number":159,"context_line":"                credentials_client \u003d cred_client.get_creds_client("},{"line_number":160,"context_line":"                    identity_client, projects_client, roles_client,"},{"line_number":161,"context_line":"                    users_client, project_domain_name\u003ddomain)"}],"source_content_type":"text/x-python","patch_set":3,"id":"7a5de9d1_45baf60b","line":158,"in_reply_to":"7a5de9d1_ec068d1f","updated":"2016-01-28 16:23:38.000000000","message":"Good idea, but let\u0027s try to merge this and use conf.\"default_credentials_domain_name\" in a follow-up patch.","commit_id":"627debfeb8139b249c48a1e4af4bb492a5d87211"}]}