)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":9545,"name":"Andriy Kurilin","email":"andr.kurilin@gmail.com","username":"akurilin"},"change_message_id":"eab67301ddf4be5306d14a831dc5a4138b6ffaa7","unresolved":false,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Add endpoint types to tempest config"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Tempest should use public endpoints by default. This commit adds"},{"line_number":10,"context_line":"both v2 and v3 endpoint types and sets them to public."},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"Change-Id: Id602b676ff08e9d2ccdf6e3f6ddfde7755449c03"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"5fc1f717_b3ee1389","line":10,"range":{"start_line":9,"start_character":0,"end_line":10,"end_character":54},"updated":"2019-04-03 15:03:43.000000000","message":"it is not true :) tempest should use such type of endpoints as it is specified in rally deployment/env . This is what this commit is actually doing.","commit_id":"8b6be2aa5aefaf0099d6961b6d700a522c006798"}],"rally_openstack/verification/tempest/config.py":[{"author":{"_account_id":9545,"name":"Andriy Kurilin","email":"andr.kurilin@gmail.com","username":"akurilin"},"change_message_id":"eab67301ddf4be5306d14a831dc5a4138b6ffaa7","unresolved":false,"context_lines":[{"line_number":126,"context_line":"        self.conf.set(section_name, \"auth_version\", \"v%s\" % target_version)"},{"line_number":127,"context_line":"        self.conf.set(section_name, \"uri\", uri)"},{"line_number":128,"context_line":"        self.conf.set(section_name, \"uri_v3\", uri_v3)"},{"line_number":129,"context_line":"        self.conf.set(section_name, \"v2_endpoint_type\","},{"line_number":130,"context_line":"                      self.credential.endpoint_type)"},{"line_number":131,"context_line":"        self.conf.set(section_name, \"v3_endpoint_type\","},{"line_number":132,"context_line":"                      self.credential.endpoint_type)"},{"line_number":133,"context_line":""},{"line_number":134,"context_line":"        self.conf.set(section_name, \"disable_ssl_certificate_validation\","},{"line_number":135,"context_line":"                      str(self.credential.https_insecure))"}],"source_content_type":"text/x-python","patch_set":2,"id":"5fc1f717_736bcbf0","line":132,"range":{"start_line":129,"start_character":8,"end_line":132,"end_character":52},"updated":"2019-04-03 15:03:43.000000000","message":"self.credential.endpoint_type can be None. In this case, tempest will use the default value which is specified in its code.\n\nCurrently, unittests caught that this case is broken since it is impossible to set config option to None. Please move new lines under `if self.credential.endpoint_type:`  statement.","commit_id":"8b6be2aa5aefaf0099d6961b6d700a522c006798"}]}