-
Notifications
You must be signed in to change notification settings - Fork 61
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鈥檒l occasionally send you account related emails.
Already on GitHub? Sign in to your account
Validate and update tutorial #455
Conversation
Co-authored-by: Chris <github.account@chrigel.net>
Co-authored-by: Chris <github.account@chrigel.net>
Co-authored-by: Chris <github.account@chrigel.net>
Co-authored-by: Chris <github.account@chrigel.net>
Co-authored-by: Chris <github.account@chrigel.net>
I only had one remark so far. It's not really important, but if you have time/wish to experiment around feel free to try it out. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM. Didn't verify whether the tutorial works though.
LGTM Maybe in the future we could migrate it away from minikube? Kind would be perfectly fine here. |
I spotted a wrong path from when I moved the backup example YAML but I think now I'm really really done with this :) |
feel free to merge then :) |
Summary
The tutorial had some parts that didn't match reality anymore. There were also some defects (missing files, wrong paths) from when the separate tutorial repo was merged in with this main repo. I tried running every step of the tutorial and fixed as I went along.
There is a pretty signifcant change to the way WordPress is restored: I couldn't get it to work as previously documented, nothing would be restored to the existing and bound PVC. A solution that worked was to bind another empty PVC to
/restore
and restore to this, then rsync the files using a shell in the container. The scripts have been adapted for this.Checklist
bug
,enhancement
,documentation
,change
,breaking
,as they show up in the changelog