-
Notifications
You must be signed in to change notification settings - Fork 62
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[Error handling] Evaluate MCAD code base for better error handling #613
Comments
Hi @asm582, just a question about this issue which I have started working on. Is it appropriate to update some functions to return errors, so that we can catch the errors when the functions are called or do we just want to add logging of errors for other function calls inside the functions? |
Hi @Fiona-Waters , I think it's a good idea to start logging errors for now and then we can design later how we can gracefully handle errors. |
WHY
This change is needed to retry or appropriately update etcd
WHAT
Review code base for error handling scenarios
HOW
Make relevant code changes and print appropriate error strings.
TESTS
Write additional tests for error handling
DONE
Increase in test cases that support error handling use case.
The text was updated successfully, but these errors were encountered: