In tests, juju status syntax is incorrect

Bug #1889132 reported by Xav Paice on 2020-07-27
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
charm-juju-controller
Medium
Unassigned

Bug Description

In src/tests/tests_juju_controller.py,

+async def juju_status(unit, inner_model, filter=""):
+ out = await model.async_run_on_unit(
+ unit,
+ "sudo -u ubuntu -H /snap/bin/juju status --format json -m {} {}".format(
+ filter, inner_model

Paul Goins (vultaire) wrote on 2020-07-24:
Guessing this was copied from somewhere.

Seems a bit odd though - shouldn't the model be the argument given to -m? Here it seems it is just because filter is empty...

This would be clear (and feel safer to me) if filter was dropped and we only put in the inner_model arg.

Xav Paice (xavpaice) on 2020-07-27
Changed in charm-juju-controller:
status: New → Triaged
importance: Undecided → Medium
To post a comment you must log in.
This report contains Public information  Edit
Everyone can see this information.

Other bug subscribers