Looking really good for a WIP, just a small comment :) Diff comments:
> diff --git a/lpbuildd/target/build_livefs.py b/lpbuildd/target/build_livefs.py > index d77e1cf..59e936b 100644 > --- a/lpbuildd/target/build_livefs.py > +++ b/lpbuildd/target/build_livefs.py > @@ -177,6 +206,10 @@ class BuildLiveFS(SnapStoreOperationMixin, Operation): > base_lb_env["REPO_SNAPSHOT_STAMP"] = ( > self.args.repo_snapshot_stamp > ) > + if self.args.snapshot_service_timestamp: > + base_lb_env["SNAPSHOT_SERVICE_TIMESTAMP"] = ( Do we need to set a env variable? Looking into https://snapshot.ubuntu.com/ it doesn't mention it been needed. If it's not needed, I would remove this bit > + self.args.snapshot_service_timestamp > + ) > if self.args.cohort_key: > base_lb_env["COHORT_KEY"] = self.args.cohort_key > lb_env = base_lb_env.copy() -- https://code.launchpad.net/~pelpsi/launchpad-buildd/+git/launchpad-buildd/+merge/472806 Your team Launchpad code reviewers is requested to review the proposed merge of ~pelpsi/launchpad-buildd:snapshot-service-integration-for-livefs-builds into launchpad-buildd:master. _______________________________________________ Mailing list: https://launchpad.net/~launchpad-reviewers Post to : launchpad-reviewers@lists.launchpad.net Unsubscribe : https://launchpad.net/~launchpad-reviewers More help : https://help.launchpad.net/ListHelp