Python 3 support and dropping cheetah

Bug #1247132 reported by Miro Hrončok
12
This bug affects 2 people
Affects Status Importance Assigned to Milestone
cloud-init
Fix Released
Low
Joshua Harlow

Bug Description

Hi,
in Fedora, we would like to use cloud-init with Python 3. While we are willing to help porting boto and cloud-init to support both Python 2.6/2.7 and 3.3, cheetah is the thing we won't like to port, as it is ooooold.

So the question is: Would you accept patches that would introduce support for Python 3 and replace cheetah with a different templating system, such as jinja? If not we would need to come with our own solution (either repalce cloud-init or fork it) and I would not like to do that very much.

Thanks for info

Related bugs:
 * bug 1219223: Is it possible that we switch from cheetah to a lighter weight template engine?

Related branches

Revision history for this message
Matthew Miller (mattdm) wrote :
Revision history for this message
Miro Hrončok (churchyard) wrote :

Oh, thanks

Scott Moser (smoser)
description: updated
Revision history for this message
Scott Moser (smoser) wrote :

I think mostly my comment in bug 1219223 comment 3 [1] covers my response.
I also have interest in python3. I do not expect to move cloud-init in Ubuntu off python2 for 14.04, but would be very happy if it supported both by then.

So, things to cover with respect to cheetah are:
 * support backwards compatible parsing (do not need to support any wheres near all of cheetah function,
 but need to support $MIRROR and other simple places)
 * declare in the format template ie, either 'startswith("#templatetypehere\n")' or possibly searching by file extension.

--
[1] https://bugs.launchpad.net/cloud-init/+bug/1219223/comments/3 mostly stands.

Scott Moser (smoser)
Changed in cloud-init:
status: New → Triaged
importance: Undecided → Critical
importance: Critical → Low
Revision history for this message
Dan Watkins (oddbloke) wrote :

This will also need mocker to be replaced with something else (presumably mock), as covered in bug 1258720.

Revision history for this message
Bohuslav "Slavek" Kabrda (bkabrda) wrote :

There is a py-2-3 patch waiting for review at https://code.launchpad.net/~harlowja/cloud-init/py2-3. It LGTM, can someone look at it and provide review to get this effor moving again? Thanks.

Scott Moser (smoser)
Changed in cloud-init:
status: Triaged → Fix Committed
Joshua Harlow (harlowja)
Changed in cloud-init:
milestone: none → 0.7.7
assignee: nobody → Joshua Harlow (harlowja)
Scott Moser (smoser)
Changed in cloud-init:
status: Fix Committed → Fix Released
Revision history for this message
James Falcon (falcojr) wrote :
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.