Merge lp:~powersj/utah/server-priority into lp:utah

Proposed by Joshua Powers
Status: Merged
Approved by: Jean-Baptiste Lallement
Approved revision: 1110
Merged at revision: 1110
Proposed branch: lp:~powersj/utah/server-priority
Merge into: lp:utah
Diff against target: 12 lines (+1/-1)
1 file modified
utah/provisioning/provisioning.py (+1/-1)
To merge this branch: bzr merge lp:~powersj/utah/server-priority
Reviewer Review Type Date Requested Status
Jean-Baptiste Lallement Approve
Canonical CI Engineering Pending
Review via email: mp+333985@code.launchpad.net

Description of the change

server: change debconf/priority to priority

As discovered with the bionic release, the value set by debconf/priority
is not being respected and therefore the priority is actually high and
not critical. This change uses priority only to properly set the
priority in a manner that will be respected.

To post a comment you must log in.
Revision history for this message
Joshua Powers (powersj) wrote :

See LP: #1732776 for more details

Revision history for this message
Jean-Baptiste Lallement (jibel) wrote :

LGTM. thanks

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'utah/provisioning/provisioning.py'
2--- utah/provisioning/provisioning.py 2017-07-06 18:03:17 +0000
3+++ utah/provisioning/provisioning.py 2017-11-20 17:59:36 +0000
4@@ -960,7 +960,7 @@
5 else:
6 parameters.extend([
7 ('DEBCONF_DEBUG', 'developer'),
8- ('debconf/priority', 'critical'),
9+ ('priority', 'critical'),
10 ])
11 for option in tuple(options):
12 if option not in self.cmdline:

Subscribers

People subscribed via source and target branches

to all changes: