lp:~pjdc/mojo/sparkle-motion

Branch merges

Propose for merging

No branches dependent on this one.

Merged into lp:mojo at revision 502

Tom Haddon: Approve on 2019-06-04

Paul Collins: Pending requested 2019-06-04

Diff: 83 lines (+11/-9)

2 files modified

docs/mojo/readme.md (+5/-1)
mojo/phase.py (+6/-8)

api

Related source package recipes

No recipes using this branch. (?)

Create packaging recipe

Related snap packages

Related bugs

Bug #1831620: support stage_parent (MOJO_STAGE_PARENT)

Medium

Fix Released

Link a bug report

Related blueprints

502. By Paul Collins on 2019-06-04

add stage_parent

Also, make things a little more consistent in anticipation of factoring.

501. By Paul Collins on 2019-05-20

readme.md: fix indentation in example secrets configs, remove redundant "charm" from secrets

Reviewed-on: https://code.launchpad.net/~pjdc/mojo/fix-secrets-readme/+merge/367604
Reviewed-by: Tom Haddon <email address hidden>
Reviewed-by: Nick Moffitt <email address hidden>

500. By Tom Haddon on 2019-04-29

[mthaddon,trivial] Increment debian changelog for release

499. By Domas Monkus on 2019-04-29

Fix regression in nagios check parallel runner.

The exception happens when the model has applications without units.

Reviewed-on: https://code.launchpad.net/~tasdomas/mojo/mojo-fix-nagios-regression/+merge/366641
Reviewed-by: Tom Haddon <email address hidden>

498. By Barry Price on 2019-04-25

Update changelog for 0.5.1-3 release

Reviewed-on: https://code.launchpad.net/~barryprice/mojo/trunk/+merge/366503
Reviewed-by: Benjamin Allot <email address hidden>

497. By Colin Watson on 2019-04-24

Remove verify check from run-jenkins-job-git-spec.

This check doesn't exist in the corresponding run-jenkins-job-bzr-spec
script, and it's buggy because it doesn't take MOJO_STAGE into consideration
when determining the path to the manifest file. Furthermore, manifests may
well do their verification by including another manifest; without expanding
includes, this script can't tell whether a manifest does any verify
operations. As such I think it's better to just drop this incorrect check.

Reviewed-on: https://code.launchpad.net/~cjwatson/mojo/remove-git-verify-check/+merge/366449
Reviewed-by: Tom Haddon <email address hidden>
Reviewed-by: Barry Price <email address hidden>

496. By Daniel Manrique on 2019-04-19

Verify nagios checks in parallel.

This uses Python's futures module for minimal-change parallel execution of nagios checks; one parallel "juju ssh" is dispatched for each machine to check, and results are presented as they arrive. The logic and presentation stay the same otherwise.

I didn't declare futures as an explicit deb dependency (only in requirements.txt) because in the deb world, mojo depends on codetree which depends on futures, so they should always be there.

I added a couple of tests to verify the expected number of juju sshs is dispatched, and that the number of workers is properly calculated.

A possible refinement would be ability to specify a maximum number of workers; this could help if e.g. simultaneous juju ssh to a large number of units on a complex deployment causes any kind of trouble.

A couple of tweaks to the "juju ssh" command line were needed to avoid the display/pty getting borked when running several tasks in parallel, but I checked and they don't affect the visual output of things.

Reviewed-on: https://code.launchpad.net/~roadmr/mojo/parallel-nagios-check/+merge/366076
Reviewed-by: Stuart Bishop <email address hidden>

495. By Chris MacNaughton on 2019-04-10

Enable Disco series

Reviewed-on: https://code.launchpad.net/~chris.macnaughton/mojo/disco-enablement/+merge/365415
Reviewed-by: Barry Price <email address hidden>
Reviewed-by: Ryan Beisner <email address hidden>

494. By Tom Haddon on 2019-02-01

Set permissions on files copied into LXC so dependencies, for instance, can be installed

Reviewed-on: https://code.launchpad.net/~mthaddon/mojo/set-perms-lxc/+merge/362583
Reviewed-by: Junien Fridrick <email address hidden>

493. By Tom Haddon on 2019-02-01

Update lxc-test to use local directory for easier testing of changes, fix juju_wait tests to use correct assertion and valid log output

Reviewed-on: https://code.launchpad.net/~mthaddon/mojo/juju-wait-tests/+merge/362532
Reviewed-by: Stuart Bishop <email address hidden>