/juju

state: Squash another unknown collection error #7385

Merged
merged 1 commit into from May 25, 2017
+12 −0

Conversation

Reviewers

@wallyworld wallyworld

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
3 participants
@babbageclunk @jujubot @wallyworld
@babbageclunk
Member

babbageclunk commented May 24, 2017 •
Edited 1 time

Description of change

We were seeing another unknown collection "remoteApplications" error which turned out to be from Relation.IsCrossModel - it needed to check the feature flag.

Dump stack when an unknown collection is requested and we're in developer mode. This will make it much easier to find them in future.

QA steps

Create a model, deploy mysql and wordpress to it, relate them together, destroy the model. The unknown collection error shouldn't appear in the logs.
@babbageclunk
Squash another unknown collection error 
This was in Relation.IsCrossModel.

Dump stack when an unknown collection is requested and we're in
developer mode. This will make it much easier to find them in future.
55073c6
@babbageclunk
Member

babbageclunk commented May 25, 2017

$$merge$$
@jujubot
Contributor

jujubot commented May 25, 2017

Status: merge request accepted. Url: http://juju-ci.vapour.ws:8080/job/github-merge-juju
@jujubot
Contributor

jujubot commented May 25, 2017

Build failed: Tests failed
build url: http://juju-ci.vapour.ws:8080/job/github-merge-juju/10971
@wallyworld
Owner

wallyworld commented May 25, 2017

$$merge$$
@jujubot
Contributor

jujubot commented May 25, 2017

Status: merge request accepted. Url: http://juju-ci.vapour.ws:8080/job/github-merge-juju
@jujubot
Contributor

jujubot commented May 25, 2017

Build failed: Tests failed
build url: http://juju-ci.vapour.ws:8080/job/github-merge-juju/10990
@babbageclunk
Member

babbageclunk commented May 25, 2017

$$merge$$
@jujubot
Contributor

jujubot commented May 25, 2017

Status: merge request accepted. Url: http://juju-ci.vapour.ws:8080/job/github-merge-juju

@jujubot jujubot merged commit 5291b1e into juju:develop May 25, 2017
1 check passed

1 check passed

github-check-merge-juju Built PR, ran unit tests, and tested LXD deploy. Use !!.*!! to request another build. IE, !!build!!, !!retry!!
Details

@babbageclunk babbageclunk deleted the babbageclunk:unknown-collection branch May 25, 2017

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