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

Utilise default labels and identifier from config for test comment #2270

Conversation

lbarcziova
Copy link
Member

@lbarcziova lbarcziova commented Dec 6, 2023

For /packit test comment command without arguments, utilise the values from the configurations.

Fixes packit/packit#2056
Merge after packit/packit#2180

TODO:

  • fix the tests
  • docs

RELEASE NOTES BEGIN
We have introduced new configuration options test_command.default_labels and test_command.default_identifier that are used by default when running /packit test comment command without any arguments (instead of specifying them via --labels/--identifier).
RELEASE NOTES END

Copy link
Contributor

@lbarcziova
Copy link
Member Author

recheck

Copy link
Contributor

For /packit test comment command without arguments utilise the values
from the configurations.

Fixes packit/packit#2056
Merge after packit/packit#2180
@lbarcziova lbarcziova force-pushed the default-test-comment-options branch from 7b1cd5a to 076eaea Compare December 7, 2023 14:06
Copy link
Contributor

@lbarcziova lbarcziova added the mergeit When set, zuul wil gate and merge the PR. label Dec 7, 2023
Copy link
Contributor

Build succeeded (gate pipeline).
https://softwarefactory-project.io/zuul/t/packit-service/buildset/cbfbc4fb2862481289de76859e037cf3

✔️ pre-commit SUCCESS in 1m 55s

@softwarefactory-project-zuul softwarefactory-project-zuul bot merged commit f20e57c into packit:main Dec 7, 2023
@lbarcziova lbarcziova deleted the default-test-comment-options branch December 7, 2023 14:24
lbarcziova added a commit to lbarcziova/packit.dev that referenced this pull request Dec 7, 2023
lbarcziova added a commit to lbarcziova/packit.dev that referenced this pull request Dec 7, 2023
lbarcziova added a commit to lbarcziova/packit.dev that referenced this pull request Dec 11, 2023
softwarefactory-project-zuul bot added a commit to packit/packit.dev that referenced this pull request Dec 12, 2023
Document new test_command options

Documentation for packit/packit-service#2270
Related to packit/packit#2056

Reviewed-by: Nikola Forró
Reviewed-by: Laura Barcziová
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
mergeit When set, zuul wil gate and merge the PR.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Allow for rerunning only automatic (not on-demand) tests upon /packit test without labels specified
2 participants