vitessio / website

vitess.io website and other project management related content
https://vitess.io
Apache License 2.0
54 stars 204 forks source link

Docs: vttablet PITR via --restore-to-timestamp and --restore-to-position #1594

Closed shlomi-noach closed 12 months ago

shlomi-noach commented 1 year ago

Documenting the changes in https://github.com/vitessio/vitess/pull/13964, and specifically these vttablet flags:

Either of which, combined with --restore_from_backup, causes the tablet to start using a point-in-time recovery.

Only merge once https://github.com/vitessio/vitess/pull/13964 is merged

netlify[bot] commented 1 year ago

Deploy Preview for vitess ready!

Name Link
Latest commit 7c0fd703d371be7d3b03b78e1b8550624c224270
Latest deploy log https://app.netlify.com/sites/vitess/deploys/652527905d3462000802faa4
Deploy Preview https://deploy-preview-1594--vitess.netlify.app
Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

shlomi-noach commented 1 year ago

Cool! This actually changed nothing, except for this line, which apparently leaks devs' home directories:

--config-path strings                                              Paths to search for config files in. (default [/Users/andrew/dev/vitess])
ajm188 commented 1 year ago

Cool! This actually changed nothing, except for this line, which apparently leaks devs' home directories:

--config-path strings                                              Paths to search for config files in. (default [/Users/andrew/dev/vitess])

yeah, we noticed this last night and @frouioui opened #1601 for it

ajm188 commented 1 year ago

hmm, this should have updated https://vitess.io/docs/18.0/reference/programs/vttablet/#options with the added flags, was your HEAD maybe not on that branch?

shlomi-noach commented 1 year ago

was your HEAD maybe not on that branch?

It was, and I also ran make build to ensure the binaries were up to date...

ajm188 commented 1 year ago

ahh, your PR is based 2 weeks ago, before the vttablet docgen stuff got added. if you merge to at least https://github.com/vitessio/vitess/commit/98c754a8fa068c7327fd05b7d83ac63da0cd80dd you should get it working

shlomi-noach commented 1 year ago

I've merged latest main, executed again, and still no dice. No change to docs other than that line I mentioned above.

shlomi-noach commented 1 year ago

Updated to reflect dashes, not underscores. Automation still does not change the vttablet docs.

shlomi-noach commented 1 year ago

Now that https://github.com/vitessio/vitess/pull/13964 is merged into main this works. I'm not sure why this wouldn't work when the branch was still unmerged?

ajm188 commented 1 year ago

I'm not sure why this wouldn't work when the branch was still unmerged?

It depends on what you used for COBRADOC_VERSION_PAIRS, and what state your local checkout was. If, for example, you did HEAD:18.0, and you had vitess checked out to some other branch that didn't have your changes, it wouldn't reflect in the docs. If instead you did <your-vitess-branch-here>:18.0 then it would.

shlomi-noach commented 1 year ago

It depends on what you used for COBRADOC_VERSION_PAIRS, and what state your local checkout was. If, for example, you did HEAD:18.0, and you had vitess checked out to some other branch that didn't have your changes, it wouldn't reflect in the docs.

I did HEAD:18.0 and my branch was checkout to the appropriate branch, and I made sure to run make build.

If instead you did :18.0 then it would.

I did that as well, and made sure to run make build

ajm188 commented 1 year ago

I did HEAD:18.0 and my branch was checkout to the appropriate branch, and I made sure to run make build.

I did that as well, and made sure to run make build

weiiiiird!

the only guess i have (and it doesn't super matter since it seems to be working for you now) is maybe you started in the website dir prior to this change? then if you had HEAD at your branch and main as the previous branch then every command after the first would have git checkout - send you back to main (and since vttablet is not first you would generate docs from main)

shlomi-noach commented 1 year ago

The website has been meanwhile frozen for v18 RC1 release. Therefore, this PR was updated to apply the changes to both 18.0 and the new 19.0 paths.