/juju

provider/maas: no need to lock an immutable value #4922

Merged
merged 1 commit into from Mar 30, 2016

Conversation

Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
2 participants
@davecheney @jujubot
@davecheney
Contributor

davecheney commented Mar 30, 2016

Updates LP 1563628

The maas storage type is immutable, its consituent fields cannot be
changed after construction so there is no need to ensure they are safely
mutated behind a mutex.

(Review request: http://reviews.vapour.ws/r/4361/)
@davecheney
provider/maas: no need to lock an immutable value 
The maas storage type is immutable, its consituent fields cannot be
changed after construction so there is no need to ensure they are safely
mutated behind a mutex.
f5a9f33
@davecheney
Contributor

davecheney commented Mar 30, 2016

$$merge$$
@jujubot
Contributor

jujubot commented Mar 30, 2016

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

jujubot commented Mar 30, 2016

Build failed: Tests failed
build url: http://juju-ci.vapour.ws:8080/job/github-merge-juju/7091
@davecheney
Contributor

davecheney commented Mar 30, 2016

$$merge$$
@jujubot
Contributor

jujubot commented Mar 30, 2016

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

@jujubot jujubot merged commit 49395e5 into juju:master Mar 30, 2016

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