Skip to content

vsphere - Clear task ID cache on PatchMachine failure#1465

Draft
jcpowermac wants to merge 1 commit intoopenshift:mainfrom
jcpowermac:vsphere-vm-not-poweredon
Draft

vsphere - Clear task ID cache on PatchMachine failure#1465
jcpowermac wants to merge 1 commit intoopenshift:mainfrom
jcpowermac:vsphere-vm-not-poweredon

Conversation

@jcpowermac
Copy link
Contributor

When PatchMachine fails during Create, the stale task ID cache entry would permanently block the machine from progressing past the stale check on every future reconcile. Delete the cache entry on PatchMachine failure so the machine can recover.

When PatchMachine fails during Create, the stale task ID cache entry
would permanently block the machine from progressing past the stale
check on every future reconcile. Delete the cache entry on PatchMachine
failure so the machine can recover.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@openshift-ci openshift-ci bot added the do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. label Feb 25, 2026
@openshift-ci
Copy link
Contributor

openshift-ci bot commented Feb 25, 2026

Skipping CI for Draft Pull Request.
If you want CI signal for your change, please convert it to an actual PR.
You can still manually trigger a test run with /test all

@jcpowermac
Copy link
Contributor Author

/test ?

@openshift-ci
Copy link
Contributor

openshift-ci bot commented Feb 25, 2026

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by:
Once this PR has been reviewed and has the lgtm label, please assign nrb for approval. For more information see the Code Review Process.

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

Details 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

@jcpowermac
Copy link
Contributor Author

/test e2e-vsphere-ovn-techpreview-serial

@jcpowermac
Copy link
Contributor Author

/test e2e-vsphere-ovn

@jcpowermac
Copy link
Contributor Author

/test unit

@jcpowermac
Copy link
Contributor Author

/test e2e-vsphere-ovn-techpreview-serial

@jcpowermac
Copy link
Contributor Author

bad claude, this is really more broken

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

Labels

do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant