/juju

[apiserver/metricsmanager] Close state objects opened. #6851

Merged
merged 1 commit into from Jan 22, 2017

Conversation

Reviewers

@reedobrien reedobrien

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
3 participants
@howbazaar @jujubot @reedobrien
@howbazaar
Owner

howbazaar commented Jan 22, 2017

The metrics manager api methods for CleanupOldMetrics and SendMetrics would open a State object with ForModel, and not close it.
@howbazaar
Owner

howbazaar commented Jan 22, 2017

At least part fixes lp:1649719.
@reedobrien

reedobrien approved these changes Jan 22, 2017
View changes

LGTM

@howbazaar
Owner

howbazaar commented Jan 22, 2017

$$merge$$
@jujubot
Contributor

jujubot commented Jan 22, 2017

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

@jujubot jujubot merged commit 032d39d into juju:2.1 Jan 22, 2017

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