Skip to content

[stable21] Fix removing remote shares when the remote server is unreachable - #29564

Closed
backportbot-nextcloud[bot] wants to merge 6 commits into
stable21from
backport/25332/stable21
Closed

[stable21] Fix removing remote shares when the remote server is unreachable#29564
backportbot-nextcloud[bot] wants to merge 6 commits into
stable21from
backport/25332/stable21

Conversation

@backportbot-nextcloud

Copy link
Copy Markdown

backport of #25332

danxuliuand others added 6 commits November 5, 2021 13:55
Signed-off-by: Daniel Calviño Sánchez <danxuliu@gmail.com>
Signed-off-by: Daniel Calviño Sánchez <danxuliu@gmail.com>
The federated server needs to be stopped during the tests, so it is now
stopped in the FederationContext for each scenario instead of just once
in the run.sh script.
Signed-off-by: Daniel Calviño Sánchez <danxuliu@gmail.com>
Technically, saying that a storage has no updates when it's not
available is correct.
This makes it possible to retrieve the cache entry for the mount point
and also to list and remove unavailable federated shares.
Signed-off-by: Vincent Petry <vincent@nextcloud.com>
Signed-off-by: Vincent Petry <vincent@nextcloud.com>
Signed-off-by: Vincent Petry <vincent@nextcloud.com>
Co-authored-by: Carl Schwan <carl@carlschwan.eu>
@PVince81

Copy link
Copy Markdown
Member

let's hold for now as this changes the behavior of unavailable external storages

@szaimen

Copy link
Copy Markdown
Contributor

let's hold for now as this changes the behavior of unavailable external storages

Does it fix #20995?

@skjnldsvskjnldsv 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.

@blizzz

Copy link
Copy Markdown
Member

moved to 21.0.7

This was referenced Jan 7, 2022
@MichaIngMichaIng added the 3. to review Waiting for reviews label Jan 12, 2022
@skjnldsvskjnldsv mentioned this pull request Jan 20, 2022
9 tasks
@skjnldsvskjnldsv mentioned this pull request Jan 25, 2022
8 tasks
@blizzz

Copy link
Copy Markdown
Member

let's hold for now as this changes the behavior of unavailable external storages

@PVince81 how to go forward with this PR?

@blizzz

Copy link
Copy Markdown
Member

no reply, closing

@blizzzblizzz closed this Mar 21, 2022
@MichaIng
MichaIng deleted the backport/25332/stable21 branch March 21, 2022 15:40
@MichaIngMichaIng removed this from the Nextcloud 21.0.10 milestone Mar 21, 2022
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants

@PVince81@szaimen@blizzz@skjnldsv@MichaIng@danxuliu