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

Proposed by Chad Smith
Status: Merged
Merge reported by: Chad Smith
Merged at revision: b1b4a04cd4ffca13bef7a36a57f51ff797e81b47
Proposed branch: ~chad.smith/ubuntu/+source/ubuntu-release-upgrader:uru-xenial-ubuntu-advantage
Merge into: ubuntu/+source/ubuntu-release-upgrader:ubuntu/xenial-proposed
Diff against target: 74 lines (+23/-3)
4 files modified
DistUpgrade/DistUpgradeController.py (+9/-1)
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+391937@code.launchpad.net

Description of the change

Release ubuntu advantage upgrade support.

git cherry-pick 3c23ab918585a9a0e72e60564ba734789f939813 # ubuntu/master
git cherry-pick 1fc540b9f5432539133db61a16af4b495be63a89 # ubuntu/master
git cherry-pick 6b6f79583687e56305a21f88d895925231e553b9 # ubuntu/master
git cherry-pick af4edfce71f3a98946fb06f823968e4aa4e5a4d7 # ubuntu/master
dch -i
git show ubuntu/master:debian/changelog

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

Uploaded to the unapproved 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 aa69430..9a212f9 100644
3--- a/DistUpgrade/DistUpgradeController.py
4+++ b/DistUpgrade/DistUpgradeController.py
5@@ -1391,7 +1391,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@@ -1404,7 +1410,9 @@ 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-
24+ if self.cache:
25+ self.cache.get_lock()
26+
27 def abort(self):
28 """ abort the upgrade, cleanup (as much as possible) """
29 logging.debug("abort called")
30diff --git a/data/mirrors.cfg b/data/mirrors.cfg
31index 3adc26d..60c8d39 100644
32--- a/data/mirrors.cfg
33+++ b/data/mirrors.cfg
34@@ -15,6 +15,8 @@ ftp://old-releases.ubuntu.com/
35 #commercial (both urls are valid)
36 http://archive.canonical.com
37 http://archive.canonical.com/ubuntu/
38+
39+http://ppa.launchpad.net/ua-client/stable/ubuntu
40 https://esm.ubuntu.com/ubuntu/
41 https://esm.ubuntu.com/apps/ubuntu/
42 https://esm.ubuntu.com/cc/ubuntu/
43diff --git a/debian/changelog b/debian/changelog
44index 5e8f70f..713c0e9 100644
45--- a/debian/changelog
46+++ b/debian/changelog
47@@ -1,3 +1,13 @@
48+ubuntu-release-upgrader (1:16.04.32) xenial; urgency=medium
49+
50+ [ Chad Smith ]
51+ * data/mirrors.cfg: add ubuntu advantage pro PPA url as valid mirror
52+ (LP: #1893717)
53+ * DistUpgrade/DistUpgradeController.py: release cache lock during
54+ runPostInstallScripts (LP: #1897778)
55+
56+ -- Chad Smith <chad.smith@canonical.com> Mon, 05 Oct 2020 14:54:27 -0600
57+
58 ubuntu-release-upgrader (1:16.04.31) xenial; urgency=medium
59
60 * data/mirrors.cfg: add all ubuntu-advantage services as valid mirrors.
61diff --git a/utils/update_mirrors.py b/utils/update_mirrors.py
62index f604ad6..05ceadb 100755
63--- a/utils/update_mirrors.py
64+++ b/utils/update_mirrors.py
65@@ -15,8 +15,8 @@ d = feedparser.parse("https://launchpad.net/ubuntu/+archivemirrors-rss")
66 #pp = pprint.PrettyPrinter(indent=4)
67 #pp.pprint(d)
68
69-# the first 29 lines are permanent
70-permanent_lines = 29
71+# the first 31 lines are permanent
72+permanent_lines = 31
73 lp_mirrors = set()
74 new_mirrors = set()
75

Subscribers

People subscribed via source and target branches