Katib job status should contain all conditions, not just the latest one

[richardsliu]

Current katib job status: https://github.com/kubeflow/katib/blob/master/pkg/api/operators/apis/studyjob/v1alpha1/studyjob_types.go#L60

Compare that with TFJob status, which has more details such as Reason and Message:
https://github.com/kubeflow/tf-operator/blob/master/pkg/apis/common/v1beta1/common_types.go#L83

We should consider using the latter model in the next version of the API.

[YujiOshima]

Should we use Events field of Studyjob?

[johnugeorge]

@YujiOshima Yes. Adding events will be also helpful to users like in TFJob

[johnugeorge]

/assign

[richardsliu]

We can close this one since the design for v1alpha2 API covers this. Implementation will be completed in 0.6.

/close

[k8s-ci-robot]

@richardsliu: Closing this issue.

In response to this:

We can close this one since the design for v1alpha2 API covers this. Implementation will be completed in 0.6.

/close

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.