add(remove)TenantAccess throws 500 Internal Error

[zhangyanzi]

The function addTenantAccess and removeTenantAccess don't check whether the parameter "tenant"is in request body. So the 500 error has been thrown.

We should catch the KeyError and transfer the KeyError to 400(HTTPBadRequest) instead of 500.

[OpenStack Infra (hudson-openstack)]

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

[OpenStack Infra (hudson-openstack)]

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

[OpenStack Infra (hudson-openstack)]

Reviewed: https://review.openstack.org/102108
Committed: https://git.openstack.org/cgit/openstack/nova/commit/?id=5d4b7365d97dd987f5af91e5800c4f3a586ef654
Submitter: Jenkins
Branch: master

commit 5d4b7365d97dd987f5af91e5800c4f3a586ef654
Author: Chris Yeoh <email address hidden>
Date: Tue Jun 24 14:53:25 2014 +0930

    Fixes internal server error for add/remove tenant flavor access request

    Fixes the internal server error (500) response when an add or remove tenant access
    to a flavor is made without specifying the tenant. Note that only the V2 API code
    change is required because the V3 API version is protected by the input validation.

    Change-Id: I7c126920016c7004fa55efb89747e97838d62f4e
    Closes-Bug: 1253014