← Back to team overview

yahoo-eng-team team mailing list archive

[Bug 1611400] Re: pep8 job failing with F821 in test_l3

 

Reviewed:  https://review.openstack.org/352941
Committed: https://git.openstack.org/cgit/openstack/neutron/commit/?id=a200f4a7b670abd7f6fc59847782719057bfe6ef
Submitter: Jenkins
Branch:    master

commit a200f4a7b670abd7f6fc59847782719057bfe6ef
Author: Ihar Hrachyshka <ihrachys@xxxxxxxxxx>
Date:   Tue Aug 9 16:32:33 2016 +0200

    pep8: fixed F821 violation in a unit test
    
    The _ symbol should have been imported explicitly. Actually, there is no
    use case for translatable messages in unit tests, it's just a waste of
    translator time to mark it as such. The patch removes the translation
    marker for good.
    
    Change-Id: I76bef710a1030dcb4eb35778ebf44ed600016f17
    Closes-Bug: #1611400


** Changed in: neutron
       Status: In Progress => Fix Released

-- 
You received this bug notification because you are a member of Yahoo!
Engineering Team, which is subscribed to neutron.
https://bugs.launchpad.net/bugs/1611400

Title:
  pep8 job failing with F821 in test_l3

Status in neutron:
  Fix Released

Bug description:
  New flake8 was released today, and probably broke neutron pep8 job
  with:

  pep8 runtests: commands[2] | flake8
  ./neutron/tests/unit/extensions/test_l3.py:2931:19: F821 undefined name '_'
              msg = _("Failure mocking...")
                    ^

To manage notifications about this bug go to:
https://bugs.launchpad.net/neutron/+bug/1611400/+subscriptions


References