actions dir not checked for lint; unused import when checked; unit test fails when lint is resolved.

Bug #1503340 reported by Ryan Beisner
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
ceilometer (Juju Charms Collection)
Fix Released
Undecided
Unassigned
ceilometer-agent (Juju Charms Collection)
Fix Released
Undecided
Unassigned
cinder (Juju Charms Collection)
Fix Released
Undecided
Unassigned
glance (Juju Charms Collection)
Fix Released
Undecided
Unassigned
heat (Juju Charms Collection)
Fix Released
Undecided
Unassigned
neutron-api (Juju Charms Collection)
Fix Released
Undecided
Unassigned
neutron-gateway (Juju Charms Collection)
Fix Released
Undecided
Unassigned
nova-cloud-controller (Juju Charms Collection)
Fix Released
Undecided
Unassigned
openstack-dashboard (Juju Charms Collection)
Fix Released
Undecided
Unassigned

Bug Description

The actions/ dir is not checked for lint by the lint make target.

When lint coverage is added for that dir, there is an unused uuid import error in actions/openstack_upgrade.py.

When that lint issue is resolved, the unit test fails with 'actions/openstack_upgrade.py'> does not have the attribute 'uuid'.

Per #juju irc discussion, I'm temporarily leaving the actions/ dir untested for lint, pending further digging -- except for those charms that are already checking the actions/ dir. They will continue to check for lint there.

Ryan Beisner (1chb1n)
description: updated
James Page (james-page)
Changed in cinder (Juju Charms Collection):
status: New → Fix Released
milestone: none → 16.01
Changed in ceilometer-agent (Juju Charms Collection):
milestone: none → 16.01
Changed in ceilometer (Juju Charms Collection):
milestone: none → 16.01
Changed in glance (Juju Charms Collection):
milestone: none → 16.01
Changed in heat (Juju Charms Collection):
milestone: none → 16.01
Changed in neutron-api (Juju Charms Collection):
milestone: none → 16.01
Changed in neutron-gateway (Juju Charms Collection):
milestone: none → 16.01
Changed in nova-cloud-controller (Juju Charms Collection):
milestone: none → 16.01
Changed in openstack-dashboard (Juju Charms Collection):
milestone: none → 16.01
Changed in glance (Juju Charms Collection):
status: New → Fix Committed
Changed in cinder (Juju Charms Collection):
status: Fix Released → Fix Committed
Changed in neutron-api (Juju Charms Collection):
status: New → Fix Committed
Changed in nova-cloud-controller (Juju Charms Collection):
status: New → Fix Committed
Changed in neutron-gateway (Juju Charms Collection):
status: New → Fix Committed
Changed in openstack-dashboard (Juju Charms Collection):
status: New → Fix Committed
Changed in heat (Juju Charms Collection):
status: New → Fix Committed
Changed in ceilometer (Juju Charms Collection):
status: New → Fix Committed
Changed in ceilometer-agent (Juju Charms Collection):
status: New → Fix Committed
James Page (james-page)
Changed in ceilometer (Juju Charms Collection):
status: Fix Committed → Fix Released
Changed in ceilometer-agent (Juju Charms Collection):
status: Fix Committed → Fix Released
Changed in cinder (Juju Charms Collection):
status: Fix Committed → Fix Released
Changed in glance (Juju Charms Collection):
status: Fix Committed → Fix Released
Changed in heat (Juju Charms Collection):
status: Fix Committed → Fix Released
Changed in neutron-api (Juju Charms Collection):
status: Fix Committed → Fix Released
Changed in neutron-gateway (Juju Charms Collection):
status: Fix Committed → Fix Released
Changed in nova-cloud-controller (Juju Charms Collection):
status: Fix Committed → Fix Released
Changed in openstack-dashboard (Juju Charms Collection):
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

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