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

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

Merged
merged 8 commits into from
Oct 10, 2023

Conversation

shlomi-noach
Copy link
Contributor

@shlomi-noach shlomi-noach commented Sep 21, 2023

Documenting the changes in vitessio/vitess#13964, and specifically these vttablet flags:

  • --restore-to-timestamp
  • --restore-to-position

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

Only merge once vitessio/vitess#13964 is merged

Signed-off-by: Shlomi Noach <2607934+shlomi-noach@users.noreply.github.com>
@netlify
Copy link

netlify bot commented Sep 21, 2023

Deploy Preview for vitess ready!

Name Link
🔨 Latest commit 7c0fd70
🔍 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.

Copy link
Contributor

@ajm188 ajm188 left a comment

Choose a reason for hiding this comment

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

vttablet now also has auto-generated docs](#1580), so you should also update those with (assuming you have your changes checked out in vitessio/vitess): COBRADOCS_VERSION_PAIRS="HEAD:18.0" VITESS_DIR=/where/does/shlomi/keep/vitess make vttablet-docs (alternatively, make generated-docs to do 'em all instead of just the one binary; same envvars apply)

@shlomi-noach
Copy link
Contributor Author

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
Copy link
Contributor

ajm188 commented Sep 26, 2023

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
Copy link
Contributor

ajm188 commented Sep 26, 2023

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
Copy link
Contributor Author

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
Copy link
Contributor

ajm188 commented Sep 26, 2023

ahh, your PR is based 2 weeks ago, before the vttablet docgen stuff got added. if you merge to at least vitessio/vitess@98c754a you should get it working

@shlomi-noach
Copy link
Contributor Author

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

Signed-off-by: Shlomi Noach <2607934+shlomi-noach@users.noreply.github.com>
@shlomi-noach
Copy link
Contributor Author

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

Signed-off-by: Shlomi Noach <2607934+shlomi-noach@users.noreply.github.com>
@shlomi-noach
Copy link
Contributor Author

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

Signed-off-by: Shlomi Noach <2607934+shlomi-noach@users.noreply.github.com>
@ajm188
Copy link
Contributor

ajm188 commented Oct 2, 2023

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
Copy link
Contributor Author

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

@shlomi-noach shlomi-noach changed the title Docs: vttablet PITR via --restore_to_timestamp and --restore_to_position Docs: vttablet PITR via --restore-to-timestamp and --restore-to-position Oct 2, 2023
@ajm188
Copy link
Contributor

ajm188 commented Oct 2, 2023

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)

Signed-off-by: Shlomi Noach <2607934+shlomi-noach@users.noreply.github.com>
Signed-off-by: Shlomi Noach <2607934+shlomi-noach@users.noreply.github.com>
Signed-off-by: Shlomi Noach <2607934+shlomi-noach@users.noreply.github.com>
@shlomi-noach
Copy link
Contributor Author

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.

Signed-off-by: Shlomi Noach <2607934+shlomi-noach@users.noreply.github.com>
@shlomi-noach shlomi-noach merged commit cc4a10c into prod Oct 10, 2023
@shlomi-noach shlomi-noach deleted the docs-pitr-tablet-init branch October 10, 2023 10:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants