Merge ~chad.smith/ubuntu/+source/ubuntu-release-upgrader:uru-bionic-ubuntu-advantage into ubuntu/+source/ubuntu-release-upgrader:ubuntu/bionic-proposed

Proposed by Chad Smith
Status: Merged
Merge reported by: Chad Smith
Merged at revision: 68454aeb9f65269187adc33785966ba5d658b9fd
Proposed branch: ~chad.smith/ubuntu/+source/ubuntu-release-upgrader:uru-bionic-ubuntu-advantage
Merge into: ubuntu/+source/ubuntu-release-upgrader:ubuntu/bionic-proposed
Diff against target: 72 lines (+22/-2)
4 files modified
DistUpgrade/DistUpgradeController.py (+8/-0)
data/mirrors.cfg (+2/-0)
debian/changelog (+10/-0)
utils/update_mirrors.py (+2/-2)
Reviewer Review Type Date Requested Status
Brian Murray (community) Approve
git-ubuntu developers Pending
Review via email: mp+391848@code.launchpad.net

Description of the change

WIP BRANCH until TBD Focal PR can be proposed

SRU of ESM upgrade support locks and official public PPA mirror

4 cherry picks:

git cherry-pick 3c23ab918585a9a0e72e60564ba734789f939813 # ubuntu/master
git cherry-pick 1fc540b9f5432539133db61a16af4b495be63a89 # ubuntu/master
git cherry-pick 6b6f79583687e56305a21f88d895925231e553b9 # ubuntu/master
git cherry-pick af4edfce71f3a98946fb06f823968e4aa4e5a4d7 # ubuntu/master

Add changlog entry:
  [ Chad Smith ]
  * data/mirrors.cfg: add ubuntu advantage pro PPA url as valid mirror
    (LP: #1893717)
  * DistUpgrade/DistUpgradeController.py: release cache lock during
    runPostInstallScripts (LP: #1897778)

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

I've uploaded these changes to the -proposed queue for review by the SRU team.

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 97eeba4..7e2bb97 100644
3--- a/DistUpgrade/DistUpgradeController.py
4+++ b/DistUpgrade/DistUpgradeController.py
5@@ -1504,7 +1504,13 @@ class DistUpgradeController(object):
6 """
7 scripts that are run in any case after the distupgrade finished
8 whether or not it was successful
9+
10+ Cache lock is released during script runs in the event that the
11+ PostInstallScripts require apt or dpkg changes.
12 """
13+ if self.cache:
14+ self.cache.release_lock()
15+ self.cache.unlock_lists_dir()
16 # now run the post-upgrade fixup scripts (if any)
17 for script in self.config.getlist("Distro","PostInstallScripts"):
18 if not os.path.exists(script):
19@@ -1517,6 +1523,8 @@ class DistUpgradeController(object):
20 self._view.getTerminal().call([script], hidden=True)
21 except Exception as e:
22 logging.error("got error from PostInstallScript %s (%s)" % (script, e))
23+ if self.cache:
24+ self.cache.get_lock()
25
26 def abort(self):
27 """ abort the upgrade, cleanup (as much as possible) """
28diff --git a/data/mirrors.cfg b/data/mirrors.cfg
29index e785a18..afb1b2b 100644
30--- a/data/mirrors.cfg
31+++ b/data/mirrors.cfg
32@@ -15,6 +15,8 @@ ftp://old-releases.ubuntu.com/
33 #commercial (both urls are valid)
34 http://archive.canonical.com
35 http://archive.canonical.com/ubuntu/
36+
37+http://ppa.launchpad.net/ua-client/stable/ubuntu
38 https://esm.ubuntu.com/ubuntu/
39 https://esm.ubuntu.com/apps/ubuntu/
40 https://esm.ubuntu.com/cc/ubuntu/
41diff --git a/debian/changelog b/debian/changelog
42index 137ee4a..d6309e9 100644
43--- a/debian/changelog
44+++ b/debian/changelog
45@@ -1,3 +1,13 @@
46+ubuntu-release-upgrader (1:18.04.40) bionic; urgency=medium
47+
48+ [ Chad Smith ]
49+ * data/mirrors.cfg: add ubuntu advantage pro PPA url as valid mirror
50+ (LP: #1893717)
51+ * DistUpgrade/DistUpgradeController.py: release cache lock during
52+ runPostInstallScripts (LP: #1897778)
53+
54+ -- Chad Smith <chad.smith@canonical.com> Mon, 05 Oct 2020 14:50:15 -0600
55+
56 ubuntu-release-upgrader (1:18.04.39) bionic; urgency=medium
57
58 * data/mirrors.cfg: add all ubuntu-advantage services as valid mirrors.
59diff --git a/utils/update_mirrors.py b/utils/update_mirrors.py
60index f604ad6..05ceadb 100755
61--- a/utils/update_mirrors.py
62+++ b/utils/update_mirrors.py
63@@ -15,8 +15,8 @@ d = feedparser.parse("https://launchpad.net/ubuntu/+archivemirrors-rss")
64 #pp = pprint.PrettyPrinter(indent=4)
65 #pp.pprint(d)
66
67-# the first 29 lines are permanent
68-permanent_lines = 29
69+# the first 31 lines are permanent
70+permanent_lines = 31
71 lp_mirrors = set()
72 new_mirrors = set()
73

Subscribers

People subscribed via source and target branches