-
Notifications
You must be signed in to change notification settings - Fork 10
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
Add posibility turn of mysql backup for specific nodes, #1
Conversation
Can one of the admins verify this patch? |
backupninja/map.jinja
Outdated
@@ -1,3 +1,4 @@ | |||
{%- from "linux/map.jinja" import system with context %} |
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.
This should not be needed, why did you add it?
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.
I have parametrized meta/backupninja.ymk for contrail formula(check contrail formula PR) so when we load it service_grains jinja should be able to render it.
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.
Yes, but grains should be used instead.
…ating remote directories, add import vars revquired for loading cassandra service_grains.
Why are you not using xtrabackup for galera/mysql backups? |
Provide access to through https://docs.google.com/a/mirantis.com/document/d/1bi2m3OZ6v2ax7Xn_A8TpmAH-0S2eDSN5aDZYU-0I9I0/edit?usp=sharing to have glance |
changes in backupninja/client.sls are really useful, because I have the same issue. I using xtrabackup for galera/mysql backups and for moving data to backup server I would like to use backupninja. I disabled backupninja for mysql in reclass model, but mysql handler was still generated. |
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, an alternative to xtrabackup
fix rsync creating remote directories, add import vars revquired for loading cassandra service_grains.
List of related PRs coming later