curtin-dev team mailing list archive

[Dan Bungert <mp+412778@xxxxxxxxxxxxxxxxxx>]

Dan Bungert has proposed merging ~dbungert/curtin:system-upgrade into curtin:master.

Commit message:
system-upgrade: lookup os family

`curtin system-upgrade` needs an osfamily lookup, as the command itself
neither takes an argument of the osfamily nor attempts to provide it.

Requested reviews:
  curtin developers (curtin-dev)

For more details, see:
https://code.launchpad.net/~dbungert/curtin/+git/curtin/+merge/412778
-- 
Your team curtin developers is requested to review the proposed merge of ~dbungert/curtin:system-upgrade into curtin:master.

diff --git a/curtin/distro.py b/curtin/distro.py
index 82a4dd5..6f94790 100644
--- a/curtin/distro.py
+++ b/curtin/distro.py
@@ -371,6 +371,9 @@ def system_upgrade(opts=None, target=None, env=None, allow_daemons=False,
                    osfamily=None):
     LOG.debug("Upgrading system in %s", target)
 
+    if not osfamily:
+        osfamily = get_osfamily(target=target)
+
     distro_cfg = {
         DISTROS.debian: {'function': run_apt_command,
                          'subcommands': ('dist-upgrade', 'autoremove')},