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

proxmox_kvm: fixes parameters for linked clones. #58846

Open
wants to merge 1 commit into
base: devel
from

Conversation

Projects
None yet
2 participants
@EnigmaCurry
Copy link

commented Jul 9, 2019

SUMMARY

proxmox_kvm has a default value for format - but format is an invalid parameter for the case that full is false. This removes the format parameter in this case.

ISSUE TYPE
  • Bugfix Pull Request
COMPONENT NAME

proxmox_kvm

ADDITIONAL INFORMATION

Before when setting full=0 this error would occur:

fatal: [nuc1]: FAILED! => {"changed": false, "msg": "Unable to clone vm testguy from vmid 9000=500 Internal Server Error: {\"data\":null}"}

This is because the format parameter is being sent. Format is only valid for a full clone, but since we request a linked clone (full=0) we do not need to specify format.

@EnigmaCurry EnigmaCurry force-pushed the EnigmaCurry:proxmox_kvm-linked-clone branch from 822f075 to f0f299a Jul 9, 2019

@ansibot

This comment has been minimized.

Copy link
Contributor

commented Jul 9, 2019

EnigmaCurry added a commit to PlenusPyramis/proxmox-ansible that referenced this pull request Jul 9, 2019

@ansibot ansibot added the stale_ci label Jul 17, 2019

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.