Skip to content

Conversation

kaovilai
Copy link
Member

Update to get latest 1.5 commits.

What this PR does / why we need it

Which issue(s) this PR fixes

Fixes #

Checklist

  • Subject and description added to both, commit and PR.
  • Relevant issues have been referenced.
  • This change includes docs.
  • This change includes unit tests.

jparrill and others added 2 commits August 20, 2025 14:03
- Increase retry attempts from 5 to 10 for better conflict handling
- Reduce initial retry delay from 1s to 500ms for faster resolution
- Add 30 second cap to maximum wait time to prevent excessive delays
- Improve logging messages for better debugging and monitoring
- Add success confirmation logs for completed operations
- Maintain consistent retry logic across both UpdateHostedCluster and UpdateNodepools functions

This addresses the 'object has been modified' conflicts that occur during
backup/restore operations when multiple processes modify HyperShift resources
concurrently. The enhanced retry mechanism provides more robust handling
of race conditions and controller conflicts.

Signed-off-by: Juan Manuel Parrilla Madrid <[email protected]>
…03-to-oadp-1.5

[oadp-1.5] : feat: improve conflict resolution for HostedCluster and NodePool updates
@openshift-ci openshift-ci bot requested review from devguyio and enxebre August 21, 2025 19:23
Copy link

openshift-ci bot commented Aug 21, 2025

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: kaovilai

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:

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

@openshift-ci openshift-ci bot added the approved Indicates a PR has been approved by an approver from all required OWNERS files. label Aug 21, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
approved Indicates a PR has been approved by an approver from all required OWNERS files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants