]> review.fuel-infra Code Review - openstack-build/cinder-build.git/commit
NetApp: E-Series fix JSONDecodeError on first add
authorMichael Price <michael.price@netapp.com>
Thu, 12 Nov 2015 22:44:07 +0000 (16:44 -0600)
committerMichael Price <michael.price@netapp.com>
Tue, 1 Dec 2015 20:27:16 +0000 (20:27 +0000)
commit489c8d2a5f829e5398d861d29d1a2beb504c6027
tree1aeddf00c2358156029a8ac934a1eae14e7d620a
parent65bed9807981c86ac1b591805c86793796fa6a4e
NetApp: E-Series fix JSONDecodeError on first add

The NetApp E-Series driver would throw a JSONDecodeError on the
addition of a new storage-system. This would occur because a
JSON response was expected, but text/html was being returned as
the Content-Type when a storage-system has been added but is still
coming online.

This patch adds a retry to the driver startup to wait for an array
to come online. It also adds code to avoid the JSONDecodeError
and fail based on the HTTP Response's return code.

Change-Id: I405723039c7a0df10d39bd2eb961ee64d71c2d01
cinder/tests/unit/volume/drivers/netapp/eseries/test_client.py
cinder/volume/drivers/netapp/eseries/client.py
cinder/volume/drivers/netapp/eseries/library.py