insights-core
insights-core copied to clipboard
Catch Timeout on test connection
All Pull Requests:
Check all that apply:
- [x] Have you followed the guidelines in our Contributing document, including the instructions about commit messages?
- [x] Is this PR to correct an issue?
- [ ] Is this PR an enhancement?
Complete Description of Additions/Changes:
The --test-connection
command only used to catch ConnectionError
, including ConnectionTimeout
, but not other timeouts such as ReadTimeoutError
. Re-used the existing REQUEST_FAILED_EXCEPTIONS
list to process all timeouts in the same way.
There is an integration test pull request https://github.com/RedHatInsights/insights-client/pull/244 for this fix.
Card IDs: