Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add a strict-migration feature flag #7176

Merged
merged 2 commits into from Mar 30, 2017
Merged

Conversation

howbazaar
Copy link
Contributor

Description of change

This branch adds a strict validation check to migrations it ensure that all status, status history, annotations, and settings are exported.

During this change, it was identified that the unit workload version was not using the status documents already retrieved, and also that the cloud instance status was being migrated from the wrong place. The Status field in the instanceDoc was depricated, but not removed, nor even mentioned that it was deprecated. This branch removes it from the instanceDoc, and updates the migration code to migrate the status and status history for the instance status.

Documentation changes

None.

@howbazaar
Copy link
Contributor Author

$$merge$$

@jujubot
Copy link
Collaborator

jujubot commented Mar 30, 2017

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

@jujubot
Copy link
Collaborator

jujubot commented Mar 30, 2017

Build failed: Tests failed
build url: http://juju-ci.vapour.ws:8080/job/github-merge-juju/10583

@anastasiamac
Copy link
Contributor

Scary but true...

I suspect that it is actually a failure related to the patch:
FAIL: migration_internal_test.go:316: MigrationSuite.TestInstanceDataFields

migration_internal_test.go:335:
s.AssertExportedFields(c, instanceData{}, fields)
migration_internal_test.go:838:
c.Assert(removed, gc.HasLen, 0)
... obtained set.Strings = set.Strings{"Status":true}
... n int = 0

OOPS: 1845 passed, 1 skipped, 1 FAILED
--- FAIL: TestPackage (679.50s)

@howbazaar
Copy link
Contributor Author

$$merge$$

@jujubot
Copy link
Collaborator

jujubot commented Mar 30, 2017

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

@jujubot
Copy link
Collaborator

jujubot commented Mar 30, 2017

Build failed: Tests failed
build url: http://juju-ci.vapour.ws:8080/job/github-merge-juju/10594

@howbazaar
Copy link
Contributor Author

$$wat?$$

@jujubot
Copy link
Collaborator

jujubot commented Mar 30, 2017

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

@jujubot jujubot merged commit 0ac6cb9 into juju:develop Mar 30, 2017
@howbazaar howbazaar deleted the strict-migration branch March 30, 2017 20:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
3 participants