wakame-vdc-agents 11.06.0 → 11.12.0

data/lib/dcmgr/cli/spec.rb

@@ -6,37 +6,37 @@ module Dcmgr::Cli
     M = Dcmgr::Models
 
     desc "add [options]", "Register a new machine spec"
-    method_option :uuid, :type => :string, :aliases => "-u", :desc => "The UUID for the new machine spec"
-    method_option :account_id, :type => :string, :aliases => "-a", :required => true, :desc => "The UUID of the account that this machine spec belongs to"
-    method_option :arch, :type => :string, :default => 'x86_64', :aliases => "-r", :desc => "The architecture for the new machine image. [#{M::HostPool::SUPPORTED_ARCH.join(', ')}]"
-    method_option(:hypervisor, :type => :string, :aliases => "-p", :default => M::HostPool::HYPERVISOR_KVM.to_s,
-                  :desc => "The hypervisor type for the new instance. [#{M::HostPool::SUPPORTED_HYPERVISOR.join(', ')}]")
-    method_option :cpu_cores, :type => :numeric, :aliases => "-c", :default => 1, :desc => "The initial cpu cores for the new instance"
-    method_option :memory_size, :type => :numeric, :aliases => "-m", :default => 1024, :desc => "The memory size for the new instance"
-    method_option :quota_weight, :type => :numeric, :aliases => "-w", :default => 1.0, :desc => "The cost weight factor for the new instance"
-    #method_option :is_public, :type => :boolean, :aliases => "-p", :default => false, :desc => "A flag that determines whether the new machine image is public or not."
+    method_option :uuid, :type => :string, :desc => "The UUID for the new machine spec"
+    method_option :account_id, :type => :string, :required => true, :desc => "The UUID of the account that this machine spec belongs to"
+    method_option :arch, :type => :string, :default => 'x86_64', :desc => "The architecture for the new machine image. [#{M::HostNode::SUPPORTED_ARCH.join(', ')}]"
+    method_option(:hypervisor, :type => :string, :default => M::HostNode::HYPERVISOR_KVM.to_s,
+                  :desc => "The hypervisor type for the new instance. [#{M::HostNode::SUPPORTED_HYPERVISOR.join(', ')}]")
+    method_option :cpu_cores, :type => :numeric, :default => 1, :desc => "The initial cpu cores for the new instance"
+    method_option :memory_size, :type => :numeric, :default => 1024, :desc => "The memory size for the new instance"
+    method_option :quota_weight, :type => :numeric, :default => 1.0, :desc => "The cost weight factor for the new instance"
     def add
       UnknownUUIDError.raise(options[:account_id]) if M::Account[options[:account_id]].nil?
-      UnsupportedArchError.raise(options[:arch]) unless M::HostPool::SUPPORTED_ARCH.member?(options[:arch])
-      UnsupportedHypervisorError.raise(options[:hypervisor]) unless M::HostPool::SUPPORTED_HYPERVISOR.member?(options[:hypervisor])
-      fields = options.dup
-      fields[:config] = {
-      }
-      puts super(M::InstanceSpec, fields)
+      UnsupportedArchError.raise(options[:arch]) unless M::HostNode::SUPPORTED_ARCH.member?(options[:arch])
+      UnsupportedHypervisorError.raise(options[:hypervisor]) unless M::HostNode::SUPPORTED_HYPERVISOR.member?(options[:hypervisor])
+      uuid = super(M::InstanceSpec,options)
+      # add one interface as default
+      invoke("addvif", [uuid, 'eth0'])
+      invoke("adddrive", [uuid, 'local', 'ephemeral1'])
+      puts uuid
     end
     
     desc "modify UUID [options]", "Modify an existing machine spec"
-    method_option :account_id, :type => :string, :aliases => "-a", :desc => "The UUID of the account that this machine spec belongs to"
-    method_option :arch, :type => :string, :aliases => "-r", :desc => "The architecture for the new machine image. [#{M::HostPool::SUPPORTED_ARCH.join(', ')}]"
-    method_option(:hypervisor, :type => :string, :aliases => "-p",
-                  :desc => "The hypervisor type for the new instance. [#{M::HostPool::SUPPORTED_HYPERVISOR.join(', ')}]")
-    method_option :cpu_cores, :type => :numeric, :aliases => "-c", :desc => "The initial cpu cores for the new instance"
-    method_option :memory_size, :type => :numeric, :aliases => "-m", :desc => "The memory size for the new instance"
-    method_option :quota_weight, :type => :numeric, :aliases => "-w", :desc => "The cost weight factor for the new instance"
+    method_option :account_id, :type => :string, :desc => "The UUID of the account that this machine spec belongs to"
+    method_option :arch, :type => :string, :desc => "The architecture for the new machine image. [#{M::HostNode::SUPPORTED_ARCH.join(', ')}]"
+    method_option(:hypervisor, :type => :string,
+                  :desc => "The hypervisor type for the new instance. [#{M::HostNode::SUPPORTED_HYPERVISOR.join(', ')}]")
+    method_option :cpu_cores, :type => :numeric, :desc => "The initial cpu cores for the new instance"
+    method_option :memory_size, :type => :numeric, :desc => "The memory size for the new instance"
+    method_option :quota_weight, :type => :numeric, :desc => "The cost weight factor for the new instance"
     def modify(uuid)
       UnknownUUIDError.raise(options[:account_id]) if options[:account_id] && M::Account[options[:account_id]].nil?
-      UnsupportedArchError.raise(options[:arch]) unless options[:arch].nil? || M::HostPool::SUPPORTED_ARCH.member?(options[:arch])
-      UnsupportedHypervisorError.raise(options[:hypervisor]) unless options[:hypervisor].nil? || M::HostPool::SUPPORTED_HYPERVISOR.member?(options[:hypervisor])
+      UnsupportedArchError.raise(options[:arch]) unless options[:arch].nil? || M::HostNode::SUPPORTED_ARCH.member?(options[:arch])
+      UnsupportedHypervisorError.raise(options[:hypervisor]) unless options[:hypervisor].nil? || M::HostNode::SUPPORTED_HYPERVISOR.member?(options[:hypervisor])
       super(M::InstanceSpec,uuid,options)
     end
     
@@ -49,24 +49,38 @@ module Dcmgr::Cli
     desc "show [UUID]", "Show list of machine spec and details"
     def show(uuid=nil)
       if uuid
-        spec = M::InstanceSpec[uuid]
+        spec = M::InstanceSpec[uuid] || UnknownUUIDError.raise(uuid)
         print ERB.new(<<__END, nil, '-').result(binding)
-UUID:
-  <%= spec.canonical_uuid %>
-Account ID:
-  <%= spec.account_id %>
-Hypervisor:
-  <%= spec.hypervisor %>
-Arch:
-  <%= spec.arch %>
-CPU Cores:
-  <%= spec.cpu_cores %>
-Memory Size:
-  <%= spec.memory_size %>
-Quota Weight:
-  <%= spec.quota_weight %>
+UUID: <%= spec.canonical_uuid %>
+Account ID: <%= spec.account_id %>
+Hypervisor: <%= spec.hypervisor %>
+Arch: <%= spec.arch %>
+CPU Cores: <%= spec.cpu_cores %>
+Memory Size: <%= spec.memory_size %>
+Quota Weight: <%= spec.quota_weight %>
+<%- unless spec.vifs.empty? -%>
+Interfaces:
+  <%- spec.vifs.each { |name, i| -%>
+  [<%= i[:index] %>] <%= name %>:
+    Bandwidth: <%= i[:bandwidth] %> kbps
+  <%- } -%>
+<%- end -%>
+<%- unless spec.drives.empty? -%>
+Drives:
+  <%- spec.drives.each { |name, i| -%>
+  [<%= i[:index] %>] <%= name %>:
+    Type: <%= i[:type] %>
+    <%- if i[:size] -%>
+    Size: <%= i[:size] %> MB
+    <%- else -%>
+    Snapshot ID: <%= i[:snapshot_id] %>
+    <%- end -%>
+  <%- } -%>
+<%- end -%>
+<%- unless spec.config.empty? -%>
 Hypervisor Configuration:
   <%= spec.config.inspect %>
+<%- end -%>
 __END
       else
         cond = {}
@@ -78,5 +92,96 @@ __END
 __END
       end
     end
+    
+    desc "addvif UUID name", "Add interfance"
+    method_option :index, :type => :numeric, :desc => "The index value for the interface. (>=0)"
+    method_option :bandwidth, :type => :numeric, :default=>100000, :desc => "The bandwidth (kbps) of the interface"
+    def addvif(uuid, name)
+      spec = M::InstanceSpec[uuid]
+
+      index = if options[:index].nil?
+                # find max index value.
+                index = spec.vifs.values.map { |i| i[:index] }.max
+                index.nil? ? 0 : (index + 1)
+              else
+                options[:index].to_i
+              end
+      
+      spec.add_vif(name, index.to_i, options[:bandwidth].to_i)
+      spec.save
+    end
+    
+    desc "delvif UUID name", "Delete interfance"
+    def delvif(uuid, name)
+      spec = M::InstanceSpec[uuid]
+      spec.remove_vif(name)
+      spec.save
+    end
+    
+    desc "modifyvif UUID name [options]", "Modify interfance parameters"
+    method_option :index, :type => :numeric, :desc => "The index value for the interface"
+    method_option :bandwidth, :type => :numeric, :desc => "The bandwidth (kbps) of the interface"
+    def modifyvif(uuid, name)
+      spec = M::InstanceSpec[uuid]
+      if options[:index]
+        spec.update_vif_index(name, options[:index].to_i)
+      end
+      if options[:bandwidth]
+        spec.update_vif_bandwidth(name, options[:bandwidth].to_i)
+      end
+      spec.save
+    end
+
+    desc "adddrive UUID TYPE NAME", "Add drive (TYPE=local,volume)"
+    method_option :index, :type => :numeric, :desc => "The index value for the interface. (>=0)"
+    method_option :size, :type => :numeric, :default=>100, :desc => "Size of the drive. (MB)"
+    def adddrive(uuid, type, name)
+      spec = M::InstanceSpec[uuid]
+
+      index = if options[:index].nil?
+                # find max index value.
+                index = spec.drives.values.map { |i| i[:index] }.max
+                index.nil? ? 0 : (index + 1)
+              else
+                options[:index].to_i
+              end
+
+      case type
+      when 'local'
+        spec.add_local_drive(name, index, options[:size].to_i)
+      when 'volume'
+        spec.add_volume_drive(name, index, options[:size].to_i)
+      else
+        raise "Unknown drive type: #{type}"
+      end
+      
+      spec.save
+    end
+
+    desc "deldrive UUID name", "Delete drive"
+    def deldrive(uuid, name)
+      spec = M::InstanceSpec[uuid]
+      spec.remove_drive(name)
+      spec.save
+    end
+    
+    desc "modifydrive UUID name [options]", "Modify drive parameters"
+    method_option :index, :type => :numeric, :desc => "The index value for the interface"
+    method_option :size, :type => :numeric, :desc => "Size of the drive. (MB)"
+    method_option :snapshot_id, :type => :string, :desc => "Snapshot ID to copy the content for new drive. Only for "
+    def modifydrive(uuid, name)
+      spec = M::InstanceSpec[uuid]
+      if options[:index]
+        spec.update_drive_index(name, options[:index].to_i)
+      end
+      if options[:size]
+        spec.update_drive_size(name, options[:size].to_i)
+      end
+      if options[:snapshot_id]
+        spec.update_drive_snapshot_id(name, options[:snapshot_id])
+      end
+      spec.save
+    end
+
   end
 end

data/lib/dcmgr/cli/storage.rb

@@ -9,20 +9,21 @@ class Storage < Base
   include Dcmgr::Models
   
   desc "add NODE_ID [options]", "Register a new storage node"
-  method_option :uuid, :type => :string, :aliases => "-u", :desc => "The uuid for the new storage pool."
-  method_option :base_path, :type => :string, :aliases => "-b", :required => true, :desc => "Base path to store volume files"
-  method_option :snapshot_base_path, :type => :string, :aliases => "-n", :required => true, :desc => "Base path to store snapshot files"
-  method_option :disk_space, :type => :numeric, :aliases => "-s", :required => true, :desc => "Amount of disk size to be exported (in MB)."
-  method_option :transport_type, :type => :string, :aliases => "-t", :default=>'iscsi', :desc => "Transport type [iscsi]"
-  method_option :ipaddr, :type => :string, :aliases => "-i", :required=>true, :desc => "IP address of transport target"
-  method_option :storage_type, :type => :string, :aliases => "-o", :default=>'zfs', :desc => "Storage type [zfs]"
-  method_option :account_id, :type => :string, :default=>'a-shpool', :aliases => "-a", :desc => "The account ID to own this."
+  method_option :uuid, :type => :string, :desc => "The uuid for the new storage node"
+  method_option :base_path, :type => :string, :required => true, :desc => "Base path to store volume files"
+  method_option :snapshot_base_path, :type => :string, :required => true, :desc => "Base path to store snapshot files"
+  method_option :disk_space, :type => :numeric, :required => true, :desc => "Amount of disk size to be exported (in MB)"
+  method_option :force, :type => :boolean, :default=>false, :desc => "Force new entry creation"
+  method_option :transport_type, :type => :string, :default=>'iscsi', :desc => "Transport type [iscsi]"
+  method_option :ipaddr, :type => :string, :required=>true, :desc => "IP address of transport target"
+  method_option :storage_type, :type => :string, :default=>'zfs', :desc => "Storage type [#{StorageNode::SUPPORTED_BACKINGSTORE.join(', ')}]"
+  method_option :account_id, :type => :string, :default=>'a-shpoolxx', :desc => "The account ID to own this"
   def add(node_id)
-    unless (options[:force] == false && Isono::Models::NodeState.exists?(:node_id=>options[:node_id]))
-      abort("Node ID is not registered yet: #{options[:node_id]}")
+    unless (options[:force] || Isono::Models::NodeState.find(:node_id=>node_id))
+      abort("Node ID is not registered yet: #{node_id}")
     end
 
-    fields = {:node_id=>options[:node_id],
+    fields = {:node_id=>node_id,
               :offering_disk_space=>options[:disk_space],
               :transport_type=>options[:transport_type],
               :storage_type=>options[:storage_type],
@@ -33,18 +34,18 @@ class Storage < Base
     }
     fields.merge!({:uuid => options[:uuid]}) unless options[:uuid].nil?
     
-    puts super(StoragePool,fields)
+    puts super(StorageNode,fields)
   end
 
   desc "del UUID", "Deregister a storage node"
   def del(uuid)
-    super(StoragePool,uuid)
+    super(StorageNode,uuid)
   end
 
   desc "show [UUID]", "Show list of storage nodes and details"
   def show(uuid=nil)
     if uuid
-      st = StoragePool[uuid]
+      st = StorageNode[uuid] || UnknownUUIDError.raise(uuid)
       puts ERB.new(<<__END, nil, '-').result(binding)
 UUID:
   <%= st.canonical_uuid %>
@@ -65,7 +66,7 @@ Snapshot base path:
 __END
     else
       cond = {}
-      all = StoragePool.filter(cond).all
+      all = StorageNode.filter(cond).all
       puts ERB.new(<<__END, nil, '-').result(binding)
 <%- all.each { |row| -%>
 <%= "%-15s %-20s %-10s" % [row.canonical_uuid, row.node_id, row.status] %>

data/lib/dcmgr/cli/tag.rb

@@ -6,11 +6,11 @@ module Dcmgr::Cli
     M = Dcmgr::Models
 
     desc "add [options]", "Create a new tag"
-    method_option :uuid, :type => :string, :aliases => "-u", :desc => "The UUID for the new tag"
-    method_option :account_id, :type => :string, :aliases => "-a", :desc => "The UUID of the account that this tag belongs to", :required => true
-    method_option :type_id, :type => :numeric, :aliases => "-t", :desc => "The type for the new tag. Valid types are '#{Dcmgr::Tags::KEY_MAP.keys.join(", ")}'", :required => true
-    method_option :name, :type => :string, :aliases => "-n", :desc => "The name for the new tag", :required => true
-    method_option :attributes, :type => :string, :aliases => "-b", :desc => "The attributes for the new tag"
+    method_option :uuid, :type => :string, :desc => "The UUID for the new tag"
+    method_option :account_id, :type => :string, :desc => "The UUID of the account that this tag belongs to", :required => true
+    method_option :type_id, :type => :numeric, :desc => "The type for the new tag. Valid types are '#{Dcmgr::Tags::KEY_MAP.keys.join(", ")}'", :required => true
+    method_option :name, :type => :string, :desc => "The name for the new tag", :required => true
+    method_option :attributes, :type => :string, :desc => "The attributes for the new tag"
     def add
       UnknownUUIDError.raise(options[:account_id]) if M::Account[options[:account_id]].nil?
       Error.raise("Invalid type_id: '#{options[:type_id]}'. Valid types are '#{Dcmgr::Tags::KEY_MAP.keys.join(", ")}'.",100) unless Dcmgr::Tags::KEY_MAP.member? options[:type_id]
@@ -19,10 +19,10 @@ module Dcmgr::Cli
     end
     
     desc "modify UUID [options]", "Modify an existing tag"
-    method_option :account_id, :type => :string, :aliases => "-a", :desc => "The UUID of the account that this tag belongs to"
-    method_option :type_id, :type => :numeric, :aliases => "-t", :desc => "The type for the new tag. Valid types are '#{Dcmgr::Tags::KEY_MAP.keys.join(", ")}'"
-    method_option :name, :type => :string, :aliases => "-n", :desc => "The name for the new tag"
-    method_option :attributes, :type => :string, :aliases => "-b", :desc => "The attributes for the new tag"
+    method_option :account_id, :type => :string, :desc => "The UUID of the account that this tag belongs to"
+    method_option :type_id, :type => :numeric, :desc => "The type for the new tag. Valid types are '#{Dcmgr::Tags::KEY_MAP.keys.join(", ")}'"
+    method_option :name, :type => :string, :desc => "The name for the new tag"
+    method_option :attributes, :type => :string, :desc => "The attributes for the new tag"
     def modify(uuid)
       UnknownUUIDError.raise(options[:account_id]) if options[:account_id] && M::Account[options[:account_id]].nil?
       Error.raise("Invalid type_id: '#{options[:type_id]}'. Valid types are '#{Dcmgr::Tags::KEY_MAP.keys.join(", ")}'.",100) unless options[:type_id].nil? || Dcmgr::Tags::KEY_MAP.member?(options[:type_id])
@@ -59,17 +59,23 @@ __END
       super(M::Tag,uuid)
     end
     
-    desc "map UUID [options]", "Map a tag to a taggable object"
-    method_option :object_id, :type => :string, :aliases => "-o", :desc => "The canonical UUID for the object to map this tag to.", :required => true
-    def map(uuid)
+    desc "map UUID OBJECT_UUID", "Map a tag to a taggable object"
+    long_desc <<__DESC
+Map a tag to a taggable object.
+
+ UUID: Tag canonical UUID. 
+ OBJECT_UUID: The canonical UUID represents the object to label this tag.
+__DESC
+    #method_option :object_id, :type => :string, :desc => "The canonical UUID for the object to map this tag to.", :required => true
+    def map(uuid, object_uuid)
       #Quick hack to get all models in Dcmgr::Models loaded in Taggable.uuid_prefix_collection
       #This is so the Taggable.find method can be used to determine the Model class based on canonical uuid
       M.constants.each {|c| eval("M::#{c}")}
       
-      object = M::Taggable.find(options[:object_id])
+      object = M::Taggable.find(object_uuid)
 
       UnknownUUIDError.raise(uuid) if M::Tag[uuid].nil?
-      UnknownUUIDError.raise(options[:object_id]) if object.nil?
+      UnknownUUIDError.raise(object_uuid) if object.nil?
       Error.raise("Tag '#{uuid}' can not be mapped to a '#{object.class}'.",100) unless M::Tag[uuid].accept_mapping?(object)
       
       M::TagMapping.create(

data/lib/dcmgr/cli/vlan.rb

@@ -6,9 +6,9 @@ module Dcmgr::Cli
     M = Dcmgr::Models
 
     desc "add [options]", "Create a new vlan lease"
-    method_option :uuid, :type => :string, :aliases => "-u", :desc => "The UUID for the new vlan lease"
-    method_option :account_id, :type => :string, :aliases => "-a", :desc => "The UUID of the account for this vlan lease"
-    method_option :tag_id, :type => :numeric, :aliases => "-t", :desc => "The ethernet tag for this vlan lease"
+    method_option :uuid, :type => :string, :desc => "The UUID for the new vlan lease"
+    method_option :account_id, :type => :string, :desc => "The UUID of the account for this vlan lease"
+    method_option :tag_id, :type => :numeric, :desc => "The ethernet tag for this vlan lease"
     def add
       UnknownUUIDError.raise(options[:account_id]) if M::Account[options[:account_id]].nil?
       Error.raise("Tag_id already exists",100) unless M::VlanLease.find(:tag_id => options[:tag_id]).nil?
@@ -22,8 +22,8 @@ module Dcmgr::Cli
     end
     
     desc "modify UUID [options]", "Modify an existing vlan lease"
-    method_option :account_id, :type => :string, :aliases => "-a", :desc => "The UUID of the account for this vlan lease"
-    method_option :tag_id, :type => :numeric, :aliases => "-t", :desc => "The ethernet tag for this vlan lease"
+    method_option :account_id, :type => :string, :desc => "The UUID of the account for this vlan lease"
+    method_option :tag_id, :type => :numeric, :desc => "The ethernet tag for this vlan lease"
     def modify(uuid)
       UnknownUUIDError.raise(options[:account_id]) if options[:account_id] && M::Account[options[:account_id]].nil?
       super(M::VlanLease,uuid,options)

data/lib/dcmgr/drivers/backing_store.rb

@@ -0,0 +1,32 @@
+# -*- coding: utf-8 -*-
+
+module Dcmgr
+  module Drivers
+    class BackingStore
+
+      def create_volume(ctx, snapshot_file=nil)
+      end
+
+      def delete_volume(ctx)
+      end
+
+      def create_snapshot(ctx, snapshot_file)
+      end
+
+      def delete_snapshot(ctx, snapshot_file)
+      end
+
+      def self.select_backing_store(backing_store)
+        case backing_store
+        when "raw"
+          bs = Dcmgr::Drivers::Raw.new
+        when "zfs"
+          bs = Dcmgr::Drivers::Zfs.new
+        else
+          raise "Unknown backing_store type: #{backing_store}"
+        end
+        bs
+      end
+    end
+  end
+end

data/lib/dcmgr/drivers/comstar.rb

@@ -0,0 +1,81 @@
+# -*- coding: utf-8 -*-
+
+module Dcmgr
+  module Drivers
+    class Comstar < IscsiTarget
+      include Dcmgr::Logger
+      include Dcmgr::Helpers::CliHelper
+
+      # ref: http://nkjmkzk.net/?p=2022
+      def create(ctx)
+        @volume = ctx.volume.dup
+
+        # $ man itadm
+        #
+        # target_node_name
+        # An iSCSI Initiator Node Context  is  identified  by
+        # its Initiator Node Name, formatted in either IQN or EUI for-
+        # mat (see RFC 3720). For example:
+        #  iqn.1986-03.com.sun:01:e00000000000.47d55444
+        #  eui.02004567A425678D
+        #
+        # [TODO]
+        iqn_prefix = "iqn.2010-09.jp.wakame"
+        target_node = "#{iqn_prefix}:#{@volume[:account_id]}.#{@volume[:uuid]}"
+
+        # target
+        sh("itadm create-target -n %s", [target_node])
+
+#        # target group
+#        target_group = "tg:#{@volume[:account_id]}.#{@volume[:uuid]}"
+#        # target must be offline.
+#        # $ stmfadm create-tg a-shpoolxx.vol-chgqqw21
+#        # => stmfadm: STMF target must be offline
+#        #
+        sh("stmfadm offline-target %s", [target_node])
+#        sh("stmfadm create-tg %s", [target_group])
+#        sh("stmfadm add-tg-member -g %s %s", [target_group, target_node])
+        sh("stmfadm online-target %s", [target_node])
+
+        sh("itadm list-target -v %s", [target_node])
+
+#        # host group
+#        host_group = "hg:#{@volume[:account_id]}.#{@volume[:uuid]}"
+#        sh("stmfadm create-hg %s", [host_group])
+#        sh("stmfadm add-hg-member -g %s %s", [host_group, target_node])
+
+        # zvol has already created by backing storep
+        zvol_path = "/dev/zvol/rdsk/#{@volume[:storage_node][:export_path]}/#{@volume[:uuid]}"
+
+        # logical_unit
+        logical_unit = sh("sbdadm create-lu %s | /usr/gnu/bin/grep -w %s", [zvol_path, zvol_path])
+        guid = logical_unit[:stdout].split(' ')[0]
+
+        # view
+#        sh("stmfadm add-view -t %s -h %s %s", [target_group, host_group, guid])
+        sh("stmfadm add-view %s", [guid])
+        logical_unit = sh("stmfadm list-view -l %s", [guid])
+        logical_unit = logical_unit[:stdout].downcase.split("\n").select {|row| row.strip!}
+
+#        opt = {:iqn => target_node, :lun=>logical_unit[2].split(": ").last, :guid => guid, :hg => host_group, :tg => target_group}
+        opt = {:iqn => target_node, :lun=>logical_unit[2].split(": ").last, :guid => guid}
+      end
+
+      def delete(ctx)
+        @volume = ctx.volume
+
+        guid = @volume[:transport_information][:guid]
+        iqn =  @volume[:transport_information][:iqn]
+        target_group =  @volume[:transport_information][:tg]
+        host_group   =  @volume[:transport_information][:hg]
+
+        sh("stmfadm offline-target %s", [iqn])
+        sh("itadm delete-target %s", [iqn])
+
+        sh("stmfadm delete-lu %s", [guid])
+#        sh("stmfadm delete-tg %s", [target_group])
+#        sh("stmfadm delete-hg %s", [host_group])
+      end
+    end
+  end
+end