Skip to content

Validate node count in create-inventory before deployment#772

Merged
openshift-merge-bot[bot] merged 1 commit intoredhat-performance:mainfrom
mcornea:validate_amount_of_nodes
Feb 23, 2026
Merged

Validate node count in create-inventory before deployment#772
openshift-merge-bot[bot] merged 1 commit intoredhat-performance:mainfrom
mcornea:validate_amount_of_nodes

Conversation

@mcornea
Copy link
Collaborator

@mcornea mcornea commented Feb 23, 2026

Adds early assertions to check that the lab allocation has enough nodes for the requested cluster type, replacing cryptic Jinja2 "list object has no element N" errors with actionable messages.

Validates minimum nodes for MNO (4), VMNO (2), and SNO (2), plus explicit worker_node_count and hv_count when set.

Adds early assertions to check that the lab allocation has enough
nodes for the requested cluster type, replacing cryptic Jinja2
"list object has no element N" errors with actionable messages.

Validates minimum nodes for MNO (4), VMNO (2), and SNO (2),
plus explicit worker_node_count and hv_count when set.

Signed-off-by: Marius Cornea <mcornea@redhat.com>
@openshift-ci openshift-ci bot requested review from akrzos and jtaleric February 23, 2026 10:22
Copy link
Member

@akrzos akrzos left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

@openshift-ci
Copy link

openshift-ci bot commented Feb 23, 2026

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: akrzos

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-merge-bot openshift-merge-bot bot merged commit 7c5bd48 into redhat-performance:main Feb 23, 2026
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants