builder-reimage: retain network configuration when commissioning

Pass skip_networking=True to machine.commission(), the API equivalent
of checking "Retain network configuration" in the MAAS UI, so a
machine commissioned by the reimage job keeps its existing interface
configuration instead of having it replaced by whatever commissioning
discovers.

Signed-off-by: David Galloway <david.galloway@ibm.com>

diff --git a/builder-reimage/build/Jenkins_builder-reimage.py b/builder-reimage/build/Jenkins_builder-reimage.py
index bc7d627f47635ceab2d21a486f4e31e1d8aa3479..32c0e79875ce52515efbcfa6151920e0eae65e03 100755 (executable)
--- a/builder-reimage/build/Jenkins_builder-reimage.py
+++ b/builder-reimage/build/Jenkins_builder-reimage.py
@@ -150,7 +150,10 @@ async def release_machine(machine):
 # -------------------------------------------------------------------------
 async def commission_machine(machine):
     if get_normalized_status(machine) == "new":
-        await machine.commission()
+        # skip_networking is the API equivalent of checking "Retain network
+        # configuration" in the MAAS UI: keep the machine's existing interface
+        # config instead of replacing it with what commissioning discovers.
+        await machine.commission(skip_networking=True)
 
 # -------------------------------------------------------------------------
 # Reimage