Replication methods return 500 status code on missing host

[Gorka Eguileor]

Current code has no check for the host field in any of the replication endpoints (failover_host, freeze, and thaw) so a call to any of these methods without the required host field will result in a KeyError exception and a 500 code being returned to the caller.

[OpenStack Infra (hudson-openstack)]

Fix proposed to branch: master
Review: https://review.openstack.org/397357

[OpenStack Infra (hudson-openstack)]

Reviewed: https://review.openstack.org/397357
Committed: https://git.openstack.org/cgit/openstack/cinder/commit/?id=57bd024fcfca5ea8827799480d62dcb3c5de01e2
Submitter: Jenkins
Branch: master

commit 57bd024fcfca5ea8827799480d62dcb3c5de01e2
Author: Gorka Eguileor <email address hidden>
Date: Mon Nov 7 12:03:07 2016 +0100

    Add host check on replication actions

    Current code has no check for the host field in any of the replication
    endpoints (failover_host, freeze, and thaw) so a call to any of these
    methods without the required host field will result in a KeyError
    exception and a 500 code being returned to the caller.

    This patch fixes this by returning expected 400 invalid error, just like
    any other action does.

    It also adds missing replication tests for contrib/services.py.

    Closes-Bug: #1641714
    Change-Id: Icf5b1277179dc7e40f53524233d57ea2e399755a

[OpenStack Infra (hudson-openstack)]

This issue was fixed in the openstack/cinder 10.0.0.0b2 development milestone.