Skip to content

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

Merged
openshift-merge-bot[bot] merged 1 commit into
openshift-metal3:masterfrom
elfosardo:fix-workers-race-condition
Aug 13, 2026
Merged

OCPBUGS-105855: Fix race condition in extra worker BMH application#1946
openshift-merge-bot[bot] merged 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

openshift-ci Bot commented Aug 12, 2026

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-ci openshift-ci Bot added the approved Indicates a PR has been approved by an approver from all required OWNERS files. label Aug 12, 2026
@tdomnesc

Copy link
Copy Markdown
Contributor

/lgtm

@openshift-ci openshift-ci Bot added the lgtm Indicates that a PR is ready to be merged. label Aug 13, 2026
@openshift-merge-bot
openshift-merge-bot Bot merged commit b322c1b into openshift-metal3:master Aug 13, 2026
19 checks passed
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. lgtm Indicates that a PR is ready to be merged.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants