Merge lp:~zyga/livecd-rootfs/base-snap into lp:livecd-rootfs

Proposed by Zygmunt Krynicki
Status: Rejected
Rejected by: Oliver Grawert
Proposed branch: lp:~zyga/livecd-rootfs/base-snap
Merge into: lp:livecd-rootfs
Diff against target: 37 lines (+16/-1)
2 files modified
debian/changelog (+5/-1)
live-build/auto/build (+11/-0)
To merge this branch: bzr merge lp:~zyga/livecd-rootfs/base-snap
Reviewer Review Type Date Requested Status
Steve Langasek Needs Fixing
Oliver Grawert Approve
Review via email: mp+295754@code.launchpad.net

Description of the change

This branch begins the rename of the "ubuntu-core" snap to "base" snap. For now both snaps are built, with identical contents. Over time the older name will go away.

To post a comment you must log in.
lp:~zyga/livecd-rootfs/base-snap updated
1392. By Zygmunt Krynicki

Build base snap alongside the existing ubuntu-core snap. Both snaps have
the same content, the only difference is in the name (base) and summary.

Revision history for this message
Michael Vogt (mvo) :
lp:~zyga/livecd-rootfs/base-snap updated
1393. By Zygmunt Krynicki

Fix missing snapcraft call, thanks to mvo

Revision history for this message
Oliver Grawert (ogra) wrote :

Looks good

review: Approve
Revision history for this message
Oliver Grawert (ogra) wrote :

Note that cdimage will also need a change to learn about the new "file type" before we can have them supported proper

Revision history for this message
Zygmunt Krynicki (zyga) wrote :

Ogra, where do I have to make the change?

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

there has been internal feedback to call this snap 'core' instead of either 'ubuntu-core' or 'base'.

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

Is there actually a reason to continue building the old snap name? I would prefer to see this replace the old one wholesale. The old name will still exist in the store and we can transition the tooling to use the new name fairly quickly.

Revision history for this message
Oliver Grawert (ogra) wrote :

No, there isn't really, we should just change the name in the meta data and do all the rest on the store side, when I'm back tomorrow I'll take care of this

Revision history for this message
Zygmunt Krynicki (zyga) wrote :

We want to publish both snaps for a while. We're not sure when the switch will happen in snapd and in any case it would be easier if we can rely on this.

Ideally this would be a name change (if we had snap id's) but it is a very tricky thing to do.

Revision history for this message
Zygmunt Krynicki (zyga) wrote :

Steve: I was asked to go ahead with the name "base".

Revision history for this message
Zygmunt Krynicki (zyga) wrote :

Please disregard my last comment.

Unmerged revisions

1393. By Zygmunt Krynicki

Fix missing snapcraft call, thanks to mvo

1392. By Zygmunt Krynicki

Build base snap alongside the existing ubuntu-core snap. Both snaps have
the same content, the only difference is in the name (base) and summary.

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'debian/changelog'
2--- debian/changelog 2016-05-25 12:30:39 +0000
3+++ debian/changelog 2016-05-25 19:59:55 +0000
4@@ -12,7 +12,11 @@
5 * adjust motd message on snappy (capitalize "Snappy Ubuntu Core" and replace
6 the "snappy --help" with "snap --help"
7
8- -- Oliver Grawert <ogra@ubuntu.com> Wed, 25 May 2016 14:26:07 +0200
9+ [ Zygmunt Krynicki ]
10+ * Build base snap alongside the existing ubuntu-core snap. Both snaps have
11+ the same content, the only difference is in the name (base) and summary.
12+
13+ -- Zygmunt Krynicki <zygmunt.krynicki@canonical.com> Wed, 25 May 2016 20:09:18 +0200
14
15 livecd-rootfs (2.410) yakkety; urgency=medium
16
17
18=== modified file 'live-build/auto/build'
19--- live-build/auto/build 2016-05-11 01:18:06 +0000
20+++ live-build/auto/build 2016-05-25 19:59:55 +0000
21@@ -342,6 +342,17 @@
22
23 snapfile="$(ls ubuntu-core*.snap)"
24 cp -a $snapfile $PREFIX.os.snap
25+ # Build the new variant (base) of the same snap too.
26+ cat > $rootfs/meta/snap.yaml <<EOF
27+name: base
28+version: $VERSION
29+summary: Base content for snapd
30+architectures: [$ARCH]
31+type: os
32+EOF
33+ snapcraft snap $rootfs
34+ snapfile="$(ls base*.snap)"
35+ cp -a $snapfile $PREFIX.base.snap
36 fi
37 fi
38

Subscribers

People subscribed via source and target branches