Skip to content

Conversation

@akalenyu
Copy link
Contributor

@akalenyu akalenyu commented Mar 19, 2023

Currently, we just dangle in Deploying phase when things go badly, although
the deployment conditions API provides more information such as Progressing=false
that we can use to better capture the status of install.

Aggregate failed workload conditions into CR

Signed-off-by: Alex Kalenyuk <[email protected]>
@kubevirt-bot kubevirt-bot added dco-signoff: yes Indicates the PR's author has DCO signed all their commits. do-not-merge/release-note-label-needed Indicates that a PR should not merge because it's missing one of the release note labels. size/M labels Mar 19, 2023
@akalenyu akalenyu force-pushed the report-degraded branch 5 times, most recently from e9ccdd6 to 519ef97 Compare March 19, 2023 20:05
@akalenyu
Copy link
Contributor Author

/retest

@akalenyu
Copy link
Contributor Author

/test all

@kubevirt-bot
Copy link

@akalenyu: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name Commit Details Required Rerun command
pull-controller-lifecycle-operator-sdk-e2e-k8s-1.18 ded2113 link false /test pull-controller-lifecycle-operator-sdk-e2e-k8s-1.18

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. I understand the commands that are listed here.

@akalenyu
Copy link
Contributor Author

/test all

1 similar comment
@akalenyu
Copy link
Contributor Author

/test all

@akalenyu
Copy link
Contributor Author

/cc @mhenriks @tiraboschi

@mhenriks
Copy link
Member

/lgtm
/approve

@kubevirt-bot kubevirt-bot added lgtm Indicates that a PR is ready to be merged. release-note Denotes a PR that will be considered when it comes time to generate release notes. and removed do-not-merge/release-note-label-needed Indicates that a PR should not merge because it's missing one of the release note labels. labels Mar 20, 2023
@kubevirt-bot
Copy link

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: akalenyu, mhenriks, tiraboschi

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

Needs approval from an approver in each of these files:

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

dco-signoff: yes Indicates the PR's author has DCO signed all their commits. lgtm Indicates that a PR is ready to be merged. release-note Denotes a PR that will be considered when it comes time to generate release notes. size/XXL

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants