get_server_version does not handle older (non-v3) servers
Bug #1694729 reported by
Matt Riedemann
This bug affects 1 person
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
python-cinderclient |
Fix Released
|
Medium
|
John Griffith |
Bug Description
This method is checking for the v3 API:
https:/
But if this is talking to an older cinder server which doesn't have the v3 API then the for loop is going to complete and return None, which could lead to a NoneType here:
https:/
The for loop should probably have an else that raises ClientException so it defaults to return 2.0.
Changed in python-cinderclient: | |
importance: | Undecided → Medium |
tags: | added: ocata-backport-potential |
To post a comment you must log in.
Fix proposed to branch: master /review. openstack. org/469520
Review: https:/