Remove deprecated .status.succeeded and .status.reason

[imjasonh]

Changes

These were deprecated in #674 and should be removed before the next release.

/kind cleanup

Submitter Checklist

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

See the contributor guide
for details on coding conventions, github and prow interactions, and the code review process.

Release Notes

The deprecated .status.succeeded and .status.reason fields are removed.

[openshift-ci-robot]

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by:
To complete the pull request process, please assign otaviof after the PR has been reviewed.
You can assign the PR to them by writing /assign @otaviof in a comment when ready.

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

Needs approval from an approver in each of these files:

• OWNERS

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

[SaschaSchwarze0]

Maybe a topic for Monday @imjasonh, I would just leave these fields in and do a larger cleanup with v1beta1.

[zhangtbj]

Just talked with our UX team, they will switch to the new condition property soon. We can double check with them next week to see when it can be ready.

Thanks for the exception! We will update once we have any news.

Also cc @qu1queee

[zhangtbj]

Just talked with our UX team, they will switch to the new condition property soon. We can double check with them next week to see when it can be ready.

Thanks for the exception! We will update once we have any news.

Also cc @qu1queee

[qu1queee]

@imjasonh do you mind adding a do-not-merge/hold label?

[qu1queee]

I think we will be able to merge this one soon. Downstream the teams are moving out of the deprecated fields by May 14th, or even before that. After that we should be able to merge this one. Sorry for the inconvenience.

[imjasonh]

No worries! I'd rather take the time to make sure this doesn't cause any pain downstream. I'll be happy as long as it makes it into the next release or two, and we're well ahead of schedule on that.

[qu1queee]

Sorry for the long wait, as today we are finally ready for this deprecation in downstream.

/lgtm
/approve

[openshift-ci]

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: qu1queee

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 [qu1queee]

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

[qu1queee]

/lgtm