Skip to content

update maxSize from 5 to 10 for 4.21 hive pool#74848

Merged
openshift-merge-bot[bot] merged 1 commit intoopenshift:mainfrom
kannon92:hive-4-21-increase-max-size
Feb 19, 2026
Merged

update maxSize from 5 to 10 for 4.21 hive pool#74848
openshift-merge-bot[bot] merged 1 commit intoopenshift:mainfrom
kannon92:hive-4-21-increase-max-size

Conversation

@kannon92
Copy link
Copy Markdown
Contributor

This matches the other pools. Not sure why it was set to 5 in the first place.

@openshift-ci-robot openshift-ci-robot added the rehearsals-ack Signifies that rehearsal jobs have been acknowledged label Feb 12, 2026
@openshift-ci-robot
Copy link
Copy Markdown
Contributor

[REHEARSALNOTIFIER]
@kannon92: no rehearsable tests are affected by this change

Note: If this PR includes changes to step registry files (ci-operator/step-registry/) and you expected jobs to be found, try rebasing your PR onto the base branch. This helps pj-rehearse accurately detect changes when the base branch has moved forward.

@openshift-ci
Copy link
Copy Markdown
Contributor

openshift-ci Bot commented Feb 12, 2026

@kannon92: all tests passed!

Full PR test history. Your PR dashboard.

Details

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository. I understand the commands that are listed here.

name: install-config-aws-us-east-1
labels:
tp.openshift.io/owner: openshift-ci
maxSize: 5
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

@deepsm007 any reason why we have 5 here? It's not always 10 though, see clusters/hosted-mgmt/hive/pools/openshift-ci/ci-ocp-4-19-0-arm64-aws-us-east-1_clusterpool.yaml

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

no reason for having 5, we can set it to 10

Copy link
Copy Markdown
Contributor

@deepsm007 deepsm007 left a comment

Choose a reason for hiding this comment

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

/label priority/ci-critical

@openshift-ci openshift-ci Bot added the priority/ci-critical Critical priority - CI infrastructure is broken or severely impacted label Feb 19, 2026
@openshift-ci openshift-ci Bot added the lgtm Indicates that a PR is ready to be merged. label Feb 19, 2026
@openshift-ci
Copy link
Copy Markdown
Contributor

openshift-ci Bot commented Feb 19, 2026

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: deepsm007, kannon92

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 Feb 19, 2026
@openshift-merge-bot openshift-merge-bot Bot merged commit 71df378 into openshift:main Feb 19, 2026
23 checks passed
theautoroboto pushed a commit to theautoroboto/release that referenced this pull request Feb 19, 2026
rrasouli pushed a commit to rrasouli/release that referenced this pull request Mar 3, 2026
kannon92 added a commit to kannon92/release that referenced this pull request Mar 3, 2026
wangke19 pushed a commit to wangke19/release that referenced this pull request Mar 4, 2026
rrasouli pushed a commit to rrasouli/release that referenced this pull request Mar 5, 2026
sdodson pushed a commit to sdodson/release that referenced this pull request Mar 8, 2026
SeanZhao-redhat pushed a commit to SeanZhao-redhat/openshift-release that referenced this pull request Mar 9, 2026
kasturinarra pushed a commit to kasturinarra/release that referenced this pull request Mar 11, 2026
tareqalayan pushed a commit to tareqalayan/release that referenced this pull request Mar 13, 2026
qiliRedHat pushed a commit to qiliRedHat/release that referenced this pull request Mar 13, 2026
MayXuQQ pushed a commit to MayXuQQ/release that referenced this pull request Mar 17, 2026
kasturinarra pushed a commit to kasturinarra/release that referenced this pull request Mar 17, 2026
sairameshv pushed a commit to sairameshv/release that referenced this pull request Mar 23, 2026
zhouying7780 pushed a commit to zhouying7780/release that referenced this pull request Mar 25, 2026
rrasouli pushed a commit to rrasouli/release that referenced this pull request Mar 25, 2026
anpingli pushed a commit to anpingli/release that referenced this pull request Mar 30, 2026
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. priority/ci-critical Critical priority - CI infrastructure is broken or severely impacted rehearsals-ack Signifies that rehearsal jobs have been acknowledged

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants