[full-ci][tests-only] test: add hook failures to the test failures list#2041
Merged
Conversation
Contributor
ScharfViktor
left a comment
ScharfViktor
left a comment
There was a problem hiding this comment.
That is correct but I'm afraid this will add instability to our CI.
Contributor
Author
something is not working here, a lot of tests failed. I will have a look. |
saw-jan
force-pushed
the
test/test-hook-failure
branch
from
f9d7fa3 to
2d26954
Compare
saw-jan
force-pushed
the
test/test-hook-failure
branch
3 times, most recently
from
4023d8f to
cb70798
Compare
saw-jan
force-pushed
the
test/test-hook-failure
branch
from
cb70798 to
b2e5fd9
Compare
Signed-off-by: Saw-jan <saw.jan.grg3e@gmail.com>
saw-jan
force-pushed
the
test/test-hook-failure
branch
from
b2e5fd9 to
cd408c6
Compare
Contributor
|
seems we need also add |
ScharfViktor
approved these changes
Jan 28, 2026
saw-jan
force-pushed
the
test/test-hook-failure
branch
from
9adecd8 to
cd408c6
Compare
renovate Bot
added a commit
to sdwilsh/ansible-playbooks
that referenced
this pull request
Feb 9, 2026
##### [\`5.0.2\`](https://github.com/opencloud-eu/opencloud/blob/HEAD/CHANGELOG.md#502---2026-02-05) ##### ❤️ Thanks to all contributors! ❤️ [@AlexAndBear](https://github.com/AlexAndBear), [@ScharfViktor](https://github.com/ScharfViktor), [@flimmy](https://github.com/flimmy), [@individual-it](https://github.com/individual-it), [@rhafer](https://github.com/rhafer), [@saw-jan](https://github.com/saw-jan) ##### 🐛 Bug Fixes - \[full-ci] reva-bump-2.42.3 \[[#2276](opencloud-eu/opencloud#2276)] ##### ✅ Tests - adapt test for [#514](opencloud-eu/opencloud#514) \[[#2255](opencloud-eu/opencloud#2255)] - api-test: upload-rename-download file with back slash \[[#2239](opencloud-eu/opencloud#2239)] - \[full-ci]\[tests-only] test: add hook failures to the test failures list \[[#2041](opencloud-eu/opencloud#2041)] ##### 📚 Documentation - docs(proxy): Clarify PROXY\_OIDC\_USERINFO\_CACHE\_TTL value \[[#2256](opencloud-eu/opencloud#2256)] ##### 📦️ Dependencies - \[full-ci] reva-bump-2.42.2 \[[#2270](opencloud-eu/opencloud#2270)] - build(deps): bump github.com/grpc-ecosystem/grpc-gateway/v2 from 2.27.5 to 2.27.6 \[[#2238](opencloud-eu/opencloud#2238)]
sdwilsh
pushed a commit
to sdwilsh/ansible-playbooks
that referenced
this pull request
Feb 9, 2026
##### [\`5.0.2\`](https://github.com/opencloud-eu/opencloud/blob/HEAD/CHANGELOG.md#502---2026-02-05) ##### ❤️ Thanks to all contributors! ❤️ [@AlexAndBear](https://github.com/AlexAndBear), [@ScharfViktor](https://github.com/ScharfViktor), [@flimmy](https://github.com/flimmy), [@individual-it](https://github.com/individual-it), [@rhafer](https://github.com/rhafer), [@saw-jan](https://github.com/saw-jan) ##### 🐛 Bug Fixes - \[full-ci] reva-bump-2.42.3 \[[#2276](opencloud-eu/opencloud#2276)] ##### ✅ Tests - adapt test for [#514](opencloud-eu/opencloud#514) \[[#2255](opencloud-eu/opencloud#2255)] - api-test: upload-rename-download file with back slash \[[#2239](opencloud-eu/opencloud#2239)] - \[full-ci]\[tests-only] test: add hook failures to the test failures list \[[#2041](opencloud-eu/opencloud#2041)] ##### 📚 Documentation - docs(proxy): Clarify PROXY\_OIDC\_USERINFO\_CACHE\_TTL value \[[#2256](opencloud-eu/opencloud#2256)] ##### 📦️ Dependencies - \[full-ci] reva-bump-2.42.2 \[[#2270](opencloud-eu/opencloud#2270)] - build(deps): bump github.com/grpc-ecosystem/grpc-gateway/v2 from 2.27.5 to 2.27.6 \[[#2238](opencloud-eu/opencloud#2238)]
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Description
Whenever any of the test hooks fail, add them to the list of test failures so that the whole scenario is reported as failed at the end of the test run.
Related Issue
How Has This Been Tested?
Screenshots (if appropriate):
Types of changes
Checklist: