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

Mastodon server migration docs do not include redis #1127

Closed
smiba opened this issue Dec 22, 2022 · 2 comments
Closed

Mastodon server migration docs do not include redis #1127

smiba opened this issue Dec 22, 2022 · 2 comments

Comments

@smiba
Copy link
Contributor

smiba commented Dec 22, 2022

(Originally posted on the mastodon/mastodon git repo: mastodon/mastodon#22556)

Pitch

Currently, if you're following the mastodon docs on migrating to a new server you're not migrating your redis database.
(https://docs.joinmastodon.org/admin/migrating/)

This however results in temporarily having reduced features. The two most prominent issues I'm seeing right after my migration:

  1. Active users is listed as "0" and will have to start counting again resulting in lower numbers
  2. Trends are reset, they will all show really low numbers as the count has started over again

The active users is a bit annoying, but doesn't necessarily reduce functionality for those already registered, but no longer having the amount of posts in a certain hashtag be accurate can be quite annoying. It also means we're skipping over trends that might have been very popular 24 hours ago, but are not seeing much action at the moment.

If I had to do another migration, I'd absolutely consider moving my redis over. Another solution would be (at least for the active user count) to have a way to re-calculate this manually instead of waiting for the next cron

Motivation

Both users and server admins will benefit from this, right now my instance will look like a ghost town for people who pass by. No one wants to register on an instance that looks like it has "0 active users" followed by trends that list only covering 3 posts (suggesting bad federation)

Other then it scaring off new users, it also makes trends temporarily inaccurate which is inconvenient for users.

@smiba
Copy link
Contributor Author

smiba commented May 19, 2024

Is this something we're considering including? I'd be happy to write the PR for this.

@smiba
Copy link
Contributor Author

smiba commented May 19, 2024

Ah sorry, it seems it has been included in the docs nowadays! It is fixed by #1390

@smiba smiba closed this as completed May 19, 2024
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

No branches or pull requests

1 participant