Added test to reproduce bug with FullStatus against latest juju. #91

Closed
wants to merge 1 commit into
from

Conversation

Projects
None yet
2 participants
Collaborator

petevg commented Mar 17, 2017

If the current python-libjuju is run against juju-2.2-alpha1 (or the
daily builds), client.FullStatus breaks due to a missing 'model-status'
key.

This is a repro for #90

@johnsca @tvansteenburgh

Added test to reproduce bug with FullStatus against latest juju.
If the current python-libjuju is run against juju-2.2-alpha1 (or the
daily builds), client.FullStatus breaks due to a missing 'model-status'
key.
Member

johnsca commented Apr 3, 2017

The test is good, and is catching the issue, but I'm unsure how to proceed with this PR since I don't want the build to be broken. AFAIU, we don't have any reasonable way to fix this without addressing #49.

Collaborator

petevg commented Apr 3, 2017

@johnsca Yeah. I don't want us to merge this.

Having an open PR kind of sucks, though, and we understand the issue pretty thoroughly. I'm going to lose this out for now -- I can resubmit once we have the bug fixed.

@petevg petevg closed this Apr 3, 2017

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment