Skip to content
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

Rework documentation for removal of streaming clustering #1207

Merged

Conversation

ThisIsMissEm
Copy link
Contributor

This PR is the companion documentation change to mastodon/mastodon#24655

@vercel
Copy link

vercel bot commented Apr 25, 2023

@ThisIsMissEm is attempting to deploy a commit to the Mastodon Team on Vercel.

A member of the Team first needs to authorize it.

- `10.0.0.0/8`
- `172.16.0.0/12`
- `192.168.0.0/16`
- `fc00::/7`
Copy link
Contributor Author

Choose a reason for hiding this comment

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

This shouldn't have been in here, as was changed automatically by prettier

@@ -59,14 +59,14 @@ systemctl reload mastodon-web
The `reload` operation is a zero-downtime restart, also called "phased restart". As such, Mastodon upgrades usually do not require any advance notice to users about planned downtime. In rare cases, you can use the `restart` operation instead, but there will be a (short) felt interruption of service for your users.
{{< /hint >}}

Rarely, the **streaming API** server is also updated and requires a restart:
Copy link
Contributor Author

Choose a reason for hiding this comment

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

@Gargron my thinking here is that we shouldn't be discouraging people from upgrading the streaming server, but just warn them that it can cause a temporary spike in resource consumption as users reconnect. (though this should be mitigated thanks to the reconnection logic)

@ClearlyClaire ClearlyClaire merged commit fe55511 into mastodon:master Apr 27, 2023
1 check failed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants