oneview-sdk 4.5.1 → 5.0.0

data/lib/oneview-sdk/resource/api200/connection_template.rb

@@ -45,7 +45,8 @@ module OneviewSDK
       # @return [OneviewSDK::ConnectionTemplate] Connection template
       def self.get_default(client)
         response = client.rest_get(BASE_URI + '/defaultConnectionTemplate')
-        OneviewSDK::ConnectionTemplate.new(client, client.response_handler(response))
+        variant = name.split('::').at(-2)
+        OneviewSDK.resource_named('ConnectionTemplate', client.api_version, variant).new(client, client.response_handler(response))
       end
     end
   end

data/lib/oneview-sdk/resource/api200/enclosure.rb

@@ -75,14 +75,10 @@ module OneviewSDK
         ensure_client && ensure_uri
         cur_state = self.class.find_by(@client, uri: @data['uri']).first
         unless cur_state[:name] == @data['name']
-          temp_data = [{ op: 'replace', path: '/name', value: @data['name'] }]
-          response = @client.rest_patch(@data['uri'], { 'body' => temp_data }, @api_version)
-          @client.response_handler(response)
+          patch('replace', '/name', @data['name'])
         end
         unless cur_state[:rackName] == @data['rackName']
-          temp_data = [{ op: 'replace', path: '/rackName', value: @data['rackName'] }]
-          response = @client.rest_patch(@data['uri'], { 'body' => temp_data }, @api_version)
-          @client.response_handler(response)
+          patch('replace', '/rackName', @data['rackName'])
         end
         self
       end
@@ -117,7 +113,7 @@ module OneviewSDK
       # @return [String] Script content
       def script
         ensure_client && ensure_uri
-        response = @client.rest_get(@data['uri'] + '/script', @api_version)
+        response = @client.rest_get(@data['uri'] + '/script', {}, @api_version)
         @client.response_handler(response)
       end
 
@@ -125,7 +121,7 @@ module OneviewSDK
       # @return [Hash] The enclosure envirnomental configuration
       def environmental_configuration
         ensure_client && ensure_uri
-        response = @client.rest_get(@data['uri'] + '/environmentalConfiguration', @api_version)
+        response = @client.rest_get(@data['uri'] + '/environmentalConfiguration', {}, @api_version)
         @client.response_handler(response)
       end
 
@@ -162,7 +158,7 @@ module OneviewSDK
           uri += '&'
         end
         uri.chop! # Get rid of trailing '&' or '?'
-        response = @client.rest_get(uri, @api_version)
+        response = @client.rest_get(uri, {}, @api_version)
         @client.response_handler(response)
       end
 

data/lib/oneview-sdk/resource/api200/enclosure_group.rb

@@ -26,6 +26,7 @@ module OneviewSDK
         # Default values:
         @data['type'] ||= 'EnclosureGroupV200'
         @data['interconnectBayMappingCount'] ||= 8
+        @data['stackingMode'] ||= 'Enclosure'
         create_interconnect_bay_mapping unless @data['interconnectBayMappings']
       end
 
@@ -33,7 +34,7 @@ module OneviewSDK
       # @return [String] The script for this enclosure group
       def get_script
         ensure_client && ensure_uri
-        response = @client.rest_get(@data['uri'] + '/script', @api_version)
+        response = @client.rest_get(@data['uri'] + '/script', {}, @api_version)
         @client.response_handler(response)
       end
 

data/lib/oneview-sdk/resource/api200/ethernet_network.rb

@@ -45,14 +45,14 @@ module OneviewSDK
       # Gets the associated profiles
       def get_associated_profiles
         ensure_client && ensure_uri
-        response = @client.rest_get("#{@data['uri']}/associatedProfiles", @api_version)
+        response = @client.rest_get("#{@data['uri']}/associatedProfiles", {}, @api_version)
         response.body
       end
 
       # Gets the associated uplink groups
       def get_associated_uplink_groups
         ensure_client && ensure_uri
-        response = @client.rest_get("#{@data['uri']}/associatedUplinkGroups", @api_version)
+        response = @client.rest_get("#{@data['uri']}/associatedUplinkGroups", {}, @api_version)
         response.body
       end
     end

data/lib/oneview-sdk/resource/api200/firmware_bundle.rb

@@ -23,7 +23,8 @@ module OneviewSDK
       def self.add(client, file_path, timeout = OneviewSDK::Rest::READ_TIMEOUT)
         options = { 'header' => { 'uploadfilename' => File.basename(file_path) } }
         result = client.upload_file(file_path, BASE_URI, options, timeout)
-        OneviewSDK::FirmwareDriver.new(client, result)
+        variant = name.split('::').at(-2)
+        OneviewSDK.resource_named('FirmwareDriver', client.api_version, variant).new(client, result)
       end
     end
   end

data/lib/oneview-sdk/resource/api200/id_pool.rb

@@ -13,7 +13,7 @@ require_relative 'resource'
 
 module OneviewSDK
   module API200
-    # Id pool resource implementation
+    # Id Pool resource implementation
     class IDPool < Resource
       BASE_URI = '/rest/id-pools'.freeze
 
@@ -32,13 +32,12 @@ module OneviewSDK
       # Gets a pool along with the list of ranges present in it.
       # @param [String] pool_type The type of the pool. Values: (ipv4, vmac, vsn, vwwn)
       # @raise [OneviewSDK::IncompleteResource] if the client
-      # @return [OneviewSDK::API200::IDPool] The response with IDs list, count and if this is a valid allocator
+      # @return [OneviewSDK::IDPool] The response with IDs list, count and if this is a valid allocator
       def get_pool(pool_type)
         ensure_client
         response = @client.rest_get("#{BASE_URI}/#{pool_type}")
         body = @client.response_handler(response)
         set_all(body)
-        self
       end
 
       # Allocates one or more IDs from a according the amount informed
@@ -66,6 +65,7 @@ module OneviewSDK
       # @return [Hash] The hash with eTag and list of ID's
       def check_range_availability(pool_type, *id_list)
         ensure_client
+        return {} if id_list.flatten.empty?
         response = @client.rest_get("#{BASE_URI}/#{pool_type}/checkrangeavailability?idList=#{id_list.flatten.join('&idList=')}")
         @client.response_handler(response)
       end
@@ -76,6 +76,7 @@ module OneviewSDK
       # @raise [OneviewSDK::IncompleteResource] if the client is not set
       # @return [Hash] The list of IDs collected
       def collect_ids(pool_type, *id_list)
+        raise IncompleteResource, 'The list of IDs informed is empty.' if id_list.flatten.empty?
         ensure_client
         response = @client.rest_put("#{BASE_URI}/#{pool_type}/collector", 'body' => { 'idList' => id_list.flatten })
         @client.response_handler(response)
@@ -99,6 +100,7 @@ module OneviewSDK
       # @return [Boolean] Returns true if is valid
       def validate_id_list(pool_type, *id_list)
         ensure_client
+        return {} if id_list.flatten.empty?
         response = @client.rest_put("#{BASE_URI}/#{pool_type}/validate", 'body' => { 'idList' => id_list.flatten })
         body = @client.response_handler(response)
         body['valid']

data/lib/oneview-sdk/resource/api200/logical_enclosure.rb

@@ -56,7 +56,7 @@ module OneviewSDK
       # @return [String] script
       def get_script
         ensure_client && ensure_uri
-        response = @client.rest_get("#{@data['uri']}/script", @api_version)
+        response = @client.rest_get("#{@data['uri']}/script", {}, @api_version)
         response.body
       end
 

data/lib/oneview-sdk/resource/api200/logical_interconnect.rb

@@ -78,13 +78,14 @@ module OneviewSDK
         ensure_client && ensure_uri
         results = OneviewSDK::Resource.find_by(@client, {}, @data['uri'] + '/internalVlans')
         internal_networks = []
+        variant = self.class.name.split('::').at(-2)
         results.each do |vlan|
           net = if vlan['generalNetworkUri'].include? 'ethernet-network'
-                  OneviewSDK::EthernetNetwork.new(@client, uri: vlan['generalNetworkUri'])
+                  OneviewSDK.resource_named('EthernetNetwork', @client.api_version, variant).new(@client, uri: vlan['generalNetworkUri'])
                 elsif vlan['generalNetworkUri'].include? 'fc-network'
-                  OneviewSDK::FCNetwork.new(@client, uri: vlan['generalNetworkUri'])
+                  OneviewSDK.resource_named('FCNetwork', @client.api_version, variant).new(@client, uri: vlan['generalNetworkUri'])
                 else
-                  OneviewSDK::FCoENetwork.new(@client, uri: vlan['generalNetworkUri'])
+                  OneviewSDK.resource_named('FCoENetwork', @client.api_version, variant).new(@client, uri: vlan['generalNetworkUri'])
                 end
           net.retrieve!
           internal_networks.push(net)

data/lib/oneview-sdk/resource/api200/logical_interconnect_group.rb

@@ -38,7 +38,7 @@ module OneviewSDK
       # @param [OneviewSDK::Client] client The client object for the OneView appliance
       # @return [Hash] The logical interconnect group settings
       def self.get_default_settings(client)
-        response = client.rest_get(BASE_URI + '/defaultSettings', client.api_version)
+        response = client.rest_get(BASE_URI + '/defaultSettings', {}, client.api_version)
         client.response_handler(response)
       end
 
@@ -79,7 +79,7 @@ module OneviewSDK
       # @return [Hash] The logical interconnect group settings
       def get_settings
         get_uri = @data['uri'] + '/settings'
-        response = @client.rest_get(get_uri, @api_version)
+        response = @client.rest_get(get_uri, {}, @api_version)
         @client.response_handler(response)
       end
 

data/lib/oneview-sdk/resource/api200/power_device.rb

@@ -155,7 +155,7 @@ module OneviewSDK
           uri += '&'
         end
         uri.chop! # Get rid of trailing '&' or '?'
-        response = @client.rest_get(uri, @api_version)
+        response = @client.rest_get(uri, {}, @api_version)
         @client.response_handler(response)
       end
 

data/lib/oneview-sdk/resource/api200/server_hardware.rb

@@ -165,7 +165,7 @@ module OneviewSDK
       # Gets the settings that describe the environmental configuration
       def environmental_configuration
         ensure_client && ensure_uri
-        response = @client.rest_get(@data['uri'] + '/environmentalConfiguration', @api_version)
+        response = @client.rest_get(@data['uri'] + '/environmentalConfiguration', {}, @api_version)
         @client.response_handler(response)
       end
 
@@ -194,7 +194,7 @@ module OneviewSDK
           uri += '&'
         end
         uri.chop! # Get rid of trailing '&' or '?'
-        response = @client.rest_get(uri, @api_version)
+        response = @client.rest_get(uri, {}, @api_version)
         @client.response_handler(response)
       end
 

data/lib/oneview-sdk/resource/api200/server_profile.rb

@@ -10,11 +10,13 @@
 # language governing permissions and limitations under the License.
 
 require_relative 'resource'
+require_relative '../../resource_helper'
 
 module OneviewSDK
   module API200
     # Server profile resource implementation
     class ServerProfile < Resource
+      include OneviewSDK::ResourceHelper
       BASE_URI = '/rest/server-profiles'.freeze
       UNIQUE_IDENTIFIERS = %w(name uri associatedServer serialNumber serverHardwareUri).freeze
 
@@ -26,30 +28,34 @@ module OneviewSDK
 
       # Sets the Server Hardware for the resource
       # @param [OneviewSDK::ServerHardware] server_hardware Server Hardware resource
+      # @raise [OneviewSDK::IncompleteResource] if server hardware not found
       def set_server_hardware(server_hardware)
-        self['serverHardwareUri'] = server_hardware['uri'] if server_hardware['uri'] || server_hardware.retrieve!
-        raise "Resource #{server_hardware['name']} could not be found!" unless server_hardware['uri']
+        raise 'Server Hardware could not be found!' unless server_hardware.retrieve!
+        self['serverHardwareUri'] = server_hardware['uri']
       end
 
       # Sets the Server Hardware Type for the resource
       # @param [OneviewSDK::ServerHardwareType] server_hardware_type Type of the desired Server Hardware
+      # @raise [OneviewSDK::IncompleteResource] if server hardware type not found
       def set_server_hardware_type(server_hardware_type)
-        self['serverHardwareTypeUri'] = server_hardware_type['uri'] if server_hardware_type['uri'] || server_hardware_type.retrieve!
-        raise "Resource #{server_hardware_type['name']} could not be found!" unless server_hardware_type['uri']
+        raise 'Server Hardware Type could not be found!' unless server_hardware_type.retrieve!
+        self['serverHardwareTypeUri'] = server_hardware_type['uri']
       end
 
       # Sets the Enclosure Group for the resource
       # @param [OneviewSDK::EnclosureGroup] enclosure_group Enclosure Group that the Server is a member
+      # @raise [OneviewSDK::IncompleteResource] if enclosure group not found
       def set_enclosure_group(enclosure_group)
-        self['enclosureGroupUri'] = enclosure_group['uri'] if enclosure_group['uri'] || enclosure_group.retrieve!
-        raise "Resource #{enclosure_group['name']} could not be found!" unless enclosure_group['uri']
+        raise 'Enclosure Group could not be found!' unless enclosure_group.retrieve!
+        self['enclosureGroupUri'] = enclosure_group['uri']
       end
 
-      # Sets the Enclosure Group for the resource
+      # Sets the Enclosure for the resource
       # @param [OneviewSDK::Enclosure] enclosure Enclosure that the Server is a member
+      # @raise [OneviewSDK::IncompleteResource] if enclosure not found
       def set_enclosure(enclosure)
-        self['enclosureUri'] = enclosure['uri'] if enclosure['uri'] || enclosure.retrieve!
-        raise "Resource #{enclosure['name']} could not be found!" unless enclosure['uri']
+        raise 'Enclosure could not be found!' unless enclosure.retrieve!
+        self['enclosureUri'] = enclosure['uri']
       end
 
       # Gets the preview of manual and automatic updates required to make the server profile consistent with its template.
@@ -88,14 +94,7 @@ module OneviewSDK
 
       # Updates the server profile from the server profile template.
       def update_from_template
-        ensure_client & ensure_uri
-        patch_operation = { 'op' => 'replace', 'path' => '/templateCompliance', 'value' => 'Compliant' }
-        patch_options = {
-          'If-Match' => self['eTag'],
-          'body' => [patch_operation]
-        }
-        response = @client.rest_patch(self['uri'], patch_options)
-        @client.response_handler(response)
+        patch('replace', '/templateCompliance', 'Compliant', 'If-Match' => self['eTag'])
       end
 
       # @!group Helpers
@@ -130,7 +129,8 @@ module OneviewSDK
           serverHardwareTypeUri: @data['serverHardwareTypeUri'],
           serverGroupUri: @data['enclosureGroupUri']
         }
-        OneviewSDK::ServerHardware.find_by(@client, params)
+        variant = self.class.name.split('::').at(-2)
+        OneviewSDK.resource_named('ServerHardware', @client.api_version, variant).find_by(@client, params)
       rescue StandardError => e
         raise IncompleteResource, "Failed to get available hardware. Message: #{e.message}"
       end
@@ -165,8 +165,9 @@ module OneviewSDK
       # @option connection_options [String] 'wwpnType' Specifies the type of WWN address to be porgrammed on the FlexNIC.
       #   The value can be 'Virtual', 'Physical' or 'UserDefined'.
       def add_connection(network, connection_options = {})
+        connection_options = Hash[connection_options.map { |k, v| [k.to_s, v] }]
         self['connections'] = [] unless self['connections']
-        connection_options['id'] ||= 0 unless connection_options[:id] # Letting OneView treat the ID registering
+        connection_options['id'] ||= 0
         connection_options['networkUri'] = network['uri'] if network['uri'] || network.retrieve!
         self['connections'] << connection_options
       end
@@ -192,13 +193,16 @@ module OneviewSDK
       # @option attachment_options [Boolean] 'permanent' Required. If true, indicates that the volume will persist when the profile is deleted.
       #   If false, then the volume will be deleted when the profile is deleted.
       # @option attachment_options [Array] 'storagePaths' A list of host-to-target path associations.
+      # @raise [OneviewSDK::IncompleteResource] if volume not found
       # @return Returns the connection hash if found, otherwise returns nil
       def add_volume_attachment(volume, attachment_options = {})
+        raise IncompleteResource, 'Volume not found!' unless volume.retrieve!
+        # Convert symbols keys to string
+        attachment_options = Hash[attachment_options.map { |k, v| [k.to_s, v] }]
         self['sanStorage'] ||= {}
         self['sanStorage']['volumeAttachments'] ||= []
+        self['sanStorage']['manageSanStorage'] ||= true
         attachment_options['id'] ||= 0
-
-        volume.retrieve! unless volume['uri'] || volume['storagePoolUri'] || volume['storageSystemUri']
         attachment_options['volumeUri'] = volume['uri']
         attachment_options['volumeStoragePoolUri'] = volume['storagePoolUri']
         attachment_options['volumeStorageSystemUri'] = volume['storageSystemUri']
@@ -207,7 +211,7 @@ module OneviewSDK
       end
 
       # Adds volume attachment entry and creates a new Volume associated in the Server profile
-      # @param [OneviewSDK::Volume] volume Volume Resource to add an attachment
+      # @param [OneviewSDK::StoragePool] storage_pool StoragePool Resource to add an attachment
       # @param [Hash] volume_options Options to create a new Volume.
       #   Please refer to OneviewSDK::Volume documentation for the data necessary to create a new Volume.
       # @param [Hash] attachment_options Options of the new attachment
@@ -217,8 +221,13 @@ module OneviewSDK
       # @option attachment_options [Boolean] 'permanent' Required. If true, indicates that the volume will persist when the profile is deleted.
       #   If false, then the volume will be deleted when the profile is deleted.
       # @option attachment_options [Array] 'storagePaths' A list of host-to-target path associations.
+      # @raise [OneviewSDK::IncompleteResource] if storage pool not found
       # @return Returns the connection hash if found, otherwise returns nil
       def create_volume_with_attachment(storage_pool, volume_options, attachment_options = {})
+        raise IncompleteResource, 'Storage Pool not found!' unless storage_pool.retrieve!
+        # Convert symbols keys to string in volume_options and attachment_options
+        volume_options = Hash[volume_options.map { |k, v| [k.to_s, v] }]
+        attachment_options = Hash[attachment_options.map { |k, v| [k.to_s, v] }]
         self['sanStorage'] ||= {}
         self['sanStorage']['volumeAttachments'] ||= []
         attachment_options['id'] ||= 0
@@ -233,7 +242,7 @@ module OneviewSDK
           attachment_options["volume#{k.to_s[0].capitalize}#{k.to_s[1, k.to_s.length - 1]}"] = v
         end
 
-        attachment_options['volumeStoragePoolUri'] = storage_pool['uri'] if storage_pool['uri'] || storage_pool.retrieve!
+        attachment_options['volumeStoragePoolUri'] = storage_pool['uri']
 
         # Since the volume is being created in this method, it needs to be nil
         attachment_options['volumeUri'] = nil
@@ -247,7 +256,9 @@ module OneviewSDK
         attachment_options['lunType'] ||= 'Auto'
         attachment_options['lun'] ||= nil
         attachment_options['storagePaths'] ||= []
+        attachment_options['volumeShareable'] = false
 
+        self['sanStorage']['manageSanStorage'] ||= true
         self['sanStorage']['volumeAttachments'] << attachment_options
       end
 
@@ -354,8 +365,7 @@ module OneviewSDK
       # @option query [String] 'sort' The number of resources to return. A count of -1 requests all the items.
       def self.get_available_storage_systems(client, query = nil)
         query_uri = build_query(query) if query
-        response = client.rest_get("#{BASE_URI}/available-storage-systems#{query_uri}")
-        client.response_handler(response)
+        find_with_pagination(client, "#{BASE_URI}/available-storage-systems#{query_uri}")
       end
 
       # Get the available targets based on the query parameters

data/lib/oneview-sdk/resource/api200/server_profile_template.rb

@@ -17,6 +17,10 @@ module OneviewSDK
     class ServerProfileTemplate < Resource
       BASE_URI = '/rest/server-profile-templates'.freeze
 
+      # Create a resource object, associate it with a client, and set its properties.
+      # @param [OneviewSDK::Client] client The client object for the OneView appliance
+      # @param [Hash] params The options for this resource (key-value pairs)
+      # @param [Integer] api_ver The api version to use when interracting with this resource.
       def initialize(client, params = {}, api_ver = nil)
         super
         # Default values
@@ -82,8 +86,9 @@ module OneviewSDK
       # @option connection_options [String] 'wwpnType' Specifies the type of WWN address to be porgrammed on the FlexNIC.
       #   The value can be 'Virtual', 'Physical' or 'UserDefined'.
       def add_connection(network, connection_options = {})
+        connection_options = Hash[connection_options.map { |k, v| [k.to_s, v] }]
         self['connections'] = [] unless self['connections']
-        connection_options['id'] = 0 # Letting OneView treat the ID registering
+        connection_options['id'] ||= 0
         connection_options['networkUri'] = network['uri'] if network['uri'] || network.retrieve!
         self['connections'] << connection_options
       end
@@ -109,22 +114,25 @@ module OneviewSDK
       # @option attachment_options [Boolean] 'permanent' Required. If true, indicates that the volume will persist when the profile is deleted.
       #   If false, then the volume will be deleted when the profile is deleted.
       # @option attachment_options [Array] 'storagePaths' A list of host-to-target path associations.
+      # @raise [OneviewSDK::IncompleteResource] if volume not found
       # @return Returns the connection hash if found, otherwise returns nil
       def add_volume_attachment(volume, attachment_options = {})
+        raise IncompleteResource, 'Volume not found!' unless volume.retrieve!
+        # Convert symbols keys to string
+        attachment_options = Hash[attachment_options.map { |k, v| [k.to_s, v] }]
         self['sanStorage'] ||= {}
         self['sanStorage']['volumeAttachments'] ||= []
+        self['sanStorage']['manageSanStorage'] ||= true
         attachment_options['id'] ||= 0
-
-        volume.retrieve! unless volume['uri'] || volume['storagePoolUri'] || volume['storageSystemUri']
         attachment_options['volumeUri'] = volume['uri']
         attachment_options['volumeStoragePoolUri'] = volume['storagePoolUri']
-        attachment_options['volumeStorageSystemUri'] = volume['storageSystemUri']
+        attachment_options['volumeStorageSystemUri'] = volume['storageSystemUri'] if volume['storageSystemUri']
 
         self['sanStorage']['volumeAttachments'] << attachment_options
       end
 
       # Adds a volume attachment entry with new volume in Server profile template
-      # @param [OneviewSDK::Volume] volume Volume Resource to add an attachment
+      # @param [OneviewSDK::StoragePool] storage_pool StoragePool Resource to add an attachment
       # @param [Hash] volume_options Options to create a new Volume.
       #   Please refer to OneviewSDK::Volume documentation for the data necessary to create a new Volume.
       # @param [Hash] attachment_options Options of the new attachment
@@ -134,8 +142,13 @@ module OneviewSDK
       # @option attachment_options [Boolean] 'permanent' Required. If true, indicates that the volume will persist when the profile is deleted.
       #   If false, then the volume will be deleted when the profile is deleted.
       # @option attachment_options [Array] 'storagePaths' A list of host-to-target path associations.
+      # @raise [OneviewSDK::IncompleteResource] if storage pool not found
       # @return Returns the connection hash if found, otherwise returns nil
       def create_volume_with_attachment(storage_pool, volume_options, attachment_options = {})
+        raise IncompleteResource, 'Storage Pool not found!' unless storage_pool.retrieve!
+        # Convert symbols keys to string in volume_options and attachment_options
+        volume_options = Hash[volume_options.map { |k, v| [k.to_s, v] }]
+        attachment_options = Hash[attachment_options.map { |k, v| [k.to_s, v] }]
         self['sanStorage'] ||= {}
         self['sanStorage']['volumeAttachments'] ||= []
         attachment_options['id'] ||= 0
@@ -150,21 +163,17 @@ module OneviewSDK
           attachment_options["volume#{k.to_s[0].capitalize}#{k.to_s[1, k.to_s.length - 1]}"] = v
         end
 
-        attachment_options['volumeStoragePoolUri'] = storage_pool['uri'] if storage_pool['uri'] || storage_pool.retrieve!
-
-        # Since the volume is being created in this method, it needs to be nil
-        attachment_options['volumeUri'] = nil
-        attachment_options['volumeStorageSystemUri'] = nil
-
+        attachment_options['volumeStoragePoolUri'] = storage_pool['uri']
         # volumeProvisionedCapacityBytes is not following the same pattern in Volume
         attachment_options['volumeProvisionedCapacityBytes'] ||= attachment_options.delete('volumeRequestedCapacity')
 
         # Defaults
+        attachment_options['volumeUri'] = nil
+        attachment_options['volumeStorageSystemUri'] = nil
         attachment_options['permanent'] ||= true
-        attachment_options['lunType'] ||= 'Auto'
-        attachment_options['lun'] ||= nil
-        attachment_options['storagePaths'] ||= []
+        attachment_options['volumeShareable'] = false
 
+        self['sanStorage']['manageSanStorage'] ||= true
         self['sanStorage']['volumeAttachments'] << attachment_options
       end
 
@@ -184,6 +193,7 @@ module OneviewSDK
       end
 
       # Gets the available server hardwares
+      # @raise [OneviewSDK::IncompleteResource] if serverHardwareTypeUri or enclosureGroupUri is missing
       # @return [Array<OneviewSDK::ServerHardware>] Array of ServerHardware resources that matches this
       #   profile template's server hardware type and enclosure group and who's state is 'NoProfileApplied'
       def get_available_hardware