yahoo-eng-team team mailing list archive

[Aaron Rosen <aaronorosen@xxxxxxxxx>]

Public bug reported:

In several places in the source tree we imports logging

https://github.com/openstack/neutron/blob/master/neutron/plugins/vmware/plugins/base.py#L16

but instead it should do:

https://github.com/openstack/neutron/blob/master/neutron/plugins/ml2/plugin.py#L52
https://github.com/openstack/neutron/blob/master/neutron/plugins/ml2/plugin.py#L63


arosen@arosen-desktop:/opt/stack/neutron/neutron$ grep -R "import logging" * 
common/utils.py:import logging as std_logging
db/migration/alembic_migrations/heal_script.py:import logging
plugins/cisco/models/virt_phy_sw_v2.py:import logging
plugins/cisco/common/cisco_credentials_v2.py:import logging as LOG
plugins/cisco/network_plugin.py:import logging
plugins/cisco/nexus/cisco_nexus_snippets.py:import logging
plugins/cisco/nexus/cisco_nexus_plugin_v2.py:import logging
plugins/cisco/nexus/cisco_nexus_network_driver_v2.py:import logging
plugins/ml2/drivers/cisco/nexus/nexus_snippets.py:import logging
plugins/vmware/plugins/base.py:import logging
service.py:import logging as std_logging
tests/base.py:import logging
tests/unit/cisco/test_network_plugin.py:import logging
tests/unit/db/metering/test_db_metering.py:import logging
tests/unit/db/firewall/test_db_firewall.py:import logging
tests/unit/db/loadbalancer/test_db_loadbalancer.py:import logging
tests/unit/ml2/test_helpers.py:import logging
tests/unit/test_servicetype.py:import logging
tests/unit/vmware/apiclient/test_api_eventlet_request.py:import logging

** Affects: neutron
     Importance: Undecided
         Status: New

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

Title:
  neutron should use openstack common logging everywhere

Status in OpenStack Neutron (virtual network service):
  New

Bug description:
  In several places in the source tree we imports logging

  https://github.com/openstack/neutron/blob/master/neutron/plugins/vmware/plugins/base.py#L16

  but instead it should do:

  https://github.com/openstack/neutron/blob/master/neutron/plugins/ml2/plugin.py#L52
  https://github.com/openstack/neutron/blob/master/neutron/plugins/ml2/plugin.py#L63

  
  arosen@arosen-desktop:/opt/stack/neutron/neutron$ grep -R "import logging" * 
  common/utils.py:import logging as std_logging
  db/migration/alembic_migrations/heal_script.py:import logging
  plugins/cisco/models/virt_phy_sw_v2.py:import logging
  plugins/cisco/common/cisco_credentials_v2.py:import logging as LOG
  plugins/cisco/network_plugin.py:import logging
  plugins/cisco/nexus/cisco_nexus_snippets.py:import logging
  plugins/cisco/nexus/cisco_nexus_plugin_v2.py:import logging
  plugins/cisco/nexus/cisco_nexus_network_driver_v2.py:import logging
  plugins/ml2/drivers/cisco/nexus/nexus_snippets.py:import logging
  plugins/vmware/plugins/base.py:import logging
  service.py:import logging as std_logging
  tests/base.py:import logging
  tests/unit/cisco/test_network_plugin.py:import logging
  tests/unit/db/metering/test_db_metering.py:import logging
  tests/unit/db/firewall/test_db_firewall.py:import logging
  tests/unit/db/loadbalancer/test_db_loadbalancer.py:import logging
  tests/unit/ml2/test_helpers.py:import logging
  tests/unit/test_servicetype.py:import logging
  tests/unit/vmware/apiclient/test_api_eventlet_request.py:import logging

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