Skip to content
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.

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

Add --ignore-for-workflow parameter #508

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

bluca
Copy link
Contributor

@bluca bluca commented Jan 16, 2025

When a _service contains multiple obs_scm entries, the OBS SCM workflow branch_package will cause all of them to be updated to the fork/commit of the workflow. But this is rarely what we want, as for example in the case where the source code to build is in one main repository (that should be overridden), but there are also auxiliary repositories where the packaging recipes are stored, and that should not be updated, as the references will be invalid.

This is making it impossible to use branch_package on PRs for the systemd upstream CI project:

https://build.opensuse.org/project/show/home:bluca:systemd

The new parameters allows me to configure the auxiliary sources to be skipped by the branch_package update.

When a _service contains multiple obs_scm entries, the OBS SCM
workflow branch_package will cause all of them to be updated to
the fork/commit of the workflow. But this is rarely what we want,
as for example in the case where the source code to build is in
one main repository (that should be overridden), but there are
also auxiliary repositories where the packaging recipes are
stored, and that should not be updated, as the references will
be invalid.

This is making it impossible to use branch_package on PRs for
the systemd upstream CI project:

https://build.opensuse.org/project/show/home:bluca:systemd

The new parameters allows me to configure the auxiliary sources
to be skipped by the branch_package update.
@bluca
Copy link
Contributor Author

bluca commented Jan 21, 2025

@M0ses gentle ping - is this an acceptable way to solve this problem with the OBS workflow? Otherwise, any other suggestions? Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant