search

theforeman / foreman_fog_proxmox

Foreman plugin to add Proxmox compute resource based on fog-proxmox gem
GNU General Public License v3.0
102 stars 30 forks source link

Editing the Proxmox Host when Powered on fails with error #251

Closed Manisha15 closed 1 year ago

Manisha15 commented 1 year ago

When editing the Proxmox Host to add CPU, Memory or OS options while the host is powered on fails with the following error:

Unable to save Failed to update a compute Proxmox (Proxmox) instance ethel-archut.deploy3.dev.atix: Expected([200]) <=> Actual(500 InternalServerError) excon.error.response :body => "{\"data\":null}" :cookies => [ ] :headers => { "Cache-Control" => "max-age=0" "Connection" => "close" "Content-Length" => "13" "Content-Type" => "application/json;charset=UTF-8" "Date" => "Wed, 22 Mar 2023 13:12:39 GMT" "Expires" => "Wed, 22 Mar 2023 13:12:39 GMT" "Pragma" => "no-cache" "Server" => "pve-api-daemon/3.0" } :host => "192.168.131.20" :local_address => "192.168.171.100" :local_port => 42140 :path => "/api2/json/nodes/proxmox-noris/qemu/104/config" :port => 443 :reason_phrase => "balloon value too large (must be smaller than assigned memory)" :remote_ip => "192.168.152.102" :status => 500 :status_line => "HTTP/1.1 500 balloon value too large (must be smaller than assigned memory)\r\n"

Expected behavior Either disallow user to add VM options when host is powered on or the error needs to be fixed

Additional context I am using Foreman3.3, foreman_fog_proxmox 0.14.2 with fog-proxmox 0.15.0

Manisha15 commented 1 year ago

If host is powered on, it automatically adds the minimum memory (balloon) value to the size of memory. But it converts the balloon memory to bytes and store it. For eg., if memory is 4096 (MB) for a host and minimum memory is not set, now, if the host is powered on it sets the minimum memory in Memory option (Advanced options ) to 4294967296 which throws the above error.