state: separate refcounts from migrations #5932

Closed
wants to merge 1 commit into
from

Conversation

Projects
None yet
1 participant
Contributor

fwereade commented Aug 4, 2016

Refcount values are (1) implied by the rest of a model description and
(2) pure implementation details necessitated by mgo/txn. They've been
dropped from description.Application; refcount sanity is now maintained
implicitly during import by the add*Ops-level funcs.

Refcount operations themselves have been extracted to refcounts_ns.go;
not all of them are used, but there's a forthcoming CL that will use
them for charm refcounts, thus helping us to safely delete
unreferenced charms in the near future.

(Review request: http://reviews.vapour.ws/r/5375/)

state: separate refcounts from migrations
Refcount values are (1) implied by the rest of a model description and
(2) pure implementation details necessitated by mgo/txn. They've been
dropped from description.Application; refcount sanity is now maintained
implicitly during import by the add*Ops-level funcs.

Refcount operations themselves have been extracted to refcounts_ns.go;
not all of them are used, but there's a forthcoming CL that will use
them for *charm* refcounts, thus helping us to safely delete
unreferenced charms in the near future.
Contributor

fwereade commented Aug 10, 2016

folded into PR5939

@fwereade fwereade closed this Aug 10, 2016

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