Merge ~vorlon/cloud-init:lp.1959343 into cloud-init:main

Proposed by Steve Langasek
Status: Rejected
Rejected by: Chad Smith
Proposed branch: ~vorlon/cloud-init:lp.1959343
Merge into: cloud-init:main
Diff against target: 18 lines (+0/-7)
1 file modified
templates/sources.list.ubuntu.tmpl (+0/-7)
Reviewer Review Type Date Requested Status
Chad Smith Needs Resubmitting
Review via email: mp+414703@code.launchpad.net

Description of the change

From Ubuntu 20.10 on, the partner archive is completely empty; we are now sunsetting it and should no longer put references to it (even commented out) in sources.list.

cloud-init currently uses a single sources.list template in its codebase, which would be used on all host releases of Ubuntu. Since the partner archive still *exists* for Ubuntu 20.04 and earlier, despite being of limited use, removing this for all releases may not be the desired behavior; however, I'm not enthusiastic about writing new code to handle this in jinja for a small corner case unless told to do so.

To post a comment you must log in.
Revision history for this message
Chad Smith (chad.smith) wrote :

I approve this message, we can easily carry a quily patch in our ubuntu/focal & ubuntu/bionic upstream branches to leave this commented out apt source unredacted. No reason to carry that template formard on upstream/main.

Since our git+launchpad repo is simply a mirror of our upstream repo, I'd prefer if we proposed and landed a PR up here https://github.com/canonical/cloud-init/pulls. Then James, Brett or myself will make sure to observe correct/unchanged behavior on 20.04 and 18.04 in our next SRU.

Revision history for this message
Chad Smith (chad.smith) wrote :

Please resubmit to https://github.com/canonical/cloud-init/pulls and we'll get that into upstream. Also, please ensure to add your github-username to tools/.github-cla-signers alphabetically so we can process that PR quickly.

review: Needs Resubmitting

Unmerged commits

6da9265... by Steve Langasek

Do not support setting up archive.canonical.com as a source

Tthe partner archive is now obsolete. LP: #1959343.

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1diff --git a/templates/sources.list.ubuntu.tmpl b/templates/sources.list.ubuntu.tmpl
2index edb92f1..6f92867 100644
3--- a/templates/sources.list.ubuntu.tmpl
4+++ b/templates/sources.list.ubuntu.tmpl
5@@ -43,13 +43,6 @@ deb {{mirror}} {{codename}}-updates multiverse
6 deb {{mirror}} {{codename}}-backports main restricted universe multiverse
7 # deb-src {{mirror}} {{codename}}-backports main restricted universe multiverse
8
9-## Uncomment the following two lines to add software from Canonical's
10-## 'partner' repository.
11-## This software is not part of Ubuntu, but is offered by Canonical and the
12-## respective vendors as a service to Ubuntu users.
13-# deb http://archive.canonical.com/ubuntu {{codename}} partner
14-# deb-src http://archive.canonical.com/ubuntu {{codename}} partner
15-
16 deb {{security}} {{codename}}-security main restricted
17 # deb-src {{security}} {{codename}}-security main restricted
18 deb {{security}} {{codename}}-security universe

Subscribers

People subscribed via source and target branches