[discuss] Improve CR Status Feedback about build

[Avni-Sharma]

It would be also good if conditions can be used as a list of Condition as mentioned here https://github.com/redhat-developer/build/blob/master/pkg/apis/core/v1alpha1/condition_types.go#L24 . and then Succeeded that is used here https://github.com/redhat-developer/build/blob/master/pkg/controller/buildrun/buildrun_controller.go#L317 can be used as a ConditionType. More conditions can then be added to the conditions list.

For more reference refer- https://github.com/kubernetes/community/blob/master/contributors/devel/sig-architecture/api-conventions.md#typical-status-properties

[zhangtbj]

Hi all,

I think we could show some basic build information in the status of the build:

1. Like using corev1alpha1.Status to show the Succeeded, Reason, StartTime and CompletionTime informations
2. If possible, provide the step lists to show each step status and result
3. If possible, show the git info (url or revision) info to let the user know which git commit they used for this build
4. If possible, show the base image/builder image info to let the user know which image (version or tag) they used to build their image.

(Item 3 and 4 can be useful for the future trigger support)

And I think the Item 1 is the first priority because right now, Build v2 just update the Status once taskrun provide the status, it cannot provide the status if there is any build native error, and cannot know the basic info like Succeeded or Time.

Hope it is helpful.

[bamachrn]

Thanks for the input, I believe we can show the logs in case of error in the build process. so that user knows what is breaking where. Build stages (steps) are available in build v1 I believe we can use some kind of similar things in build v2 as well.

[qu1queee]

This topic have been widely discussed, the latest state around the main enhancements are:

• BuildRun supports conditions(already in master branch)
• Build is gonna get an enhancement inside its Status field, WIP at #463 (comment)