yahoo-eng-team team mailing list archive

[Paul Michali <1521766@xxxxxxxxxxxxxxxxxx>]

Public bug reported:

The astroid package, used by pylint, was recently updated to version
1.4.1. Both 1.4.0 and 1.4.1 do not work with pylint 1.4.4, which is
being used by LB, VPN, and neutron. astroid 1.3.8 works with pylint
1.4.4.

For neutron:
master and liberty gate uses pep8-constraints, which has a pin for astroid
kilo needs pinning (proposal is to add to requriements)
juno doesn't use pylint, so works

Note: Can modify pep8 target to do same as pep8-constraints, so that
developers can use same command they are used to.

To add users
For LB:
master - Temp workaround was to remove pylint, can pin, if desired.
liberty, kilo? - will need to pin
juno?


For VPN:
master, liberty, and kilo need to pin pylint and astroid
juno does not use pylint

NOTE: Can migrate to using pep8-constraints job and target.


For FW:
does not do pylint for pep8, so no problem seen (but no coverage).

Because this broke gate, there are several patches already in play for
this.

** Affects: neutron
     Importance: Undecided
     Assignee: Paul Michali (pcm)
         Status: In Progress


** Tags: neutron neutron-lbaas neutron-vpnaas

** Changed in: neutron
     Assignee: (unassigned) => Paul Michali (pcm)

** Changed in: neutron
       Status: New => In Progress

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

Title:
  pylint breakage in neutron and neutron-vpnaas

Status in neutron:
  In Progress

Bug description:
  The astroid package, used by pylint, was recently updated to version
  1.4.1. Both 1.4.0 and 1.4.1 do not work with pylint 1.4.4, which is
  being used by LB, VPN, and neutron. astroid 1.3.8 works with pylint
  1.4.4.

  For neutron:
  master and liberty gate uses pep8-constraints, which has a pin for astroid
  kilo needs pinning (proposal is to add to requriements)
  juno doesn't use pylint, so works

  Note: Can modify pep8 target to do same as pep8-constraints, so that
  developers can use same command they are used to.

  To add users
  For LB:
  master - Temp workaround was to remove pylint, can pin, if desired.
  liberty, kilo? - will need to pin
  juno?

  
  For VPN:
  master, liberty, and kilo need to pin pylint and astroid
  juno does not use pylint

  NOTE: Can migrate to using pep8-constraints job and target.

  
  For FW:
  does not do pylint for pep8, so no problem seen (but no coverage).

  Because this broke gate, there are several patches already in play for
  this.

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