-
Notifications
You must be signed in to change notification settings - Fork 291
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
DAOS-15111 cq: fix thollander/actions-comment-pull-request #13820
Conversation
Test-tag: pr,vm comment_includes is now comment_tag Required-githooks: true Signed-off-by: Dalton Bohning <dalton.bohning@intel.com>
Bug-tracker data: |
Skip-test: true Skip-tests: true Skip-build: true Skip-unit-tests: true Skip-fault-injection-test: true Signed-off-by: Dalton Bohning <dalton.bohning@intel.com>
Skip-test: true Skip-tests: true Skip-build: true Skip-unit-tests: true Skip-fault-injection-test: true Signed-off-by: Dalton Bohning <dalton.bohning@intel.com>
Skip-test: true Skip-tests: true Skip-build: true Skip-unit-tests: true Skip-fault-injection-test: true Signed-off-by: Dalton Bohning <dalton.bohning@intel.com>
Changes verified on #13845: |
thollander/actions-comment-pull-request#160 looks to be the change that triggered this, what we're setting is now a tag rather than a string match on the comment, as such it's probably better to change the string as well so that it's meaningful as a tag. You could probably also remove that string from the comment as well, it was only added there so that there was a substring which was always included so there was something to match. |
Skip-test: true Skip-tests: true Skip-unit-tests: true Skip-fault-injection-test: true Required-githooks: true Signed-off-by: Dalton Bohning <dalton.bohning@intel.com>
Skip-test: true Skip-tests: true Skip-build: true Skip-unit-tests: true Skip-fault-injection-test: true Required-githooks: true Signed-off-by: Dalton Bohning <dalton.bohning@intel.com>
Bug-tracker data: |
Verified again: https://github.com/daos-stack/daos/actions/runs/8008062802 |
Jenkins testing was skipped since only a GHA was modified |
Test-tag: pr,vm
comment_includes is now comment_tag
Required-githooks: true
Before requesting gatekeeper:
Features:
(orTest-tag*
) commit pragma was used or there is a reason documented that there are no appropriate tags for this PR.Gatekeeper: