FS#46543 - [cloud-init] Add snoonan's patches

Attached to Project: Community Packages
Opened by Johannes Ernst (jernst) - Saturday, 03 October 2015, 01:30 GMT
Last edited by Jonathan Steel (jsteel) - Wednesday, 09 November 2016, 19:31 GMT
Task Type Bug Report
Category Packages
Status Closed
Assigned To Jonathan Steel (jsteel)
Architecture All
Severity Low
Priority Normal
Reported Version
Due in Version Undecided
Due Date Undecided
Percent Complete 100%
Votes 0
Private No

Details

I'd like to suggest that the patches here [1] should be merged into this package. They are sensible and work well on EC2.
Note that [2] is currently the only maintained Arch build that I'm aware of that works on EC2.

[1] http://git.uplinklabs.net/snoonan/projects/archlinux/ec2/ec2-packages.git/tree/cloud-init
[2] https://www.uplinklabs.net/projects/arch-linux-on-ec2/
This task depends upon

Closed by  Jonathan Steel (jsteel)
Wednesday, 09 November 2016, 19:31 GMT
Reason for closing:  Implemented
Additional comments about closing:  0.7.8-2
Comment by Doug Newgard (Scimmia) - Saturday, 03 October 2015, 02:16 GMT
Disclaimer: I have not looked at the patches

Arch tends not to patch packages unless it's critical for usability, especially if those patches haven't been accepted upstream. Can you explain why these patches are critical?
Comment by Johannes Ernst (jernst) - Sunday, 04 October 2015, 21:02 GMT
* without fix-logging.patch, errors occur when attempting to set up logging on Arch (cannot write to file)
* fix-python-version.patch makes it invoke python2 instead of python (I don't have first-hand knowledge of whether that is needed but chances are it is)
* systemd-network-online.patch: wait until network is present before attempting to run cloud-init, otherwise may fail
* packaged file archlinux.cloud.cfg actually claims it is Ubuntu, not Arch (snoonan's patches don't fix that either :-()

Three (perhaps all) of those are Arch-specific.
Comment by Jonathan Steel (jsteel) - Wednesday, 09 November 2016, 18:41 GMT
The systemd-network-online patch doesn't look necessary now, the changes must have been made upstream since then.

The Arch PKGBUILD already changes the python version.

I'll add the logging patch.

Loading...