Exception handling needs to be fixed
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
Python client library for Zaqar |
Fix Released
|
Medium
|
Eva Balycheva |
Bug Description
I noticed some problems:
1. In some cases no information is provided for the CLI user these exceptions is raised: https:/
2. When the CLI user experiences the error not from python-zaqarclient code, but, for example, from keystoneclient when credentials are bad, only the exception's message is shown. The name of the exception remains hidden, though it can give additional hint to the user.
3. Only in few kinds of error situations the CLI user gets zaqar-related error. Example is "openstack queue get metadata --os-queues-
4. Some of the error messages will never be shown to the CLI user, but are intended to be shown. Examples:
Pool creation: https:/
Subscription creation: https:/
5. The information about what exceptions can be raised by "Client" class's methods is missing in method docstrings.
Possible solution:
1. Catch each exception in the CLI code to provide the CLI user this information wrapped in nice message: exception's name and exception's message. This can be done by some new decorator.
2. Make all exceptions in zaqarclient/
3. Update docstrings of the Client class's methods to list possible exceptions that can be raised. (Maybe)
4. Fix error messages that never show themselves.
Changed in python-zaqarclient: | |
assignee: | nobody → Eva Balycheva (ubershy) |
description: | updated |
description: | updated |
description: | updated |
summary: |
- Exception handling needs to be improved + Exception handling needs to be fixed |
Changed in python-zaqarclient: | |
status: | New → Confirmed |
importance: | Undecided → Medium |
description: | updated |
description: | updated |
description: | updated |
description: | updated |
Changed in python-zaqarclient: | |
status: | In Progress → Fix Released |
Fix proposed to branch: master /review. openstack. org/282114
Review: https:/