From 201f59c37624e9acc2de53b2ae397efeaa2715ba Mon Sep 17 00:00:00 2001
From: Brian Cain <brianccain@gmail.com>
Date: Mon, 14 Sep 2015 21:45:33 -0700
Subject: [PATCH] Add verbose to methods

---
 lib/vmfloaty.rb        | 26 +++++++++++++-------------
 lib/vmfloaty/auth.rb   | 12 ++++++------
 lib/vmfloaty/http.rb   |  4 ++--
 lib/vmfloaty/pooler.rb | 36 ++++++++++++++++++------------------
 4 files changed, 39 insertions(+), 39 deletions(-)

diff --git a/lib/vmfloaty.rb b/lib/vmfloaty.rb
index 39049ea..deab8ec 100644
--- a/lib/vmfloaty.rb
+++ b/lib/vmfloaty.rb
@@ -35,11 +35,11 @@ class Vmfloaty
         pass = password "Enter your password please:", '*'
 
         unless options.token
-          token = Auth.get_token(url, user, pass)
+          token = Auth.get_token(verbose, url, user, pass)
         end
 
         unless os_types.nil?
-          response = Pooler.retrieve(os_types, token, url)
+          response = Pooler.retrieve(verbose, os_types, token, url)
           puts response
         else
           puts 'You did not provide an OS to get'
@@ -60,7 +60,7 @@ class Vmfloaty
         filter = options.filter
         url = options.url ||= config['url']
 
-        os_list = Pooler.list(url, filter)
+        os_list = Pooler.list(verbose, url, filter)
         puts os_list
       end
     end
@@ -78,7 +78,7 @@ class Vmfloaty
         url = options.url ||= config['url']
         hostname = options.hostname
 
-        query = Pooler.query(url, hostname)
+        query = Pooler.query(verbose, url, hostname)
         puts query
       end
     end
@@ -102,7 +102,7 @@ class Vmfloaty
         tags = options.tags
         token = options.token
 
-        res_body = Pooler.modify(url, hostname, token, lifetime, tags)
+        res_body = Pooler.modify(verbose, url, hostname, token, lifetime, tags)
         puts res_body
       end
     end
@@ -120,7 +120,7 @@ class Vmfloaty
         hosts = options.hosts
         url = options.url ||= config['url']
 
-        Pool.delete(url, hosts)
+        Pool.delete(verbose, url, hosts)
       end
     end
 
@@ -139,7 +139,7 @@ class Vmfloaty
         hostname = options.hostname
         token = options.token
 
-        res_body = Pooler.snapshot(url, hostname, token)
+        res_body = Pooler.snapshot(verbose, url, hostname, token)
         puts res_body
       end
     end
@@ -161,7 +161,7 @@ class Vmfloaty
         token = options.token
         snapshot_sha = options.snapshot
 
-        res_body = Pooler.revert(url, hostname, token, snapshot_sha)
+        res_body = Pooler.revert(verbose, url, hostname, token, snapshot_sha)
         puts res_body
       end
     end
@@ -177,7 +177,7 @@ class Vmfloaty
         verbose = options.verbose || config['verbose']
         url = options.url ||= config['url']
 
-        status = Pooler.status(url)
+        status = Pooler.status(verbose, url)
         puts status
       end
     end
@@ -193,7 +193,7 @@ class Vmfloaty
         verbose = options.verbose || config['verbose']
         url = options.url ||= config['url']
 
-        summary = Pooler.summary(url)
+        summary = Pooler.summary(verbose, url)
         puts summary
       end
     end
@@ -217,11 +217,11 @@ class Vmfloaty
 
         case action
         when "get"
-          puts Auth.get_token(url, user, pass)
+          puts Auth.get_token(verbose, url, user, pass)
         when "delete"
-          Auth.delete_token(url, user, pass, token)
+          Auth.delete_token(verbose, url, user, pass, token)
         when "status"
-          Auth.token_status(url, user, pass, token)
+          Auth.token_status(verbose, url, user, pass, token)
         else
           puts "Unknown action: #{action}"
         end
diff --git a/lib/vmfloaty/auth.rb b/lib/vmfloaty/auth.rb
index d799153..29ecaa4 100644
--- a/lib/vmfloaty/auth.rb
+++ b/lib/vmfloaty/auth.rb
@@ -3,8 +3,8 @@ require 'json'
 require 'vmfloaty/http'
 
 class Auth
-  def self.get_token(url, user, password)
-    conn = Http.get_conn(url)
+  def self.get_token(verbose, url, user, password)
+    conn = Http.get_conn(verbose, url)
 
     resp = conn.post do |req|
             req.url '/v1/token'
@@ -15,26 +15,26 @@ class Auth
     resp_body
   end
 
-  def self.delete_token(url, user, pass, token)
+  def self.delete_token(verbose, url, user, pass, token)
     if token.nil?
       puts 'You did not provide a token'
       return
     end
 
-    conn = Http.get_conn(url)
+    conn = Http.get_conn(verbose, url)
 
     response = conn.delete "/v1/token/#{token}"
     res_body = JSON.parse(response)
     puts res_body
   end
 
-  def self.token_status(url, user, pass, token)
+  def self.token_status(verbose, url, user, pass, token)
     if token.nil?
       puts 'You did not provide a token'
       return
     end
 
-    conn = Http.get_conn(url)
+    conn = Http.get_conn(verbose, url)
 
     response = conn.get "/v1/token/#{token}"
     res_body = JSON.parse(response.body)
diff --git a/lib/vmfloaty/http.rb b/lib/vmfloaty/http.rb
index 086f0c0..2750cbb 100644
--- a/lib/vmfloaty/http.rb
+++ b/lib/vmfloaty/http.rb
@@ -1,10 +1,10 @@
 require 'faraday'
 
 class Http
-  def self.get_conn(url)
+  def self.get_conn(verbose, url)
     conn = Faraday.new(:url => url) do |faraday|
       faraday.request :url_encoded
-      faraday.response :logger
+      faraday.response :logger if verbose
       faraday.adapter Faraday.default_adapter
     end
 
diff --git a/lib/vmfloaty/pooler.rb b/lib/vmfloaty/pooler.rb
index 68c5988..bdcf37c 100644
--- a/lib/vmfloaty/pooler.rb
+++ b/lib/vmfloaty/pooler.rb
@@ -3,8 +3,8 @@ require 'vmfloaty/http'
 require 'json'
 
 class Pooler
-  def self.list(url, os_filter=nil)
-    conn = Http.get_conn(url)
+  def self.list(verbose, url, os_filter=nil)
+    conn = Http.get_conn(verbose, url)
 
     response = conn.get '/vm'
     response_body = JSON.parse(response.body)
@@ -18,9 +18,9 @@ class Pooler
     hosts
   end
 
-  def self.retrieve(os_type, token, url)
+  def self.retrieve(verbose, os_type, token, url)
     os = os_type.split(',')
-    conn = Http.get_conn(url)
+    conn = Http.get_conn(verbose, url)
     os_body = {}
 
     os.each do |os_type|
@@ -41,9 +41,9 @@ class Pooler
     res_body
   end
 
-  def self.modify(url, hostname, token, lifetime, tags)
+  def self.modify(verbose, url, hostname, token, lifetime, tags)
     modify_body = {'lifetime'=>lifetime, 'tags'=>tags}
-    conn = Http.get_conn(url)
+    conn = Http.get_conn(verbose, url)
 
     # need to use token
     response = conn.put "/#{hostname}"
@@ -52,9 +52,9 @@ class Pooler
     res_body
   end
 
-  def self.delete(url, hostname)
+  def self.delete(verbose, url, hostname)
     hosts = hostnames.split(',')
-    conn = Http.get_conn(url)
+    conn = Http.get_conn(verbose, url)
 
     hosts.each do |host|
       puts "Scheduling host #{host} for deletion"
@@ -64,24 +64,24 @@ class Pooler
     end
   end
 
-  def self.status(url)
-    conn = Http.get_conn(url)
+  def self.status(verbose, url)
+    conn = Http.get_conn(verbose, url)
 
     response = conn.get '/status'
     res_body = JSON.parse(response.body)
     res_body
   end
 
-  def self.summary(url)
-    conn = Http.get_conn(url)
+  def self.summary(verbose, url)
+    conn = Http.get_conn(verbose, url)
 
     response = conn.get '/summary'
     res_body = JSON.parse(response.body)
     res_body
   end
 
-  def self.query(url, hostname)
-    conn = Http.get_conn(url)
+  def self.query(verbose, url, hostname)
+    conn = Http.get_conn(verbose, url)
 
     response = conn.get "/vm/#{hostname}"
     res_body = JSON.parse(response.body)
@@ -89,8 +89,8 @@ class Pooler
     res_body
   end
 
-  def self.snapshot(url, hostname, token)
-    conn = Http.get_conn(url)
+  def self.snapshot(verbose, url, hostname, token)
+    conn = Http.get_conn(verbose, url)
 
     # need to use token
     response = conn.post "/#{hostname}/snapshot"
@@ -98,8 +98,8 @@ class Pooler
     res_body
   end
 
-  def self.revert(url, hostname, token, snapshot_sha)
-    conn = Http.get_conn(url)
+  def self.revert(verbose, url, hostname, token, snapshot_sha)
+    conn = Http.get_conn(verbose, url)
 
     # need to use token
     response = conn.post "/#{hostname}/snapshot/#{snapshot}"