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

dbctl for dqlite backup and restore #1435

Merged
merged 6 commits into from
Aug 10, 2020
Merged

Conversation

ktsakalozos
Copy link
Member

@ktsakalozos ktsakalozos commented Jul 26, 2020

A dbctl command is added to take backups and restore the dqlite db.

$ microk8s dbctl --help
usage: microk8s dbctl [-h] [--debug] {restore,backup} ...

backup and restore the Kubernetes datastore

optional arguments:
  -h, --help        show this help message and exit
  --debug           print debug output

commands:
  {restore,backup}  backup and restore operations

@ktsakalozos
Copy link
Member Author

@sstubbs here is the PR with a command to handle dqlite backups.

@stephenstubbs
Copy link

Oh brilliant! thanks a lot. This is such a great feature to have.

@@ -96,6 +96,8 @@ apps:
command: microk8s-cilium.wrapper
juju:
command: microk8s-juju.wrapper
dbctl:
command: microk8s-dbctl.wrapper
Copy link
Contributor

Choose a reason for hiding this comment

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

If you have time, can you please consider which plugs would be needed for strict?

Copy link
Member Author

@ktsakalozos ktsakalozos Jul 29, 2020

Choose a reason for hiding this comment

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

I think we would need this https://snapcraft.io/docs/personal-files-interface to be able to write the backup file.

This is only by looking the code. I did not run any tests.

Copy link
Contributor

@joedborg joedborg left a comment

Choose a reason for hiding this comment

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

Looks good to me, thanks.

@ktsakalozos ktsakalozos merged commit 4cb316d into feature/ha-enable Aug 10, 2020
@ktsakalozos ktsakalozos deleted the feature/dbctl branch August 11, 2020 13:28
ktsakalozos added a commit that referenced this pull request Aug 17, 2020
ktsakalozos added a commit that referenced this pull request Aug 20, 2020
* First pass on microk8s enable

* Add pie chart to the grafana of prometheus operator

* Revert "Add pie chart to the grafana of prometheus operator"

This reverts commit 1c28dc6.

* Handle distribute an operation across nodes (#1297)

* Handle dqlite and cluster agent ports on HA cluster

* Fix distributed ops on single node cluster (#1320)

* Start dqlite and calico by default

* Do not try to distribute configuration when cluster agent is stopped

* Dqlite without REST interface

* Remove the ha-cluster subcommand from status (#1374)

* Remove the ha-cluster subcommand from status

* fix enabling of dns when running HA. (#1403)

* Fix LXC deployments, umount kubelet when removing microk8s (#1424)

* Stopping control plane on HA (#1406)

* Change ownership of files under the dqlite path (#1429)

* Change ownership of files under the dqlite path

* Address review comments

* Add message in joining (#1452)

* Handle not reachable nodes (#1450)

* Fixes #1448 (#1457)

* microk8s.leave single leader guard (#1462)

* Fix cilium in an ha cluster (#1453)

* dbctl for dqlite backup and restore (#1435)

Co-authored-by: balchua <balchua@yahoo.com>
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