distro/arch.py fixes

Bug #1555605 reported by Paul Melis
This bug report is a duplicate of:  Bug #1706593: Archlinux Bugs. Edit Remove
10
This bug affects 2 people
Affects Status Importance Assigned to Milestone
cloud-init
Incomplete
Medium
Unassigned

Bug Description

I needed to patch a few places in distros/arch.py to get cloud-init to work on Arch. This is with an OpenNebula-based cloud, but that shouldn't matter. In /etc/cloud/cloud.cfg I obviously used "arch" for system_info -> distro.

I checked against current trunk version of distros/arch.py. There's 3 issues in arch.py that this patch fixes (in order of appearance in the patch):

1. The loopback device is returned as an entry by net_util.translate_network(), leading to it gettingprocessed to configure settings like IP address, netmask, etc, but these aren't available and leads to an exception. I simply removed the "lo" entry from the dictionary.

2. The netctl command expects its profile files to be named /etc/netctl/<iface>. The code in arch.py leaves out the last separator, leading to files like /etc/netctleth0, which doesn't work

3. The function convert_resolv_conf() can't possibly work, as it tries to iterate over the "list" type instead of the "settings" list.

Revision history for this message
Paul Melis (paul-floorball-flamingos) wrote :
Revision history for this message
Scott Moser (smoser) wrote :

Paul,
Thanks for the patch. I can pull this, but you'll need to sign the Canonical contributors agreement.
http://www.ubuntu.com/legal/contributors
please feel free to ping me in irc (freenode smoser) or email if you have questions.

Revision history for this message
Joshua Powers (powersj) wrote :

Hey Paul, any chance you were able to sign the contributor agreement?

Changed in cloud-init:
status: New → Incomplete
importance: Undecided → Medium
Revision history for this message
Paul Melis (paul-floorball-flamingos) wrote :

I'd rather not sign the CLA. Is placing the patch in the public domain a good alternative?

Revision history for this message
Rich Lees (rlees85) wrote :

I really would like to see this merged. Can I do anything to help? I will agree to the agreement as long as there is nothing stupid in there.

Revision history for this message
Rich Lees (rlees85) wrote :

Update: I signed the agreement. I was going to make a patch file exactly the same as this for the same reasons. If it could be assumed "mine" and merged that would be great. Alternatively, I would raise another bug with an identical patch file attached.

Please advice on how to proceed... Would love for this to make the next release...

Revision history for this message
Rich Lees (rlees85) wrote :
Revision history for this message
Scott Moser (smoser) wrote :

I'm marking this a duplicate of the newly filed 1706593. See discussion there.

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.