gogetit 0.2.1 → 0.3.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: a8894161933329fe5c330fdfe273572eaca29169
-  data.tar.gz: b1ece96587307c03ed4c858b5c4568bdffc789b3
+  metadata.gz: 9d5861321df7c324e2ed483329663114406a308b
+  data.tar.gz: 996bd6ae6cf0f34a38fb36f91abff1d94d06b34b
 SHA512:
-  metadata.gz: 579d18217f3eba138849cff0c7e3e06430843ff629c3e7a3a0e65f7d9655e0a066adbbea8ba012bd1467311a33a28a849a4caf93360bf83c42f4531967a6708a
-  data.tar.gz: 15b7091187727b452fa188a3d6cb7d70fa52270947c1f1a59b6a6c81bbc1dc11244c0d25703ffba319d01894c797c198518ebcdbf987e823393eee947f4fcbe8
+  metadata.gz: 61507178c9a1ad0aedb2d969a5714f8d9c3197e75fd016aaa974a51f6d860e9e2d65bbdd45680ce72e48600db02dde295a47d626dafc902d482cda99b4795aeb
+  data.tar.gz: e517a9a7000d81f192a7e456e572933408d3a843536e3e7e7d8fb0f98c25323a517080632d71306be4b6ed6650c5f6aa1639b0e0091f66cdd7fa83df16c7f3cf

data/README.md

@@ -9,6 +9,12 @@ By using this, you will get to use them all together in automated and efficient
 - Aware of Chef knife and its sub commands such as Vault to automate routine tasks.
 - Being used by [kitchen-gogetkitchen](https://github.com/itisnotdone/kitchen-gogetkitchen) as a driver for Chef Test Kitchen.
 
+## Limitations
+- Network resource awareness is only provided by MAAS.
+- Only LXD and Libvirt(KVM) are available as provider.
+- Only IPv4 is available for IP assignment.
+- It is tested only on Ubuntu 16.04 with OVS as virtual switch.
+
 ## Installation
 
 ### dependent packages
@@ -31,17 +37,41 @@ $ gem install gogetit --no-ri --no-rdoc
 ## Usage
 ```bash
 gogetit list
-gogetit create lxd lxd01
-gogetit create libvirt kvm01
+gogetit create lxd01
+gogetit create lxd01 --provider lxd
+
+# For advanced network configuration
+gogetit create lxd01 -p lxd -i 192.168.0.10
+
+# When specifying multiple IPs, the first one will be chosen as gateway.
+# And the IP which belongs to the gateway interface will be the IP of the FQDN
+# of the container which is set by MAAS.
+# The IPs should belong to networks defined and recognized by MAAS.
+gogetit create lxd01 -p lxd -i 192.168.10.10 10.0.0.2
+
+# When specifying multiple VLANs, the first one will be chosen as gateway.
+# gogetit create lxd01 -p lxd -v 0 10 12
+# gogetit create lxd01 -p lxd -v 10 11
+
+gogetit create kvm01 -p libvirt
+gogetit create kvm01 -p libvirt -i 192.168.10.10 10.0.0.2
+
+# to provision with a bare metal machine
+# gogetit create kvm01 -p bare
 
 gogetit destroy lxd01
 gogetit rebuild kvm01
 
 # to create a container bootstrapping as a chef node
-gogetit create --chef chef01
+gogetit create chef01 --chef
+
+or
+
+CHEF=true gogetit create chef01
+
 
 # to destroy a container deleting corresponding chef node and client
-gogetit destroy --chef chef01
+gogetit destroy chef01 --chef
 ```
 
 ```ruby
@@ -49,8 +79,9 @@ require 'gogetit'
 ```
 
 ## TODO
-- Network subnets and space aware via MAAS
-- Static IP allocation
+- Provisioning LXD with useful cloud-init user-data.
+- Deploying bare metal machine from machine pool.
+- Static IP auto-assignment with VLAN info.
 
 ## Development and Contributing
 Clone and then execute followings:

data/lib/gogetit/cli.rb

@@ -17,16 +17,28 @@ module Gogetit
     end
 
     desc 'create (TYPE) NAME', 'Create either a container or KVM domain.'
-    method_option :chef, :type => :boolean, :desc => "Enable chef awareness."
-    def create(type='lxd', name)
-      case type
+    method_option :provider, :aliases => '-p', :type => :string, \
+      :default => 'lxd', :desc => 'A provider such as lxd and libvirt'
+    method_option :chef, :aliases => '-c', :type => :boolean, \
+      :default => false, :desc => 'Chef awareness'
+
+    method_option :vlans, :aliases => '-v', :type => :array, \
+      :desc => 'A list of VLAN IDs to connect to'
+    method_option :ipaddresses, :aliases => '-i', :type => :array, \
+      :desc => 'A list of static IPs to assign'
+    def create(name)
+      abort("vlans and ipaddresses can not be used at the same time.") \
+        if options['vlans'] and options['ipaddresses']
+
+      case options[:provider]
       when 'lxd'
-        Gogetit.lxd.create(name)
+        Gogetit.lxd.create(name, options.to_hash)
       when 'libvirt'
-        Gogetit.libvirt.create(name)
+        Gogetit.libvirt.create(name, options.to_hash)
       else
         abort('Invalid argument entered.')
       end
+
       # post-tasks
       if options[:chef]
         knife_bootstrap(name, type, Gogetit.config)
@@ -34,6 +46,7 @@ module Gogetit
       end
       Gogetit.config[:default][:user] ||= ENV['USER']
       puts "ssh #{Gogetit.config[:default][:user]}@#{name}"
+      print "ssh #{Gogetit.config[:default][:user]}@#{name}"
     end
 
     desc 'destroy NAME', 'Destroy either a container or KVM domain.'

data/lib/gogetit/config.rb

@@ -61,23 +61,5 @@ module Gogetit
       abort('Please define default configuration for GoGetIt at ~/.gogetit/gogetit.yml.')
     end
     config.merge!(symbolize_keys(YAML.load_file(conf_file)))
-
-    logger.debug('Define provider configuration directory..')
-    provider_conf_dir = user_gogetit_home + '/conf'
-    config[:provider_conf_dir] = provider_conf_dir
-    default_provider_conf_file = provider_conf_dir + '/default.yml'
-    config[:default_provider_conf_file] = default_provider_conf_file
-    if not File.exists?(default_provider_conf_file)
-      if not File.directory?(provider_conf_dir)
-        logger.debug('Creating provider configuration directory..')
-        FileUtils.mkdir(provider_conf_dir)
-      end
-      src = File.new(lib_dir + '/sample_conf/default.yml')
-      dst = Dir.new(provider_conf_dir)
-      logger.debug('Copying provider configuration file..')
-      FileUtils.cp(src, dst)
-      abort('Please define default configuration for providers at ~/.gogetit/conf/default.yml.')
-    end
-
   end
 end

data/lib/gogetit/maas.rb

@@ -1,4 +1,5 @@
 require 'maas/client'
+require 'ipaddr'
 
 module Gogetit
   class GogetMAAS
@@ -36,48 +37,142 @@ module Gogetit
     end
 
     def domain_name_exists?(name)
+      logger.info("Calling <#{__method__.to_s}>")
       return true if dnsresource_exists?(name) or machine_exists?(name)
     end
 
+    def ip_reserved?(addresses)
+      logger.info("Calling <#{__method__.to_s}>")
+      ips = Set.new
+      addresses.each do |ip|
+        ips.add(IPAddr.new(ip))
+      end
+
+      reserved_ips = Set.new
+      conn.request(:get, ['ipaddresses']).each do |ip|
+        reserved_ips.add(IPAddr.new(ip['ip']))
+      end
+
+      rackcontroller_ips = Set.new
+      conn.request(:get, ['rackcontrollers']).each do |rctrl|
+        rctrl['ip_addresses'].each do |ip|
+          rackcontroller_ips.add(IPAddr.new(ip))
+        end
+      end
+
+      # reserved_ips | rackcontroller_ips
+      # Returns a new array by joining ary with other_ary, 
+      # excluding any duplicates and preserving the order from the original array.
+      if ips.disjoint? reserved_ips | rackcontroller_ips
+        subnets = conn.request(:get, ['subnets'])
+        ipranges = conn.request(:get, ['ipranges'])
+
+        ifaces = []
+
+        ips.each do |ip|
+          available = false
+          subnets.each do |subnet|
+            if IPAddr.new(subnet['cidr']).include?(ip)
+              ipranges.each do |range|
+                if range['subnet']['id'] == subnet['id']
+                  first = IPAddr.new(range['start_ip']).to_i
+                  last = IPAddr.new(range['end_ip']).to_i
+                  if (first..last) === ip.to_i
+                    logger.info("#{ip} is available.")
+                    available = true
+                    subnets.delete(subnet)
+                    subnet['ip'] = ip.to_s
+                    ifaces << subnet
+                    break
+                  end
+                end
+              end
+            end
+            break if available
+          end
+
+          if not available
+            logger.info("#{ip.to_s} does not belong to any subnet pre-defined.")
+            return false
+          end
+        end
+
+      else
+        logger.info("#{(ips & (reserved_ips | rackcontroller_ips)).to_a.join(', ')}\
+                    is already reserved.")
+        return false
+      end
+
+      return ifaces
+    end
+
+    def ipaddresses_reserved?(ip)
+      logger.info("Calling <#{__method__.to_s}>")
+      conn.request(:get, ['ipaddresses']).each do |address|
+        if address['ip'] == ip
+          logger.info("#{ip} is reserved.")
+          return true
+        end
+      end
+      return false
+    end
+
     def ipaddresses(op = nil, params = nil)
+      logger.info("Calling <#{__method__.to_s}>")
       case op
       when nil
         conn.request(:get, ['ipaddresses'])
       when 'reserve'
-        # sample = {
+        # params = {
         #   'subnet' => '10.1.2.0/24',
         #   'ip' => '10.1.2.8',
         #   'hostname' => 'hostname',
         #   'mac' => 'blahblah'
         # }
         default_param = { 'op' => op }
+        logger.info("#{params['ip']} is being reserved..")
         conn.request(:post, ['ipaddresses'], default_param.merge!(params))
       when 'release'
         # Gogetit.maas.ipaddresses('release', {'ip' => '10.1.2.8'})
-        # sample = {
+        # params = {
         #   'ip' => '10.1.2.8',
-        #   'hostname' => 'hostname',
-        #   'mac' => 'blahblah'
         # }
         default_param = { 'op' => op }
+        logger.info("#{params['ip']} is being released..")
         conn.request(:post, ['ipaddresses'], default_param.merge!(params))
       end
     end
 
+    def interfaces(url = [], params = {})
+      logger.info("Calling <#{__method__.to_s}>")
+
+      url.insert(0, 'nodes')
+      url.insert(2, 'interfaces')
+      # ['nodes', system_id, 'interfaces']
+
+      case params['op']
+      when nil
+        conn.request(:get, url)
+      when 'create_vlan'
+        logger.info("Creating a vlan interface for id: #{params['vlan']}..")
+        conn.request(:post, url, params)
+      when 'link_subnet'
+        logger.info("Linking a subnet for id: #{url[3]}..")
+        conn.request(:post, url, params)
+      when 'unlink_subnet'
+        logger.info("Linking a subnet for id: #{url[3]}..")
+        conn.request(:post, url, params)
+      end
+    end
+
     def delete_dns_record(name)
       logger.info("Calling <#{__method__.to_s}>")
-      id = nil
       conn.request(:get, ['dnsresources']).each do |item|
         if item['fqdn'] == name + '.' + get_domain
-          id = item['id']
+          logger.info("#{item['fqdn']} is being deleted..")
+          conn.request(:delete, ['dnsresources', item['id']])
         end
       end
-
-      if ! id.nil?
-        conn.request(:delete, ['dnsresources', id.to_s])
-      else
-        logger.warn('No such record found.')
-      end
     end
 
     def refresh_pods

data/lib/gogetit/util.rb

@@ -91,7 +91,7 @@ module Gogetit
 		end
 
     def wait_until_available(fqdn, logger)
-      until ping_available?(fqdn)
+      until ping_available?(fqdn, logger)
         logger.info("Calling <#{__method__.to_s}> for ping to be ready..")
         sleep 3
       end
@@ -104,8 +104,10 @@ module Gogetit
       logger.info("#{fqdn} is now available to ssh..")
     end
 
-    def ping_available?(fqdn)
-      `ping -c 1 -W 1 #{fqdn}`
+    def ping_available?(host, logger)
+      # host can be both IP and FQDN.
+      logger.info("Calling <#{__method__.to_s}> for #{host}")
+      `ping -c 1 -W 1 #{host}`
       $?.exitstatus == 0
     end
 
@@ -116,5 +118,19 @@ module Gogetit
         puts e
       end
     end
+
+    def check_ip_available(addresses, maas, logger)
+      logger.info("Calling <#{__method__.to_s}>")
+      # to do a ping test
+      addresses.each do |ip|
+        abort("#{ip} is already being used.") if ping_available?(ip, logger)
+      end
+      # to check with MAAS
+      ifaces = maas.ip_reserved?(addresses)
+      abort("one of #{addresses.join(', ')} is already being used.") \
+        unless ifaces
+      return ifaces
+    end
+
   end
 end

data/lib/gogetit/version.rb

@@ -1,3 +1,3 @@
 module Gogetit
-  VERSION = "0.2.1"
+  VERSION = "0.3.0"
 end

data/lib/providers/libvirt.rb

@@ -42,16 +42,70 @@ module Gogetit
         .value
     end
 
+    def generate_nics(ifaces, domain)
+      abort("There is no dns server specified for the gateway network.") \
+        unless ifaces[0]['dns_servers'][0]
+      abort("There is no gateway specified for the gateway network.") \
+        unless ifaces[0]['gateway_ip']
+
+      # It seems the first IP has to belong to the untagged VLAN in the Fabric.
+      abort("The first IP you entered does not belong to the untagged VLAN in the Fabric.") \
+        unless ifaces[0]['vlan']['name'] == 'untagged'
+
+      domain[:ifaces] = ifaces
+      domain[:nic] = []
+
+      ifaces.each_with_index do |iface,index|
+        if index == 0
+          if iface['vlan']['name'] == 'untagged'
+            nic = {
+              network: config[:default][:native_bridge],
+              portgroup: config[:default][:native_bridge]
+            }
+          elsif iface['vlan']['name'] != 'untagged'
+            nic = {
+              network: config[:default][:native_bridge],
+              portgroup: config[:default][:native_bridge] + '-' + iface['vlan']['vid'].to_s
+            }
+          end
+          domain[:nic].push(nic)
+        elsif index > 0
+          # Only if the fisrt interface has untagged VLAN,
+          # it will be configured with VLANs.
+          # This will not be hit as of now and might be deprecated.
+          if ifaces[0]['vlan']['name'] != 'untagged'
+            nic = {
+              network: config[:default][:native_bridge],
+              portgroup: config[:default][:native_bridge] + '-' + iface['vlan']['vid'].to_s
+            }
+            domain[:nic].push(nic)
+          end
+        end
+      end
+      return domain
+    end
+
     # subject.create(name: 'test01')
-    def create(name, conf_file = nil)
+    def create(name, options = nil)
       logger.info("Calling <#{__method__.to_s}>")
-      if maas.domain_name_exists?(name) or domain_exists?(name)
-        puts "Domain #{name} already exists! Please check both on MAAS and libvirt."
-        return false
+      abort("Domain #{name} already exists! Please check both on MAAS and libvirt.") \
+        if maas.domain_name_exists?(name) or domain_exists?(name)
+
+      domain = config[:libvirt][:specs][:default]
+      if options['ipaddresses']
+        ifaces = check_ip_available(options['ipaddresses'], maas, logger)
+        domain = generate_nics(ifaces, domain)
+      elsif options[:vlans]
+        #check_vlan_available(options[:vlans])
+      else
+        domain[:nic] = [
+          {
+            network: config[:default][:native_bridge],
+            portgroup: config[:default][:native_bridge]
+          }
+        ]
       end
 
-      conf_file ||= config[:default_provider_conf_file]
-      domain = symbolize_keys(YAML.load_file(conf_file))
       domain[:name] = name
       domain[:uuid] = SecureRandom.uuid
 
@@ -60,6 +114,65 @@ module Gogetit
 
       system_id = maas.get_system_id(domain[:name])
       maas.wait_until_state(system_id, 'Ready')
+
+      # To configure interfaces
+      if options['ipaddresses']
+
+        # It assumes you only have a physical interfaces.
+        interfaces = maas.interfaces([system_id])
+        maas.interfaces(
+          [system_id, interfaces[0]['id']],
+          {
+            'op' => 'unlink_subnet',
+            'id' => interfaces[0]['links'][0]['id']
+          }
+        )
+
+        maas.interfaces(
+          [system_id, interfaces[0]['id']],
+          {
+            'op' => 'link_subnet',
+            'mode' => 'STATIC',
+            'subnet' => ifaces[0]['id'],
+            'ip_address' => ifaces[0]['ip'],
+            'default_gateway' => 'True',
+            'force' => 'False'
+          }
+        )
+
+        if domain[:ifaces].length > 1
+          ifaces.shift
+
+          # VLAN configuration
+          ifaces.each_with_index do |iface,index|
+            params = {
+              'op' => 'create_vlan',
+              'vlan' => iface['vlan']['id'],
+              'parent' => interfaces[0]['id']
+            }
+            maas.interfaces([system_id], params)
+
+            interfaces = maas.interfaces([system_id])
+            interfaces.shift
+
+            maas.interfaces([system_id, interfaces[index]['id']],
+              {
+                'op' => 'link_subnet',
+                'mode' => 'STATIC',
+                'subnet' => ifaces[index]['id'],
+                'ip_address' => ifaces[index]['ip'],
+                'default_gateway' => 'False',
+                'force' => 'False'
+              }
+            )
+          end
+        end
+
+      elsif options[:vlans]
+        #check_vlan_available(options[:vlans])
+      else
+      end
+
       logger.info("Calling to deploy...")
       maas.conn.request(:post, ['machines', system_id], {'op' => 'deploy'})
       maas.wait_until_state(system_id, 'Deployed')
@@ -123,10 +236,10 @@ module Gogetit
       doc = define_volumes(doc, domain)
       doc = add_nic(doc, domain[:nic])
 
-      #print_xml(doc)
-      #volumes.each do |v|
-      #  print_xml(v)
-      #end
+      # print_xml(doc)
+      # volumes.each do |v|
+      #   print_xml(v)
+      # end
 
       return Oga::XML::Generator.new(doc).to_xml
     end

data/lib/providers/lxd.rb

@@ -1,5 +1,6 @@
 require 'hyperkit'
 require 'gogetit/util'
+require 'yaml'
 
 module Gogetit
   class GogetLXD
@@ -41,17 +42,179 @@ module Gogetit
       end
     end
 
-    def create(name, args = {})
+    def generate_args(options)
+      args = {}
+      args[:devices] = {}
+
+      ifaces = check_ip_available(options['ipaddresses'], maas, logger)
+      abort("There is no dns server specified for the gateway network.") \
+        unless ifaces[0]['dns_servers'][0]
+      abort("There is no gateway specified for the gateway network.") \
+        unless ifaces[0]['gateway_ip']
+      args[:ifaces] = ifaces
+      args[:config] = {
+        'user.network-config': {
+          'version' => 1,
+          'config' => [
+            {
+              'type' => 'nameserver',
+              'address' => ifaces[0]['dns_servers'][0]
+            }
+          ]
+        }
+      }
+
+      ifaces.each_with_index do |iface,index|
+        if index == 0
+          iface_conf = {
+            'type' => 'physical',
+            'name' => "eth#{index}",
+            'subnets' => [
+              {
+                'type' => 'static',
+                'ipv4' => true,
+                'address' => iface['ip'] + '/' + iface['cidr'].split('/')[1],
+                'gateway' => iface['gateway_ip'],
+                'mtu' => iface['vlan']['mtu'],
+                'control' => 'auto'
+              }
+            ]
+          }
+        elsif index > 0
+          if ifaces[0]['vlan']['name'] != 'untagged'
+            iface_conf = {
+              'type' => 'physical',
+              'name' => "eth#{index}",
+              'subnets' => [
+                {
+                  'type' => 'static',
+                  'ipv4' => true,
+                  'address' => iface['ip'] + '/' + iface['cidr'].split('/')[1],
+                  'mtu' => iface['vlan']['mtu'],
+                  'control' => 'auto'
+                }
+              ]
+            }
+          elsif ifaces[0]['vlan']['name'] == 'untagged'
+            iface_conf = {
+              'type' => 'vlan',
+              'name' => "eth0.#{iface['vlan']['vid'].to_s}",
+              'vlan_id' => iface['vlan']['vid'].to_s,
+              'vlan_link' => 'eth0',
+              'subnets' => [
+                {
+                  'type' => 'static',
+                  'ipv4' => true,
+                  'address' => iface['ip'] + '/' + iface['cidr'].split('/')[1],
+                  'mtu' => iface['vlan']['mtu'],
+                  'control' => 'auto'
+                }
+              ]
+            }
+          end
+        end
+
+        args[:config][:'user.network-config']['config'].push(iface_conf)
+      end
+
+      args[:config][:"user.network-config"] = \
+        YAML.dump(args[:config][:"user.network-config"])[4..-1]
+
+      # To configure devices
+      ifaces.each_with_index do |iface,index|
+        if index == 0
+          if iface['vlan']['name'] == 'untagged' # or vid == 0
+            args[:devices][:"eth#{index}"] = {
+              mtu: iface['vlan']['mtu'].to_s,   #This must be string
+              name: "eth#{index}",
+              nictype: 'bridged',
+              parent: config[:default][:native_bridge],
+              type: 'nic'
+            }
+          elsif iface['vlan']['name'] != 'untagged' # or vid != 0
+            args[:devices][:"eth#{index}"] = {
+              mtu: iface['vlan']['mtu'].to_s,   #This must be string
+              name: "eth#{index}",
+              nictype: 'bridged',
+              parent: config[:default][:native_bridge] + "-" + iface['vlan']['vid'].to_s,
+              type: 'nic'
+            }
+          end
+        # When ifaces[0]['vlan']['name'] == 'untagged' and index > 0,
+        # it does not need to generate more devices 
+        # since it will configure the IPs with tagged VLANs.
+        elsif ifaces[0]['vlan']['name'] != 'untagged'
+          args[:devices][:"eth#{index}"] = {
+            mtu: iface['vlan']['mtu'].to_s,   #This must be string
+            name: "eth#{index}",
+            nictype: 'bridged',
+            parent: config[:default][:native_bridge] + "-" + iface['vlan']['vid'].to_s,
+            type: 'nic'
+          }
+        end
+      end
+
+      return args
+    end
+
+    def create(name, options = {})
       logger.info("Calling <#{__method__.to_s}>")
-      if container_exists?(name) or maas.domain_name_exists?(name)
-        puts "Container #{name} already exists!"
-        return false
+      abort("Container or Hostname #{name} already exists!") \
+        if container_exists?(name) or maas.domain_name_exists?(name)
+
+      args = {}
+      if options['ipaddresses']
+        args = generate_args(options)
+      elsif options[:vlans]
+        #check_vlan_available(options[:vlans])
+      else
+        args[:profiles] ||= config[:lxd][:profiles]
       end
 
       args[:alias] ||= config[:lxd][:default_alias]
-      args[:profiles] ||= config[:lxd][:profiles]
       args[:sync] ||= true
+
       conn.create_container(name, args)
+      container = conn.container(name)
+
+      if options['vlans'] or options['ipaddresses']
+        container.devices = args[:devices].merge!(container.devices.to_hash)
+        conn.update_container(name, container)
+        # Fetch container object again
+        container = conn.container(name)
+
+        # Generate params to reserve IPs
+        args[:ifaces].each_with_index do |iface,index|
+          if index == 0
+            params = {
+              'subnet' => iface['cidr'],
+              'ip' => iface['ip'],
+              'hostname' => name,
+              'mac' => container[:expanded_config][:"volatile.eth#{index}.hwaddr"]
+            }
+          elsif index > 0
+            # if dot, '.', is used as a conjunction instead of '-', it fails ocuring '404 not found'.
+            # if under score, '_', is used as a conjunction instead of '-', it breaks MAAS DNS somehow..
+            if args[:ifaces][0]['vlan']['name'] == 'untagged'
+              params = {
+                'subnet' => iface['cidr'],
+                'ip' => iface['ip'],
+                'hostname' => 'eth0' + '-' + iface['vlan']['vid'].to_s  + '-' + name,
+                'mac' => container[:expanded_config][:"volatile.eth0.hwaddr"]
+              }
+            elsif args[:ifaces][0]['vlan']['name'] != 'untagged'
+              params = {
+                'subnet' => iface['cidr'],
+                'ip' => iface['ip'],
+                'hostname' => "eth#{index}" + '-' + name,
+                'mac' => container[:expanded_config][:"volatile.eth#{index}.hwaddr"]
+              }
+            end
+          end
+          maas.ipaddresses('reserve', params)
+        end
+      end
+
       conn.start_container(name, :sync=>"true")
 
       fqdn = name + '.' + maas.get_domain
@@ -62,12 +225,39 @@ module Gogetit
 
     def destroy(name, args = {})
       logger.info("Calling <#{__method__.to_s}>")
+
+      container = conn.container(name)
       args[:sync] ||= true
+
       if get_state(name) == 'Running'
         conn.stop_container(name, args)
       end
+
       wait_until_state(name, 'Stopped')
       conn.delete_container(name, args)
+
+      if container[:config][:"user.network-config"]
+        net_conf = YAML.load(
+          container[:config][:"user.network-config"]
+        )['config']
+        # To remove DNS configuration
+        net_conf.shift
+
+        net_conf.each do |nic|
+          if nic['subnets'][0]['type'] == 'static'
+            # It assumes we only assign a single subnet on a VLAN.
+            # Subnets in a VLAN, VLANs in a Fabric
+            ip = nic['subnets'][0]['address'].split('/')[0]
+
+            if maas.ipaddresses_reserved?(ip)
+              maas.ipaddresses('release', { 'ip' => ip })
+            end
+          end
+        end
+      end
+
+      # When multiple static IPs were reserved, it will not delete anything
+      # since they are deleted when releasing the IPs above.
       maas.delete_dns_record(name)
       logger.info("#{name} has been destroyed.")
       true

data/lib/sample_conf/gogetit.yml

@@ -1,18 +1,45 @@
 default:
   user: ubuntu
-etcd:
-  url: http://etcd.example.com:2379
+  native_bridge: my_favorite_bridge
+# etcd:
+#   url: http://etcd.example.com:2379
+# about MAAS
 maas:
   key: K:E:Y
   url: http://maas.example.com/MAAS/api/2.0
+# about LXD
 lxd:
   url: https://lxd.example.com:8443
   name: REMOTE_NAME
   default_alias: ubuntu-16.04
   profiles:
   - my_favorite_bridge
+# about Libvirt
 libvirt:
   url: qemu+ssh://WHOAMI@kvm.example.com/system
+  specs:
+    default:
+      vcpu: 1
+      memory: 1
+      disk:
+       root:
+        pool: ssd
+        capacity: 4
+    ceph:
+      vcpu: 4
+      memory: 8
+      disk:
+       root:
+        pool: ssd
+        capacity: 4
+       data:
+        -
+          pool: hdd
+          capacity: 8
+        -
+          pool: hdd
+          capacity: 8
+# about Chef
 chef:
   chef_repo_root: /SOMEWHERE/CHEF_REPO
   bootstrap:

data/lib/sample_conf/lxd_profile.yaml

@@ -0,0 +1,17 @@
+config:
+  #   raw.lxc: lxc.aa_profile=unconfined
+  #   security.privileged: "true"
+  user.user-data: |
+    #cloud-config
+    ssh_authorized_keys:
+     - ssh-rsa blahblah someone@somewhere
+    apt_mirror: http://repo.example.com/ubuntu/
+description: ""
+devices:
+  eth0:
+    mtu: "8954"
+    name: eth0
+    nictype: bridged
+    parent: somebr
+    type: nic
+name: somebr

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: gogetit
 version: !ruby/object:Gem::Version
-  version: 0.2.1
+  version: 0.3.0
 platform: ruby
 authors:
 - Don Draper
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2017-08-07 00:00:00.000000000 Z
+date: 2017-09-07 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: bundler
@@ -215,9 +215,8 @@ files:
 - lib/gogetit/version.rb
 - lib/providers/libvirt.rb
 - lib/providers/lxd.rb
-- lib/sample_conf/ceph.yml
-- lib/sample_conf/default.yml
 - lib/sample_conf/gogetit.yml
+- lib/sample_conf/lxd_profile.yaml
 - lib/template/disk.xml
 - lib/template/domain.xml
 - lib/template/nic.xml

data/lib/sample_conf/ceph.yml

@@ -1,17 +0,0 @@
-vcpu: 4
-memory: 8
-disk:
- root:
-  pool: ssd
-  capacity: 4
- data:
-  -
-    pool: hdd
-    capacity: 8
-  -
-    pool: hdd
-    capacity: 8
-nic:
- -
-   network: br-one
-   portgroup: all

data/lib/sample_conf/default.yml

@@ -1,10 +0,0 @@
-vcpu: 1
-memory: 1
-disk:
- root:
-  pool: ssd
-  capacity: 4
-nic:
- -
-   network: br-one
-   portgroup: all