Merge ~mwhudson/curtin:chrootabletarget-non-root into curtin:master

[Michael Hudson-Doyle]

have ChrootableTarget ignore allow_daemons when target == "/"

For one thing, disabling deamons in the root filesystem is almost
certainly never wanted and for another this allows ChrootableTarget to
be used with "/" as non-root (for example, allowing you to run
get_efibootmgr("/") as non-root when running subiquity integration
tests).

[Server Team CI bot (server-team-bot)]

PASSED: Continuous integration, rev:3a2577cd816e24848537f9e8ecb3f75f8e2c8c1c
https://jenkins.canonical.com/server-team/job/curtin-ci/156/
Executed test runs:
    SUCCESS: https://jenkins.canonical.com/server-team/job/curtin-ci/nodes=metal-amd64/156/
    SUCCESS: https://jenkins.canonical.com/server-team/job/curtin-ci/nodes=metal-arm64/156/
    SUCCESS: https://jenkins.canonical.com/server-team/job/curtin-ci/nodes=metal-ppc64el/156/
    SUCCESS: https://jenkins.canonical.com/server-team/job/curtin-ci/nodes=metal-s390x/156/

Click here to trigger a rebuild:
https://jenkins.canonical.com/server-team/job/curtin-ci/156//rebuild

review: Approve (continuous-integration)

[Dan Bungert (dbungert)]

I feel like doing curtin stuff in a "chroot" to / is quite strange anyhow, but if this helps tests let's do it.

review: Approve

diff --git a/curtin/util.py b/curtin/util.py
index 025b306..005a521 100644
--- a/curtin/util.py
+++ b/curtin/util.py
@@ -740,7 +740,7 @@ class ChrootableTarget(object):
             if do_mount(p, tpath, opts='--bind'):
                 self.umounts.append(tpath)
 
-        if not self.allow_daemons:
+        if self.target != "/" and not self.allow_daemons:
             self.disabled_daemons = disable_daemons_in_root(self.target)
 
         rconf = paths.target_path(self.target, "/etc/resolv.conf")