kvm test failing on 12.04.5 because there is no enum module
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
Checkbox Provider - Base |
Fix Released
|
Critical
|
Sylvain Pineau |
Bug Description
As of 12.04.5, the kvm test is failing with a traceback:
Traceback (most recent call last):
File "/tmp/nest-
from enum import Enum
ImportError: No module named enum
This occurs when running checkbox-ng (canonical-
root@droplet-
DISTRIB_ID=Ubuntu
DISTRIB_
DISTRIB_
DISTRIB_
root@droplet-
Traceback (most recent call last):
File "./virtualization", line 28, in <module>
from enum import Enum
ImportError: No module named enum
It does work in the older version included in checkbox-
root@droplet-
Executing KVM Test
WARNING:
I: -input-charset not specified, using utf-8 (detected in locale settings)
Total translation table size: 0
Total rockridge attributes bytes: 331
Total directory bytes: 0
Path table size(bytes): 10
Max brk space used 0
183 extents written (0 MB)
INFO:root:Storing VM console output in /usr/lib/
It looks like the enum changes were introduced by Dann Frazier in his ARM updates according to bzr blame:
2957.1.4 dann.fr | from enum import Enum
So that's the point in which we broke the test for 12.04.
Marking this as Critical as it's actually causing havoc on 12.04.5 certifications where we're trying to use c-c-s instead of Checkbox.
Related branches
- Sylvain Pineau (community): Approve
- Jeff Lane : Pending requested
- dann frazier: Pending requested
-
Diff: 126 lines (+22/-19)1 file modifiedproviders/plainbox-provider-checkbox/bin/virtualization (+22/-19)
- Pierre Equoy: Approve
-
Diff: 99 lines (+8/-8)8 files modifiedcheckbox-ng/checkbox_ng/__init__.py (+1/-1)
checkbox-ng/setup.py (+1/-1)
checkbox-support/setup.py (+1/-1)
plainbox/plainbox/__init__.py (+1/-1)
plainbox/setup.py (+1/-1)
providers/plainbox-provider-checkbox/manage.py (+1/-1)
providers/plainbox-provider-resource-generic/manage.py (+1/-1)
providers/plainbox-provider-sru/manage.py (+1/-1)
Changed in plainbox-provider-checkbox: | |
milestone: | none → 0.17 |
Changed in plainbox-provider-checkbox: | |
assignee: | nobody → Sylvain Pineau (sylvain-pineau) |
status: | Confirmed → In Progress |
Changed in plainbox-provider-checkbox: | |
status: | In Progress → Fix Committed |
Changed in plainbox-provider-checkbox: | |
status: | Fix Committed → Fix Released |
Yeah.
Please replace enum by regular integers there. There's no need for the extra magic in this small script IMHO.