Skip to content

Decrease s2i image build flavor when running in service operators - #257

Merged
openshift-merge-bot[bot] merged 1 commit into
openstack-k8s-operators:mainfrom
amoralej:resize-build-nodeset
Oct 13, 2026
Merged

openshift-merge-bot[bot] merged 1 commit into
openstack-k8s-operators:mainfrom
amoralej:resize-build-nodeset

Conversation

@amoralej

@amoralej amoralej commented Oct 7, 2026 •

Copy link
Copy Markdown
Contributor

This patch is changing the default nodeset to one with with 2vCPUs, 4GB RAM and 40GB HDD so that it's used in s2i content provide jobs defined in the service operators while keeping the existing one when running in the s2i-openstack-operators repo as it builds all the containers.

I'm also keeping the content for the old nodeset name unchanged as it's used in a job defined out of this repo [1] which also build all containers.

[1] https://review.rdoproject.org/r/plugins/gitiles/config/+/refs/heads/master/zuul.d/s2i-openstack-build-push.yaml

Depends-On: #239

This patch is changing the default nodeset to one with with 2vCPUs, 4GB RAM
and 40GB HDD so that it's used in s2i content provide jobs defined in
the service operators while keeping the existing one when running in the
s2i-openstack-operators repo as it builds all the containers.

I'm also keeping the content for the old nodeset name unchanged as it's
used in a job defined out of this repo [1] which also build all
containers.

[1] https://review.rdoproject.org/r/plugins/gitiles/config/+/refs/heads/master/zuul.d/s2i-openstack-build-push.yaml

Signed-off-by: Alfredo Moralejo <amoralej@redhat.com>
Co-Authored-By: Daniel Pawlik <dpawlik@redhat.com>
@centosinfra-prod-github-app

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/b6a3a96f2e8c4ff48a51fbc7e71e1abc

❌ s2i-openstack-containers-molecule NODE_FAILURE Node(set) request 100-0000226126 failed in 0s
❌ s2i-openstack-container-content-provider NODE_FAILURE Node(set) request 100-0000226127 failed in 0s
⚠️ s2i-openstack-container-consumer-smoke SKIPPED Skipped due to failed job s2i-openstack-container-content-provider
⚠️ s2i-openstack-deploy-validation SKIPPED Skipped due to failed job s2i-openstack-container-content-provider

@amoralej

amoralej commented Oct 7, 2026

Copy link
Copy Markdown
Contributor Author

recheck

@centosinfra-prod-github-app

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/d26e4047a5f749fa84447da757bf2e25

❌ s2i-openstack-containers-molecule NODE_FAILURE Node(set) request 100-0000226130 failed in 0s
❌ s2i-openstack-container-content-provider NODE_FAILURE Node(set) request 100-0000226132 failed in 0s
⚠️ s2i-openstack-container-consumer-smoke SKIPPED Skipped due to failed job s2i-openstack-container-content-provider
⚠️ s2i-openstack-deploy-validation SKIPPED Skipped due to failed job s2i-openstack-container-content-provider

amoralej added a commit to amoralej/watcher-operator that referenced this pull request Oct 7, 2026
@amoralej

amoralej commented Oct 7, 2026

Copy link
Copy Markdown
Contributor Author

recheck

@centosinfra-prod-github-app

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/5382329595654989bd459579e22b3e0a

✔️ s2i-openstack-containers-molecule SUCCESS in 3m 05s
❌ s2i-openstack-container-content-provider FAILURE in 7m 40s
⚠️ s2i-openstack-container-consumer-smoke SKIPPED Skipped due to failed job s2i-openstack-container-content-provider
⚠️ s2i-openstack-deploy-validation SKIPPED Skipped due to failed job s2i-openstack-container-content-provider

@amoralej

amoralej commented Oct 7, 2026

Copy link
Copy Markdown
Contributor Author

recheck

@centosinfra-prod-github-app

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/00ba5747eac14ba49e76ad30909abea0

✔️ s2i-openstack-containers-molecule SUCCESS in 5m 09s
❌ s2i-openstack-container-content-provider FAILURE in 8m 09s
⚠️ s2i-openstack-container-consumer-smoke SKIPPED Skipped due to failed job s2i-openstack-container-content-provider
⚠️ s2i-openstack-deploy-validation SKIPPED Skipped due to failed job s2i-openstack-container-content-provider

@amoralej

amoralej commented Oct 7, 2026

Copy link
Copy Markdown
Contributor Author

Tested in openstack-k8s-operators/watcher-operator#486 . For a provider job building 5 containers (nova + watcher), the runtime job moved from 6 to 10 minutes by decreasing the flavor which seems perfectly fine.

https://gateway-cloud-softwarefactory.apps.ocp.cloud.ci.centos.org/logs//559/rdoproject.org/5597fbacc35d4e8ead0ef6d1b0800b54/job-output.txt.gz

@amoralej

amoralej commented Oct 7, 2026

Copy link
Copy Markdown
Contributor Author

recheck

@amoralej amoralej closed this Oct 7, 2026
@amoralej amoralej reopened this Oct 7, 2026
@centosinfra-prod-github-app

Copy link
Copy Markdown

@elfiesmelfie
elfiesmelfie requested a review from sdatko October 12, 2026 11:35

@sdatko sdatko 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

@openshift-ci

openshift-ci Bot commented Oct 13, 2026

Copy link
Copy Markdown
Contributor

[APPROVALNOTIFIER] This PR is APPROVED

Approval requirements bypassed by manually added approval.

This pull-request has been approved by:

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 503f42a into openstack-k8s-operators:main Oct 13, 2026
4 checks 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.

3 participants