Comment 19 for bug 1406333

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to neutron-vpnaas-dashboard (master)

Reviewed: https://review.openstack.org/492044
Committed: https://git.openstack.org/cgit/openstack/neutron-vpnaas-dashboard/commit/?id=66c1e460865d64a6ce8ce454a81048e2bdafa9eb
Submitter: Jenkins
Branch: master

commit 66c1e460865d64a6ce8ce454a81048e2bdafa9eb
Author: Akihiro Motoki <email address hidden>
Date: Wed Aug 9 08:43:49 2017 +0000

    Ensure log messages are not translated

    This is VPNaaS dashboard version of the horizon patch
    https://review.openstack.org/#/c/455635/.
    The horizon patch was merged before VPNaaS dashboard split out,
    but unfortunately I failed to import the change.

    The following describes the rational of this change
    (quoted from the horizon change).
    ----
    Previously translated messages are included in log messages
    and it was determined what language is chosen by users.
    It makes difficult for operators to understand log messgaes.

    This commit tries to use English messages for all log messages.
    The following policies are applied based on the past discussions
    in the bug 1406333 and related reviews.

    - English messages are used for log messages.
    - log messages include exception messages if possible
      to help operators identify what happens.
    - Use ID rather than name for log messages
      as ID is much more unique compared to name.
    - LOG.debug() in success code path are deleted.
      We don't log success messages in most places and API calls to
      back-end services can be logged from python bindings.

    Change-Id: I1a37b7ccbfa29cd83456931f7864ad9ce31aa570
    Closes-Bug: #1406333