Skip to content
This repository has been archived by the owner on May 6, 2022. It is now read-only.

Move "Checksum" to *Status resource #852

Closed
duglin opened this issue May 15, 2017 · 1 comment
Closed

Move "Checksum" to *Status resource #852

duglin opened this issue May 15, 2017 · 1 comment
Assignees
@pmorie
Labels
api
Milestone
0.0.7

Comments

@duglin
Copy link
Contributor

duglin commented May 15, 2017

Right now we have InstanceSpec.Checksum and BindingSpec.Checksum - these should be moved to *Status instead since they're really part of the user-facing data and only used by internal processing.
@duglin duglin added the api label May 15, 2017
@duglin duglin added this to the 0.1.0 milestone May 15, 2017
@duglin duglin mentioned this issue May 15, 2017
Closed
@pmorie
Copy link
Contributor

pmorie commented May 15, 2017

I will take this one, there are a couple subtleties about it I have already spent some time thinking about, and I added these fields in the first place.

@pmorie pmorie self-assigned this May 15, 2017
@pmorie pmorie modified the milestones: 0.0.7, 0.1.0 May 15, 2017
@pmorie pmorie mentioned this issue May 15, 2017
Merged
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@pmorie pmorie

Labels
api
Projects
None yet
Milestone
0.0.7
Development

No branches or pull requests
2 participants
@pmorie @duglin