-
Notifications
You must be signed in to change notification settings - Fork 368
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
Reduce downtime with POWER_VM_DOWN_BEFORE_BACKUP=1 #166
Comments
@ctsde-markus, thank you for sharing your code. |
Nonono! Don't apologise for that. Indeed i didn't consider passthrough (i'm not using it at all). Making a new configuration variable for snapshotting and restarting a powered down VM before the backup on request is the better approach here. PS: If you already have working code, it's really simple to create a pull request. You just need an account on github (you obviously have that). Then you just fork the original project, checkout a local working copy of your fork on your device (git clone $URL) and create a branch for the changes (git branch $BRANCHNAME). Then you change the code, commit the changes locally (git add $CHANGED_FILE && git commit) and push them back to github (git push). An when you think it's good enough, you just make a pull request online on github. Those helped me:
|
Thank you for the kind words @ctsde-markus and the much needed help with git. I am going to follow your instructions and attempt to push that set of changes. Regards. |
Hello @ctsde-markus , I am hopping to draw from your knowledge for this problem that I am having with git. I created a branch added and committed the code without a problem. But the push is not working for me.
remote: Permission to lamw/ghettoVCB.git denied to cognoquest. fatal: unable to access 'https://github.com/lamw/ghettoVCB.git/': The requested URL returned error: 403 Obviously I am having permission issues. I see in documentation that I need the permission from the owner but I am a little lost on how to proceed? Thanks in advance for the help |
I don't really use git much, so my information is somewhat generic.. You can't push to "lamw", that belongs to another user.. If you haven't done so, you need to fork the project, and then you will have your own location to push changes.. Will be something like https://github.com/cognoquest/ghettoVCB.git/ i think you need to push there, then ask lamw to do a pull request. |
Oh I believe I missed the following step: 'forking the project' first, oops, yes indeed that makes sense to me now. @ffejtable , you may not use git much but you are allot better at this than me. Thanks |
By moving the power-on block up to right before the actual backup with vmkfstools takes place and taking a snapshot even for VMs with POWER_VM_DOWN_BEFORE_BACKUP=1 the downtime for clean (or "cold") backups can be greatly reduced. This makes sense for db servers and Windows domain controllers.
Lately i had a few unsuccessful (or painful) restores of Windows VMs (refusing to boot after restore) and so i increasingly tend to set POWER_VM_DOWN_BEFORE_BACKUP=1 for almost every Windows VM. Some of them are Servers, so short downtimes make a difference.
I made a pull request #165 to implement this. Please review and test it to make sure i didn't forget anything.
The text was updated successfully, but these errors were encountered: