lib/dcmgr/models/instance_nic.rb in wakame-vdc-dcmgr-11.06.0 vs lib/dcmgr/models/instance_nic.rb in wakame-vdc-dcmgr-11.12.0

@@ -3,71 +3,94 @@
 module Dcmgr::Models
   # Network interface for running instance.
   class InstanceNic < BaseNew
     taggable 'vif'
 
-    inheritable_schema do
-      Fixnum :instance_id, :null=>false
-      Fixnum :network_id, :null=>false
-      Fixnum :nat_network_id
-      Fixnum :bandwidth_group_id
-      String :mac_addr, :null=>false, :size=>12
-      
-      index :mac_addr
-    end
-    with_timestamps
-
-    many_to_one :bandwidth_group
     many_to_one :instance
     many_to_one :nat_network, :key => :nat_network_id, :class => Network
     many_to_one :network
     one_to_many :ip, :class=>IpLease
     one_to_many(:direct_ip_lease, :class=>IpLease, :read_only=>true) do |ds|
       ds.where(:network_id=>self.network_id)
     end
     one_to_many(:nat_ip_lease, :class=>IpLease, :read_only=>true) do |ds|
       ds.where(:network_id=>self.nat_network_id)
     end
+    
+    subset(:alives, {:deleted_at => nil})
 
-    def to_hash
-      h = values.dup.merge(super)
-      h.delete(:instance_id)
-      h
+    def to_api_document
+      hash = super
+      hash.delete(instance_id)
+      hash
     end
 
+    def release_ip_lease
+      ip_dataset.destroy
+    end
+
+    #Override the delete method to keep the row and just mark it as deleted
+    def delete
+      self.deleted_at ||= Time.now
+      self.save
+    end
+
     def before_validation
-      newlease=nil
-      m = self[:mac_addr] = normalize_mac_addr(self[:mac_addr])
-      if m
-        if m.size == 6
-          newlease = MacLease.lease(m)
-        else
-          MacLease.create(:mac_addr=>m)
-        end
-      else
-        newlease = MacLease.lease()
+      self[:mac_addr] = normalize_mac_addr(self[:mac_addr])
+
+      # set maximum index number if the nic has no index value and
+      # is attached to instance.
+      if self.instance_id && self.device_index.nil?
+        max_idx = self.class.alives.filter(:instance_id=>self.instance_id).max(:device_index)
+        self.device_index = max_idx.nil? ? 0 : (max_idx + 1)
       end
-      self[:mac_addr] = newlease.mac_addr if newlease
       
       super
     end
 
     def before_destroy
-      MacLease.find(:mac_addr=>self.mac_addr).destroy
-      ip_dataset.destroy
+      maclease = MacLease.find(:mac_addr=>self.mac_addr)
+      maclease.destroy if maclease
+      release_ip_lease
       super
     end
 
     def validate
       super
 
+      # do not run validation if the row is maked as deleted.
+      return true if self.deleted_at
+
       unless self.mac_addr.size == 12 && self.mac_addr =~ /^[0-9a-f]{12}$/
         errors.add(:mac_addr, "Invalid mac address syntax: #{self.mac_addr}")
       end
+      if MacLease.find(:mac_addr=>self.mac_addr).nil?
+        errors.add(:mac_addr, "MAC address is not on the MAC lease database.")
+      end
+
+      # find duplicate device index.
+      if self.instance_id
+        idx = self.class.alives.filter(:instance_id=>self.instance_id).select(:device_index).all
+        if idx.uniq.size != idx.size
+          errors.add(:device_index, "Duplicate device index.")
+        end
+      end
     end
 
     def pretty_mac_addr(delim=':')
       self.mac_addr.unpack('A2'*6).join(delim)
+    end
+
+    def fqdn_hostname
+      raise "Instance is not associated: #{self.canonical_uuid}" if self.instance.nil?
+      raise "Network is not associated: #{self.canonical_uuid}" if self.network.nil?
+      sprintf("%s.%s.%s", self.instance.hostname, self.instance.account.uuid, self.network.domain_name)
+    end
+
+    def nat_fqdn_hostname
+      raise "Instance is not associated: #{self.canonical_uuid}" if self.instance.nil?
+      raise "Network is not associated: #{self.canonical_uuid}" if self.network.nil?
+      sprintf("%s.%s.%s", self.instance.hostname, self.instance.account.uuid, self.nat_network.domain_name)
     end
 
     private
     def normalize_mac_addr(str)
       str = str.downcase.gsub(/[^0-9a-f]/, '')