Skip to content

refactor(ci_dcn_site): replace hardcoded /home/zuul paths #3395

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

refactor(ci_dcn_site): replace hardcoded /home/zuul paths #3395

wants to merge 1 commit into from
+1 −1

Conversation

Valkyrie00
Copy link
Contributor

@Valkyrie00 Valkyrie00 commented Oct 9, 2025 •
edited
Loading

Description

Replace hardcoded /home/zuul paths with ansible_user_dir variable in the ci_dcn_site role to support configurable user paths.

Changes

  • roles/ci_dcn_site/tasks/ceph.yml:
    • output_dir in Deploy Ceph task: Use ansible_user_dir instead of/home/zuul

@Valkyrie00 Valkyrie00 requested a review from a team October 9, 2025 22:17
@openshift-ci OpenShift CI
Copy link
Contributor

openshift-ci bot commented Oct 9, 2025

Skipping CI for Draft Pull Request.
If you want CI signal for your change, please convert it to an actual PR.
You can still manually trigger a test run with /test all

@openshift-ci OpenShift CI
Copy link
Contributor

openshift-ci bot commented Oct 9, 2025

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by:

The full list of commands accepted by this bot can be found here.

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

@Valkyrie00 Valkyrie00 changed the title [DNM] refactor(ci_dcn_site): replace hardcoded /home/zuul paths refactor(ci_dcn_site): replace hardcoded /home/zuul paths Oct 13, 2025
@Valkyrie00 Valkyrie00 marked this pull request as ready for review October 13, 2025 14:07
@Valkyrie00 Valkyrie00 requested a review from a team as a code owner October 13, 2025 14:07
bshewale
bshewale previously approved these changes Oct 14, 2025
View reviewed changes
evallesp
evallesp previously approved these changes Oct 14, 2025
View reviewed changes
@evallesp evallesp enabled auto-merge (rebase) October 14, 2025 09:22
danpawlik
danpawlik previously approved these changes Oct 17, 2025
View reviewed changes
@Valkyrie00
refactor(ci_dcn_site): replace hardcoded /home/zuul paths
c8f939d 
Replace hardcoded '/home/zuul' paths with ansible_user_dir variable to support non-zuul users in ci_cdn_role

# Conflicts:
#	roles/ci_dcn_site/defaults/main.yml
@Valkyrie00 Valkyrie00 dismissed stale reviews from danpawlik, evallesp, and bshewale via c8f939d October 20, 2025 08:09
@Valkyrie00 Valkyrie00 force-pushed the feat/replace-hardcoded-zuul-path-ci-dcn-site branch from c4eda71 to c8f939d Compare October 20, 2025 08:09
@softwarefactory-project-zuul softwarefactory-project-zuul
Copy link

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://softwarefactory-project.io/zuul/t/rdoproject.org/buildset/9eeed932391d4da2a7da675660c250bb

❌ openstack-k8s-operators-content-provider FAILURE in 7m 16s
⚠️ podified-multinode-edpm-deployment-crc SKIPPED Skipped due to failed job openstack-k8s-operators-content-provider
⚠️ cifmw-crc-podified-edpm-baremetal SKIPPED Skipped due to failed job openstack-k8s-operators-content-provider
✔️ cifmw-pod-zuul-files SUCCESS in 4m 26s
✔️ noop SUCCESS in 0s
❌ cifmw-pod-ansible-test TIMED_OUT in 30m 56s
✔️ cifmw-pod-pre-commit SUCCESS in 7m 47s
❌ build-push-container-cifmw-client FAILURE in 5m 21s
✔️ cifmw-molecule-ci_dcn_site SUCCESS in 2m 24s

@Valkyrie00
Copy link
Contributor Author

recheck

@Valkyrie00 Valkyrie00 closed this Oct 20, 2025
auto-merge was automatically disabled October 20, 2025 09:00

Pull request was closed

@Valkyrie00 Valkyrie00 reopened this Oct 20, 2025
@softwarefactory-project-zuul softwarefactory-project-zuul
Copy link

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://softwarefactory-project.io/zuul/t/rdoproject.org/buildset/1a3b1d52c18045d2847ce1352d9fd6a0

✔️ openstack-k8s-operators-content-provider SUCCESS in 1h 55m 13s
✔️ podified-multinode-edpm-deployment-crc SUCCESS in 1h 22m 13s
✔️ cifmw-crc-podified-edpm-baremetal SUCCESS in 1h 32m 11s
✔️ cifmw-pod-zuul-files SUCCESS in 4m 16s
✔️ noop SUCCESS in 0s
❌ cifmw-pod-ansible-test TIMED_OUT in 30m 56s
✔️ cifmw-pod-pre-commit SUCCESS in 7m 29s
✔️ build-push-container-cifmw-client SUCCESS in 17m 17s
✔️ cifmw-molecule-ci_dcn_site SUCCESS in 2m 20s

@Valkyrie00
Copy link
Contributor Author

/retest

1 similar comment
@Valkyrie00
Copy link
Contributor Author

/retest

@Valkyrie00
Copy link
Contributor Author

recheck

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@evallesp evallesp evallesp approved these changes

@danpawlik danpawlik danpawlik left review comments

@bshewale bshewale bshewale left review comments

At least 2 approving reviews are required to merge this pull request.

Assignees

No one assigned

Labels

Ready For Review

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@Valkyrie00 @danpawlik @evallesp @bshewale