-
Notifications
You must be signed in to change notification settings - Fork 104
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
Bugfix: IntegrationTest TestRestartController #1313
Conversation
@@ -72,7 +72,7 @@ type PlanStatus struct { | |||
Status ExecutionStatus `json:"status,omitempty"` | |||
Message string `json:"message,omitempty"` // more verbose explanation of the status, e.g. a detailed error message | |||
// +nullable | |||
LastFinishedRun metav1.Time `json:"lastFinishedRun,omitempty"` | |||
LastFinishedRun *metav1.Time `json:"lastFinishedRun,omitempty"` |
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.
As this can now be nil
, we should have checks for nil
in places where this field is used, right?
I.e. in instance_types_helpers.go wasRunAfter
and plan_history.go planHistory
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.
yep other than this it lgtm
…that it can be nil Signed-off-by: Ken Sipe <kensipe@gmail.com>
Signed-off-by: Ken Sipe <kensipe@gmail.com>
d3a1cc8
to
49304f9
Compare
Signed-off-by: Ken Sipe <kensipe@gmail.com>
…ullable but now actually is Signed-off-by: Ken Sipe <kensipe@gmail.com>
Signed-off-by: Ken Sipe <kensipe@gmail.com> Signed-off-by: Andreas Neumann <aneumann@mesosphere.com>
Signed-off-by: Ken Sipe <kensipe@gmail.com> Signed-off-by: Thomas Runyon <runyontr@gmail.com>
lastFinishedRun
a ptr so that it can benil
Fixes #1312
Signed-off-by: Ken Sipe kensipe@gmail.com