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

Adding steps for a server migration. #739

Open
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

ndeet
Copy link
Contributor

@ndeet ndeet commented Dec 16, 2022

No description provided.

Copy link
Member

@dennisreimann dennisreimann left a comment

Choose a reason for hiding this comment

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

I'd say the installation on the new server could happen before the (DNS) switch, so that only the restore process has to be run as the final step.

docs/backup-restore.md Show resolved Hide resolved
docs/backup-restore.md Show resolved Hide resolved
docs/backup-restore.md Show resolved Hide resolved
docs/backup-restore.md Outdated Show resolved Hide resolved
@ndeet
Copy link
Contributor Author

ndeet commented Dec 16, 2022

Thanks a lot for your fixes 💚

I'd say the installation on the new server could happen before the (DNS) switch, so that only the restore process has to be run as the final step.

I was not sure if the letsencrypt certs nginx vhosts etc. will work with the backup & restore. But thinking about it if everything is copied over they should and there is no need to have a new cert to be issued on the new server 🤔 ... what will happen in that case though is that the letsencrypt cert creation attempt will fail and show some error during installation? Not sure what is better.

@dennisreimann
Copy link
Member

Hmm, good point, I haven't tried either — I think the cert creation just fails without interupting the setup and it'll try again later.

@ndeet
Copy link
Contributor Author

ndeet commented Dec 21, 2022

Yes, it for sure does not interrupt the setup but likely shows some error messages which might confuse users or if everything is alright.

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