)]}'
{"refstack_client/refstack_client.py":[{"author":{"_account_id":10526,"name":"Catherine Diep","email":"cdiep@us.ibm.com","username":"cdiep"},"change_message_id":"e0b0f4a7ba94ab6aa212a9a8f5ef3b44605cb2c7","unresolved":false,"context_lines":[{"line_number":148,"context_line":"                         and conf_file.getboolean(\u0027identity-feature-enabled\u0027,"},{"line_number":149,"context_line":"                                                  \u0027api_v3\u0027)"},{"line_number":150,"context_line":"                         and conf_file.has_option(\u0027identity\u0027, \u0027uri_v3\u0027))"},{"line_number":151,"context_line":"                else \u0027v2\u0027)"},{"line_number":152,"context_line":""},{"line_number":153,"context_line":"            if conf_file.has_option(\u0027auth\u0027, \u0027test_accounts_file\u0027):"},{"line_number":154,"context_line":"                account_file \u003d os.path.expanduser("}],"source_content_type":"text/x-python","patch_set":4,"id":"7a740942_0bb7d9d8","side":"PARENT","line":151,"updated":"2015-12-11 21:51:49.000000000","message":"There is a bit of history why this code was written as it is to \"prefer using v3 to fetch the Identity ID\" whenever possible regardless of the setting of the auth_version parameter.\n\nOur reason at the time was because: Using v2,  the ID returned is a random selection among the endpoint ID of either admin, private or public endpoints.  For v3, the ID is always the ID of the Keystone service ID.  So, we feel like this is the number that we prefer.","commit_id":"1d392df77f1b88edee17de89690414e5bf7681b3"},{"author":{"_account_id":10526,"name":"Catherine Diep","email":"cdiep@us.ibm.com","username":"cdiep"},"change_message_id":"e0b0f4a7ba94ab6aa212a9a8f5ef3b44605cb2c7","unresolved":false,"context_lines":[{"line_number":150,"context_line":"                  and conf_file.getboolean(\u0027identity-feature-enabled\u0027,"},{"line_number":151,"context_line":"                                           \u0027api_v3\u0027)"},{"line_number":152,"context_line":"                  and conf_file.has_option(\u0027identity\u0027, \u0027uri_v3\u0027)):"},{"line_number":153,"context_line":"                auth_version \u003d \u0027v3\u0027"},{"line_number":154,"context_line":"            else:"},{"line_number":155,"context_line":"                auth_version \u003d \u0027v2\u0027"},{"line_number":156,"context_line":""}],"source_content_type":"text/x-python","patch_set":4,"id":"7a740942_0bbd9967","line":153,"updated":"2015-12-11 21:51:49.000000000","message":"I have no problem to update the code as suggested.  Just want to remind everyone on the intention of the original code.","commit_id":"97eabc64e9e2d8c6835456af1d4f7cfa0cab6121"},{"author":{"_account_id":10526,"name":"Catherine Diep","email":"cdiep@us.ibm.com","username":"cdiep"},"change_message_id":"e0b0f4a7ba94ab6aa212a9a8f5ef3b44605cb2c7","unresolved":false,"context_lines":[{"line_number":187,"context_line":"                insecure_opt \u003d self.conf.getboolean("},{"line_number":188,"context_line":"                    \u0027identity\u0027, \u0027disable_ssl_certificate_validation\u0027)"},{"line_number":189,"context_line":"            else:"},{"line_number":190,"context_line":"                insecure_opt \u003d self.args.insecure"},{"line_number":191,"context_line":""},{"line_number":192,"context_line":"            args \u003d {"},{"line_number":193,"context_line":"                \u0027insecure\u0027: insecure_opt,"}],"source_content_type":"text/x-python","patch_set":4,"id":"7a740942_4b1de1bc","line":190,"updated":"2015-12-11 21:51:49.000000000","message":"This change looks good to me. Thanks!","commit_id":"97eabc64e9e2d8c6835456af1d4f7cfa0cab6121"}]}