Bug 1132677 - Backport patches fixing Python 2.6 compatibility and errors at vdsm-tool rolling out certificates
Summary: Backport patches fixing Python 2.6 compatibility and errors at vdsm-tool roll...
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: oVirt
Classification: Retired
Component: vdsm
Version: 3.5
Hardware: Unspecified
OS: Unspecified
unspecified
high
Target Milestone: ---
: 3.5.0
Assignee: Yaniv Bronhaim
QA Contact: Jiri Belka
URL:
Whiteboard: infra
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2014-08-21 19:36 UTC by Nir Soffer
Modified: 2016-02-10 19:32 UTC (History)
9 users (show)

Fixed In Version: v4.16.4
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2014-10-17 12:44:05 UTC
oVirt Team: Infra
Embargoed:

Attachments (Terms of Use)


Links
System ID Private Priority Status Summary Last Updated
oVirt gerrit 31734 0 None None None Never
oVirt gerrit 31738 0 None None None Never
oVirt gerrit 31785 0 None None None Never
oVirt gerrit 32053 0 ovirt-3.5 MERGED tool: Raise UsageError when used incorrectly Never
oVirt gerrit 32055 0 ovirt-3.5 MERGED tool: Fix error message for non-existing module Never
oVirt gerrit 32056 0 ovirt-3.5 MERGED tool: Fix compatibility with Python 2.6 Never
oVirt gerrit 32057 0 ovirt-3.5 MERGED tool: Fix another Python 2.6 compatibility issue Never
oVirt gerrit 32058 0 ovirt-3.5 MERGED tool: Use space after comma when formatting lists Never
oVirt gerrit 32059 0 ovirt-3.5 MERGED tool: Fix help message when is-configured fails Never
oVirt gerrit 32060 0 None NEW tool: Remove pointless __init__ methods Never
oVirt gerrit 32061 0 ovirt-3.5 MERGED tool: Fix TypeError when configuration check fails Never
oVirt gerrit 32102 0 ovirt-3.5 MERGED tests: Fix Python 2.6 compatibility Never
oVirt gerrit 32103 0 None NEW el6_tests: fix toolTests to use VdsmTestCase so that assertIn is available Never

Description Nir Soffer 2014-08-21 19:36:38 UTC 
Description of problem:
Recent changes in master introduced Python 2.6 compatibility regressions:

commit 096cc03d2f69c04e84fc737104c0c0f4b08ad704
Date:   Wed Aug 13 09:33:22 2014 +0300

    vdsm-tool: simplify getting modules by names.

commit 398d016b58c2fb423be35f7cf1357e2e1c06e64f
Date:   Thu Aug 14 20:46:19 2014 +0300

    vdsm-tool: roll out self signed certificates in vdsm-tool.

How reproducible:
Always

Steps to Reproduce:
1. Run the vdsm-tool on system with Pyhton 2.6 (EL 6.5)

Actual results:
Tool fails with syntax error
Comment 1 Nir Soffer 2014-08-21 19:41:12 UTC 
http://gerrit.ovirt.org/31785 fixes an issue in the tests, not in the tool. But we like to have it as well since without this builds in the CI on EL slaves will break.
Comment 2 Jiri Belka 2014-09-22 10:52:16 UTC 
ok, rhel with python 2.6.6 and vdsm-4.16.4-0.el6.x86_64

details https://bugzilla.redhat.com/show_bug.cgi?id=1127877#c10
Comment 3 Sandro Bonazzola 2014-10-17 12:44:05 UTC 
oVirt 3.5 has been released and should include the fix for this issue.
Note You need to log in before you can comment on or make changes to this bug.