yahoo-eng-team team mailing list archive

[Petri Kemppainen <petri.kemppainen@xxxxxxxxx>]

Thanks for your comments. Should the status be changed to Invalid?


** Changed in: neutron
       Status: Triaged => Invalid

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

Title:
  pylint problem importing module classes.py

Status in neutron:
  Invalid

Bug description:
  Execution of tox fails every time in pep8 check.
  Seems that importing of classes.py fails, preventing pep8 test execution.

  ...
  pep8 runtests: commands[2] | flake8
  pep8 runtests: commands[3] | sh ./tools/coding-checks.sh --pylint 
  Running pylint...
  You can speed this up by running it on 'HEAD~[0-9]' (e.g. HEAD~1, this change only)...
  Problem importing module classes.py: cannot import name InferenceContext
  Problem importing module classes.pyc: cannot import name InferenceContext
  ************* Module neutron.auth
  E: 27,29: Undefined variable 'base' (undefined-variable)
  E: 30, 5: Undefined variable 'webob' (undefined-variable)
  E: 33,18: Undefined variable 'req' (undefined-variable)
  ...

  Here is the sequence of used commands:
  1) git clone https://git.openstack.org/openstack/neutron.git
  2) cd neutron
  3) tox

  
  And here are the logs just before error: 
  ..
  ======
  Totals
  ======
  Ran: 6513 tests in 403.0000 sec.
   - Passed: 6449
   - Skipped: 64
   - Expected Fail: 0
   - Unexpected Success: 0
   - Failed: 0
  Sum of execute time for each test: 1564.6589 sec.

  ==============
  Worker Balance
  ==============
   - Worker 0 (1627 tests) => 0:06:33.766003
   - Worker 1 (1629 tests) => 0:06:32.369482
   - Worker 2 (1629 tests) => 0:06:35.912263
   - Worker 3 (1628 tests) => 0:06:32.684246

  Slowest Tests:

  Test id                                                                                                        Runtime (s)
  -------------------------------------------------------------------------------------------------------------  -----------
  neutron.tests.unit.extensions.test_extraroute.ExtraRouteDBSepTestCase.test_router_add_interface_ipv6_subnet    3.800
  neutron.tests.unit.extensions.test_l3.L3NatDBIntTestCase.test_router_add_interface_ipv6_subnet                 3.330
  neutron.tests.unit.extensions.test_extraroute.ExtraRouteDBIntTestCase.test_router_add_interface_ipv6_subnet    3.325
  neutron.tests.unit.extensions.test_l3.L3NatDBSepTestCase.test_router_add_interface_ipv6_subnet                 3.217
  neutron.tests.unit.extensions.test_extraroute.ExtraRouteDBSepTestCase.test_floatingip_update_different_router  2.733
  neutron.tests.unit.plugins.ml2.test_agent_scheduler.Ml2AgentSchedulerTestCase.test_router_sync_data            2.624
  neutron.tests.unit.extensions.test_extraroute.ExtraRouteDBIntTestCase.test_floatingip_update_different_router  2.505
  neutron.tests.unit.db.test_agentschedulers_db.OvsAgentSchedulerTestCase.test_router_sync_data                  2.443
  neutron.tests.unit.plugins.ml2.test_plugin.TestMl2PortsV2.test_requested_ips_only                              2.423
  neutron.tests.unit.extensions.test_l3.L3NatDBSepTestCase.test_floatingip_update_different_router               2.405
  pep8 create: /home/petrkemp/devstack/neutron/.tox/pep8
  pep8 installdeps: -r/home/petrkemp/devstack/neutron/requirements.txt, -r/home/petrkemp/devstack/neutron/test-requirements.txt
  pep8 develop-inst: /home/petrkemp/devstack/neutron
  pep8 installed: aioeventlet==0.4,alembic==0.8.3,amqp==1.4.7,anyjson==0.3.3,appdirs==1.4.0,astroid==1.4.1,Babel==2.1.1,beautifulsoup4==4.4.1,cachetools==1.1.5,cliff==1.15.0,cmd2==0.6.8,contextlib2==0.4.0,coverage==4.0.3,ddt==1.0.1,debtcollector==1.0.0,decorator==4.0.4,docutils==0.12,ecdsa==0.13,enum34==1.1.1,eventlet==0.17.4,extras==0.0.3,fasteners==0.14.1,fixtures==1.4.0,flake8==2.2.4,funcsigs==0.4,functools32==3.2.3.post2,futures==3.0.3,futurist==0.8.0,greenlet==0.4.9,hacking==0.10.2,httplib2==0.9.2,iso8601==0.1.11,Jinja2==2.8,jsonschema==2.5.1,keystoneauth1==2.0.0,keystonemiddleware==4.0.0,kombu==3.0.29,lazy-object-proxy==1.2.1,linecache2==1.0.0,logilab-common==1.1.0,logutils==0.3.3,Mako==1.0.3,MarkupSafe==0.23,mccabe==0.2.1,mock==1.3.0,monotonic==0.4,mox3==0.12.0,msgpack-python==0.4.6,netaddr==0.7.18,netifaces==0.10.4,-e git+https://git.openstack.org/openstack/neutron.git@81a4aac8d429a5d8b2874c4fce0ffd6498f5b1c6#egg=neutron-master,ordereddict==1.1,os-client-config==1.12.0,os-testr==0.4.2,oslo.concurrency==3.0.0,oslo.config==3.0.0,oslo.context==1.0.0,oslo.db==4.0.0,oslo.i18n==3.0.0,oslo.log==2.0.0,oslo.messaging==3.0.0,oslo.middleware==3.0.0,oslo.policy==1.0.0,oslo.rootwrap==3.0.1,oslo.serialization==2.0.0,oslo.service==1.0.0,oslo.utils==3.0.0,oslo.versionedobjects==1.0.0,oslosphinx==4.0.0,oslotest==2.0.0,paramiko==1.16.0,Paste==2.0.2,PasteDeploy==1.5.2,pbr==1.8.1,pecan==1.0.3,pep8==1.5.7,prettytable==0.7.2,pycadf==2.0.0,pycrypto==2.6.1,pyflakes==0.8.1,Pygments==2.0.2,pyinotify==0.9.6,pylint==1.4.4,PyMySQL==0.6.7,pyparsing==2.0.6,python-editor==0.4,python-keystoneclient==2.0.0,python-mimeparse==0.1.4,python-neutronclient==3.1.0,python-novaclient==2.35.0,python-subunit==1.2.0,pytz==2015.7,PyYAML==3.11,reno==1.1.0,repoze.lru==0.6,requests==2.8.1,requests-mock==0.7.0,requestsexceptions==1.1.1,retrying==1.3.3,Routes==2.2,ryu==3.26,simplejson==3.8.1,singledispatch==3.4.0.3,six==1.10.0,Sphinx==1.2.3,SQLAlchemy==1.0.9,sqlalchemy-migrate==0.10.0,sqlparse==0.1.18,stevedore==1.9.0,tempest-lib==0.11.0,Tempita==0.5.2,testrepository==0.0.20,testresources==0.2.7,testscenarios==0.5.0,testtools==1.8.1,traceback2==1.4.0,trollius==2.0,unicodecsv==0.14.1,unittest2==1.1.0,waitress==0.8.10,WebOb==1.5.1,WebTest==2.0.20,wheel==0.24.0,wrapt==1.10.5
  pep8 runtests: PYTHONHASHSEED='270373021'
  pep8 runtests: commands[0] | sh ./tools/misc-sanity-checks.sh
  pep8 runtests: commands[1] | /home/petrkemp/devstack/neutron/tools/check_unit_test_structure.sh
  10 unmatched test modules were ignored
  Success!  All test modules match targets in the code tree.
  pep8 runtests: commands[2] | flake8
  pep8 runtests: commands[3] | sh ./tools/coding-checks.sh --pylint 
  Running pylint...
  You can speed this up by running it on 'HEAD~[0-9]' (e.g. HEAD~1, this change only)...
  Problem importing module classes.py: cannot import name InferenceContext
  Problem importing module classes.pyc: cannot import name InferenceContext
  ************* Module neutron.auth
  E: 27,29: Undefined variable 'base' (undefined-variable)
  E: 30, 5: Undefined variable 'webob' (undefined-variable)
  E: 33,18: Undefined variable 'req' (undefined-variable)
  E: 34,15: Undefined variable 'user_id' (undefined-variable)
  E: 39,20: Undefined variable 'req' (undefined-variable)
  E: 42,17: Undefined variable 'r' (undefined-variable)
  E: 42,36: Undefined variable 'req' (undefined-variable)
  E: 45,22: Undefined variable 'req' (undefined-variable)
  E: 46,20: Undefined variable 'req' (undefined-variable)
  E: 49,17: Undefined variable 'req' (undefined-variable)
  E: 52,21: Undefined variable 'req' (undefined-variable)
  E: 53,37: Undefined variable 'req' (undefined-variable)
  E: 56,30: Undefined variable 'user_id' (undefined-variable)
  E: 56,39: Undefined variable 'tenant_id' (undefined-variable)
  E: 56,56: Undefined variable 'roles' (undefined-variable)
  E: 57,40: Undefined variable 'user_name' (undefined-variable)
  E: 57,63: Undefined variable 'tenant_name' (undefined-variable)
  E: 58,41: Undefined variable 'req_id' (undefined-variable)
  E: 58,60: Undefined variable 'auth_token' (undefined-variable)
  E: 61, 8: Undefined variable 'req' (undefined-variable)
  E: 61,41: Undefined variable 'ctx' (undefined-variable)
  E: 63,15: Undefined variable 'self' (undefined-variable)
  E: 68,15: Undefined variable 'local_conf' (undefined-variable)
  E: 69,15: Undefined variable 'pipeline' (undefined-variable)
  E: 70,15: Undefined variable 'loader' (undefined-variable)
  E: 70,33: Undefined variable 'n' (undefined-variable)
  E: 70,45: Undefined variable 'pipeline' (undefined-variable)
  E: 71,10: Undefined variable 'loader' (undefined-variable)
  E: 71,25: Undefined variable 'pipeline' (undefined-variable)
  E: 72, 4: Undefined variable 'filters' (undefined-variable)
  E: 73,18: Undefined variable 'filters' (undefined-variable)
  E: 74,21: Undefined variable 'app' (undefined-variable)
  E: 75,11: Undefined variable 'app' (undefined-variable)
  ************* Module neutron.context
  ...

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