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

Documentation for MySQL Shell backup engine #1846

Merged
merged 4 commits into from
Sep 24, 2024

Conversation

rvrangel
Copy link
Contributor

This PR adds the documentation needed for the new MySQL Shell Backup engine that is being added in vitessio/vitess#16295

Signed-off-by: Renan Rangel <rrangel@slack-corp.com>
Copy link

netlify bot commented Sep 13, 2024

Deploy Preview for vitess ready!

Name Link
🔨 Latest commit 2417665
🔍 Latest deploy log https://app.netlify.com/sites/vitess/deploys/66ed74a3999038000881161e
😎 Deploy Preview https://deploy-preview-1846--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.

…estore/creating-a-backup.md

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

@shlomi-noach @deepthi is there anything missing you would like me to cover here?

@shlomi-noach
Copy link
Contributor

@rvrangel this looks good to me. Only did a brief review a couple days ago as the PR is still Draft - sounds like it's ready for a review now?

@rvrangel rvrangel marked this pull request as ready for review September 18, 2024 09:24
@rvrangel
Copy link
Contributor Author

ah yes, marked it as ready for review now

Copy link
Contributor

@shlomi-noach shlomi-noach left a comment

Choose a reason for hiding this comment

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

Just one clarification, otherwise looks good. I'm not marking as "approved" yet, because our flow is to first merge the code PR, then the docs PR, so I just don't want to accidentally merge this PR ahead of time.

* you can control how many threads and other options with this flag. See the [MySQL docs](https://dev.mysql.com/doc/mysql-shell/8.0/en/mysql-shell-utilities-dump-instance-schema.html) for other
options
* `--mysql-shell-load-flags="{\"updateGtidSet\": \"replace\", \"progressFile\": \"\", \"skipBinlog\": true}"`
* `updateGtidSet` must be set to `true` so MySQL Shell updates the executed GTID at the
Copy link
Contributor

Choose a reason for hiding this comment

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

A bit confused about this, because the preceding line says "{\"updateGtidSet\": \"replace\", ; "replace" is not said true and is not even a boolean.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

that's a good catch, thanks! I have updated it :)

rvrangel and others added 2 commits September 18, 2024 12:45
Signed-off-by: Renan Rangel <rvrangel@users.noreply.github.com>
Signed-off-by: Renan Rangel <rrangel@slack-corp.com>
@frouioui frouioui merged commit 4652f55 into vitessio:prod Sep 24, 2024
5 checks passed
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.

3 participants