tox -epep8 fails in trove due to missing gettext pkg

Bug #1317652 reported by Peter Stachowski
16
This bug affects 2 people
Affects Status Importance Assigned to Milestone
Trove Integration
Fix Committed
Low
Peter Stachowski

Bug Description

A change introduced to check the validity of po/pot files (https://review.openstack.org/#/c/84239 ) has the side-effect of requiring the msgfmt utility, found in the gettext pkg. We need to include this in redstack install, or running 'tox -epep8' in the trove project will result in an error:

ERROR: InvocationError: "/bin/bash -c find trove -type f -regex '.*\\.pot?' -print0 | xargs -0 -n 1 msgfmt --check-format -o /dev/null"

Changed in trove-integration:
assignee: nobody → Peter Stachowski (petel)
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to trove-integration (master)

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

Changed in trove-integration:
status: New → In Progress
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to trove-integration (master)

Reviewed: https://review.openstack.org/92917
Committed: https://git.openstack.org/cgit/openstack/trove-integration/commit/?id=72e7a6bf9da9fac87282b4eab2318263725a9d40
Submitter: Jenkins
Branch: master

commit 72e7a6bf9da9fac87282b4eab2318263725a9d40
Author: Peter Stachowski <email address hidden>
Date: Thu May 8 17:16:55 2014 -0400

    Added gettext to install_prep_packages

    To verify the validity of po/pot files, there is now a dependency
    on the msgfmt utility found in the gettext package.
    (This was introduced by https://review.openstack.org/#/c/84239 )

    The gettext package has therefore been added to the redstack
    install_prep_packages function.

    Change-Id: Iba6688dc3ef77ce6669253115d70a908f089ccd6
    Closes-Bug: #1317652

Changed in trove-integration:
status: In Progress → Fix Committed
Revision history for this message
Mike Spreitzer (mike-spreitzer) wrote :

Hey, don't we have to fix all installs, not just redstack?

Revision history for this message
Peter Stachowski (peterstac) wrote :

That could very well be true. This was caused by a change to redstack though, so I thought it was local to it. If devstack needs to be fixed instead, maybe it should be done there. I don't recall seeing people complain that devstack was broken though ...

Revision history for this message
Nikhil Manchanda (slicknik) wrote :

OpenStack Infra isn't broken since I believe their Puppet scripts take care of ensuring this is installed on every devstack-gate node.

Changed in trove-integration:
milestone: none → juno-2
importance: Undecided → Low
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Duplicates of this bug

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.