Skip to content

Decrease flavor for content provider CI job - #4225

Merged
Valkyrie00 merged 1 commit into
openstack-k8s-operators:mainfrom
danpawlik:decrease-flavor
Oct 7, 2026
Merged

Decrease flavor for content provider CI job#4225
Valkyrie00 merged 1 commit into
openstack-k8s-operators:mainfrom
danpawlik:decrease-flavor

Conversation

@danpawlik

@danpawlik danpawlik commented Oct 2, 2026

Copy link
Copy Markdown
Contributor

The new flavor would have 2vCPUs, 4GB RAM and 40GB HDD. Before flavor have twice more resources.
It has been done because the CI infra has insufficient resources.

@danpawlik
danpawlik requested review from a team, morucci, rabi, raukadah, sdatko and tosky October 2, 2026 07:46
@qodo-code-review

qodo-code-review Bot commented Oct 2, 2026 •
edited
Loading

Copy link
Copy Markdown

PR Summary by Qodo

Reduce content-provider CI job resource allocation

⚙️ Configuration changes 🕐 Less than 10 minutes

Grey Divider

AI Description 

• Assign the content-provider job a dedicated, smaller CentOS Stream 9 nodeset.
• Reduce its requested resources to ease pressure on CI infrastructure without changing other jobs.
Diagram 

graph TD
  A["Content-provider job"] --> B["Dedicated nodeset"] --> C["Medium cloud label"]
Loading
High-Level Assessment

A dedicated nodeset is appropriate: it reduces resources for this job without changing the shared CentOS Stream 9 nodeset used elsewhere.

Files changed (2) +17 / -1

Other (2) +17 / -1
base.yamlSelect the content-provider-specific nodeset +1/-1

Select the content-provider-specific nodeset

• Points the content-provider job at the new nodeset instead of the shared CentOS Stream 9 nodeset. Other job settings remain unchanged.

zuul.d/base.yaml

nodeset.yamlDefine a smaller content-provider nodeset +16/-0

Define a smaller content-provider nodeset

• Adds a single-controller nodeset using the existing medium cloud label for the requested 2-vCPU, 4-GB RAM, 40-GB disk flavor. It retains the switch and peers groups of the shared nodeset.

zuul.d/nodeset.yaml

@danpawlik
danpawlik requested a review from hjensas October 2, 2026 07:48
morucci
morucci previously approved these changes Oct 2, 2026
View reviewed changes
@danpawlik
danpawlik force-pushed the decrease-flavor branch 2 times, most recently from 7540f8c to 5eb3096 Compare October 2, 2026 08:11
@sdatko
sdatko enabled auto-merge (rebase) October 2, 2026 08:12
@sdatko

sdatko commented Oct 2, 2026

Copy link
Copy Markdown
Contributor

/lgtm /approve /yadayada

@evallesp

evallesp commented Oct 2, 2026

Copy link
Copy Markdown
Contributor

/override rdoproject.org/github-check

@openshift-ci

openshift-ci Bot commented Oct 2, 2026

Copy link
Copy Markdown
Contributor

@evallesp: Overrode contexts on behalf of evallesp: rdoproject.org/github-check

Details

In response to this:

/override rdoproject.org/github-check

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.

evallesp
evallesp approved these changes Oct 2, 2026
View reviewed changes

@evallesp evallesp left a comment

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.

/lgtm
/approve

@danpawlik
danpawlik marked this pull request as draft October 2, 2026 10:01
auto-merge was automatically disabled October 2, 2026 10:01

Pull request was converted to draft

@danpawlik
danpawlik marked this pull request as ready for review October 2, 2026 11:44
@qodo-code-review

qodo-code-review Bot commented Oct 2, 2026 •
edited
Loading

Copy link
Copy Markdown

Code Review by Qodo

Grey Divider

Sorry, something went wrong

We weren't able to complete the code review on our side. Please try again manually by commenting /agentic_review on this PR.

Grey Divider

Qodo Logo

michburk
michburk approved these changes Oct 2, 2026
View reviewed changes

@michburk michburk left a comment

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.

/lgtm
/approve

@openshift-ci

openshift-ci Bot commented Oct 2, 2026

Copy link
Copy Markdown
Contributor

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: evallesp, michburk

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

@sdatko

sdatko commented Oct 2, 2026

Copy link
Copy Markdown
Contributor

recheck

@centosinfra-prod-github-app

centosinfra-prod-github-app Bot commented Oct 2, 2026

Copy link
Copy Markdown

Build succeeded (check pipeline).
https://gateway-cloud-softwarefactory.apps.ocp.cloud.ci.centos.org/zuul/t/rdoproject.org/buildset/abbcb9376b0a4f5c864bac57ca38e8f7

✔️ openstack-k8s-operators-content-provider SUCCESS in 2h 56m 54s
✔️ podified-multinode-edpm-deployment-crc SUCCESS in 1h 20m 38s
✔️ podified-multinode-edpm-deployment-crc-centos-10 SUCCESS in 1h 23m 00s
✔️ cifmw-crc-podified-edpm-baremetal SUCCESS in 1h 35m 27s
✔️ cifmw-crc-podified-edpm-baremetal-minor-update SUCCESS in 2h 00m 23s
✔️ cifmw-pod-zuul-files SUCCESS in 6m 29s
✔️ openstack-k8s-operators-content-provider-bootc SUCCESS in 3h 02m 04s
✔️ cifmw-crc-podified-edpm-baremetal-bootc SUCCESS in 1h 30m 34s
✔️ noop SUCCESS in 0s
✔️ cifmw-pod-ansible-test SUCCESS in 10m 31s
✔️ cifmw-pod-pre-commit SUCCESS in 11m 16s
✔️ ci-framework-openstack-meta-content-provider SUCCESS in 17m 53s

@Valkyrie00

Valkyrie00 commented Oct 5, 2026

Copy link
Copy Markdown
Contributor

recheck

@centosinfra-prod-github-app

centosinfra-prod-github-app Bot commented Oct 5, 2026

Copy link
Copy Markdown

Build failed (check pipeline). Post recheck (without leading slash)
to rerun all jobs. Make sure the failure cause has been resolved before
you rerun jobs.

https://gateway-cloud-softwarefactory.apps.ocp.cloud.ci.centos.org/zuul/t/rdoproject.org/buildset/e50a04d8f3ee4e5987e53a58542db0ea

✔️ openstack-k8s-operators-content-provider SUCCESS in 3h 42m 41s
✔️ podified-multinode-edpm-deployment-crc SUCCESS in 1h 22m 14s
✔️ podified-multinode-edpm-deployment-crc-centos-10 SUCCESS in 1h 21m 56s
✔️ cifmw-crc-podified-edpm-baremetal SUCCESS in 1h 33m 39s
❌ cifmw-crc-podified-edpm-baremetal-minor-update FAILURE in 2h 08m 32s
✔️ cifmw-pod-zuul-files SUCCESS in 4m 38s
✔️ openstack-k8s-operators-content-provider-bootc SUCCESS in 3h 08m 51s
✔️ cifmw-crc-podified-edpm-baremetal-bootc SUCCESS in 1h 26m 21s
✔️ noop SUCCESS in 0s
✔️ cifmw-pod-ansible-test SUCCESS in 8m 23s
✔️ cifmw-pod-pre-commit SUCCESS in 8m 34s
✔️ ci-framework-openstack-meta-content-provider SUCCESS in 18m 58s

@Valkyrie00

Valkyrie00 commented Oct 5, 2026

Copy link
Copy Markdown
Contributor

recheck

@danpawlik @Valkyrie00
Decrease flavor for content provider CI job
9377c99 
The new flavor would have 2vCPUs, 4GB RAM and 40GB HDD. Before
flavor have twice more resources.
It has been done because the CI infra has insufficient resources.

Signed-off-by: Daniel Pawlik <dpawlik@redhat.com>
@openshift-ci openshift-ci Bot removed the lgtm label Oct 6, 2026
@openshift-ci

openshift-ci Bot commented Oct 6, 2026

Copy link
Copy Markdown
Contributor

New changes are detected. LGTM label has been removed.

@centosinfra-prod-github-app

centosinfra-prod-github-app Bot commented Oct 7, 2026

Copy link
Copy Markdown

Build failed (check pipeline). Post recheck (without leading slash)
to rerun all jobs. Make sure the failure cause has been resolved before
you rerun jobs.

https://gateway-cloud-softwarefactory.apps.ocp.cloud.ci.centos.org/zuul/t/rdoproject.org/buildset/04d6e18aa437460d889520e60a3c8c3b

✔️ openstack-k8s-operators-content-provider SUCCESS in 3h 28m 00s
✔️ podified-multinode-edpm-deployment-crc SUCCESS in 1h 20m 53s
✔️ podified-multinode-edpm-deployment-crc-centos-10 SUCCESS in 1h 21m 03s
✔️ cifmw-crc-podified-edpm-baremetal SUCCESS in 1h 36m 19s
❌ cifmw-crc-podified-edpm-baremetal-minor-update FAILURE in 2h 08m 50s
✔️ cifmw-pod-zuul-files SUCCESS in 4m 43s
✔️ openstack-k8s-operators-content-provider-bootc SUCCESS in 2h 58m 48s
✔️ cifmw-crc-podified-edpm-baremetal-bootc SUCCESS in 1h 26m 49s
✔️ noop SUCCESS in 0s
✔️ cifmw-pod-ansible-test SUCCESS in 10m 21s
✔️ cifmw-pod-pre-commit SUCCESS in 9m 38s
✔️ ci-framework-openstack-meta-content-provider SUCCESS in 19m 03s

@Valkyrie00

Valkyrie00 commented Oct 7, 2026

Copy link
Copy Markdown
Contributor

/override rdoproject.org/github-check

@openshift-ci

openshift-ci Bot commented Oct 7, 2026

Copy link
Copy Markdown
Contributor

@Valkyrie00: Overrode contexts on behalf of Valkyrie00: rdoproject.org/github-check

Details

In response to this:

/override rdoproject.org/github-check

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.

@Valkyrie00
Valkyrie00 merged commit d2dc22d into openstack-k8s-operators:main Oct 7, 2026
7 of 8 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@sdatko sdatko sdatko approved these changes
@evallesp evallesp evallesp approved these changes
@michburk michburk michburk approved these changes
@morucci morucci morucci left review comments
@rabi rabi Awaiting requested review from rabi
@tosky tosky Awaiting requested review from tosky
@raukadah raukadah Awaiting requested review from raukadah
@hjensas hjensas Awaiting requested review from hjensas

Assignees

@evallesp evallesp

@michburk michburk

Labels

approved Ready For Review top-quality

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

6 participants

@danpawlik @sdatko @evallesp @Valkyrie00 @morucci @michburk