Merge ~chad.smith/ubuntu-release-upgrader:esm-support-pro-upgrades into ubuntu-release-upgrader:ubuntu/master

Proposed by Chad Smith
Status: Merged
Merged at revision: d1ee995bfd8f3e7768af2c6bcd19aa23e7da0e69
Proposed branch: ~chad.smith/ubuntu-release-upgrader:esm-support-pro-upgrades
Merge into: ubuntu-release-upgrader:ubuntu/master
Diff against target: 37 lines (+3/-2)
3 files modified
DistUpgrade/DistUpgradeController.py (+1/-0)
data/mirrors.cfg (+1/-1)
debian/changelog (+1/-1)
Reviewer Review Type Date Requested Status
Brian Murray Approve
Review via email: mp+391725@code.launchpad.net

Description of the change

Followup fixes for ubuntu advantage upgrade support

1. Instead of a private stable PPA for ubuntu-advantage we needed to create a public PPA that didn't require authentication. update the PPA URL in mirrors.cfg to http://ppa.launchpad.net/ua-client/stable/ubuntu

2. DistUpgradeController.py also needs to release the lock /var/lib/apt/lists/lock in runPostInstallScripts using self.cache.unlock_lists_dir()

3. fix d/changelog bug reference for cache lock release

To post a comment you must log in.
Revision history for this message
Brian Murray (brian-murray) wrote :

One in-line comment.

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

> One in-line comment.

@brian-murray:

Confirmed that all public PPAs are only http (and don't support https). So this will remain http unless the following bug gets addressed for public PPAs.

https://bugs.launchpad.net/launchpad/+bug/1473091

Revision history for this message
Chad Smith (chad.smith) :
Revision history for this message
Brian Murray (brian-murray) :
review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1diff --git a/DistUpgrade/DistUpgradeController.py b/DistUpgrade/DistUpgradeController.py
2index 36e8520..18e1ac3 100644
3--- a/DistUpgrade/DistUpgradeController.py
4+++ b/DistUpgrade/DistUpgradeController.py
5@@ -1524,6 +1524,7 @@ class DistUpgradeController(object):
6 """
7 if self.cache:
8 self.cache.release_lock()
9+ self.cache.unlock_lists_dir()
10 # now run the post-upgrade fixup scripts (if any)
11 for script in self.config.getlist("Distro","PostInstallScripts"):
12 if not os.path.exists(script):
13diff --git a/data/mirrors.cfg b/data/mirrors.cfg
14index 54397dd..2d25217 100644
15--- a/data/mirrors.cfg
16+++ b/data/mirrors.cfg
17@@ -18,7 +18,7 @@ ftp://ddebs.ubuntu.com/
18 http://archive.canonical.com
19 http://archive.canonical.com/ubuntu/
20
21-https://private-ppa.launchpad.net/ubuntu-advantage/stable/ubuntu
22+http://ppa.launchpad.net/ua-client/stable/ubuntu
23 https://esm.ubuntu.com/ubuntu/
24 https://esm.ubuntu.com/apps/ubuntu/
25 https://esm.ubuntu.com/cc/ubuntu/
26diff --git a/debian/changelog b/debian/changelog
27index 671d3e8..d3dbae8 100644
28--- a/debian/changelog
29+++ b/debian/changelog
30@@ -4,7 +4,7 @@ ubuntu-release-upgrader (1:20.10.12) groovy; urgency=medium
31 * data/mirrors.cfg: add ubuntu advantage pro PPA url as valid mirror
32 (LP: #1893717)
33 * DistUpgrade/DistUpgradeController.py: release cache lock during
34- runPostInstallScripts (LP: #1893717)
35+ runPostInstallScripts (LP: #1897778)
36
37 -- Chad Smith <chad.smith@canonical.com> Tue, 22 Sep 2020 16:59:52 -0600
38

Subscribers

People subscribed via source and target branches