Merge ~ahasenack/ubuntu-seeds:groovy-motd-news-config-ubuntu-server into ~ubuntu-core-dev/ubuntu-seeds/+git/ubuntu:groovy

Proposed by Andreas Hasenack on 2020-07-30
Status: Merged
Approved by: Andreas Hasenack on 2020-08-13
Approved revision: 7708cae6f924cafdadd56a16ffc6c8b30b83870b
Merge reported by: Andreas Hasenack
Merged at revision: 7708cae6f924cafdadd56a16ffc6c8b30b83870b
Proposed branch: ~ahasenack/ubuntu-seeds:groovy-motd-news-config-ubuntu-server
Merge into: ~ubuntu-core-dev/ubuntu-seeds/+git/ubuntu:groovy
Diff against target: 9 lines (+1/-0)
1 file modified
server (+1/-0)
Reviewer Review Type Date Requested Status
Bryce Harrington 2020-07-30 Approve on 2020-07-31
Canonical Server Core Reviewers 2020-07-30 Pending
Review via email: mp+388403@code.launchpad.net

This proposal supersedes a proposal from 2020-07-30.

Description of the change

See https://code.launchpad.net/~ahasenack/ubuntu/+source/base-files/+git/base-files/+merge/388400 for details.

The overall plan is to have motd-news only enabled on server systems after this upgrade. What it does:
- the /etc/default/motd-news config file is moved from base-files to a NEW package motd-news-config. If it has modifications, they are preserved
- ubuntu-server has a dependency on motd-news-config (this MP)
- new base-files breaks old ubuntu-server, forcing an upgrade if it is installed

This MP cannot land until motd-news-config is accepted into the archive.

To post a comment you must log in.
Bryce Harrington (bryce) wrote :

LGTM, +1

Ack that the motd-news-config package needs to land first.

review: Approve
Andreas Hasenack (ahasenack) wrote :

base-files for groovy uploaded, merging this.

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1diff --git a/server b/server
2index 9521b15..dcde59b 100644
3--- a/server
4+++ b/server
5@@ -64,3 +64,4 @@ This seed lists the packages we want to install by default in the new server ins
6 * gnupg
7 * dirmngr
8 * (landscape-common) # LP: #1756261, recommends to be removable for minimal (LP: #1763182)
9+ * motd-news-config # LP: #1888575

Subscribers

People subscribed via source and target branches