Strings passed to InvalidAggregateAction should be translated

[Mark McLoughlin]

InvalidAggregateAction .looks like:

 class InvalidAggregateAction(Invalid):
     msg_fmt = _("Cannot perform action '%(action)s' on aggregate "
     "%(aggregate_id)s. Reason: %(reason)s.")

The values for action are:

  delete
  add_host_to_aggregate
  update aggregate
  update aggregate metadata

Also we use 'not empty' untranslated as a reason string in one place

We should standardize these strings a little more and always translate them

[OpenStack Infra (hudson-openstack)]

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

[Sean Dague (sdague)]

stalled until kilo

[OpenStack Infra (hudson-openstack)]

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

commit d330f56ec45ba6b1101eb379523bc4fca98b0f0c
Author: jichenjc <email address hidden>
Date: Mon Oct 27 07:38:47 2014 +0800

    Correct InvalidAggregateAction translation&format.

    InvalidAggregateAction .looks like:
     class InvalidAggregateAction(Invalid):
         msg_fmt = _("Cannot perform action '%(action)s' on aggregate "
         "%(aggregate_id)s. Reason: %(reason)s.")

    The values for action are not well formed and reason are not translated.
    This patch fixs those problems

    Change-Id: I9ed20badde8a38eb71c5f121f30e4d54ad50022e
    Closes-Bug: #1278736