Skip to content

Conversation

bshewale
Copy link
Collaborator

@bshewale bshewale commented Oct 9, 2025

Replace hardcoded /home/zuul/ paths with ansible_user_dir variable in kustomize_deploy & ci_gen_kustomize_values role 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 13:25
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

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

✔️ openstack-k8s-operators-content-provider SUCCESS in 1h 43m 54s
✔️ podified-multinode-edpm-deployment-crc SUCCESS in 1h 19m 42s
cifmw-crc-podified-edpm-baremetal FAILURE in 43m 00s
✔️ cifmw-pod-zuul-files SUCCESS in 4m 39s
✔️ noop SUCCESS in 0s
✔️ cifmw-pod-ansible-test SUCCESS in 8m 35s
cifmw-pod-pre-commit FAILURE in 6m 13s
✔️ cifmw-architecture-validate-hci SUCCESS in 5m 47s
✔️ build-push-container-cifmw-client SUCCESS in 20m 39s
cifmw-molecule-ci_gen_kustomize_values FAILURE in 3m 25s
✔️ cifmw-molecule-kustomize_deploy SUCCESS in 4m 38s

@bshewale bshewale force-pushed the replace-hardcoded-zuul-ci_gen_kustomize branch from 47f7310 to d334ded Compare October 13, 2025 12:14
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/72ba670fbc3642d1af7041e3d6608b7a

✔️ openstack-k8s-operators-content-provider SUCCESS in 1h 41m 23s
✔️ podified-multinode-edpm-deployment-crc SUCCESS in 1h 14m 49s
✔️ cifmw-crc-podified-edpm-baremetal SUCCESS in 1h 27m 40s
✔️ cifmw-pod-zuul-files SUCCESS in 4m 34s
✔️ noop SUCCESS in 0s
✔️ cifmw-pod-ansible-test SUCCESS in 8m 24s
✔️ cifmw-pod-pre-commit SUCCESS in 7m 46s
✔️ cifmw-architecture-validate-hci SUCCESS in 5m 07s
✔️ build-push-container-cifmw-client SUCCESS in 18m 46s
cifmw-molecule-ci_gen_kustomize_values FAILURE in 5m 49s
✔️ cifmw-molecule-kustomize_deploy SUCCESS in 6m 22s

@bshewale bshewale force-pushed the replace-hardcoded-zuul-ci_gen_kustomize branch from d334ded to 1190962 Compare October 14, 2025 12:40
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/f1683ad050ca40d98be9f3f59ef22d1d

✔️ openstack-k8s-operators-content-provider SUCCESS in 1h 41m 23s
✔️ podified-multinode-edpm-deployment-crc SUCCESS in 1h 17m 41s
✔️ cifmw-crc-podified-edpm-baremetal SUCCESS in 1h 25m 59s
✔️ cifmw-pod-zuul-files SUCCESS in 4m 47s
✔️ noop SUCCESS in 0s
✔️ cifmw-pod-ansible-test SUCCESS in 9m 42s
✔️ cifmw-pod-pre-commit SUCCESS in 7m 49s
✔️ cifmw-architecture-validate-hci SUCCESS in 5m 16s
✔️ build-push-container-cifmw-client SUCCESS in 20m 14s
cifmw-molecule-ci_gen_kustomize_values FAILURE in 3m 48s
✔️ cifmw-molecule-kustomize_deploy SUCCESS in 4m 54s

@bshewale bshewale force-pushed the replace-hardcoded-zuul-ci_gen_kustomize branch 14 times, most recently from 878243d to ff40abc Compare October 15, 2025 11:56
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/de18c89c6f8b49be80aac12f58e7f158

✔️ openstack-k8s-operators-content-provider SUCCESS in 1h 55m 41s
✔️ podified-multinode-edpm-deployment-crc SUCCESS in 1h 17m 34s
✔️ cifmw-crc-podified-edpm-baremetal SUCCESS in 1h 39m 31s
✔️ cifmw-pod-zuul-files SUCCESS in 4m 40s
✔️ noop SUCCESS in 0s
✔️ cifmw-pod-ansible-test SUCCESS in 9m 20s
✔️ cifmw-pod-pre-commit SUCCESS in 8m 40s
✔️ cifmw-architecture-validate-hci SUCCESS in 6m 17s
✔️ build-push-container-cifmw-client SUCCESS in 20m 52s
cifmw-molecule-ci_gen_kustomize_values FAILURE in 3m 17s
✔️ cifmw-molecule-kustomize_deploy SUCCESS in 4m 20s

@bshewale bshewale force-pushed the replace-hardcoded-zuul-ci_gen_kustomize branch 3 times, most recently from 90c7208 to 9a49bd4 Compare October 16, 2025 10:04
@bshewale bshewale force-pushed the replace-hardcoded-zuul-ci_gen_kustomize branch 4 times, most recently from fb8e8bc to c89796a Compare October 16, 2025 10:36
@bshewale bshewale force-pushed the replace-hardcoded-zuul-ci_gen_kustomize branch 2 times, most recently from 9c10c85 to 548fe9c Compare October 16, 2025 11:07
@danpawlik
Copy link
Contributor

diff --git a/roles/ci_gen_kustomize_values/molecule/default/prepare.yml b/roles/ci_gen_kustomize_values/molecule/default/prepare.yml
index 34cbe58d..b11327d2 100644
--- a/roles/ci_gen_kustomize_values/molecule/default/prepare.yml
+++ b/roles/ci_gen_kustomize_values/molecule/default/prepare.yml
@@ -20,7 +20,7 @@
   vars:
     cifmw_ci_gen_kustomize_values_src_dir: >-
       {{
-        (lookup('env', 'HOME', '/home/zuul'),
+        (lookup('env', 'HOME'),
          'ci-framework-data',
          'artifacts', 'ci_k8s_snippets') | path_join
       }}
@@ -47,5 +47,6 @@

 - name: Download tools for later testing and validations
   ansible.builtin.import_playbook: >-
-    {{ ('/home/zuul/src/github.com/openstack-k8s-operators',
+    {{ (lookup('env', 'HOME'),
+        'src/github.com/openstack-k8s-operators',
         'install_yamls/devsetup/download_tools.yaml') | path_join }}

that works on my side

@bshewale bshewale force-pushed the replace-hardcoded-zuul-ci_gen_kustomize branch from 548fe9c to 70ac86f Compare October 16, 2025 11:30
@bshewale bshewale changed the title Replace hardcoded /home/zuul paths in kustomize_deploy & ci_gen_kusto… Replace hardcoded /home/zuul paths in kustomize_deploy & ci_gen_kustomize_values Oct 16, 2025
Copy link

This change depends on a change that failed to merge.

Change #3411 is needed.

@bshewale
Copy link
Collaborator Author

recheck

@bshewale bshewale force-pushed the replace-hardcoded-zuul-ci_gen_kustomize branch from 70ac86f to 817b68b Compare October 16, 2025 11:42
@bshewale
Copy link
Collaborator Author

diff --git a/roles/ci_gen_kustomize_values/molecule/default/prepare.yml b/roles/ci_gen_kustomize_values/molecule/default/prepare.yml
index 34cbe58d..b11327d2 100644
--- a/roles/ci_gen_kustomize_values/molecule/default/prepare.yml
+++ b/roles/ci_gen_kustomize_values/molecule/default/prepare.yml
@@ -20,7 +20,7 @@
   vars:
     cifmw_ci_gen_kustomize_values_src_dir: >-
       {{
-        (lookup('env', 'HOME', '/home/zuul'),
+        (lookup('env', 'HOME'),
          'ci-framework-data',
          'artifacts', 'ci_k8s_snippets') | path_join
       }}
@@ -47,5 +47,6 @@

 - name: Download tools for later testing and validations
   ansible.builtin.import_playbook: >-
-    {{ ('/home/zuul/src/github.com/openstack-k8s-operators',
+    {{ (lookup('env', 'HOME'),
+        'src/github.com/openstack-k8s-operators',
         'install_yamls/devsetup/download_tools.yaml') | path_join }}

that works on my side

Working.

…mize_values

Replace hardcoded /home/zuul/ paths with ansible_user_dir variable
in kustomize_deploy & ci_gen_kustomize_values role to support different
user environments and improve consistency with configurable user
variables pattern.
@bshewale bshewale force-pushed the replace-hardcoded-zuul-ci_gen_kustomize branch from 817b68b to dd4eb1d Compare October 16, 2025 12:03
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.

2 participants