Skip to content

added possibility of creating clone without specifying resource pool.#33

Merged
jswager merged 3 commits intojenkinsci:masterfrom
Ichai5:master
Nov 20, 2015
Merged

added possibility of creating clone without specifying resource pool.#33
jswager merged 3 commits intojenkinsci:masterfrom
Ichai5:master

Conversation

@Ichai5
Copy link
Copy Markdown
Contributor

@Ichai5 Ichai5 commented Jun 14, 2015

Hi vsphere-cloud-plugin team,

I’m currently need that the cloned VM will be created on the same host as the source.
When I browse the vmware documentation on VirtualMachineRelocateSpec I see that if resource pool is not specified and host also, the current host is used.
So If I have understood, enable cloning the VM without specifying the Resource Pool will clone it to the same host.
For a clone operation from a template to a virtual machine, this pool is required according to the vmWare Doc.
So we can enable cloning a VM without specifying a resource pool when the source is not a template.
Please find the changeset that let me get this feature.
Are these changes acceptable for you ?
If yes can you add them to the next plugin version.

Thanks in advance,

Ichai

Comment thread .project Outdated
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This file should not be in this commit.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

You are right,
How can I remove from the pull request ?

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Make a commit to Ichai5:master that deletes the file. The PR will be automatically updated when the change is pushed to GitHub Ichai5:master.

@daniel-beck
Copy link
Copy Markdown
Member

Please note that I'm not a developer of this plugin, so my review comments are just offered as advice, nothing more.

@jswager
Copy link
Copy Markdown
Member

jswager commented Jun 15, 2015

@daniel-beck Maybe not a maintainer yet - want to be? Pretty easy to fix that. Your ongoing input has been valuable.

@daniel-beck
Copy link
Copy Markdown
Member

@jswager Thanks, but my plate is full, and I don't use vSphere anyway, so I doubt I'd make a good maintainer. I'm still available if you have specific questions though.

@jswager
Copy link
Copy Markdown
Member

jswager commented Jun 15, 2015

@daniel-beck Reviewing pull requests is always a great way to help! Thanks!

2. restore doCheckResourcePool to check that resource pool is specified if source is template.
jswager added a commit that referenced this pull request Nov 20, 2015
added possibility of creating clone without specifying resource pool.
@jswager jswager merged commit 133b11b into jenkinsci:master Nov 20, 2015
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.

3 participants