Merge lp:~mthaddon/mojo/ntp-focal into lp:mojo/mojo-specs

Proposed by Tom Haddon
Status: Merged
Approved by: Gareth Woolridge
Approved revision: 155
Merged at revision: 153
Proposed branch: lp:~mthaddon/mojo/ntp-focal
Merge into: lp:mojo/mojo-specs
Diff against target: 54 lines (+25/-17)
1 file modified
charm-testing/ntp/post-deploy (+25/-17)
To merge this branch: bzr merge lp:~mthaddon/mojo/ntp-focal
Reviewer Review Type Date Requested Status
Mojo Maintainers Pending
Review via email: mp+379512@code.launchpad.net

Commit message

charm-testing/ntp: if we're on focal, only check chrony

Description of the change

charm-testing/ntp: if we're on focal, only check chrony.

Also some re-indenting for clarity.

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

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

Revision history for this message
🤖 Canonical IS Merge Bot (canonical-is-mergebot) wrote :

Merge proposal is approved, but source revision has changed, setting status to needs review.

Revision history for this message
🤖 Canonical IS Merge Bot (canonical-is-mergebot) wrote :

Change successfully merged at revision 153

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'charm-testing/ntp/post-deploy'
2--- charm-testing/ntp/post-deploy 2018-09-03 09:35:45 +0000
3+++ charm-testing/ntp/post-deploy 2020-02-19 23:01:18 +0000
4@@ -14,25 +14,33 @@
5 }
6
7 case $MOJO_SERIES in
8+ focal)
9+ # After bionic: only chrony
10+ chronyd_check
11+ ;;
12 trusty|xenial)
13- # Before bionic: only ntpd
14- ntpd_check
15- ;;
16+ # Before bionic: only ntpd
17+ ntpd_check
18+ ;;
19+ bionic)
20+ # bionic: pre-upgrade == ntpd, post-upgrade == chronyd
21+ case $PHASE in
22+ upgrade|redeploy)
23+ chronyd_check
24+ ;;
25+ initial)
26+ ntpd_check
27+ ;;
28+ *)
29+ echo "ERROR: unknown phase"
30+ exit 1
31+ ;;
32+ esac
33+ ;;
34 *)
35- # bionic and later: pre-upgrade == ntpd, post-upgrade == chronyd
36- case $PHASE in
37- upgrade|redeploy)
38- chronyd_check
39- ;;
40- initial)
41- ntpd_check
42- ;;
43- *)
44- echo "ERROR: unknown phase"
45- exit 1
46- ;;
47- esac
48- ;;
49+ echo "ERROR: unknown series"
50+ exit 1
51+ ;;
52 esac
53
54 echo "######################### Post-deploy done #########################"

Subscribers

People subscribed via source and target branches

to all changes: