Merge ~paride/ubuntu-seeds/+git/platform:seed-probert into ~ubuntu-core-dev/ubuntu-seeds/+git/platform:groovy

Proposed by Paride Legovini
Status: Merged
Approved by: Christian Ehrhardt 
Approved revision: fb93501a65eef2e90192d7a16845c6bdee4f99cc
Merged at revision: fb93501a65eef2e90192d7a16845c6bdee4f99cc
Proposed branch: ~paride/ubuntu-seeds/+git/platform:seed-probert
Merge into: ~ubuntu-core-dev/ubuntu-seeds/+git/platform:groovy
Diff against target: 26 lines (+5/-1)
2 files modified
STRUCTURE (+2/-1)
supported-installer-server (+3/-0)
Reviewer Review Type Date Requested Status
Christian Ehrhardt  Approve
Review via email: mp+390737@code.launchpad.net
To post a comment you must log in.
Revision history for this message
Paride Legovini (paride) wrote :

Prior to this change germinate pulled in:

probert-common | probert | probert-storage | Ubuntu Developers <email address hidden> | 17064 | 118

probert-storage | probert | curtin | Ubuntu Developers <email address hidden> | 13864 | 63

with this change:

probert | probert | Ubuntu.Groovy supported-installer-server seed | Ubuntu Developers <email address hidden> | 1768 | 12

probert-common | probert | probert-storage | Ubuntu Developers <email address hidden> | 17064 | 118

probert-storage | probert | probert | Ubuntu Developers <email address hidden> | 13864 | 63

with:

? Unknown dependency probert-network by probert

which has to be included in main (change-override).

Revision history for this message
Paride Legovini (paride) wrote :

First time I touch the seeds, so pedantic reviews are appreciated :)

Revision history for this message
Christian Ehrhardt  (paelzer) wrote :

I was looking into the underlying case which is #1890301 and agree with the approach in general.

I was doing a pre/post germinate run as well:
+probert | probert | Ubuntu.Groovy supported-installer-server seed
Ok we want that (more clear than the current off i386 only dependency)
+? Unknown dependency probert-network by probert on arch amd64
This is due tot he component mismatch and running main-only.

Running the same pre/post with universe enabled:
Now we get the expected component mismatch:
  10 +probert | probert | Ubuntu.Groovy supported-installer-server seed | Ubuntu Developers <<email address hidden>
  11 +probert-common | probert | probert-network | Ubuntu Developers <<email address hidden>
  12 +probert-network | probert | probert | Ubuntu Developers <<email address hidden>
  13 +probert-storage | probert | probert | Ubuntu Developers

Addition of the new supported-installer-server - ack
It is listed in the right places.

I see no warnings about this new seed:
 117 +* Downloading file:///home/paelzer/work/seeds/ubuntu.groovy/supported-installer-server
 118 +* Downloading file:///home/paelzer/work/seeds/ubuntu.groovy/languages/supported-installer-server
 119 +* Downloading file:///home/paelzer/work/seeds/platform.groovy/supported-installer-server
works fine and is used
We get all the auxiliary files for supported-installer-server that I'd expect like e.g. supported-installer-server.depends

As a side effect probert source will move from the general "supported.sources" to "supported-installer-server.sources" which is fine I think

The probert meta package LGTM as start for all subpackages and the source.
Much better than missing it like now but still having it in the installer snap build.

Overall LGTM +1

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1diff --git a/STRUCTURE b/STRUCTURE
2index 2924088..22cc1c0 100644
3--- a/STRUCTURE
4+++ b/STRUCTURE
5@@ -26,10 +26,11 @@ supported-hardware-desktop: standard
6 supported-hardware-server: standard
7 supported-installer-common: standard
8 supported-installer-desktop: standard
9+supported-installer-server: standard
10 supported-network-common: standard
11 supported-network-client: standard
12 supported-sysadmin-common: standard
13 supported-sysadmin-server: standard
14-supported-server: supported-misc-servers supported-hardware-common supported-hardware-server supported-installer-common supported-network-common supported-sysadmin-common supported-sysadmin-server build-essential supported-development-common supported-kernel-common supported-cloud supported-maas supported-landscape supported-openstack
15+supported-server: supported-misc-servers supported-hardware-common supported-hardware-server supported-installer-common supported-installer-server supported-network-common supported-sysadmin-common supported-sysadmin-server build-essential supported-development-common supported-kernel-common supported-cloud supported-maas supported-landscape supported-openstack
16 supported-desktop: supported-hardware-desktop supported-installer-desktop supported-hardware-common supported-installer-common supported-network-common supported-network-client supported-sysadmin-common build-essential supported-development-common supported-development-desktop supported-kernel-common supported-kernel-desktop
17 supported-common: supported-server supported-desktop
18diff --git a/supported-installer-server b/supported-installer-server
19new file mode 100644
20index 0000000..ef12be9
21--- /dev/null
22+++ b/supported-installer-server
23@@ -0,0 +1,3 @@
24+= Installer =
25+
26+ * probert # needed by subiquity

Subscribers

People subscribed via source and target branches