kvm test failing on 12.04.5 because there is no enum module

Bug #1403933 reported by Jeff Lane 
6
This bug affects 1 person
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-1tko1_.918c2f61bdf3e4aaad0bddf1cf88f95d3bdcb1dd5e30ec7e0d579ed2543510b4/virtualization", line 28, in <module>
    from enum import Enum
ImportError: No module named enum

This occurs when running checkbox-ng (canonical-certification-server) on 12.04 systems.
root@droplet-20141217143817:/usr/lib/2013.com.canonical.certification:checkbox/bin# cat /etc/lsb-release
DISTRIB_ID=Ubuntu
DISTRIB_RELEASE=12.04
DISTRIB_CODENAME=precise
DISTRIB_DESCRIPTION="Ubuntu 12.04.5 LTS"
root@droplet-20141217143817:/usr/lib/2013.com.canonical.certification:checkbox/bin# sudo ./virtualization -h
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-certification.
root@droplet-20141217143817:/usr/share/checkbox/scripts# sudo ./virtualization kvm
Executing KVM Test
WARNING:root:Invalid or Empty timeout in config file. Falling back to default
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/checkbox/bin/virt_debug

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

Changed in plainbox-provider-checkbox:
milestone: none → 0.17
Revision history for this message
Zygmunt Krynicki (zyga) wrote :

Yeah.

Please replace enum by regular integers there. There's no need for the extra magic in this small script IMHO.

Changed in plainbox-provider-checkbox:
status: New → Confirmed
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
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.