Skip to content

OCPBUGS-105855: Fix race condition in extra worker BMH application - #1946

Open
elfosardo wants to merge 1 commit into
openshift-metal3:masterfrom
elfosardo:fix-workers-race-condition
Open

OCPBUGS-105855: Fix race condition in extra worker BMH application#1946
elfosardo wants to merge 1 commit into
openshift-metal3:masterfrom
elfosardo:fix-workers-race-condition

Conversation

@elfosardo

Copy link
Copy Markdown
Member

Wait for all worker Machines to have a BareMetalHost assigned before applying extra worker manifests. Without this, hosts that register quickly (e.g. via IPMI) can be claimed by the primary worker MachineSet when a regular worker is slow to register (e.g. via Redfish), leaving the cluster in a degraded state with an unclaimed worker BMH.

Assisted-By: Claude Opus 4.6

Wait for all worker Machines to have a BareMetalHost assigned before
applying extra worker manifests. Without this, hosts that register
quickly (e.g. via IPMI) can be claimed by the primary worker MachineSet
when a regular worker is slow to register (e.g. via Redfish), leaving
the cluster in a degraded state with an unclaimed worker BMH.
Assisted-By: Claude Opus 4.6
@dtantsur

Copy link
Copy Markdown
Member

/approve

@openshift-ci

Copy link
Copy Markdown

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: dtantsur

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

The pull request process is described 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

@openshift-ciopenshift-ciBot added the approved Indicates a PR has been approved by an approver from all required OWNERS files. label Aug 12, 2026
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

approvedIndicates 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.

2 participants

@elfosardo@dtantsur