Merge lp:~mthaddon/mojo/remove-check-juju into lp:mojo/mojo-specs

Proposed by Tom Haddon
Status: Merged
Merged at revision: 15
Proposed branch: lp:~mthaddon/mojo/remove-check-juju
Merge into: lp:mojo/mojo-specs
Diff against target: 45 lines (+3/-8)
4 files modified
font-ubuntu-com/manifest-verify (+1/-1)
juju-intro/manifest-verify (+1/-1)
mojo-how-to/manifest-verify (+1/-1)
mojo-spec-helpers/tests/check-juju (+0/-5)
To merge this branch: bzr merge lp:~mthaddon/mojo/remove-check-juju
Reviewer Review Type Date Requested Status
Laurent Sesquès Approve
Review via email: mp+281073@code.launchpad.net

Description of the change

Mojo now supports a "juju-check-wait" phase, which confirms the environment has reached a steady state and that juju status is clean. Update fonts.u.c and mojo.c.c specs to use that.

To post a comment you must log in.
lp:~mthaddon/mojo/remove-check-juju updated
16. By Tom Haddon

Also need to remove from juju-intro

Revision history for this message
Laurent Sesquès (sajoupa) wrote :

looks good.

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'font-ubuntu-com/manifest-verify'
--- font-ubuntu-com/manifest-verify 2015-11-27 13:16:34 +0000
+++ font-ubuntu-com/manifest-verify 2015-12-21 09:52:48 +0000
@@ -1,4 +1,4 @@
1# Check juju1# Check juju
2verify config=scripts/check-juju2juju-check-wait
3# The service is up and running, let's verify it3# The service is up and running, let's verify it
4verify config=scripts/verify-nrpe4verify config=scripts/verify-nrpe
55
=== removed symlink 'font-ubuntu-com/scripts/check-juju'
=== target was u'../../mojo-spec-helpers/tests/check-juju'
=== removed symlink 'juju-intro/check-juju'
=== target was u'../mojo-spec-helpers/tests/check-juju'
=== modified file 'juju-intro/manifest-verify'
--- juju-intro/manifest-verify 2015-09-08 08:23:46 +0000
+++ juju-intro/manifest-verify 2015-12-21 09:52:48 +0000
@@ -1,4 +1,4 @@
1# Checking juju status1# Checking juju status
2verify config=check-juju2juju-check-wait
3# Running all naigos checks to confirm service is working as expected3# Running all naigos checks to confirm service is working as expected
4verify config=verify-nrpe4verify config=verify-nrpe
55
=== removed symlink 'mojo-how-to/check-juju'
=== target was u'../mojo-spec-helpers/tests/check-juju'
=== modified file 'mojo-how-to/manifest-verify'
--- mojo-how-to/manifest-verify 2015-09-03 06:57:32 +0000
+++ mojo-how-to/manifest-verify 2015-12-21 09:52:48 +0000
@@ -1,4 +1,4 @@
1# Check juju1# Check juju
2verify config=check-juju2juju-check-wait
3# The service is up and running, let's verify it3# The service is up and running, let's verify it
4verify config=verify-nrpe4verify config=verify-nrpe
55
=== removed file 'mojo-spec-helpers/tests/check-juju'
--- mojo-spec-helpers/tests/check-juju 2015-09-03 06:32:08 +0000
+++ mojo-spec-helpers/tests/check-juju 1970-01-01 00:00:00 +0000
@@ -1,5 +0,0 @@
1#!/usr/bin/python
2import utils.mojo_utils as mojo_utils
3
4mojo_utils.juju_check_hooks_complete()
5mojo_utils.juju_status_check_and_wait()

Subscribers

People subscribed via source and target branches

to all changes: