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
diff --git a/DistUpgrade/DistUpgradeController.py b/DistUpgrade/DistUpgradeController.py
index aa69430..9a212f9 100644
--- a/DistUpgrade/DistUpgradeController.py
+++ b/DistUpgrade/DistUpgradeController.py
@@ -1391,7 +1391,13 @@ class DistUpgradeController(object):
1391 """ 1391 """
1392 scripts that are run in any case after the distupgrade finished 1392 scripts that are run in any case after the distupgrade finished
1393 whether or not it was successful1393 whether or not it was successful
1394
1395 Cache lock is released during script runs in the event that the
1396 PostInstallScripts require apt or dpkg changes.
1394 """1397 """
1398 if self.cache:
1399 self.cache.release_lock()
1400 self.cache.unlock_lists_dir()
1395 # now run the post-upgrade fixup scripts (if any)1401 # now run the post-upgrade fixup scripts (if any)
1396 for script in self.config.getlist("Distro","PostInstallScripts"):1402 for script in self.config.getlist("Distro","PostInstallScripts"):
1397 if not os.path.exists(script):1403 if not os.path.exists(script):
@@ -1404,7 +1410,9 @@ class DistUpgradeController(object):
1404 self._view.getTerminal().call([script], hidden=True)1410 self._view.getTerminal().call([script], hidden=True)
1405 except Exception as e:1411 except Exception as e:
1406 logging.error("got error from PostInstallScript %s (%s)" % (script, e))1412 logging.error("got error from PostInstallScript %s (%s)" % (script, e))
1407 1413 if self.cache:
1414 self.cache.get_lock()
1415
1408 def abort(self):1416 def abort(self):
1409 """ abort the upgrade, cleanup (as much as possible) """1417 """ abort the upgrade, cleanup (as much as possible) """
1410 logging.debug("abort called")1418 logging.debug("abort called")
diff --git a/data/mirrors.cfg b/data/mirrors.cfg
index 3adc26d..60c8d39 100644
--- a/data/mirrors.cfg
+++ b/data/mirrors.cfg
@@ -15,6 +15,8 @@ ftp://old-releases.ubuntu.com/
15#commercial (both urls are valid)15#commercial (both urls are valid)
16http://archive.canonical.com16http://archive.canonical.com
17http://archive.canonical.com/ubuntu/17http://archive.canonical.com/ubuntu/
18
19http://ppa.launchpad.net/ua-client/stable/ubuntu
18https://esm.ubuntu.com/ubuntu/20https://esm.ubuntu.com/ubuntu/
19https://esm.ubuntu.com/apps/ubuntu/21https://esm.ubuntu.com/apps/ubuntu/
20https://esm.ubuntu.com/cc/ubuntu/22https://esm.ubuntu.com/cc/ubuntu/
diff --git a/debian/changelog b/debian/changelog
index 5e8f70f..713c0e9 100644
--- a/debian/changelog
+++ b/debian/changelog
@@ -1,3 +1,13 @@
1ubuntu-release-upgrader (1:16.04.32) xenial; urgency=medium
2
3 [ Chad Smith ]
4 * data/mirrors.cfg: add ubuntu advantage pro PPA url as valid mirror
5 (LP: #1893717)
6 * DistUpgrade/DistUpgradeController.py: release cache lock during
7 runPostInstallScripts (LP: #1897778)
8
9 -- Chad Smith <chad.smith@canonical.com> Mon, 05 Oct 2020 14:54:27 -0600
10
1ubuntu-release-upgrader (1:16.04.31) xenial; urgency=medium11ubuntu-release-upgrader (1:16.04.31) xenial; urgency=medium
212
3 * data/mirrors.cfg: add all ubuntu-advantage services as valid mirrors.13 * data/mirrors.cfg: add all ubuntu-advantage services as valid mirrors.
diff --git a/utils/update_mirrors.py b/utils/update_mirrors.py
index f604ad6..05ceadb 100755
--- a/utils/update_mirrors.py
+++ b/utils/update_mirrors.py
@@ -15,8 +15,8 @@ d = feedparser.parse("https://launchpad.net/ubuntu/+archivemirrors-rss")
15#pp = pprint.PrettyPrinter(indent=4)15#pp = pprint.PrettyPrinter(indent=4)
16#pp.pprint(d)16#pp.pprint(d)
1717
18# the first 29 lines are permanent18# the first 31 lines are permanent
19permanent_lines = 2919permanent_lines = 31
20lp_mirrors = set()20lp_mirrors = set()
21new_mirrors = set()21new_mirrors = set()
2222

Subscribers

People subscribed via source and target branches