-
Notifications
You must be signed in to change notification settings - Fork 989
Make droplet's lifecycle information available #3634
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
Make droplet's lifecycle information available #3634
Conversation
anujc25
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Couple of comments but otherwise looks good.
It might be good to rebase this PR as well so that it can take the improvements done in the integration tests pipeline. (same comments are applicable to #3633 as well)
resources/droplet_resource.go
Outdated
| // An object describing the lifecycle that was used when staging the droplet | ||
| // possible values for type: "buildpack", "cnd", "docker" | ||
| type DropletLifecycle struct { | ||
| Type string `json:"name"` |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Should we call the json tag as type instead of name here? That is how the cloud-controller api returns the response as well.
resources/droplet_resource.go
Outdated
| } | ||
|
|
||
| // An object describing the lifecycle that was used when staging the droplet | ||
| // possible values for type: "buildpack", "cnd", "docker" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
| // possible values for type: "buildpack", "cnd", "docker" | |
| // possible values for type: "buildpack", "cnb", "docker" |
92140fa to
297ece9
Compare
anujc25
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM.
gururajsh
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Description of the Change
The PR makes the droplet's lifecycle information available as a property of the droplet object.
Why Is This PR Valuable?
Within cf_exporter we want to specify different actions based on which droplet lifecycle is used. Therefore we need to make that information available.
Applicable Issues
How Urgent Is The Change?
not urgent
Other Relevant Parties
Who else is affected by the change?