Merge lp:~xnox/utah/fix-chroot into lp:utah

[Dimitri John Ledkov]

This should hopefully fix bug 1061242 .
Instead of using manual chroot use the supported wrapper that does a log of checks and correctly enters /target in a way that will allow package installation. In particular it bind-mounts /run for DNS resolution.

This backs out the previous "fix" which overrode /etc/resolv.conf symlink pointing into /run.

I have tried to run the desktop smoke run_list locally, but I did not succeed in setting up utah to execute the test correctly for me.

Please test this about raring desktop smoke test, or review for sanity & merge and re-trigger jenkins to run to see if it finally resolves the desktop smoke jobs.

[Max Brustkern (nuclearbob)]

I'm testing this on my workstation on VMs, and I'm preparing to test it on bare metal. I'll update this with the results.

[Max Brustkern (nuclearbob)]

This has tested okay on conventional hardware. Unless anyone else on the team objects, I'd like to merge it.

review: Approve

[Javier Collado (javier.collado)]

Merging as per comment above.

=== modified file 'utah/provisioning/provisioning.py'
--- utah/provisioning/provisioning.py	2012-12-06 15:19:55 +0000
+++ utah/provisioning/provisioning.py	2012-12-06 17:50:31 +0000
@@ -1043,9 +1043,8 @@
                 .format(install_commands)
             )
             chroot_command = ' '.join([install_commands, postinstall_commands])
-            question.value.prepend('cp /etc/resolv.conf '
-                '/target/etc/resolv.conf ; chroot /target sh -c \'{}\'; '
-                .format(chroot_command))
+            question.value.prepend('in-target sh -c \'{}\'; '
+                                   .format(chroot_command))
             question.prepend("ubiquity ubiquity/summary note")
             question.prepend("ubiquity ubiquity/reboot boolean true")
         else: