)]}'
{"designate/cmd/manage.py":[{"author":{"_account_id":10224,"name":"Feodor Tersin","email":"ftersin@hotmail.com","username":"ftersin"},"change_message_id":"11929570fe324efd70fac0440da76a0c737113d6","unresolved":false,"context_lines":[{"line_number":108,"context_line":"    try:"},{"line_number":109,"context_line":"        utils.read_config(\u0027designate\u0027, sys.argv)"},{"line_number":110,"context_line":"        logging.setup(CONF, \u0027designate\u0027)"},{"line_number":111,"context_line":"    except cfg.ConfigFilesNotFoundError as e:"},{"line_number":112,"context_line":"        cfg_files \u003d \", \".join(e.config_files)"},{"line_number":113,"context_line":"        print(_(\"Failed to read configuration file(s): %s\") % cfg_files)"},{"line_number":114,"context_line":"        sys.exit(1)"}],"source_content_type":"text/x-python","patch_set":2,"id":"5a8bc5a2_3713685e","line":111,"updated":"2016-09-22 07:39:24.000000000","message":"What now is the reason to handle this? Why is this better than default behavior, i.e. in ConfigFilesPermissionDeniedError case?","commit_id":"939c392613b48d075a18ad6da43c3cce49235e0b"},{"author":{"_account_id":13252,"name":"Dr. Jens Harbott","display_name":"Jens Harbott (frickler)","email":"frickler@offenerstapel.de","username":"jrosenboom"},"change_message_id":"8b53d3ddd05b3e1e970618d96807b35ffe13e174","unresolved":false,"context_lines":[{"line_number":108,"context_line":"    try:"},{"line_number":109,"context_line":"        utils.read_config(\u0027designate\u0027, sys.argv)"},{"line_number":110,"context_line":"        logging.setup(CONF, \u0027designate\u0027)"},{"line_number":111,"context_line":"    except cfg.ConfigFilesNotFoundError as e:"},{"line_number":112,"context_line":"        cfg_files \u003d \", \".join(e.config_files)"},{"line_number":113,"context_line":"        print(_(\"Failed to read configuration file(s): %s\") % cfg_files)"},{"line_number":114,"context_line":"        sys.exit(1)"}],"source_content_type":"text/x-python","patch_set":2,"id":"7f287b81_ff611dd5","line":111,"in_reply_to":"5a8bc5a2_3713685e","updated":"2017-09-01 08:30:23.000000000","message":"After looking at the output, I have to agree with Feodor here. If you want to avoid users/admins to see the full python traceback, you should be consistent for all possible error scenarios.\n\nBut I\u0027d also say that the error messages generated by oslo_config are good enough and consistent with things people may see in other OpenStack applications, so why not just stick with them and drop the exception handling completely?","commit_id":"939c392613b48d075a18ad6da43c3cce49235e0b"},{"author":{"_account_id":10224,"name":"Feodor Tersin","email":"ftersin@hotmail.com","username":"ftersin"},"change_message_id":"731f5eb7fdcad191421196670bb2f59e4aa3b4f7","unresolved":false,"context_lines":[{"line_number":108,"context_line":"    try:"},{"line_number":109,"context_line":"        utils.read_config(\u0027designate\u0027, sys.argv)"},{"line_number":110,"context_line":"        logging.setup(CONF, \u0027designate\u0027)"},{"line_number":111,"context_line":"    except cfg.ConfigFilesNotFoundError as e:"},{"line_number":112,"context_line":"        cfg_files \u003d \", \".join(e.config_files)"},{"line_number":113,"context_line":"        print(_(\"Failed to read configuration file(s): %s\") % cfg_files)"},{"line_number":114,"context_line":"        sys.exit(1)"}],"source_content_type":"text/x-python","patch_set":2,"id":"5a8bc5a2_b72bf832","line":111,"in_reply_to":"5a8bc5a2_3713685e","updated":"2016-09-22 07:45:24.000000000","message":"i.e. -\u003e e.g.","commit_id":"939c392613b48d075a18ad6da43c3cce49235e0b"}]}