Do not upload an existing iso_url about packer-plugin-proxmox HOT 5 CLOSED

This is probably a duplicate of this issue: #49

from packer-plugin-proxmox.

The change for proxmox-api-go got merged already, so I created a draft here: https://github.com/sebastian-de/packer-plugin-proxmox/commit/9761f385397dd974980d304643d2780eb2d695ab
It works and has the added benefit that ISOs get downloaded to the PVE node directly. Currently they get downloaded to the host packer is running on and then uploaded to the node from there.

I have to add test and clean it up a little.

Since #130 is near the finish line (hopefully!), I want to merge my Cloud-Init changes first. After that I'll finish the work here.

from packer-plugin-proxmox.

I absolutely agree that this should be improved.

Meanwhile, as a workaround you could check the existence of the ISO file via the Proxmox API. See here.

This may or may not be useful, depending on how you run Packer. Personally I run it from an Ansible playbook, so I just created a task before running Packer, which checks if the ISO is already present on the host. Depending on the result I am either using iso-file when its present or iso-url and iso-checksum when it needs to get downloaded.

Example:

pvesh get /nodes/proxmox/storage/local-iso/content | grep 'debian-11.5.0-amd64-netinst.iso'

from packer-plugin-proxmox.

I absolutely agree that this should be improved.

Meanwhile, as a workaround you could check the existence of the ISO file via the Proxmox API. See here.

It's unfortunate that Packer doesn't have a preprocessor function that would act in a similar fashion.

from packer-plugin-proxmox.

After having a look at this upstream bug report, I guess this should be possible.
Two steps are required:

• add support for download-url to proxmox-api-go
• rewrite step_upload_iso.go to use download-url

The PVE would then skip the download if the file with the correct checksum is already present. If I find the time, I'll have a look at this.

from packer-plugin-proxmox.