Comment 5 for bug 1551037

Revision history for this message
Erno Kuvaja (jokke) wrote :

Comparing the bug description and the change this does not make sense to me. Apart from the fact that this bug is against glance and the change is to python-glanceclient, The change is in a place where output would never be as described in the bug description.

Is this correct bug for https://review.openstack.org/#/c/290882/ & https://review.openstack.org/#/c/291118 ?

If the server is throwing 409 it should have been catched here https://github.com/openstack/python-glanceclient/blob/master/glanceclient/v2/shell.py#L345-L348 and the output certainly should not be the 500 as in bug description.

Even more convinced about this as the tests passes on master even lines https://github.com/openstack/python-glanceclient/blob/master/glanceclient/v2/shell.py#L341-L344 are removed but the new test introduced in 290882 is kept in place.