azure-armrest 0.3.0 → 0.3.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: d6e051e8b1aafc0e56dead25b4b049a1db77991d
-  data.tar.gz: 6cd67f1ba82fc44da524373ae20f68b6e0a29737
+  metadata.gz: 81c1664938977f85f9b79045d208233777851e8a
+  data.tar.gz: 4418b381009c2c0274e7fd45f217ca114a44ae8e
 SHA512:
-  metadata.gz: 2243372f9f4b26501539f91db5d91be939454c7608a6feee25d9addd8aae16239c986b76555286a1f19e384ce4fc7d77511701f30ae5104145b8da3003671bc2
-  data.tar.gz: c139739090ad6fa30c5e86e61197db7cf09c7695d79ad4ee6aa728281294653234728cb782542f7a0633b1deede6d10258107211d51588ff1afcc2964aa7ee03
+  metadata.gz: b101ce1ef690827027dfd59ab2c509064afd8f488cc96b6879466ae29d9dca2b699a4fa6227022b5594e8f575f173f14e1a6c66805baa0d8cbdeee377c9ea236
+  data.tar.gz: 274f153ce11e6628d91bdf0bc21ef0dc15ad88c2bdf801705a5cdc14d159d3fff5ef1a61dbe370fb57d8a56b61facee9d67d89569fc9be0e6cc2a661bf1dd1a7

data/.travis.yml

@@ -1,8 +1,7 @@
 language: ruby
 rvm:
-- "2.0"
-- "2.1"
-- "2.2"
+- "2.2.5"
+- "2.3.1"
 - ruby-head
 - jruby-head
 matrix:

data/CHANGES

@@ -1,3 +1,12 @@
+= 0.3.1 - 14-Jul-2016
+* Now uses the parallel gem and thread pools for those few methods where
+  we were using threaded methods internally in order to reduce errors
+  caused by too many requests.
+* Updated the rest-client dependency to 2.0.0 now that it has been released.
+* Updated travis.yml to only test 2.2.5 and 2.3.1 for now. Partly due to a
+  change in the activesupport gem.
+* One rspec fix for 2.3.x.
+
 = 0.3.0 - 28-Jun-2016
 * The configuration handling has been moved into its own class called
   Azure::Armrest::Configuration. You can call this explicitly using :new,

data/azure-armrest.gemspec

@@ -20,12 +20,13 @@ behind the scenes.
   EOF
 
   spec.add_dependency 'json'
-  spec.add_dependency 'rest-client', '~> 2.0.0.rc'
+  spec.add_dependency 'rest-client', '~> 2.0.0'
   spec.add_dependency 'cache_method', '~> 0.2.7'
   spec.add_dependency 'azure-signature', '~> 0.2.0'
   spec.add_dependency 'activesupport', '>= 1.2.0'
   spec.add_dependency 'nokogiri', '~> 1.6.0'
   spec.add_dependency 'addressable', '~> 2.4.0'
+  spec.add_dependency 'parallel', '~> 1.9.0'
 
   spec.add_development_dependency 'bundler'
   spec.add_development_dependency 'rake'

data/lib/azure/armrest.rb

@@ -2,6 +2,7 @@ require 'rest-client'
 require 'json'
 require 'thread'
 require 'addressable'
+require 'parallel'
 
 # The Azure module serves as a namespace.
 module Azure

data/lib/azure/armrest/availability_set_service.rb

@@ -9,10 +9,6 @@ module Azure
       def initialize(configuration, options = {})
         super(configuration, 'availabilitySets', 'Microsoft.Compute', options)
       end
-
-      def list_all
-        list_in_all_groups
-      end
     end # AvailabilitySetService
   end # Armrest
 end # Azure

data/lib/azure/armrest/model/storage_account.rb

@@ -215,14 +215,12 @@ module Azure
       def all_blobs(key = nil)
         key ||= properties.key1
         array = []
-        threads = []
+        mutex = Mutex.new
 
-        containers(key).each do |container|
-          threads << Thread.new(container, key) { |c, k| array << blobs(c.name, k) }
+        Parallel.each(containers(key), :in_threads => 10) do |container|
+          mutex.synchronize { array << blobs(container.name, key) }
         end
 
-        threads.each(&:join)
-
         array.flatten
       end
 

data/lib/azure/armrest/resource_group_based_service.rb

@@ -75,23 +75,20 @@ module Azure
         @model_class ||= Object.const_get(self.class.to_s.sub(/Service$/, ''))
       end
 
-      # Aggregate resources from each group
-      # To be used in the case that API does not support list_all with one call
+      # Aggregate resources from all resource groups.
+      #
+      # To be used in the cases where the API does not support list_all with one call.
+      #
       def list_in_all_groups
         array = []
-        threads = []
         mutex = Mutex.new
 
-        resource_groups.each do |rg|
-          threads << Thread.new(rg.name) do |group|
-            response = rest_get(build_url(group))
-            results = JSON.parse(response)['value'].map { |hash| model_class.new(hash) }
-            mutex.synchronize { array << results } unless results.blank?
-          end
+        Parallel.each(list_resource_groups, :in_threads => 10) do |rg|
+          response = rest_get(build_url(rg.name))
+          results = JSON.parse(response)['value'].map { |hash| model_class.new(hash) }
+          mutex.synchronize { array << results } unless results.blank?
         end
 
-        threads.each(&:join)
-
         array.flatten
       end
     end

data/lib/azure/armrest/storage_account_service.rb

@@ -246,30 +246,25 @@ module Azure
       #
       def get_private_images(storage_accounts)
         results = []
-        threads = []
         mutex = Mutex.new
 
-        storage_accounts.each do |lstorage_account|
-          threads << Thread.new(lstorage_account) do |storage_account|
-            key = get_account_key(storage_account)
+        Parallel.each(storage_accounts, :in_threads => 10) do |storage_account|
+          key = get_account_key(storage_account)
 
-            storage_account.all_blobs(key).each do |blob|
-              next unless File.extname(blob.name).casecmp('.vhd') == 0
-              next unless blob.properties.lease_state.casecmp('available') == 0
+          storage_account.all_blobs(key).each do |blob|
+            next unless File.extname(blob.name).casecmp('.vhd') == 0
+            next unless blob.properties.lease_state.casecmp('available') == 0
 
-              blob_properties = storage_account.blob_properties(blob.container, blob.name, key)
-              next unless blob_properties.respond_to?(:x_ms_meta_microsoftazurecompute_osstate)
-              next unless blob_properties.x_ms_meta_microsoftazurecompute_osstate.casecmp('generalized') == 0
+            blob_properties = storage_account.blob_properties(blob.container, blob.name, key)
+            next unless blob_properties.respond_to?(:x_ms_meta_microsoftazurecompute_osstate)
+            next unless blob_properties.x_ms_meta_microsoftazurecompute_osstate.casecmp('generalized') == 0
 
-              mutex.synchronize do
-                results << blob_to_private_image_object(storage_account, blob, blob_properties)
-              end
+            mutex.synchronize do
+              results << blob_to_private_image_object(storage_account, blob, blob_properties)
             end
           end
         end
 
-        threads.each(&:join)
-
         results
       end
 

data/lib/azure/armrest/version.rb

@@ -1,5 +1,5 @@
 module Azure
   module Armrest
-    VERSION = '0.3.0'.freeze
+    VERSION = '0.3.1'.freeze
   end
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: azure-armrest
 version: !ruby/object:Gem::Version
-  version: 0.3.0
+  version: 0.3.1
 platform: ruby
 authors:
 - Daniel J. Berger
@@ -11,7 +11,7 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2016-06-28 00:00:00.000000000 Z
+date: 2016-07-14 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: json
@@ -33,14 +33,14 @@ dependencies:
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 2.0.0.rc
+        version: 2.0.0
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 2.0.0.rc
+        version: 2.0.0
 - !ruby/object:Gem::Dependency
   name: cache_method
   requirement: !ruby/object:Gem::Requirement
@@ -111,6 +111,20 @@ dependencies:
     - - "~>"
       - !ruby/object:Gem::Version
         version: 2.4.0
+- !ruby/object:Gem::Dependency
+  name: parallel
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: 1.9.0
+  type: :runtime
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: 1.9.0
 - !ruby/object:Gem::Dependency
   name: bundler
   requirement: !ruby/object:Gem::Requirement