Skip to content

[stable18] set the mount id before calling storage wrapper - #22518

Merged
rullzer merged 1 commit into
stable18from
backport/22517/stable18
Sep 1, 2020
Merged

[stable18] set the mount id before calling storage wrapper#22518
rullzer merged 1 commit into
stable18from
backport/22517/stable18

Conversation

@backportbot-nextcloud

Copy link
Copy Markdown

backport of #22517

this allows the storage wrapper to use the mount id for it's own logic
Signed-off-by: Robin Appelman <robin@icewind.nl>
@faily-bot

faily-botBot commented Aug 31, 2020

Copy link
Copy Markdown

🤖 beep boop beep 🤖

Here are the logs for the failed build:

Status of 32482: failure

mariadb10.4-php7.3

Show full log
There were 2 warnings:
1) Test\Files\ViewTest::testRenameFailDeleteTargetKeepSource
Trying to configure method "writeStream" which cannot be configured because it does not exist, has not been specified, is final, or is static
2) Test\Files\ViewTest::testCopyFailDeleteTargetKeepSource
Trying to configure method "writeStream" which cannot be configured because it does not exist, has not been specified, is final, or is static
--
There was 1 failure:
1) Test\User\DatabaseTest::testSearch
Failed asserting that actual size 3 matches expected size 2.
/drone/src/tests/lib/User/Backend.php:114
/drone/src/tests/lib/User/DatabaseTest.php:125

@rullzerrullzer added the 3. to review Waiting for reviews label Aug 31, 2020
@juliusknorrjuliusknorr added 4. to release Ready to be released and/or waiting for tests to finish and removed 3. to review Waiting for reviews labels Aug 31, 2020
@rullzer
rullzer merged commit f9fb6ca into stable18Sep 1, 2020
@rullzer
rullzer deleted the backport/22517/stable18 branch September 1, 2020 06:05
@rullzerrullzer mentioned this pull request Sep 4, 2020
@MorrisJobkeMorrisJobke mentioned this pull request Sep 9, 2020
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

4. to releaseReady to be released and/or waiting for tests to finish

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants

@rullzer@juliusknorr@icewind1991