Join GitHub today
GitHub is home to over 20 million developers working together to host and review code, manage projects, and build software together.
Fix LP:#1696828 by sanitizing meter status code inputs on import. #7479
Conversation
|
!!build!! |
|
I was able to reproduce the issue by attempting the migration from a 2.1.3 controller to 2.2-rc1. With the above fix, the migration succeeded. $$merge$$ |
|
Status: merge request accepted. Url: http://juju-ci.vapour.ws:8080/job/github-merge-juju |
|
Build failed: Tests failed |
|
$$merge$$ |
|
Status: merge request accepted. Url: http://juju-ci.vapour.ws:8080/job/github-merge-juju |
jujubot
merged commit 1cb9c09
into
juju:develop
Jun 9, 2017
1 check passed
github-check-merge-juju
Ran tests against PR. Use !!.*!! to request another build. IE, !!build!!, !!retry!!
Details
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
cmars commentedJun 9, 2017
Cherry-picking #7478 into develop
Description of change
Meter status codes need to be sanitized to a set of enumerated values before setting them in state during a model migration.
QA steps
Export a model from a 2.1.3 controller and import into a 2.2-rc1 controller, per LP:#1696828.
Documentation changes
No.
Bug reference
LP:#1696828