fog 1.29.0 → 1.30.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 450665b606e103251fe5b57065161cb47a810cd0
-  data.tar.gz: e3a0cbc70aed6d1a5063a82fcf75eb0dbe0b8f57
+  metadata.gz: 541b4b4ec03b04b08598ee747100a9492dcf4e73
+  data.tar.gz: 3a1e2471be28574eb326288a1688dfd8af1f54a8
 SHA512:
-  metadata.gz: 9a3502e0de24115509ecf04e5b898db461e2147f4dcdbd366917a43ab1a074dc8430ba25ca5f52b63c21c2984af7bc9d96e66175d08a9f2874902b356ece19c8
-  data.tar.gz: 3e9d7533fb040cdafc27e583ab4fe58ebd4aee60cc7580aebc180d39afe334755c4610d5da0cd2c6b70803352318a1ab813cf6c98482872d0d2ee30cb91cfda4
+  metadata.gz: 1802c2977fcd068ecdc7a080555d258fc92a37418001ac50b51fb84c1509692d2dc197c5d2814bdd67175ca87827a83c931e9331cf89b285f09195b1796a2e35
+  data.tar.gz: 3203ecb30b25ce2dac15913c0fca2f5dc3beda29aaa339674740e0bc6fbe236f1099436eb7f0e742db232a50ad9ab9d561da1f1897bc16333bd6bfa9a6a419fe

data/CHANGELOG.md

@@ -1,3 +1,91 @@
+## 1.30.0 05/07/2015
+*Hash* 7de752a9554c0437a7112ec47829f52988ef1444
+
+Statistic     | Value
+------------- | --------:
+Collaborators | 2
+Downloads     | 9587472
+Forks         | 1436
+Open Issues   | 85
+Watchers      | 3479
+
+**MVP!** Josef Stribny
+
+#### [Brightbox]
+*   Remove provider specific helper test. thanks Paul Thornthwaite
+
+#### [GleSYS]
+*   server/create: sync required and optional args with API spec. thanks Tobias Nygren
+
+#### [Rackspace|Documentation]
+*   Documenting the Networks option. thanks James Belchamber
+
+#### [cloudstack|compute]
+*   Public IP Address model improvement. thanks Atsushi Sasaki
+
+#### [dynect|dns]
+*   include SOA and NS records in #records response. thanks Richard Henning
+
+#### [misc]
+*   Add missing attributes to stack and resource. thanks Bill Wei
+*   Supporting openstack auth v3 - project/user domain for compute/network. thanks Christophe Larsonneur
+*   Introduce disk-snapshot and VM shutdown functionality. thanks Evgeny Yurchenko
+*   image_pool request introduced. thanks Evgeny Yurchenko
+*   Fixes #3543, adds support for cpuHotAdd and memoryHotAdd. thanks Francois Herbert
+*   update CONTRIBUTING to point to design document in wiki. thanks Isaac Hollander McCreery
+*   Hash#reject! returns nil if no changes were made; it is not correct behavior in assignment. thanks Jaroslav Barton
+*   Fix the 'redirect to master' functionality. When authenticating towards a slave host, the error HOST_IS_SLAVE is returned by the slave host together with the IP address of the master host. Previously this resulted in an InvalidLogin exception which was hiding the actual cause. Now it results in an HostIsSlave exception which is then used to redirect the connection to the master host in case :xenserver_redirect_to_master is true. thanks Jonas Kongslund
+*   Remove libvirt provider specific files. thanks Josef Stribny
+*   Add modular dependency on extracted fog-libvirt. thanks Josef Stribny
+*   Remove libvirt bin file. thanks Josef Stribny
+*   Travis: Install libvirt header files to compile ruby-libvirt. thanks Josef Stribny
+*   Remove dependency on fog-libvirt altogether. thanks Josef Stribny
+*   Remove libvirt tests from fog. thanks Josef Stribny
+*   Added vsphere_debug flag for stderr debugging. thanks Lukas Zapletal
+*   add Linode Singapore DC to mock data. thanks Marques Johansson
+*   Set proper content type and clear operations after request. thanks Matt Darby
+*   Fixing Unrecognized argument: path warning message. thanks Miguel Martinez
+*   name and storage profile VM configuraiton during vApp instantiation. thanks Miguel Martinez
+*   vcloud-token on instantiation. thanks Miguel Martinez
+*   Remove Google code. thanks Paulo Henrique Lopes Ribeiro
+*   Remove Google tests. thanks Paulo Henrique Lopes Ribeiro
+*   Remove Google main file. thanks Paulo Henrique Lopes Ribeiro
+*   Remove Google Binary. thanks Paulo Henrique Lopes Ribeiro
+*   Remove mocks for Google. thanks Paulo Henrique Lopes Ribeiro
+*   Depend on fog-google. thanks Paulo Henrique Lopes Ribeiro
+*   Force version 0.0.2 of fog-google. thanks Paulo Henrique Lopes Ribeiro
+*   add firmware support for vsphere. thanks Samuel Keeley
+*   Allow passing query options to snapshots endpoint. thanks Sean Handley
+*   Change openstack_prj to openstack_project. thanks TerryHowe
+*   Host and port options support for create_temp_url. thanks biomancer
+*   Fix hp storage tests. thanks biomancer
+*   Use sane default ports for get_object_http(s)_url. thanks biomancer
+*   Compatibility fix for ruby 1.8.7 that does not have URI.encode_www_form method. thanks biomancer
+*   Make port option overridable for get_object_http(s)_url. thanks biomancer
+*   Fixing indentation. thanks eyurchenko
+*   Tests added for vm_disk_snapshot, vm_shutdown and image_pool requests. thanks eyurchenko
+*   Remove empty Mock classes. thanks eyurchenko
+*   Return to "standard" template name. thanks eyurchenko
+*   Remove puts (leftovers from debugging tests). thanks eyurchenko
+
+#### [opennebula]
+*   added get_by_name for network model. thanks ller
+*   identify the network by id or name within a flavor. thanks ller
+*   fixed networks.get_by_name. thanks ller
+*   changed list_networks mock names. thanks ller
+*   added mock for template_pool. thanks ller
+*   added flavor tests. thanks ller
+*   flavor: fixed get_vcpu and get_memory; added tests. thanks ller
+*   removed dead debug code. thanks ller
+*   added better string escaping for schedule requirements and raw sections. thanks ller
+*   fix: escape RAW parameter the right way. thanks ller
+*   fixed networks.get_by_name, added tests. thanks ller
+
+#### [vsphere|compute]
+*   Find virtual machines and templates under a folder. thanks Kevin Menard
+*   Fixed spacing. thanks Kevin Menard
+
+
 ## 1.29.0 04/02/2015
 *Hash* 90e96f36e6f70a12f0c6794a149231c337293887
 

data/CONTRIBUTING.md

@@ -29,7 +29,7 @@ New contributors are always welcome, when it doubt please ask questions. We stri
 
 Before we go into more detail, please be aware that Fog is going through some growing pains right now.  A lot of things are moving right now, and things may not be documented as well as we would like.
 
-It might be wise to check out the [Roadmap](https://github.com/fog/fog/wiki/Roadmap) in the Wiki for more information.
+It might be wise to check out the [Roadmap](https://github.com/fog/fog/wiki/Roadmap) and [Design Document](https://github.com/fog/fog/wiki/fog-design-document) in the Wiki for more information.
 
 For right now, there are three major things in motion:
 

data/fog.gemspec

@@ -11,8 +11,8 @@ Gem::Specification.new do |s|
   ## If your rubyforge_project name is different, then edit it and comment out
   ## the sub! line in the Rakefile
   s.name              = "fog"
-  s.version           = "1.29.0"
-  s.date              = "2015-04-02"
+  s.version           = "1.30.0"
+  s.date              = "2015-05-07"
   s.rubyforge_project = "fog"
 
   ## Make sure your summary is short. The description may be as long
@@ -56,6 +56,7 @@ Gem::Specification.new do |s|
   s.add_dependency("fog-aws", "~> 0.0")
   s.add_dependency("fog-brightbox", "~> 0.4")
   s.add_dependency("fog-ecloud")
+  s.add_dependency("fog-google", ">= 0.0.2")
   s.add_dependency("fog-local")
   s.add_dependency("fog-powerdns", ">= 0.1.1")
   s.add_dependency("fog-profitbricks")
@@ -71,7 +72,6 @@ Gem::Specification.new do |s|
 
   s.add_development_dependency("docker-api", ">= 1.13.6")
   s.add_development_dependency("fission")
-  s.add_development_dependency("google-api-client", "~> 0.6", ">= 0.6.2")
   s.add_development_dependency("minitest")
   s.add_development_dependency("minitest-stub-const")
   s.add_development_dependency("opennebula")
@@ -85,10 +85,6 @@ Gem::Specification.new do |s|
   s.add_development_dependency("thor")
   s.add_development_dependency("yard")
 
-  if ENV["FOG_USE_LIBVIRT"]
-    s.add_development_dependency("ruby-libvirt","~> 0.5.0")
-  end
-
   s.files = `git ls-files`.split("\n")
   s.test_files = `git ls-files -- {spec,tests}/*`.split("\n")
 end

data/lib/fog.rb

@@ -39,7 +39,6 @@ require 'fog/hp'
 require 'fog/ibm'
 require 'fog/internet_archive'
 require 'fog/joyent'
-require 'fog/libvirt'
 require 'fog/linode'
 require 'fog/local'
 require 'fog/bare_metal_cloud'

data/lib/fog/bin.rb

@@ -71,7 +71,6 @@ require 'fog/bin/hp'
 require 'fog/bin/ibm'
 require 'fog/bin/internet_archive'
 require 'fog/bin/joyent'
-require 'fog/bin/libvirt'
 require 'fog/bin/linode'
 require 'fog/bin/local'
 require 'fog/bin/bare_metal_cloud'

data/lib/fog/cloudstack/models/compute/public_ip_address.rb

@@ -2,51 +2,94 @@ module Fog
   module Compute
     class Cloudstack
       class PublicIpAddress < Fog::Model
-        identity  :id,                    :aliases => 'id'
-        attribute :network_id,            :aliases => 'networkid'
-        attribute :associated_network_id, :aliases => 'associatednetworkid'
-        attribute :physical_network_id,   :aliases => 'physicalnetworkid'
-        attribute :ip_address,            :aliases => 'ipaddress'
-        attribute :mac_address,           :aliases => 'macaddress'
-        attribute :state,                 :aliases => 'state'
-        attribute :traffic_type,          :aliases => 'traffictype'
-        attribute :is_default,            :aliases => 'isdefault', :type => :boolean
-        attribute :is_source_nat,         :aliases => 'issourcenat', :type => :boolean
-        attribute :is_static_nat,         :aliases => 'isstaticnat', :type => :boolean
-        attribute :is_system,             :aliases => 'issytem', :type => :boolean
-        attribute :is_portable,           :aliases => 'isportable', :type => :boolean
-        attribute :allocated,             :aliases => 'allocated', :type => :time
-        attribute :zone_id,               :aliases => 'zone_id'
-        attribute :domain_id,             :aliases => 'domain_id'
-        attribute :tags,                  :type => :array
-        attribute :type
+        identity  :id
 
-       def save
-          requires :display_text, :name
+        attribute :account
+        attribute :allocated,               :type => :time
+        attribute :associated_network_id,                      :aliases => 'associatednetworkid'
+        attribute :associated_network_name,                    :aliases => 'associatednetworkname'
+        attribute :domain
+        attribute :domain_id,                                  :aliases => 'domainid'
+        attribute :for_virtual_network,     :type => :boolean, :aliases => 'forvirtualnetwork'
+        attribute :ip_address,                                 :aliases => 'ipaddress'
+        attribute :is_portable,             :type => :boolean, :aliases => 'isportable'
+        attribute :is_source_nat,           :type => :boolean, :aliases => 'issourcenat'
+        attribute :is_static_nat,           :type => :boolean, :aliases => 'isstaticnat'
+        attribute :is_system,               :type => :boolean, :aliases => 'issystem'
+        attribute :network_id,              :type => :boolean, :aliases => 'networkid'
+        attribute :physical_network_id,                        :aliases => 'physicalnetworkid'
+        attribute :project
+        attribute :project_id,                                 :aliases => 'projectid'
+        attribute :purpose
+        attribute :state
+        attribute :server_display_name,                        :aliases => 'virtualmachinedisplayname'
+        attribute :server_id,                                  :aliases => 'virtualmachineid'
+        attribute :server_name,                                :aliases => 'virtualmachinename'
+        attribute :vlan_id,                                    :aliases => 'vlanid'
+        attribute :vlan_name,                                  :aliases => 'vlanname'
+        attribute :server_ip_address,                          :aliases => 'vmipaddress'
+        attribute :vpc_id,                                     :aliases => 'vpcid'
+        attribute :zone_id,                                    :aliases => 'zoneid'
+        attribute :zone_name,                                  :aliases => 'zonename'
+        attribute :tags,                    :type => :array
+        attribute :job_id,                                     :aliases => 'jobid'   # only on associate
 
-          options = {
-            'displaytext' => display_text,
-            'name'        => name,
-            'customized'  => is_customized,
-            'disksize'    => disk_size,
-            'domain_id'   => domain_id,
-            'storagetype' => storage_type,
-            'tags'        => tags
-          }
+        def initialize(attributes = {})
+          # assign server first to prevent race condition with persisted?
+          self.server = attributes.delete(:server)
+          super
+        end
 
-          response = service.associate_ip_address(options)
-          merge_attributes(response['associateipaddressresponse'])
+        def ready?
+          state == 'Allocated'
         end
 
         def destroy
-          requires :id
+          requires :identity
+          service.disassociate_ip_address('id' => id)
+          true
+        end
 
-          response = service.disassociate_ip_address('id' => id )
-          success_status = response['disassociateipaddressresponse']['success']
+        def server=(new_server)
+          @server = new_server
+          if persisted?
+            if !server_id.nil? && (new_server.nil? || server_id != new_server.id)
+              service.disable_static_nat('ipaddressid' => id)
+              self.server_display_name = nil
+              self.server_id = nil
+              self.server_name = nil
+              self.server_ip_address = nil
+              self.is_static_nat = false
+            end
+            unless new_server.nil?
+              service.enable_static_nat(
+                'ipaddressid' => id, 'virtualmachineid' => new_server.id)
+            end
+          end
+        end
 
-          success_status == 'true'
+        def server
+          service.servers.get(server_id)
         end
 
+        def save
+          raise Fog::Errors::Error.new('Resaving an existing object may create a duplicate') if persisted?
+          options = {
+            'account'    => account,
+            'domainid'   => domain_id,
+            'isportable' => is_portable,
+            'networkid'  => network_id,
+            'projectid'  => project_id,
+            'vpcid'      => vpc_id,
+            'zoneid'     => zone_id,
+          }
+          response = service.associate_ip_address(options)
+          merge_attributes(response['associateipaddressresponse'])
+          if @server
+            self.server = @server
+          end
+          true
+        end
       end
     end
   end

data/lib/fog/cloudstack/models/compute/public_ip_addresses.rb

@@ -1,5 +1,5 @@
 require 'fog/core/collection'
-require 'fog/cloudstack/models/compute/address'
+require 'fog/cloudstack/models/compute/public_ip_address'
 
 module Fog
   module Compute
@@ -14,10 +14,12 @@ module Fog
         end
 
         def get(address_id)
-          options = { 'id' => address_id }
-          response = service.list_public_ip_addresses(options)
-          public_ip_addresses = response["listpublicipaddressesresponse"]["publicipaddress"].first
-          new(public_ip_addresses)
+          response = service.list_public_ip_addresses('id' => address_id)
+          if public_ip_address = response["listpublicipaddressesresponse"]["publicipaddress"].first
+            new(public_ip_address)
+          end
+        rescue Fog::Compute::Cloudstack::BadRequest
+          nil
         end
       end
     end

data/lib/fog/cloudstack/requests/compute/associate_ip_address.rb

@@ -22,8 +22,9 @@ module Fog
 
       class Mock
         def associate_ip_address(*args)
+          public_ip_address_id = Fog::Cloudstack.uuid
           public_ip_address = {
-            "id"                => "f2f2f2f2-f2f2-f2f2-f2f2-f2f2f2f2f2",
+            "id"                => public_ip_address_id,
             "ipaddress"         => "192.168.200.3",
             "allocated"         => "2014-12-22T22:32:39+0000",
             "zoneid"            => "0e276270-7950-4483-bf21-3dc897dbe08a",
@@ -48,7 +49,6 @@ module Fog
           {'associateipaddressresponse' => public_ip_address}
         end
       end
-
     end
   end
 end

data/lib/fog/cloudstack/requests/compute/disable_static_nat.rb

@@ -19,6 +19,32 @@ module Fog
         end
       end
 
+      class Mock
+        def disable_static_nat(*args)
+          ip_address_id = args[0].is_a?(Hash) ? args[0]['ipaddressid'] : args[0]
+
+          address = self.data[:public_ip_addresses][ip_address_id]
+
+          unless address
+            raise Fog::Compute::Cloudstack::BadRequest.new(
+"Unable to execute API command disablestaticnat due to invalid value. \
+Invalid parameter ipaddressid value=#{ip_address_id} due to incorrect long value format, \
+or entity does not exist or due to incorrect parameter annotation for the field in api cmd class.")
+          end
+
+          if address['virtualmachineid'].nil?
+            raise Fog::Compute::Cloudstack::BadRequest.new(
+"Specified IP address id is not associated with any vm Id")
+          end
+
+          address.merge!(
+            'virtualmachineid'          => nil,
+            'virtualmachinname'         => nil,
+            'virtualmachinedisplayname' => nil
+          )
+          {'enablestaticnatresponse' => {'success' => 'true'}}
+        end
+      end
     end
   end
 end

data/lib/fog/cloudstack/requests/compute/disassociate_ip_address.rb

@@ -21,7 +21,7 @@ module Fog
 
       class Mock
         def disassociate_ip_address(*args)
-          public_ip_address_id = options['id']
+          public_ip_address_id = args[0].is_a?(Hash) ? args[0]['id'] : args[0]
           if self.data[:public_ip_addresses][public_ip_address_id]
             self.data[:public_ip_addresses].delete(public_ip_address_id)
             { "disassociateipaddressresponse" => { "success" => "true" }}

data/lib/fog/cloudstack/requests/compute/enable_static_nat.rb

@@ -20,6 +20,49 @@ module Fog
         end
       end
 
+      class Mock
+        def enable_static_nat(*args)
+          ip_address_id = nil
+          virtual_machine_id = nil
+          if args[0].is_a? Hash
+            ip_address_id = args[0]['ipaddressid']
+            virtual_machine_id = args[0]['virtualmachineid']
+          else
+            ip_address_id = args[0]
+            virtual_machine_id = args[1]
+          end
+
+          server = self.data[:servers][virtual_machine_id]
+          address = self.data[:public_ip_addresses][ip_address_id]
+
+          unless server
+            raise Fog::Compute::Cloudstack::BadRequest.new(
+"Unable to execute API command enablestaticnat due to invalid value. \
+Invalid parameter virtualmachineid value=#{virtual_machine_id} due to incorrect long value format, \
+or entity does not exist or due to incorrect parameter annotation for the field in api cmd class.")
+          end
+
+          unless address
+            raise Fog::Compute::Cloudstack::BadRequest.new(
+"Unable to execute API command enablestaticnat due to invalid value. \
+Invalid parameter ipaddressid value=#{ip_address_id} due to incorrect long value format, \
+or entity does not exist or due to incorrect parameter annotation for the field in api cmd class.")
+          end
+
+          unless address['virtualmachineid'].nil?
+            raise Fog::Compute::Cloudstack::BadRequest.new(
+"Failed to enable static nat for the ip address id=#{ip_address_id} \
+as vm id=#{virtual_machine_id} is already associated with ip id=#{ip_address_id}")
+          end
+
+          address.merge!(
+            'virtualmachineid'          => server['id'],
+            'virtualmachinname'         => server['name'],
+            'virtualmachinedisplayname' => server['displayname']
+          )
+          {'enablestaticnatresponse' => {'success' => 'true'}}
+        end
+      end
     end
   end
 end

data/lib/fog/cloudstack/requests/compute/list_public_ip_addresses.rb

@@ -23,12 +23,22 @@ module Fog
       end
 
       class Mock
-        def list_public_ip_addresses(*arg)
-          public_ip_addresses = self.data[:public_ip_addresses]
-          { "listpublicipaddressesresponse" => { "count"=> public_ip_addresses.count, "publicipaddress"=> public_ip_addresses.values } }
+        def list_public_ip_addresses(*args)
+          public_ip_address_id = args[0].is_a?(Hash) ? args[0]['id'] : nil
+          if public_ip_address_id
+            public_ip_addresses = [self.data[:public_ip_addresses][public_ip_address_id]]
+          else
+            public_ip_addresses = self.data[:public_ip_addresses].values
+          end
+
+          {
+            'listpublicipaddressesresponse' => {
+              'count' => public_ip_addresses.size,
+              'publicipaddress' => public_ip_addresses
+            }
+          }
         end
       end
-
     end
   end
 end