-
-
Notifications
You must be signed in to change notification settings - Fork 4.7k
Disable fragile tests for now #14541
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
Conversation
Signed-off-by: Morris Jobke <[email protected]>
|
🤖 beep boop beep 🤖 Here are the logs for the failed build: Status of 16768: failure
Show full log
Show full log
Show full log
Show full log |
ChristophWurst
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Fine by me. But will CI pass on this one? 🙈
Now there are other failing ones 😭 Let me retrigger the job. |
|
🤖 beep boop beep 🤖 Here are the logs for the failed build: Status of 16789: failureTESTS=acceptance, TESTS-ACCEPTANCE=app-comments
Show full logTESTS=acceptance, TESTS-ACCEPTANCE=app-files
Show full logTESTS=acceptance, TESTS-ACCEPTANCE=app-files-tags
Show full logTESTS=acceptance, TESTS-ACCEPTANCE=header
Show full logTESTS=sqlite-php7.1-samba-non-nativeShow full log |
|
And there is the revert: #14557 |
Those are the ones that fail on all the branches and just hide the real errors. After merging this I will open a revert of this PR to properly analyze where this broke.