Support update VM snapshot

Create new snapshot was disabled when VM already had an existing snapshot. VCloud only supports having one snapshot per VM, but you can still create new snapshot, which updates the existing one. So now VM supports `snapshot_create` even when it already has a snapshot. Signed-off-by: sasoc <[email protected]>
ManageIQ · Mar 5, 2018 · 6e2cb29 · 6e2cb29
1 parent e1fcf19
commit 6e2cb29
Show file tree

Hide file tree

Showing 2 changed files with 10 additions and 0 deletions.
diff --git a/app/models/manageiq/providers/vmware/cloud_manager/vm.rb b/app/models/manageiq/providers/vmware/cloud_manager/vm.rb
@@ -4,6 +4,7 @@ class ManageIQ::Providers::Vmware::CloudManager::Vm < ManageIQ::Providers::Cloud
   supports :snapshots
   supports :remove_all_snapshots
   supports_not :remove_snapshot
+  supports :snapshot_create
 
   def provider_object(connection = nil)
     connection ||= ext_management_system.connect

diff --git a/spec/models/manageiq/providers/vmware/cloud_manager_spec.rb b/spec/models/manageiq/providers/vmware/cloud_manager_spec.rb
@@ -120,6 +120,15 @@
 
         @ems.vm_create_snapshot(vm, snapshot_options)
       end
+
+      it 'supports snapshot create' do
+        expect(vm.supports_snapshot_create?).to be_truthy
+      end
+
+      it 'supports snapshot update' do
+        FactoryGirl.create(:snapshot, :vm_or_template_id => vm.id)
+        expect(vm.supports_snapshot_create?).to be_truthy
+      end
     end
 
     context ".vm_revert_to_snapshot" do