filter services and resources by capabilities on volume create:

- Basic: choose an existing service
- Advanced: choose a name for a new service and choose existing resources to attach to it

app/models/manageiq/providers/autosde/inventory/collector/storage_manager.rb

@@ -42,4 +42,8 @@ def wwpn_candidates
   def host_initiator_groups
     @host_initiator_groups ||= @manager.autosde_client.HostClusterApi.host_clusters_get
   end
+
+  def capability_values
+    @capability_values ||= @manager.autosde_client.ServiceAbstractCapabilityValueApi.service_abstract_capability_values_get
+  end
 end

app/models/manageiq/providers/autosde/inventory/collector/target_collection.rb

@@ -56,6 +56,10 @@ def host_initiator_groups
     []
   end
 
+  def capability_values
+    []
+  end
+
   private
 
   def parse_targets!

app/models/manageiq/providers/autosde/inventory/parser/storage_manager.rb

@@ -11,6 +11,7 @@ class ManageIQ::Providers::Autosde::Inventory::Parser::StorageManager < ManageIQ
   #     :ems_ref => "s2", :name => "Storage 3", :ems_id=>persister.manager.id
   # )#
   def parse
+    ext_management_system
     physical_storage_families
     physical_storages
     storage_resources
@@ -23,6 +24,13 @@ def parse
     wwpn_candidates
   end
 
+  def ext_management_system
+    persister.ext_management_system.build(
+      :guid         => persister.manager.guid,
+      :capabilities => collector.capability_values
+    )
+  end
+
   def physical_storage_families
     collector.physical_storage_families.each do |storage_family|
       persister.physical_storage_families.build(
@@ -116,10 +124,11 @@ def volume_mappings
   def storage_services
     collector.storage_services.each do |service|
       persister.storage_services.build(
-        :name        => service.name,
-        :description => service.description,
-        :version     => service.version,
-        :ems_ref     => service.uuid
+        :name         => service.name,
+        :description  => service.description,
+        :version      => service.version,
+        :ems_ref      => service.uuid,
+        :capabilities => parse_capabilities(service.capability_values_json)
       )
     end
   end

app/models/manageiq/providers/autosde/inventory/persister/storage_manager.rb

@@ -16,5 +16,6 @@ def initialize_inventory_collections
     add_collection(storage, :storage_resources)
     add_collection(storage, :storage_services)
     add_collection(storage, :cloud_volumes)
+    add_collection(storage, :ext_management_system)
   end
 end

app/models/manageiq/providers/autosde/storage_manager/cloud_volume.rb

@@ -10,12 +10,24 @@ class ManageIQ::Providers::Autosde::StorageManager::CloudVolume < ::CloudVolume
 
   def self.raw_create_volume(ext_management_system, options = {})
     # @type [StorageService]
-    vol_to_create = ext_management_system.autosde_client.VolumeCreate(
-      :service => ext_management_system.storage_services.find(options["storage_service_id"]).ems_ref,
+    creation_hash = {
+      :service => "",
       :name    => options["name"],
       :size    => options["size"],
       :count   => options["count"]
-    )
+    }
+
+    if options['mode'] == 'Basic'
+      creation_hash[:service] = ext_management_system.storage_services.find(options["storage_service_id"]).ems_ref
+    else
+      creation_hash[:service_name] = options["new_service_name"]
+      creation_hash[:resources] = []
+      options["storage_resource_id"].each do |resource|
+        creation_hash[:resources].push(ext_management_system.storage_resources.find(resource['value']).ems_ref)
+      end
+    end
+
+    vol_to_create = ext_management_system.autosde_client.VolumeCreate(creation_hash)
     task_id = ext_management_system.autosde_client.VolumeApi.volumes_post(vol_to_create).task_id
 
     options = {
@@ -100,20 +112,22 @@ def params_for_update
   end
 
   def self.params_for_create(provider)
-    services = provider.storage_services.map { |service| {:value => service.id.to_s, :label => service.name} }
+    capabilities = []
+    provider.capabilities.map do |capability|
+      capabilities.push({:label => "#{capability['abstract_capability']}: #{capability['value']}", :value => capability['uuid']})
+    end
 
     {
       :fields => [
         {
-          :component    => "select",
-          :name         => "storage_service_id",
-          :id           => "storage_service_id",
-          :label        => _("Storage Pool"),
-          :isRequired   => true,
-          :validate     => [{:type => "required"}],
-          :options      => services,
-          :includeEmpty => true,
-          :isDisabled   => false
+          :component  => "select",
+          :name       => "required_capabilities",
+          :id         => "required_capabilities",
+          :label      => _("Required Capabilities"),
+          :options    => capabilities,
+          :isRequired => true,
+          :isMulti    => true,
+          :validate   => [{:type => "required"}]
         },
         {
           :component  => "text-field",

spec/models/manageiq/providers/autosde/storage_manager/refresher_spec.rb

@@ -212,7 +212,7 @@ def run_targeted_refresh(targets = [])
     def assert_ems
       expect(ems.physical_storages.count).to(eq(1))
       expect(ems.physical_storage_families.count).to(eq(2))
-      expect(ems.storage_resources.count).to(eq(8))
+      expect(ems.storage_resources.count).to(eq(9))
       expect(ems.storage_services.count).to(eq(7))
       expect(ems.cloud_volumes.count).to(eq(13))
       expect(ems.wwpn_candidates.count).to(eq(2))
@@ -251,15 +251,15 @@ def assert_specific_physical_storage_family
     end
 
     def assert_specific_storage_resource
-      storage_resource = ems.storage_resources.find_by(:ems_ref => "e6833c27-374b-4a4a-8d76-455cfe5f4270")
+      storage_resource = ems.storage_resources.find_by(:ems_ref => "41e4e6fb-d670-4db9-8325-464c138dee6f")
       expect(storage_resource).to(have_attributes(
-                                    :name             => "9.151.159.178:ilyak_test_pool",
-                                    :ems_ref          => "e6833c27-374b-4a4a-8d76-455cfe5f4270",
-                                    :logical_free     => 601_295_421_440,
-                                    :logical_total    => 0,
-                                    :physical_storage => ems.physical_storages.find_by(:ems_ref => "980f3ceb-c599-49c4-9db3-fdc793cb8666"),
+                                    :name             => "test_or_30:test_pool",
+                                    :ems_ref          => "41e4e6fb-d670-4db9-8325-464c138dee6f",
+                                    :logical_free     => 472_446_402_560,
+                                    :logical_total    => 515_396_075_520,
+                                    :physical_storage => ems.physical_storages.find_by(:ems_ref => "9b1cedc0-b476-47f1-8f25-7a7da2b7d91c"),
                                     :type             => "ManageIQ::Providers::Autosde::StorageManager::StorageResource",
-                                    :capabilities     => [{"name" => "compression", "uuid" => "29ac2ba7-1e55-411e-9828-472c935bb882", "value" => "True"}, {"name" => "thin_provision", "uuid" => "63a2dfbe-bb1b-408c-8e94-36c9d7791df2", "value" => "True"}]
+                                    :capabilities     => [{"name" => "compression", "uuid" => "45e0cf41-1842-45e8-b83b-17de54be406d", "value" => "True"}, {"name" => "thin_provision", "uuid" => "2fddfc81-2561-40d8-bd84-cec6ec26d551", "value" => "True"}]
                                   ))
     end