Set user-friendly BuildRun status when Pod gets evicted

[MayukhSobo]

Changes

When a BuildRun Pod gets evicted, then the status of the BuildRun is:

  status:
    buildSpec: [...]
    conditions:
    - lastTransitionTime: "2021-08-11T09:54:37Z"
      message: 'buildrun failed due to an unexpected error in pod disk-full-run-p5psw-kmx9f-pod-x7ng9:
        for detailed information: kubectl --namespace 9sj9w1nrbzt logs disk-full-run-p5psw-kmx9f-pod-x7ng9
        --all-containers'
      reason: Failed
      status: "False"
      type: Succeeded
    failedAt:
      pod: disk-full-run-p5psw-kmx9f-pod-x7ng9

This is mis-leading because an evicted pod has no logs that one can look at. The PR changes the BuildRun status to:

  status:
    buildSpec: [...]
    conditions:
    - lastTransitionTime: "2021-08-11T09:54:37Z"
      message: 'Container step-build-and-push exceeded its local ephemeral storage limit "1460M".'
      reason: PodEvicted
      status: "False"
      type: Succeeded
    failedAt:
      pod: disk-full-run-p5psw-kmx9f-pod-x7ng9

The message is coming from the Pod's status.

Submitter Checklist

• Includes tests if functionality changed/was added
• Includes docs if changes are user-facing
• Set a kind label on this PR
• Release notes block has been filled in, or marked NONE

Release Notes

The BuildRun status is improved when a BuildRun Pod gets evicted.

[MayukhSobo]

/assign @SaschaSchwarze0

[qu1queee]

@MayukhSobo some sanity checks are failing in this PR, also can you add pls the proper PR template, e.g. (release notes if needed, etc)

[SaschaSchwarze0]

Code is fine. Please check the validation failure in https://github.com/shipwright-io/build/pull/858/checks?check_run_id=3447027690. Please add the new PodEvicted reason to the table in https://github.com/shipwright-io/build/blob/main/docs/buildrun.md#understanding-the-state-of-a-buildrun.

[gabemontero]

/approve

thanks!

aside from the verify failure (got a double import being flagged by make staticcheck) will defer lgtm to @SaschaSchwarze0 once he is happy with the doc update

[openshift-ci]

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: gabemontero

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Needs approval from an approver in each of these files:

• OWNERS [gabemontero]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

[SaschaSchwarze0]

/lgtm