Merge pull request #922 from agrare/fix_vm_set_description

Fix Vm#set_description error (cherry picked from commit b070f71)
ManageIQ · Oct 8, 2024 · 1f1ec1e · 1f1ec1e
1 parent 3bc40b8
commit 1f1ec1e
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 3 deletions.
diff --git a/app/models/manageiq/providers/vmware/infra_manager.rb b/app/models/manageiq/providers/vmware/infra_manager.rb
@@ -656,9 +656,9 @@ def vm_quick_stats(obj, options = {})
     end
     alias_method :host_quick_stats, :vm_quick_stats
 
-    def vm_set_description(vm, new_description, options = {})
+    def vm_set_description(vm, options = {})
       options[:spec] = VimHash.new("VirtualMachineConfigSpec") do |spec|
-        spec.annotation = new_description
+        spec.annotation = options.delete(:new_description)
       end
 
       vm_reconfigure(vm, options)

diff --git a/app/models/manageiq/providers/vmware/infra_manager/vm_or_template_shared/operations.rb b/app/models/manageiq/providers/vmware/infra_manager/vm_or_template_shared/operations.rb
@@ -40,7 +40,7 @@ def raw_rename(new_name)
   end
 
   def raw_set_description(new_description)
-    run_command_via_parent(:vm_set_description, new_description)
+    run_command_via_parent(:vm_set_description, :new_description => new_description)
   end
 
   def log_user_event(event_message)