Better API Exception rendering

exoscale / cs

A simple, yet powerful CloudStack API client for python and the command-line.

BSD 3-Clause "New" or "Revised" License

88 stars 36 forks source link

Better API Exception rendering #117

Closed igrishaev closed 4 years ago

igrishaev commented 4 years ago

While investigating compute-tests failures, we faced poor HTTP 431 response from CloudStack exceptions which do not expose the error field. This PR makes overrides the __str__ method so that we expose the error field so we will see it in CI.

Travis is not working anymore, but I run tox locally which includes tests and linters, and they work fine. I only had to comment out 27 in the tox file as there is one place which ruins Python 2 support.

igrishaev commented 4 years ago

@brutasse ok 2.7 and 3.5 were removed.

brutasse commented 4 years ago

Thanks @igrishaev!

synergiator commented 3 years ago

awesome thank you! this solves my issue #110