search

Tendrl / ceph-integration

Extracts all data from a Ceph cluster for consumption by Tendrl
GNU Lesser General Public License v2.1
3 stars 11 forks source link

Handling NoneType return in ceph_integration/ceph.py #308

Closed rishubhjain closed 7 years ago

rishubhjain commented 7 years ago

https://github.com/Tendrl/ceph-integration/blob/develop/tendrl/ceph_integration/ceph.py#L723 might return ceph_version and empty dict, which will be followed by returning a None Type by https://github.com/Tendrl/ceph-integration/blob/develop/tendrl/ceph_integration/ceph.py#L847. I feel returning NoneType should be handled before https://github.com/Tendrl/ceph-integration/blob/develop/tendrl/ceph_integration/ceph.py#L873

rishubhjain commented 7 years ago

@shtripat If possible could you look into this as well. would like to discuss this.

r0h4n commented 7 years ago

Valid bug