Skip to content

Log when a storage is marked as unavailable - #26299

Merged
MorrisJobke merged 1 commit into
masterfrom
enh/noid/log-storage-unavailable
Mar 25, 2021
Merged

Log when a storage is marked as unavailable#26299
MorrisJobke merged 1 commit into
masterfrom
enh/noid/log-storage-unavailable

Conversation

@MorrisJobke

Copy link
Copy Markdown
Member

Sometimes it might be good to trace when a storage gets unavailable.

@MorrisJobkeMorrisJobke added the 3. to review Waiting for reviews label Mar 25, 2021
@MorrisJobkeMorrisJobke added this to the Nextcloud 22 milestone Mar 25, 2021
@MorrisJobke

Copy link
Copy Markdown
MemberAuthor

/backport to stable21

@MorrisJobke

Copy link
Copy Markdown
MemberAuthor

/backport to stable20

@MorrisJobke

Copy link
Copy Markdown
MemberAuthor

the 20 backport needs most likely an adjustment for the used class.

Signed-off-by: Morris Jobke <hey@morrisjobke.de>
@MorrisJobke
MorrisJobkeforce-pushed the enh/noid/log-storage-unavailable branch from 7fca4d2 to 81fef4dCompareMarch 25, 2021 15:19

@rullzerrullzer left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

But php-cs is not happy

@MorrisJobke

Copy link
Copy Markdown
MemberAuthor

the 20 backport needs most likely an adjustment for the used class.

Was added to 20 - so all good.

@MorrisJobke

Copy link
Copy Markdown
MemberAuthor

But php-cs is not happy

But php-cs was not happy

@MorrisJobkeMorrisJobke added 4. to release Ready to be released and/or waiting for tests to finish and removed 3. to review Waiting for reviews labels Mar 25, 2021
@MorrisJobke
MorrisJobke merged commit c25daf5 into masterMar 25, 2021
@MorrisJobke
MorrisJobke deleted the enh/noid/log-storage-unavailable branch March 25, 2021 16:08
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

@MorrisJobke@rullzer@icewind1991