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

Change shutdown snapshot backup logic #165

Closed
wants to merge 6 commits into from
Closed

Change shutdown snapshot backup logic #165

wants to merge 6 commits into from

Conversation

ctsde-markus
Copy link

@ctsde-markus ctsde-markus commented Feb 8, 2019

I moved the block powering back on a VM with POWER_VM_DOWN_BEFORE_BACKUP=1 right above the actual backup takes place to shorten the downtime. Also made sure that such VMs are snapshotted too (otherwise it wouldn't work).

While i was here i documented the -e parameter and corrected some indentation (most of it was auto-corrected by Atom editor).

I ran a few tests and they worked for me so far. I will use this version on an internal production server starting 2/12/2019 and report back after the first backup runs were made.

My intention to make this change are database servers and Windows domain controllers to ensure a clean ("cold") backup in a guaranteed clean state while keeping downtimes still short.

@ctsde-markus
Copy link
Author

I think @cognoquest has a better approach for this feature with his pull request #167

I'll review his pull request in the next days.

@ctsde-markus
Copy link
Author

Closing this as i moved on to Proxmox

@ctsde-markus ctsde-markus deleted the change-shutdown-snapshot-backup-logic branch January 15, 2021 23:34
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.

1 participant