foreman_reserve 0.1.7 → 0.1.8

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: edaeb4e6dd24d5ca57297860ae94f8e49f47ac39
-  data.tar.gz: 79c1507a10634769373a92ba9f9d256d862ae155
+  metadata.gz: bb84c64163abd43a3594ba08ec85ba60af1422ad
+  data.tar.gz: be9a4bb75f9d66261e45731fb109876b3ad20bae
 SHA512:
-  metadata.gz: 1c929a0f33b04449f517359cd7cb292cf1abb9e41fb1968e114e6fd6b1ed3d814c653da9be2715a26fe0510c4691eee39c47b77209627ffcddf03aa5a6c1da55
-  data.tar.gz: 63c57ee74c1a651740656ed52defbc1ee28de60e869e04e1b755424693ceecd7153c42f97ac52c38b8fc26aea2c37e41ad18145f02acb2a187822e258bbe5b9d
+  metadata.gz: 8dcbbb655cd823f700c8d4d29dbbcc8763fb893f065299af666935cb84c7fe6c32f64f637170c5de2f37e381e5dffc406d646ebfeefa570041984947d4ea58e3
+  data.tar.gz: 55e934ec17a96fa945b8f589574f0fd5923537c62a2221a51e1ffd9f3c0be9e52c882f59ffe8321202dbefa3700987c3af7aaaa7602e28bb87b13039a7d76fbf

data/VERSION

@@ -1 +1 @@
-0.1.7
+0.1.8

data/app/controllers/foreman_reserve/api/hosts_controller.rb

@@ -7,14 +7,12 @@ module Api
       # @param query [String] Query to filter the hosts with
       def get_reserved(query='')
         hosts = User.current.admin? ? Host::Managed : Host::Managed.my_hosts
-        hosts.search_for(query)
-        hosts.each do |host|
+        hosts = hosts.search_for(query)
+        hosts.find_all do |host|
           params = host.info()['parameters']
-          if params and
+          params and
               params.key?('RESERVED') and
               params['RESERVED'] != 'false'
-            host
-          end
         end
       end
 
@@ -22,14 +20,12 @@ module Api
       # @param query [String] Query to filter the hosts with
       def get_free(query='')
         hosts = User.current.admin? ? Host::Managed : Host::Managed.my_hosts
-        hosts.search_for(query)
-        hosts.each do |host|
+        hosts = hosts.search_for(query)
+        hosts.find_all do |host|
           params = host.info()['parameters']
-          if params and
-              params.has_key('RESERVED') and
+          params and
+              params.key?('RESERVED') and
               params['RESERVED'] == 'false'
-            host
-          end
         end
       end
 
@@ -72,9 +68,6 @@ module Api
           return not_found if potential_hosts.empty?
           return not_acceptable if potential_hosts.count < amount
           @hosts = potential_hosts[0..(amount-1)].each do |host|
-            logger.error host.class.name
-            logger.error host.class.instance_methods(false)
-            logger.error Host.class.instance_methods(false)
             host.reserve!(reason)
           end
         ensure

data/foreman_reserve.gemspec

@@ -2,7 +2,7 @@
 
 Gem::Specification.new do |s|
   s.name = "foreman_reserve"
-  s.version = "0.1.7"
+  s.version = "0.1.8"
 
   s.required_rubygems_version = Gem::Requirement.new(">= 0") if s.respond_to? :required_rubygems_version=
   s.authors = ["Joseph Mitchell Magen", "David Caro"]

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: foreman_reserve
 version: !ruby/object:Gem::Version
-  version: 0.1.7
+  version: 0.1.8
 platform: ruby
 authors:
 - Joseph Mitchell Magen