Skip to content
This repository has been archived by the owner on Mar 8, 2023. It is now read-only.

Feature stepremoteupload #146

Conversation

remijouannet
Copy link
Contributor

Sorry this PR also include everything in #144

this PR include a step to upload an iso to vpshere for the builder vsphere-iso

@smitche1 smitche1 mentioned this pull request Oct 9, 2018
@steven-sheehy
Copy link

I was looking for this feature as well to have feature parity with vmware-iso. Any update when this could be reviewed? Thanks @remijouannet for your contribution.

@remijouannet
Copy link
Contributor Author

@steven-sheehy
Copy link

@remijouannet Thanks, I tried it and it does not allow you to use the old approach of only specifying iso_paths:

1 error(s) occurred:
* One of iso_url or iso_urls must be specified.

@remijouannet
Copy link
Contributor Author

remijouannet commented Oct 10, 2018

here is a working example, don't use iso_path

"iso_urls": [
"https://mirrors.ircam.fr/pub/CentOS/7/isos/x86_64/CentOS-7-x86_64-Minimal-1804.iso"
],
"iso_checksum_type" : "sha256",
"iso_checksum" : "714acc0aefb32b7d51b515e25546835e55a90da9fb00417fbee2d03a62801efd",
"iso_target_path" : "/tmp/CentOS-7-x86_64-Minimal-1804.iso",

@steven-sheehy
Copy link

Yes, I also have the iso_urls approach working. My point was I think the maintainers will probably want to keep the old approach as well. iso_paths is also still mentioned in README and in examples as supported.

@remijouannet
Copy link
Contributor Author

Yeah but iso_paths is use for mouting a ISO that is already upload on the esxi, my patch doesn't remove that you juste have too choose between the two methods

Anyway i will wait for the review to see if this behavior is ok or not

@segator
Copy link

segator commented Oct 10, 2018

it works nicely!!!

are you going to integrate it? I'm planning to use this plugin for my development automatic proceses with jenkins.

@mkuzmin
Copy link
Contributor

mkuzmin commented Oct 16, 2018

I'd happy to merge this! please rebase the branch onto master

@remijouannet
Copy link
Contributor Author

Will work on that ☺️

@remijouannet
Copy link
Contributor Author

new PR #164

@swerveshot
Copy link

Great work on this issue guys. Now with this feature added combining Packer with Terraform almost brings a Vagrant-like experience to vSphere. I'm amazed that it takes a community project to achieve this and Hashicorp still requires you use a nasty SSH backdoor.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Development

Successfully merging this pull request may close these issues.

5 participants