launchpad-reviewers team mailing list archive

[Tobias Koch <tobias.koch@xxxxxxxxxxxxx>]

Tobias Koch has proposed merging lp:~tobijk/launchpad-buildd/repo-snapshot-stamp into lp:launchpad-buildd.

Commit message:
buildlivefs: support passing REPO_SNAPSHOT_STAMP variable into environment
  
This is used to generate images with identical version of Debian packages
during parallelized image builds.


Requested reviews:
  Launchpad code reviewers (launchpad-reviewers)

For more details, see:
https://code.launchpad.net/~tobijk/launchpad-buildd/repo-snapshot-stamp/+merge/361191

The corresponding MP for livecd-rootfs is here:

https://code.launchpad.net/~tobijk/livecd-rootfs/+git/livecd-rootfs/+merge/360984
-- 
Your team Launchpad code reviewers is requested to review the proposed merge of lp:~tobijk/launchpad-buildd/repo-snapshot-stamp into lp:launchpad-buildd.

=== modified file 'debian/changelog'
--- debian/changelog	2018-11-07 11:48:18 +0000
+++ debian/changelog	2018-12-20 13:13:14 +0000
@@ -5,6 +5,9 @@
 
   [ Tobias Koch ]
   * Update LXD backend to work with LXD 3.
+  * buildlivefs: support passing a REPO_SNAPSHOT_STAMP variable into the
+    environment, used to generate images with identical version of Debian
+    packages during parallelized image builds.
 
  -- Colin Watson <cjwatson@xxxxxxxxxx>  Thu, 25 Oct 2018 10:03:14 +0100
 

=== modified file 'lpbuildd/livefs.py'
--- lpbuildd/livefs.py	2018-10-19 06:46:23 +0000
+++ lpbuildd/livefs.py	2018-12-20 13:13:14 +0000
@@ -38,6 +38,7 @@
         self.extra_ppas = extra_args.get("extra_ppas", [])
         self.channel = extra_args.get("channel")
         self.image_targets = extra_args.get("image_targets", [])
+        self.repo_snapshot_stamp = extra_args.get("repo_snapshot_stamp")
         self.debug = extra_args.get("debug", False)
 
         super(LiveFilesystemBuildManager, self).initiate(
@@ -65,6 +66,8 @@
             args.extend(["--channel", self.channel])
         for image_target in self.image_targets:
             args.extend(["--image-target", image_target])
+        if self.repo_snapshot_stamp:
+            args.extend(["--repo-snapshot-stamp", self.repo_snapshot_stamp])
         if self.debug:
             args.append("--debug")
         self.runTargetSubProcess("buildlivefs", *args)

=== modified file 'lpbuildd/target/build_livefs.py'
--- lpbuildd/target/build_livefs.py	2018-10-09 09:46:07 +0000
+++ lpbuildd/target/build_livefs.py	2018-12-20 13:13:14 +0000
@@ -53,6 +53,10 @@
             action="append", metavar="TARGET",
             help="produce image for TARGET")
         parser.add_argument(
+            "--repo-snapshot-stamp", dest="repo_snapshot_stamp",
+            metavar="TIMESTAMP",
+            help="build against package repo state at TIMESTAMP"
+        parser.add_argument(
             "--proposed", default=False, action="store_true",
             help="enable use of -proposed pocket")
         parser.add_argument(
@@ -131,6 +135,9 @@
             if self.args.image_targets:
                 base_lb_env["IMAGE_TARGETS"] = " ".join(
                     self.args.image_targets)
+            if self.args.repo_snapshot_stamp:
+                base_lb_env["REPO_SNAPSHOT_STAMP"] = \
+                        self.args.repo_snapshot_stamp
             lb_env = base_lb_env.copy()
             lb_env["SUITE"] = self.args.series
             if self.args.datestamp is not None: