Skip to content

WIP: Use group_vars in ci/playbooks#3237

Closed
amartyasinha wants to merge 1 commit intoopenstack-k8s-operators:mainfrom
amartyasinha:global-vars
Closed

WIP: Use group_vars in ci/playbooks#3237
amartyasinha wants to merge 1 commit intoopenstack-k8s-operators:mainfrom
amartyasinha:global-vars

Conversation

@amartyasinha
Copy link
Copy Markdown
Contributor

@amartyasinha amartyasinha commented Aug 29, 2025 •
edited
Loading

Since Zuul is not upgraded, we will have to do the workaround to read group_vars in ci/playbooks executed by Zuul executor. This commit tries to do so.

If Zuul is upgraded, then we can drop this PR and modify jobs definition to include group_vars files.

@openshift-ci
Copy link
Copy Markdown
Contributor

openshift-ci Bot commented Aug 29, 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
Copy link
Copy Markdown
Contributor

openshift-ci Bot commented Aug 29, 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.

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

@softwarefactory-project-zuul

This comment was marked as outdated.

Sign in to view
@amartyasinha
Copy link
Copy Markdown
Contributor Author

amartyasinha commented Sep 1, 2025

recheck

@danpawlik
Copy link
Copy Markdown
Contributor

danpawlik commented Sep 1, 2025

@amartyasinha about adoption-standalone-to-crc-ceph-provider I think it might be better when #3193 would be working... OR https://review.rdoproject.org/r/c/rdo-jobs/+/58147

@softwarefactory-project-zuul

This comment was marked as outdated.

Sign in to view
@amartyasinha
Copy link
Copy Markdown
Contributor Author

amartyasinha commented Sep 1, 2025

@amartyasinha about adoption-standalone-to-crc-ceph-provider I think it might be better when #3193 would be working... OR https://review.rdoproject.org/r/c/rdo-jobs/+/58147

Should I add Depends-On and then recheck?

@softwarefactory-project-zuul

This comment was marked as outdated.

Sign in to view
@amartyasinha
Copy link
Copy Markdown
Contributor Author

amartyasinha commented Sep 3, 2025

recheck

@softwarefactory-project-zuul

This comment was marked as outdated.

Sign in to view
@softwarefactory-project-zuul

This comment was marked as outdated.

Sign in to view
@amartyasinha amartyasinha force-pushed the global-vars branch 3 times, most recently from 921fdb6 to 1fd631f Compare September 4, 2025 10:35
@softwarefactory-project-zuul

This comment was marked as outdated.

Sign in to view
@michburk
Copy link
Copy Markdown
Contributor

michburk commented Sep 5, 2025

recheck

@softwarefactory-project-zuul

This comment was marked as outdated.

Sign in to view
@amartyasinha
Copy link
Copy Markdown
Contributor Author

amartyasinha commented Sep 8, 2025

recheck

@softwarefactory-project-zuul

This comment was marked as outdated.

Sign in to view
This was referenced Sep 11, 2025
Merged
Merged
Merged
@softwarefactory-project-zuul

This comment was marked as outdated.

Sign in to view
@amartyasinha
Copy link
Copy Markdown
Contributor Author

amartyasinha commented Sep 16, 2025

recheck

@softwarefactory-project-zuul

This comment was marked as outdated.

Sign in to view
@amartyasinha amartyasinha force-pushed the global-vars branch 8 times, most recently from bfffe19 to b7deffb Compare October 13, 2025 06:58
@softwarefactory-project-zuul

This comment was marked as outdated.

Sign in to view
@softwarefactory-project-zuul

This comment was marked as outdated.

Sign in to view
@danpawlik danpawlik mentioned this pull request Oct 13, 2025
Closed
@amartyasinha
Copy link
Copy Markdown
Contributor Author

amartyasinha commented Oct 14, 2025

recheck

@softwarefactory-project-zuul

This comment was marked as outdated.

Sign in to view
@amartyasinha amartyasinha force-pushed the global-vars branch 2 times, most recently from 06c067e to 78e1115 Compare October 22, 2025 10:05
@amartyasinha amartyasinha changed the title WIP: Move common variable to group_vars: cifmw_project_dir & cifmw_project_dir_absolute WIP: Use group_vars in ci/playbooks: cifmw_project_dir & cifmw_project_dir_absolute Oct 22, 2025
@amartyasinha
Use group_vars in ci/playbooks: cifmw_project_dir & cifmw_project_dir…
70f81c8 
…_absolute

Since Zuul is not upgraded, we will have to do the workaround to read group_vars in ci/playbooks executed by Zuul executor. This commit tries to do so.

Signed-off-by: Amartya Sinha <amsinha@redhat.com>
@softwarefactory-project-zuul

This comment was marked as outdated.

Sign in to view
@amartyasinha amartyasinha changed the title WIP: Use group_vars in ci/playbooks: cifmw_project_dir & cifmw_project_dir_absolute WIP: Use group_vars in ci/playbooks Oct 29, 2025
@github-actions
Copy link
Copy Markdown

github-actions Bot commented Nov 14, 2025

This PR is stale because it has been for over 15 days with no activity.
Remove stale label or comment or this will be closed in 7 days.

@github-actions github-actions Bot added the Stale label Nov 14, 2025
@github-actions github-actions Bot closed this Nov 22, 2025
@amartyasinha amartyasinha deleted the global-vars branch January 9, 2026 06:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

do-not-merge/work-in-progress Stale

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@amartyasinha @danpawlik @michburk @openshift-merge-robot