Merge lp:~hloeung/ubuntu-repository-cache/units-stuck-in-maintenance into lp:ubuntu-repository-cache

Proposed by Haw Loeung
Status: Merged
Approved by: Barry Price
Approved revision: 333
Merged at revision: 329
Proposed branch: lp:~hloeung/ubuntu-repository-cache/units-stuck-in-maintenance
Merge into: lp:ubuntu-repository-cache
Diff against target: 19 lines (+2/-0)
1 file modified
reactive/ubuntu_repository_cache.py (+2/-0)
To merge this branch: bzr merge lp:~hloeung/ubuntu-repository-cache/units-stuck-in-maintenance
Reviewer Review Type Date Requested Status
Barry Price Approve
Canonical IS Reviewers Pending
Review via email: mp+399324@code.launchpad.net

Commit message

Fix issue with units stuck in maintenance - LP:1918199

To post a comment you must log in.
Revision history for this message
Canonical IS Mergebot (canonical-is-mergebot) wrote :

This merge proposal is being monitored by mergebot. Change the status to Approved to merge.

Revision history for this message
Barry Price (barryprice) wrote :

+1

review: Approve
Revision history for this message
Canonical IS Mergebot (canonical-is-mergebot) wrote :

Change successfully merged at revision 329

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'reactive/ubuntu_repository_cache.py'
2--- reactive/ubuntu_repository_cache.py 2021-02-16 21:29:13 +0000
3+++ reactive/ubuntu_repository_cache.py 2021-03-08 22:19:08 +0000
4@@ -112,6 +112,7 @@
5 @reactive.when('ubuntu-repository-cache.apache2-restart-required')
6 def apache2_restart():
7 status.maintenance('Restarting apache2 as MPM settings have changed')
8+ reactive.clear_flag('ubuntu-repository-cache.active')
9 host.service_restart('apache2')
10 reactive.clear_flag('ubuntu-repository-cache.apache2-restart-required')
11
12@@ -131,6 +132,7 @@
13 '''Start the charm's services'''
14
15 status.maintenance('Starting %s' % SERVICE)
16+ reactive.clear_flag('ubuntu-repository-cache.active')
17 service.render_configs()
18
19 sync_host = hookenv.config()['sync-host']

Subscribers

People subscribed via source and target branches