Skip to content

[s2i] Fix the NFS job by setting the right volume - #766

Merged
openshift-merge-bot[bot] merged 1 commit into
openstack-k8s-operators:mainfrom
tosky:zuul-fix-nfs-job
Oct 22, 2026
Merged

[s2i] Fix the NFS job by setting the right volume#766
openshift-merge-bot[bot] merged 1 commit into
openstack-k8s-operators:mainfrom
tosky:zuul-fix-nfs-job

Conversation

@tosky

@tosky tosky commented Oct 22, 2026

Copy link
Copy Markdown
Contributor

The cinder-volume container that should be consumed from the newly-built containers is nfs, not volume1.

@tosky
[s2i] Fix the NFS job by setting the right volume
7ab45c6 
The cinder-volume container that should be consumed
from the newly-built containers is nfs, not volume1.

Signed-off-by: Luigi Toscano <ltoscano@redhat.com>
@openshift-ci
openshift-ci Bot requested review from dprince and fmount October 22, 2026 09:17
@coderabbitai

coderabbitai Bot commented Oct 22, 2026

Copy link
Copy Markdown

Important

  • 🔍 Trigger review

This repository does not receive automatic reviews because it has fewer than 10 stars.

⚙️ Run configuration
  • Source mode: Auto
  • Repository: Unavailable
  • Branch: Unavailable
  • File: No configuration file from this repository applied
  • Commit: Unavailable
  • Layers: Central YAML (base), Organization UI (inherited)
  • Review profile: CHILL
  • Plan: Advanced
  • Run ID: 2332cf5d-09c2-45ea-a6c6-a4b6c9d5d705
  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

Comment @coderabbitai help to get the list of available commands.

fmount
fmount approved these changes Oct 22, 2026
View reviewed changes

@fmount fmount 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 makes sense, thanks @tosky

@openshift-ci openshift-ci Bot added the lgtm label Oct 22, 2026
@openshift-ci

openshift-ci Bot commented Oct 22, 2026

Copy link
Copy Markdown
Contributor

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: fmount, tosky

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

@centosinfra-prod-github-app

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

Copy link
Copy Markdown

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

✔️ cinder-s2i-content-provider SUCCESS in 3h 41m 05s
✔️ cinder-s2i-test-ceph SUCCESS in 1h 46m 42s
✔️ cinder-s2i-test-nfs SUCCESS in 1h 43m 18s
✔️ openstack-k8s-operators-content-provider SUCCESS in 3h 41m 01s
✔️ cinder-operator-kuttl SUCCESS in 37m 24s

Warning:
File ansible/tmp/86839a4b2183453d80b9fc0ea0747f95 found in build home directory during playbook github.com/openstack-k8s-operators/ci-framework/ci/playbooks/collect-logs.yml for job cinder-operator-kuttl. Add to preserve-home-paths if safe, or otherwise remove.
File ansible/tmp/b34de1d204a14800b2ee7329d4d7599e found in build home directory during playbook github.com/openstack-k8s-operators/ci-framework/ci/playbooks/collect-logs.yml for job cinder-operator-kuttl. Add to preserve-home-paths if safe, or otherwise remove.
File ansible/tmp/6f805a3f89dd45dcb01b4cac5a29b05d found in build home directory during playbook github.com/openstack-k8s-operators/ci-framework/ci/playbooks/collect-logs.yml for job cinder-operator-kuttl. Add to preserve-home-paths if safe, or otherwise remove.
File ansible/tmp/122fe5278dd94a018d6ad14202b93ce8 found in build home directory during playbook github.com/openstack-k8s-operators/ci-framework/ci/playbooks/collect-logs.yml for job cinder-operator-kuttl. Add to preserve-home-paths if safe, or otherwise remove.
File ansible/tmp/86839a4b2183453d80b9fc0ea0747f95 found in build home directory during playbook github.com/openstack-k8s-operators/ci-framework/ci/playbooks/multinode-autohold.yml for job cinder-operator-kuttl. Add to preserve-home-paths if safe, or otherwise remove.
File ansible/tmp/132b632704324681afcee380efb3c16b found in build home directory during playbook github.com/openstack-k8s-operators/ci-framework/ci/playbooks/collect-logs.yml for job cinder-s2i-test-nfs. Add to preserve-home-paths if safe, or otherwise remove.
File ansible/tmp/512df7048a1f43938f5eff8604c18a6f found in build home directory during playbook github.com/openstack-k8s-operators/ci-framework/ci/playbooks/collect-logs.yml for job cinder-s2i-test-nfs. Add to preserve-home-paths if safe, or otherwise remove.
File ansible/tmp/132b632704324681afcee380efb3c16b found in build home directory during playbook github.com/openstack-k8s-operators/ci-framework/ci/playbooks/multinode-autohold.yml for job cinder-s2i-test-nfs. Add to preserve-home-paths if safe, or otherwise remove.
File ansible/tmp/512df7048a1f43938f5eff8604c18a6f found in build home directory during playbook github.com/openstack-k8s-operators/ci-framework/ci/playbooks/multinode-autohold.yml for job cinder-s2i-test-nfs. Add to preserve-home-paths if safe, or otherwise remove.
File ansible/tmp/132b632704324681afcee380efb3c16b found in build home directory during playbook review.rdoproject.org/config/playbooks/crc/print-machineconfigpool.yaml for job cinder-s2i-test-nfs. Add to preserve-home-paths if safe, or otherwise remove.
File .ssh/known_hosts.old found in build home directory during playbook review.rdoproject.org/config/playbooks/base-minimal/post-logs.yaml for job cinder-s2i-test-nfs. Add to preserve-home-paths if safe, or otherwise remove.
File ansible/tmp/9140ae9135c54dc7aa4dfc7476d0b541 found in build home directory during playbook github.com/openstack-k8s-operators/ci-framework/ci/playbooks/collect-logs.yml for job cinder-s2i-test-ceph. Add to preserve-home-paths if safe, or otherwise remove.
File ansible/tmp/56d3e9c847a34edea41d9aae48d9f028 found in build home directory during playbook github.com/openstack-k8s-operators/ci-framework/ci/playbooks/collect-logs.yml for job cinder-s2i-test-ceph. Add to preserve-home-paths if safe, or otherwise remove.
File ansible/tmp/9140ae9135c54dc7aa4dfc7476d0b541 found in build home directory during playbook github.com/openstack-k8s-operators/ci-framework/ci/playbooks/multinode-autohold.yml for job cinder-s2i-test-ceph. Add to preserve-home-paths if safe, or otherwise remove.
File ansible/tmp/56d3e9c847a34edea41d9aae48d9f028 found in build home directory during playbook github.com/openstack-k8s-operators/ci-framework/ci/playbooks/multinode-autohold.yml for job cinder-s2i-test-ceph. Add to preserve-home-paths if safe, or otherwise remove.
File ansible/tmp/9140ae9135c54dc7aa4dfc7476d0b541 found in build home directory during playbook review.rdoproject.org/config/playbooks/crc/print-machineconfigpool.yaml for job cinder-s2i-test-ceph. Add to preserve-home-paths if safe, or otherwise remove.
File .ssh/known_hosts.old found in build home directory during playbook review.rdoproject.org/config/playbooks/base-minimal/post-logs.yaml for job cinder-s2i-test-ceph. Add to preserve-home-paths if safe, or otherwise remove.
File .ssh/config found in build home directory during playbook github.com/openstack-k8s-operators/s2i-openstack-containers/playbooks/container-ci/zuul/pre.yaml for job cinder-s2i-content-provider. Add to preserve-home-paths if safe, or otherwise remove.

@openshift-merge-bot
openshift-merge-bot Bot merged commit e1f1c09 into openstack-k8s-operators:main Oct 22, 2026
6 checks passed
@tosky
tosky deleted the zuul-fix-nfs-job branch October 22, 2026 17:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@fmount fmount fmount approved these changes
@dprince dprince Awaiting requested review from dprince

Assignees

@fmount fmount

Labels

approved lgtm

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@tosky @fmount