Skip to content

Replace SSH with Proxmox Upload API for bootdisk ISO uploads - #195

Open
EnigmaXV wants to merge 1 commit into
theforeman:masterfrom
ATIX-AG:iso-upload-api
Open

Replace SSH with Proxmox Upload API for bootdisk ISO uploads#195
EnigmaXV wants to merge 1 commit into
theforeman:masterfrom
ATIX-AG:iso-upload-api

Conversation

@EnigmaXV

@EnigmaXV EnigmaXV commented Aug 26, 2026

Copy link
Copy Markdown
  • Replaced the SSH based bootdisk ISO upload with the Proxmox Upload API.
  • Updated iso_upload and iso_attach to accept and use the ISO Upload Storage selected by the user.
  • Updated iso_delete to search all ISO-capable storages instead of using the first available ISO storage, which can be problematic when multiple ISO-capable storages exist. This is necessary because the selected storage_id is no longer available when the host is reloaded after provisioning.
  • Added unit tests to cover the changes.

This PR depends on => theforeman/foreman_fog_proxmox#543

@stejskalleos

Copy link
Copy Markdown
Collaborator

@sbernhard I don't have access to the Proxmox instance on my setup; could you review it?

@stejskalleos
stejskalleos self-requested a review August 27, 2026 06:54
Comment thread app/models/concerns/foreman_bootdisk/compute_resources/proxmox.rb
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