Skip to content

Conversation

markgoddard
Copy link

Running the full workflows can take a long time, which may be
undesirable when we want to get changes only for a small number of
package repositories (e.g. for a security or bug fix release).

This change adds the ability to specify a filter to apply to the list of
package repositories in the various playbooks. The filter is also
exposed in the Github actions workflows.

Running the full workflows can take a long time, which may be
undesirable when we want to get changes only for a small number of
package repositories (e.g. for a security or bug fix release).

This change adds the ability to specify a filter to apply to the list of
package repositories in the various playbooks. The filter is also
exposed in the Github actions workflows.
@markgoddard markgoddard requested a review from a team as a code owner August 19, 2022 15:50
@markgoddard markgoddard self-assigned this Aug 19, 2022
ansible-playbook -i ansible/inventory \
ansible/test-pulp-repo-version-query.yml \
ansible/test-kayobe-repo-version-generate.yml \
-e deb_package_repo_filter="'$FILTER'" \
Copy link
Contributor

Choose a reason for hiding this comment

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

Does something swallow the single quotes? Otherwise, I'd expect the variable not to be interpreted 🤔

Copy link
Author

Choose a reason for hiding this comment

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

Seems not

foo=bar
echo "'$foo'"
'bar'

Copy link
Contributor

Choose a reason for hiding this comment

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

Didn't know that! Found the table in: https://stackoverflow.com/a/42082956 useful.

Copy link
Author

Choose a reason for hiding this comment

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

nice find

@markgoddard markgoddard merged commit 95293a8 into main Aug 30, 2022
@markgoddard markgoddard deleted the package-filters branch August 30, 2022 15:10
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.

2 participants