Skip to content

Replace hardcoded /home/zuul paths in federation role#3390

Merged
amartyasinha merged 1 commit intoopenstack-k8s-operators:mainfrom
bshewale:replace-hardcoded-zuul-federation
Oct 15, 2025
Merged

Replace hardcoded /home/zuul paths in federation role#3390
amartyasinha merged 1 commit intoopenstack-k8s-operators:mainfrom
bshewale:replace-hardcoded-zuul-federation

Conversation

@bshewale
Copy link
Copy Markdown
Contributor

@bshewale bshewale commented Oct 9, 2025

Replace hardcoded /home/zuul/ paths with ansible_user_dir variable in hook source paths to support different user environments and improve consistency with configurable user variables pattern.

@bshewale bshewale requested a review from a team as a code owner October 9, 2025 10:03
@softwarefactory-project-zuul
Copy link
Copy Markdown

softwarefactory-project-zuul Bot commented Oct 9, 2025

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

✔️ openstack-k8s-operators-content-provider SUCCESS in 1h 35m 18s
✔️ podified-multinode-edpm-deployment-crc SUCCESS in 1h 21m 27s
✔️ cifmw-crc-podified-edpm-baremetal SUCCESS in 1h 15m 44s
✔️ cifmw-pod-zuul-files SUCCESS in 4m 39s
✔️ noop SUCCESS in 0s
✔️ cifmw-pod-ansible-test SUCCESS in 8m 41s
❌ cifmw-pod-pre-commit FAILURE in 6m 05s
✔️ build-push-container-cifmw-client SUCCESS in 20m 12s
✔️ cifmw-molecule-federation SUCCESS in 2m 38s

@bshewale
Replace hardcoded /home/zuul paths in federation role
ac8d70f 
Replace hardcoded /home/zuul/ paths with ansible_user_dir
variable in hook source paths to support different user
environments and improve consistency with configurable user
variables pattern.
@bshewale bshewale force-pushed the replace-hardcoded-zuul-federation branch from f0180cf to ac8d70f Compare October 13, 2025 04:22
@Deydra71
Copy link
Copy Markdown

Deydra71 commented Oct 13, 2025

/lgtm

@openshift-ci openshift-ci Bot added the lgtm label Oct 13, 2025
@Deydra71
Copy link
Copy Markdown

Deydra71 commented Oct 13, 2025

/approve

@openshift-ci
Copy link
Copy Markdown
Contributor

openshift-ci Bot commented Oct 13, 2025

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: Deydra71

The full list of commands accepted by this bot can be found 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

dmendiza
dmendiza approved these changes Oct 14, 2025
View reviewed changes
Copy link
Copy Markdown

@dmendiza dmendiza left a comment

Choose a reason for hiding this comment

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

lgtm

@jagee
Copy link
Copy Markdown
Contributor

jagee commented Oct 14, 2025

lgtm

@amartyasinha amartyasinha merged commit 12ad066 into openstack-k8s-operators:main Oct 15, 2025
5 checks passed
@xek xek mentioned this pull request Feb 5, 2026
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@dmendiza dmendiza dmendiza approved these changes

@amartyasinha amartyasinha amartyasinha approved these changes

@Deydra71 Deydra71 Deydra71 approved these changes

Assignees

@Deydra71 Deydra71

Labels

lgtm Ready For Review

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

5 participants

@bshewale @Deydra71 @jagee @dmendiza @amartyasinha