Skip to content

VSphere folder parameter#61

Merged
jswager merged 1 commit intojenkinsci:masterfrom
maleix:master
Dec 6, 2016
Merged

VSphere folder parameter#61
jswager merged 1 commit intojenkinsci:masterfrom
maleix:master

Conversation

@maleix
Copy link
Copy Markdown
Contributor

@maleix maleix commented Nov 16, 2016

A new folder parameter has been added for templates and clone/deploy
actions.

This parameter creates the new VM in the specified folder of vSphere
(which can be a path of folders or a unique name folder).

If left blank or folder not available, it will create the VM in the same
folder as the original VM.

It is specially interesting for organizing and separating templates from
autodeployed slaves.
For example, we have daily backups and we want separate folders for
templates and autodeployed dynamic slaves, so we can specify to backup
only the folders that contain the templates and omit folders that
contain the dynamic slaves.

A new folder parameter has been added for templates and clone/deploy
actions.

This parameter creates the new VM in the specified folder of vSphere
(which can be a path of folders or a unique name folder).

If left blank or folder not available, it will create the VM in the same
folder as the original VM.

It is specially interesting for organizing and separating templates from
autodeployed slaves.
For example, we have daily backups and we want separate folders for
templates and autodeployed dynamic slaves, so we can specify to backup
only the folders that contain the templates and omit folders that
contain the dynamic slaves.
@segator
Copy link
Copy Markdown

segator commented Nov 19, 2016

I just give a try to your PR and works perfect!

@jswager jswager merged commit 8dcf749 into jenkinsci:master Dec 6, 2016
@pjdarton
Copy link
Copy Markdown
Member

pjdarton commented Jun 8, 2017

@maleix @jswager src/main/resources/org/jenkinsci/plugins/vsphere/builders/Clone/config.jelly looks incorrect.
The <f:entry...> with attribute title="${%Folder}" has field="datastore", but the Clone.java's folder field is actually called "folder".
(looks like this was a copy/paste from the <f:entry...> title="${%Datastore}" immediately above it)

Also, src/main/resources/org/jenkinsci/plugins/vsphere/builders/Deploy/config.jelly specifies field="Folder" instead of "folder" (that'll probably work, but it's not pretty).

(Apologies for the delay in noticing this; it's been 6 months since I last followed this plugin but I'm catching up now)

@pjdarton
Copy link
Copy Markdown
Member

pjdarton commented Jun 8, 2017

I've raised #72 as a prospective fix for these issues.

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.

4 participants