)]}'
{"oslo_config/validator.py":[{"author":{"_account_id":28522,"name":"Hervé Beraud","email":"herveberaud.pro@gmail.com","username":"hberaud"},"change_message_id":"0721429c93fb69b3096402c7611c9ce988efe305","unresolved":false,"context_lines":[{"line_number":87,"context_line":""},{"line_number":88,"context_line":"def load_opt_data(conf):"},{"line_number":89,"context_line":"    with open(conf.opt_data) as f:"},{"line_number":90,"context_line":"        return yaml.safe_load(f)"},{"line_number":91,"context_line":""},{"line_number":92,"context_line":""},{"line_number":93,"context_line":"def _validate(conf):"}],"source_content_type":"text/x-python","patch_set":2,"id":"9fb8cfa7_22234d58","line":90,"updated":"2019-06-12 10:15:53.000000000","message":"LGTM but maybe we can use \u003d\u003e https://github.com/openstack/oslo.serialization/blob/master/oslo_serialization/yamlutils.py#L24","commit_id":"dc9b711a3fa494e6d363657eeb8c2049585d5976"},{"author":{"_account_id":6928,"name":"Ben Nemec","email":"openstack@nemebean.com","username":"bnemec"},"change_message_id":"c605b4572fab06aa4a714d112f8e2d1d63b84c69","unresolved":false,"context_lines":[{"line_number":87,"context_line":""},{"line_number":88,"context_line":"def load_opt_data(conf):"},{"line_number":89,"context_line":"    with open(conf.opt_data) as f:"},{"line_number":90,"context_line":"        return yaml.safe_load(f)"},{"line_number":91,"context_line":""},{"line_number":92,"context_line":""},{"line_number":93,"context_line":"def _validate(conf):"}],"source_content_type":"text/x-python","patch_set":2,"id":"9fb8cfa7_c5103651","line":90,"in_reply_to":"9fb8cfa7_22234d58","updated":"2019-06-12 15:02:41.000000000","message":"This is just a refactoring of the existing code to make it easier to unit test. Feel free to propose that as a followup though.","commit_id":"dc9b711a3fa494e6d363657eeb8c2049585d5976"}]}