Merge lp:~xnox/debian-cd/subiquity-maas into lp:~ubuntu-cdimage/debian-cd/ubun3

Proposed by Dimitri John Ledkov
Status: Merged
Approved by: Łukasz Zemczak
Approved revision: 1988
Merged at revision: 1989
Proposed branch: lp:~xnox/debian-cd/subiquity-maas
Merge into: lp:~ubuntu-cdimage/debian-cd/ubun3
Diff against target: 18 lines (+8/-0)
1 file modified
tools/add_live_filesystem (+8/-0)
To merge this branch: bzr merge lp:~xnox/debian-cd/subiquity-maas
Reviewer Review Type Date Requested Status
Łukasz Zemczak Approve
Michael Hudson-Doyle (community) Approve
Steve Langasek Needs Information
Review via email: mp+340419@code.launchpad.net

Description of the change

Add maas squashfs for subiquity MAAS install support.

To post a comment you must log in.
lp:~xnox/debian-cd/subiquity-maas updated
1986. By Steve Langasek

Merge lp:~mwhudson/debian-cd/live-server-cmdline-2 (with fixups)

Revision history for this message
Steve Langasek (vorlon) wrote :

Should this be generalized, rather than continuing to hard-code a list of squashfs names? (For comparison, casper itself uses a glob, it does not have a hard-coded list.)

review: Needs Information
Revision history for this message
Michael Hudson-Doyle (mwhudson) :
Revision history for this message
Dimitri John Ledkov (xnox) :
Revision history for this message
Steve Langasek (vorlon) :
lp:~xnox/debian-cd/subiquity-maas updated
1988. By Dimitri John Ledkov

Ship maas squashfs in a subdir.

Revision history for this message
Dimitri John Ledkov (xnox) wrote :

> Should this be generalized, rather than continuing to hard-code a list of squashfs names? (For comparison, casper
> itself uses a glob, it does not have a hard-coded list.)

No, as these are not to be mounted by casper nor to be used as a "live" environment. The live system will mount these elsewhere, as of when needed, for the installation.

Revision history for this message
Dimitri John Ledkov (xnox) wrote :

moved these into a subdir, to avoid casper mounting these.

Revision history for this message
Michael Hudson-Doyle (mwhudson) wrote :

This looks fine to me now.

review: Approve
Revision history for this message
Łukasz Zemczak (sil2100) wrote :

Should be fine.

review: Approve
Revision history for this message
Łukasz Zemczak (sil2100) wrote :

(this is now merged into the main debian-cd branch that's on nusakan).

Revision history for this message
Steve Langasek (vorlon) wrote :

On Tue, Apr 10, 2018 at 09:09:28AM -0000, Dimitri John Ledkov wrote:
> > Should this be generalized, rather than continuing to hard-code a list of squashfs names? (For comparison, casper
> > itself uses a glob, it does not have a hard-coded list.)

> No, as these are not to be mounted by casper nor to be used as a "live"
> environment. The live system will mount these elsewhere, as of when
> needed, for the installation.

Right, but should we need to change debian-cd every time a new squashfs is
added to the output, if it's *not* going to be mounted at boot? Or should
debian-cd take all extra livefses produced and by default stick them in this
subdirectory?

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'tools/add_live_filesystem'
--- tools/add_live_filesystem 2018-02-23 13:07:01 +0000
+++ tools/add_live_filesystem 2018-04-10 09:08:11 +0000
@@ -18,6 +18,14 @@
18 if [ -e "$LIVEIMAGES/$FULLARCH.squashfs.gpg" ]; then18 if [ -e "$LIVEIMAGES/$FULLARCH.squashfs.gpg" ]; then
19 ln "$LIVEIMAGES/$FULLARCH.squashfs.gpg" "$BDIR/CD1/$DIR/filesystem.squashfs.gpg"19 ln "$LIVEIMAGES/$FULLARCH.squashfs.gpg" "$BDIR/CD1/$DIR/filesystem.squashfs.gpg"
20 fi20 fi
21 if [ -e "$LIVEIMAGES/$FULLARCH.maas-rack.squashfs" ]; then
22 mkdir -p "$BDIR/CD1/$DIR/maas/"
23 ln "$LIVEIMAGES/$FULLARCH.maas-rack.squashfs" "$BDIR/CD1/$DIR/maas/maas-rack.squashfs"
24 fi
25 if [ -e "$LIVEIMAGES/$FULLARCH.maas-region.squashfs" ]; then
26 mkdir -p "$BDIR/CD1/$DIR/maas/"
27 ln "$LIVEIMAGES/$FULLARCH.maas-region.squashfs" "$BDIR/CD1/$DIR/maas/maas-region.squashfs"
28 fi
2129
22 if [ -e "$LIVEIMAGES/$FULLARCH.installer.squashfs" ]; then30 if [ -e "$LIVEIMAGES/$FULLARCH.installer.squashfs" ]; then
23 ln "$LIVEIMAGES/$FULLARCH.installer.squashfs" "$BDIR/CD1/$DIR/installer.squashfs"31 ln "$LIVEIMAGES/$FULLARCH.installer.squashfs" "$BDIR/CD1/$DIR/installer.squashfs"

Subscribers

People subscribed via source and target branches