crowbar-client 3.1.7 → 3.1.8

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 3f864e6715677523b58fc601bce43448d4a3faa1
-  data.tar.gz: bbc62230ae538cdb6737677166bfcd22a4d0f210
+  metadata.gz: 72c316559100e694e34b4d071b41bad26af9e7ae
+  data.tar.gz: 3a4277d825d8d7f9739c12797195c5920de03d4e
 SHA512:
-  metadata.gz: 160d5b924836dd0cb4fc3347b4842f86f0f6ae6e4f67f686173708adb679ab17220a483369612538041d316860ff06c5545e3d0601f2bd3b22a6eba4f3e7f048
-  data.tar.gz: 62e76b59a457801d66cc3c32c57c1eb07109054076a64821fb0f4b4aac0660958f2f6be50a2af43c6a543056f3d194d52333aea8a7ab525270959edfa3d85afd
+  metadata.gz: 67ac11b8b97dbcce0bbe2f7df46a86fe915850b299a7b8b00902cf2fcb01471ae7fe339266d604e3235a91a099efd626fdd78ebc58ac6442499a2b6079e95534
+  data.tar.gz: d1c8d291d299f0a742a18dc515bd734c27d8acc5c94488038be315220cb500febeecfe3e035ded13f8acfaf7cd521844cbfe3a8697ce5e102016d24bec54ac57

data/CHANGELOG.md

@@ -1,5 +1,15 @@
 # Changelog
 
+## [3.1.8](https://github.com/crowbar/crowbar-client/releases/tag/v3.1.8) - 2017-02-02
+
+* ENHANCEMENT
+  * Inform the user that OpenStack backup is ongoing (@jsuchome)
+  * Allow granular node upgrade (@MaximilianMeister)
+* BUGFIX
+  * Display an error when a false parameter gets passed (@MaximilianMeister)
+  * Change next step after 'services' to openstack DB backup (@jsuchome)
+  * Correctly format database error (@MaximilianMeister)
+
 ## [3.1.7](https://github.com/crowbar/crowbar-client/releases/tag/v3.1.7) - 2017-01-13
 
 * ENHANCEMENT

data/lib/crowbar/client/app/upgrade.rb

@@ -184,16 +184,23 @@ module Crowbar
           catch_errors(e)
         end
 
-        desc "nodes",
-          "Trigger the upgrade on all nodes"
+        desc "nodes COMPONENT",
+          "Trigger the node upgrade (all|controllers|NODENAME)"
 
         long_desc <<-LONGDESC
-          `nodes` will upgrade all nodes.
+          `nodes all` will upgrade all nodes.
+
+          `nodes controllers` will upgrade all controller nodes.
+
+          `nodes NODENAME` will upgrade the node with the NODENAME.
+          This only works for compute nodes.
         LONGDESC
 
-        def nodes
+        def nodes(component)
           Command::Upgrade::Nodes.new(
-            *command_params
+            *command_params(
+              component: component
+            )
           ).execute
         rescue => e
           catch_errors(e)

data/lib/crowbar/client/command/upgrade/backup.rb

@@ -34,11 +34,14 @@ module Crowbar
             request.process do |request|
               case request.code
               when 200
-                say "Successfully created backup for #{args.component}"
                 if args.component == "crowbar"
+                  say "Successfully created backup for #{args.component}."
                   say "Next step: 'crowbarctl upgrade repocheck crowbar'"
+                else
+                  say "Creating a backup of OpenStack database. "
+                  say "Query the action progress with 'crowbarctl upgrade status'."
+                  say "Next step: 'crowbarctl upgrade nodes'"
                 end
-                say "Next step: 'crowbarctl upgrade nodes'" if args.component == "openstack"
               else
                 case args.component
                 when "crowbar"
@@ -50,7 +53,8 @@ module Crowbar
                     request.parsed_response["error"], "backup_openstack"
                   )
                 else
-                  request.parsed_response["error"]
+                  err "No such component '#{args.component}'. " \
+                    "Only 'crowbar' and 'openstack' are valid components."
                 end
               end
             end

data/lib/crowbar/client/command/upgrade/database.rb

@@ -25,6 +25,7 @@ module Crowbar
         #
         class Database < Base
           include Mixin::Database
+          include Mixin::UpgradeError
 
           def args_with_options
             args.easy_merge!(
@@ -47,7 +48,9 @@ module Crowbar
 
             request.process do |request|
               unless request.code == 200
-                err request.parsed_response["error"]
+                err format_error(
+                  request.parsed_response["error"], "database"
+                )
               end
 
               response = JSON.parse(request.body)

data/lib/crowbar/client/command/upgrade/nodes.rb

@@ -34,8 +34,16 @@ module Crowbar
             request.process do |request|
               case request.code
               when 200
-                say "Successfully triggered the upgrade of the nodes. " \
-                  "Query the upgrade status to follow the process with 'crowbarctl upgrade status'."
+                case args.component
+                when "all"
+                  say "Successfully triggered the upgrade of the nodes. "
+                when "controllers"
+                  say "Successfully triggered the upgrade of the controller nodes. "
+                else
+                  say "Successfully triggered the upgrade of node #{args.component}. "
+                end
+                say "Query the upgrade status to follow the process with " \
+                  "'crowbarctl upgrade status'."
               else
                 err format_error(
                   request.parsed_response["error"], "nodes"

data/lib/crowbar/client/command/upgrade/repocheck.rb

@@ -64,7 +64,8 @@ module Crowbar
                     request.parsed_response["error"], "repocheck_nodes"
                   )
                 else
-                  err request.parsed_response["error"]
+                  err "No such component '#{args.component}'. " \
+                    "Only 'admin' and 'nodes' are valid components."
                 end
               end
             end

data/lib/crowbar/client/command/upgrade/services.rb

@@ -34,9 +34,9 @@ module Crowbar
             request.process do |request|
               case request.code
               when 200
-                say "Stopping related services on all nodes." \
+                say "Stopping related services on all nodes. " \
                   "Query the upgrade status to follow the process with 'crowbarctl upgrade status'."
-                say "Next step: 'crowbarctl upgrade nodes'"
+                say "Next step: 'crowbarctl upgrade backup openstack'"
               else
                 err format_error(
                   request.parsed_response["error"], "services"

data/lib/crowbar/client/request/upgrade/nodes.rb

@@ -35,6 +35,17 @@ module Crowbar
             )
           end
 
+          #
+          # Override the request content
+          #
+          # @return [Hash] the content for the request
+          #
+          def content
+            super.easy_merge!(
+              component: attrs.component
+            )
+          end
+
           #
           # HTTP method that gets used by the request
           #

data/lib/crowbar/client/version.rb

@@ -33,7 +33,7 @@ module Crowbar
       #
       # Patch version
       #
-      PATCH = 7
+      PATCH = 8
 
       #
       # Optional suffix

data/spec/crowbar/client/request/upgrade/nodes_spec.rb

@@ -27,11 +27,13 @@ describe "Crowbar::Client::Request::Upgrade::Nodes" do
 
     let!(:attrs) do
       {
+        component: "all"
       }
     end
 
     let!(:params) do
       {
+        component: "all"
       }
     end
 

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: crowbar-client
 version: !ruby/object:Gem::Version
-  version: 3.1.7
+  version: 3.1.8
 platform: ruby
 authors:
 - Thomas Boerger
@@ -10,7 +10,7 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2017-01-13 00:00:00.000000000 Z
+date: 2017-02-02 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: bundler