Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Fix/template downloads new secstorvm #909

Merged
merged 3 commits into from
Nov 3, 2020

Conversation

ddegoede
Copy link
Contributor

@ddegoede ddegoede commented Oct 5, 2020

Fix for #889, I now correctly recognises existing templates, and removes records in the DB of previously existing and or failed downloads when attemping to download again.

Before:

#
#Mon Oct 05 07:34:42 GMT 2020
filename=6c545b64-5572-3546-93d7-275f534f95e8.qcow2
id=201
qcow2.size=2037579776
public=true
uniquename=centos
qcow2.virtualsize=8589934592
virtualsize=8589934592
checksum=f05daf6ed4c97c387cd6fdac6386d671
hvm=false
description=centos
qcow2=true
qcow2.filename=6c545b64-5572-3546-93d7-275f534f95e8.qcow2
size=2037579776

After:

#
#Mon Oct 05 08:47:59 GMT 2020
filename=e1a6da25-c7db-3e7a-b96f-49dc2c20dff0.qcow2
id=201
qcow2.size=2037579776
public=true
uniquename=201-2-f217520e-6dd3-3823-a4c2-56359ceccc0d
qcow2.virtualsize=8589934592
virtualsize=8589934592
checksum=f05daf6ed4c97c387cd6fdac6386d671
hvm=false
description=centos
qcow2=true
qcow2.filename=e1a6da25-c7db-3e7a-b96f-49dc2c20dff0.qcow2
size=2037579776

@ddegoede ddegoede force-pushed the fix/template-downloads-new-secstorvm branch from bf88d11 to 9dcdb55 Compare October 5, 2020 09:16
@ddegoede
Copy link
Contributor Author

ddegoede commented Nov 2, 2020

go build

@ddegoede
Copy link
Contributor Author

ddegoede commented Nov 2, 2020

go build

@sanderv32 sanderv32 merged commit 7f1c2f1 into master Nov 3, 2020
@sanderv32 sanderv32 deleted the fix/template-downloads-new-secstorvm branch November 3, 2020 09:16
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants