knife-ucs 0.0.2 → 0.0.3

data/README.md

@@ -1,7 +1,7 @@
-Knife UCS (alpha)
+Knife UCS (beta)
 ===============
 
-This is a Chef Knife plugin for Cisco UCS inspired by knife-openstack (Matt Ray @Opscode). This plugin gives knife the ability to provision, list, and manage Cisco UCS. It leverages UCSlib (http://github.com/murraju/ucslib). 
+This is a Chef Knife plugin for Cisco UCS. This plugin gives knife the ability to provision, list, and manage Cisco UCS. It leverages UCSlib (http://github.com/murraju/ucslib). 
 
 # Installation #
 
@@ -41,7 +41,10 @@ Example usage can be found here -> https://github.com/murraju/knife-ucs/wiki
 
 Author:: Murali Raju <murali.raju@appliv.com>
 
-Copyright:: Copyright (c) 2012 Murali Raju.
+Author:: Velankani Engineering Team <eng@velankani.net>
+
+Copyright:: Copyright (c) 2011 Murali Raju <murali.raju@appliv.com>
+Copyright:: Copyright (c) 2012 Velankani Information Systems, Inc.
 
 License:: Apache License, Version 2.0
 

data/configs/hosfw.json

@@ -0,0 +1,18 @@
+{
+	"hostfw":{
+				"host_firmware_pkg_name" 	: "EngHostFW",
+				"hardware_model"		 	: "N20-AC0002",
+				"hardware_type"				: "adaptor",
+				"hardware_vendor"			: "Cisco Systems Inc",
+				"firmware_version"			: "2.0(2q)",
+				"org"						: "Eng"
+	},
+	"hostfw":{
+				"host_firmware_pkg_name" 	: "EngHostFW",
+				"hardware_model"		 	: "N20-AC0002",
+				"hardware_type"				: "adaptor",
+				"hardware_vendor"			: "Cisco Systems Inc",
+				"firmware_version"			: "2.0(2q)",
+				"org"						: "Eng"
+	}	
+}

data/knife-ucs.gemspec

@@ -8,9 +8,9 @@ Gem::Specification.new do |s|
   s.platform    = Gem::Platform::RUBY
   s.has_rdoc = true
   s.extra_rdoc_files = ["README.md", "LICENSE" ]
-  s.authors     = ["Murali Raju"]
-  s.email       = ["murraju@appliv.com"]
-  s.homepage    = "https://github.com/murraju/knife-ucs"
+  s.authors     = ["Murali Raju", "Velankani Engineering"]
+  s.email       = ["murraju@appliv.com", "eng@velankani.net"]
+  s.homepage    = "https://github.com/velankanisys/knife-ucs"
   s.summary     = %q{Cisco UCS Support for Chef's Knife Command}
   s.description = s.summary
 
@@ -21,5 +21,6 @@ Gem::Specification.new do |s|
 
   s.add_dependency "nokogiri", "~> 1.5.2"
   s.add_dependency "rest-client", "~> 1.6.7"
-  s.add_dependency "ucslib", "~> 0.0.6"
+  s.add_dependency "ucslib", "~> 0.1.6"
+  s.add_dependency "chef", "~>10.16.2"
 end

data/lib/chef/knife/ucs_base.rb

@@ -1,5 +1,7 @@
 # Author:: Murali Raju (<murali.raju@appliv.com>)
+# Author:: Velankani Engineering <eng@velankani.net>
 # Copyright:: Copyright (c) 2012 Murali Raju.
+# Copyright:: Copyright (c) 2012 Velankani Information Systems, Inc
 # License:: Apache License, Version 2.0
 #
 # Licensed under the Apache License, Version 2.0 (the "License");
@@ -54,6 +56,9 @@ class Chef
 
       def connection
         ucs_session = UCSToken.new
+        Chef::Log.debug("username: #{Chef::Config[:knife][:ucsm_username]}")
+        Chef::Log.debug("password: #{Chef::Config[:knife][:ucsm_password]}")
+        Chef::Log.debug("host:     #{Chef::Config[:knife][:ucsm_host]}")
         @connection ||= begin
           connection = ucs_session.get_token({
             :username => Chef::Config[:knife][:ucsm_username],
@@ -76,6 +81,13 @@ class Chef
           provisioner = UCSProvision.new(connection)
         end 
       end
+
+      def manager
+        ucs_manager = UCSManage.new(connection)
+        @manager ||= begin
+          manager = ucs_manager.discover_state
+        end 
+      end
       
       def destroyer
         @destroyer ||= begin

data/lib/chef/knife/ucs_blades_list.rb

@@ -1,5 +1,7 @@
 # Author:: Murali Raju (<murali.raju@appliv.com>)
+# Author:: Velankani Engineering <eng@velankani.net>
 # Copyright:: Copyright (c) 2012 Murali Raju.
+# Copyright:: Copyright (c) 2012 Velankani Information Systems, Inc
 # License:: Apache License, Version 2.0
 #
 # Licensed under the Apache License, Version 2.0 (the "License");

data/lib/chef/knife/ucs_cpus_list.rb

@@ -1,5 +1,7 @@
 # Author:: Murali Raju (<murali.raju@appliv.com>)
+# Author:: Velankani Engineering <eng@velankani.net>
 # Copyright:: Copyright (c) 2012 Murali Raju.
+# Copyright:: Copyright (c) 2012 Velankani Information Systems, Inc
 # License:: Apache License, Version 2.0
 #
 # Licensed under the Apache License, Version 2.0 (the "License");

data/lib/chef/knife/ucs_org_create.rb

@@ -1,5 +1,7 @@
 # Author:: Murali Raju (<murali.raju@appliv.com>)
+# Author:: Velankani Engineering <eng@velankani.net>
 # Copyright:: Copyright (c) 2012 Murali Raju.
+# Copyright:: Copyright (c) 2012 Velankani Information Systems, Inc
 # License:: Apache License, Version 2.0
 #
 # Licensed under the Apache License, Version 2.0 (the "License");
@@ -44,15 +46,15 @@ class Chef
       def run
         $stdout.sync = true
         
-        xml_response = provisioner.create_org({:org => Chef::Config[:knife][:org]}.to_json)
+        xml_response = provisioner.set_org({:org => Chef::Config[:knife][:org]}.to_json)
         xml_doc = Nokogiri::XML(xml_response)
         
         xml_doc.xpath("configConfMos/outConfigs/pair/orgOrg").each do |org|
             puts ''
-            puts "Org: #{ui.color("#{org.attributes['name']}", :magenta)} status: #{ui.color("#{org.attributes['status']}", :green)}"
+            puts "Org: #{ui.color("#{org.attributes['name']}", :blue)} status: #{ui.color("#{org.attributes['status']}", :green)}"
         end
 
-        #Ugly...refactor later to parse error with better exception handling. Nokogiri xpath search for elements might be an option
+        
         xml_doc.xpath("configConfMos").each do |org|
            puts "#{org.attributes['errorCode']} #{ui.color("#{org.attributes['errorDescr']}", :red)}"
         end

data/lib/chef/knife/ucs_org_delete.rb

@@ -1,5 +1,7 @@
 # Author:: Murali Raju (<murali.raju@appliv.com>)
+# Author:: Velankani Engineering <eng@velankani.net>
 # Copyright:: Copyright (c) 2012 Murali Raju.
+# Copyright:: Copyright (c) 2012 Velankani Information Systems, Inc
 # License:: Apache License, Version 2.0
 #
 # Licensed under the Apache License, Version 2.0 (the "License");
@@ -49,7 +51,7 @@ class Chef
         
         xml_doc.xpath("configConfMos/outConfigs/pair/orgOrg").each do |org|
             puts ''
-            puts "Org: #{ui.color("#{org.attributes['name']}", :magenta)} status: #{ui.color("#{org.attributes['status']}", :red)}"
+            puts "Org: #{ui.color("#{org.attributes['name']}", :blue)} status: #{ui.color("#{org.attributes['status']}", :red)}"
         end
 
         #Ugly...refactor later to parse error with better exception handling. Nokogiri xpath search for elements might be an option

data/lib/chef/knife/ucs_orgs_list.rb

@@ -1,5 +1,7 @@
 # Author:: Murali Raju (<murali.raju@appliv.com>)
+# Author:: Velankani Engineering <eng@velankani.net>
 # Copyright:: Copyright (c) 2012 Murali Raju.
+# Copyright:: Copyright (c) 2012 Velankani Information Systems, Inc
 # License:: Apache License, Version 2.0
 #
 # Licensed under the Apache License, Version 2.0 (the "License");

data/lib/chef/knife/ucs_policy_create.rb

@@ -0,0 +1,108 @@
+# Author:: Murali Raju (<murali.raju@appliv.com>)
+# Author:: Velankani Engineering <eng@velankani.net>
+# Copyright:: Copyright (c) 2012 Murali Raju.
+# Copyright:: Copyright (c) 2012 Velankani Information Systems, Inc
+# License:: Apache License, Version 2.0
+#
+# Licensed under the Apache License, Version 2.0 (the "License");
+# you may not use this file except in compliance with the License.
+# You may obtain a copy of the License at
+#
+# http://www.apache.org/licenses/LICENSE-2.0
+#
+# Unless required by applicable law or agreed to in writing, software
+# distributed under the License is distributed on an "AS IS" BASIS,
+# WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+# See the License for the specific language governing permissions and
+# limitations under the License.
+#
+
+require 'chef/knife/ucs_base'
+
+class Chef
+  class Knife
+    class UcsPolicyCreate < Knife
+
+      include Knife::UCSBase
+
+      deps do
+        require 'readline'
+        require 'chef/json_compat'
+        require 'chef/knife/bootstrap'
+        Chef::Knife::Bootstrap.load_deps
+      end
+
+      banner "knife ucs policy create (options)"
+
+      attr_accessor :initial_sleep_delay
+
+      option :policy,
+        :long => "--policy-type POLICY",
+        :description => "The policy type <boot,host-firmware,mgmt-firmware>",
+        :proc => Proc.new { |f| Chef::Config[:knife][:policy] = f }
+
+      option :org,
+        :long => "--org ORG",
+        :description => "The organization to use",
+        :proc => Proc.new { |f| Chef::Config[:knife][:org] = f }
+
+      option :name,
+        :long => "--policy-name POLICYNAME",
+        :description => "The policy name to use",
+        :proc => Proc.new { |f| Chef::Config[:knife][:name] = f }
+
+      option :hardwaremodel,
+        :long => "--hardware-model MODEL",
+        :description => "The hardware model used with firmware policies",
+        :proc => Proc.new { |f| Chef::Config[:knife][:hardwaremodel] = f }
+
+      option :hardwaretype,
+        :long => "--hardware-type TYPE",
+        :description => "The hardware type used with firmware policies",
+        :proc => Proc.new { |f| Chef::Config[:knife][:hardwaretype] = f }
+
+      option :hardwarevendor,
+        :long => "--hardware-vendor VENDOR",
+        :description => "The hardware vendor used with firmware policies",
+        :proc => Proc.new { |f| Chef::Config[:knife][:hardwarevendor] = f }
+
+      option :firmwareversion,
+        :long => "--firmware-version VERSION",
+        :description => "The firmware version used with firmware policies",
+        :proc => Proc.new { |f| Chef::Config[:knife][:firmwareversion] = f }
+
+
+
+      def run
+        $stdout.sync = true
+        
+        policy = "#{Chef::Config[:knife][:policy]}".downcase
+        case policy
+        when 'host-firmware'
+
+          json = { :host_firmware_pkg_name => Chef::Config[:knife][:name], :hardware_model => Chef::Config[:knife][:hardwaremodel].to_s,
+                   :hardware_type => Chef::Config[:knife][:hardwaretype].to_s,  :hardware_vendor => Chef::Config[:knife][:hardwarevendor].to_s, 
+                   :firmware_version => Chef::Config[:knife][:firmwareversion].to_s, :org => Chef::Config[:knife][:org] }.to_json
+
+          xml_response = provisioner.set_host_firmware_package(json)
+          xml_doc = Nokogiri::XML(xml_response)
+          
+          xml_doc.xpath("configConfMos/outConfigs/pair/firmwareComputeHostPack").each do |hostfw|
+            puts ''
+            puts "Host Firmware Pack: #{ui.color("#{hostfw.attributes['name']}", :blue)}" + 
+                  " status: #{ui.color("#{hostfw.attributes['status']}", :green)}"
+          end
+          
+          xml_doc.xpath("configConfMos").each do |hostfw|
+             puts "#{hostfw.attributes['errorCode']} #{ui.color("#{hostfw.attributes['errorDescr']}", :red)}"
+          end
+          
+        else
+          puts "Incorrect options. Please make sure you are using one of the following: host-firmware, mgmt-firmware"
+        end     
+        
+      end
+      
+    end
+  end
+end

data/lib/chef/knife/ucs_policy_delete.rb

@@ -0,0 +1,57 @@
+# Author:: Murali Raju (<murali.raju@appliv.com>)
+# Author:: Velankani Engineering <eng@velankani.net>
+# Copyright:: Copyright (c) 2012 Murali Raju.
+# Copyright:: Copyright (c) 2012 Velankani Information Systems, Inc
+# License:: Apache License, Version 2.0
+#
+# Licensed under the Apache License, Version 2.0 (the "License");
+# you may not use this file except in compliance with the License.
+# You may obtain a copy of the License at
+#
+# http://www.apache.org/licenses/LICENSE-2.0
+#
+# Unless required by applicable law or agreed to in writing, software
+# distributed under the License is distributed on an "AS IS" BASIS,
+# WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+# See the License for the specific language governing permissions and
+# limitations under the License.
+#
+
+require 'chef/knife/ucs_base'
+
+class Chef
+  class Knife
+    class UcsPolicyDelete < Knife
+
+      include Knife::UCSBase
+
+      deps do
+        require 'readline'
+        require 'chef/json_compat'
+        require 'chef/knife/bootstrap'
+        Chef::Knife::Bootstrap.load_deps
+      end
+
+      banner "knife ucs policy delete (options)"
+
+      attr_accessor :initial_sleep_delay
+
+      option :policy,
+        :long => "--policy-type POLICY",
+        :description => "The policy type <boot,hostfirmware,managementfirmware>",
+        :proc => Proc.new { |f| Chef::Config[:knife][:policy] = f }
+
+      option :name,
+        :long => "--policy-name POLICYNAME",
+        :description => "The policy name",
+        :proc => Proc.new { |f| Chef::Config[:knife][:name] = f }
+
+      def run
+        $stdout.sync = true
+               
+        #TBD
+
+      end
+    end
+  end
+end

data/lib/chef/knife/ucs_policy_update.rb

@@ -0,0 +1,107 @@
+# Author:: Murali Raju (<murali.raju@appliv.com>)
+# Author:: Velankani Engineering <eng@velankani.net>
+# Copyright:: Copyright (c) 2012 Murali Raju.
+# Copyright:: Copyright (c) 2012 Velankani Information Systems, Inc
+# License:: Apache License, Version 2.0
+#
+# Licensed under the Apache License, Version 2.0 (the "License");
+# you may not use this file except in compliance with the License.
+# You may obtain a copy of the License at
+#
+# http://www.apache.org/licenses/LICENSE-2.0
+#
+# Unless required by applicable law or agreed to in writing, software
+# distributed under the License is distributed on an "AS IS" BASIS,
+# WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+# See the License for the specific language governing permissions and
+# limitations under the License.
+#
+require 'chef/knife/ucs_base'
+
+class Chef
+  class Knife
+    class UcsPolicyUpdate < Knife
+
+      include Knife::UCSBase
+
+      deps do
+        require 'readline'
+        require 'chef/json_compat'
+        require 'chef/knife/bootstrap'
+        Chef::Knife::Bootstrap.load_deps
+      end
+
+      banner "knife ucs policy update (options)"
+
+      attr_accessor :initial_sleep_delay
+
+      option :policy,
+        :long => "--policy-type POLICY",
+        :description => "The policy type <boot,host-firmware,mgmt-firmware>",
+        :proc => Proc.new { |f| Chef::Config[:knife][:policy] = f }
+
+      option :org,
+        :long => "--org ORG",
+        :description => "The organization to use",
+        :proc => Proc.new { |f| Chef::Config[:knife][:org] = f }
+
+      option :name,
+        :long => "--policy-name POLICYNAME",
+        :description => "The policy name to use",
+        :proc => Proc.new { |f| Chef::Config[:knife][:name] = f }
+
+      option :hardwaremodel,
+        :long => "--hardware-model MODEL",
+        :description => "The hardware model used with firmware policies",
+        :proc => Proc.new { |f| Chef::Config[:knife][:hardwaremodel] = f }
+
+      option :hardwaretype,
+        :long => "--hardware-type TYPE",
+        :description => "The hardware type used with firmware policies",
+        :proc => Proc.new { |f| Chef::Config[:knife][:hardwaretype] = f }
+
+      option :hardwarevendor,
+        :long => "--hardware-vendor VENDOR",
+        :description => "The hardware vendor used with firmware policies",
+        :proc => Proc.new { |f| Chef::Config[:knife][:hardwarevendor] = f }
+
+      option :firmwareversion,
+        :long => "--firmware-version VERSION",
+        :description => "The firmware version used with firmware policies",
+        :proc => Proc.new { |f| Chef::Config[:knife][:firmwareversion] = f }
+
+
+      def run
+        $stdout.sync = true
+        
+        policy = "#{Chef::Config[:knife][:policy]}".downcase
+        case policy
+        when 'host-firmware'
+        
+          json = { :host_firmware_pkg_name => Chef::Config[:knife][:name], :hardware_model => Chef::Config[:knife][:hardwaremodel].to_s,
+                   :hardware_type => Chef::Config[:knife][:hardwaretype].to_s,  :hardware_vendor => Chef::Config[:knife][:hardwarevendor].to_s, 
+                   :firmware_version => Chef::Config[:knife][:firmwareversion].to_s, :org => Chef::Config[:knife][:org] }.to_json
+          
+
+          xml_response = updater.update_host_firmware_package(json)
+          xml_doc = Nokogiri::XML(xml_response)
+          
+          xml_doc.xpath("configConfMos/outConfigs/pair/firmwareComputeHostPack").each do |hostfw|
+            puts ''
+            puts "Host Firmware Pack: #{ui.color("#{hostfw.attributes['name']}", :blue)}" + 
+                  " status: #{ui.color("#{hostfw.attributes['status']}", :green)}"
+          end
+          
+          xml_doc.xpath("configConfMos").each do |hostfw|
+             puts "#{hostfw.attributes['errorCode']} #{ui.color("#{hostfw.attributes['errorDescr']}", :red)}"
+          end
+          
+        else
+          puts "Incorrect options. Please make sure you are using one of the following: host-firmware, mgmt-firmware"
+        end     
+        
+      end
+      
+    end
+  end
+end

data/lib/chef/knife/ucs_pool_create.rb

@@ -1,5 +1,7 @@
 # Author:: Murali Raju (<murali.raju@appliv.com>)
+# Author:: Velankani Engineering <eng@velankani.net>
 # Copyright:: Copyright (c) 2012 Murali Raju.
+# Copyright:: Copyright (c) 2012 Velankani Information Systems, Inc
 # License:: Apache License, Version 2.0
 #
 # Licensed under the Apache License, Version 2.0 (the "License");
@@ -14,7 +16,6 @@
 # See the License for the specific language governing permissions and
 # limitations under the License.
 #
-
 require 'chef/knife/ucs_base'
 
 class Chef
@@ -34,45 +35,38 @@ class Chef
       
       attr_accessor :initial_sleep_delay
       
-      option :pool,
-        :short => "-P POOL",
-        :long => "--pool POOLTYPE",
+      option :pooltype,
+        :long => "--pool-type POOLTYPE",
         :description => "UCS pool types <mac,uuid,wwpn,wwnn,managementip>",
-        :proc => Proc.new { |f| Chef::Config[:knife][:pool] = f }
+        :proc => Proc.new { |f| Chef::Config[:knife][:pooltype] = f }
 
       option :name,
-        :short => "-N NAME",
         :long => "--pool-name POOLNAME",
         :description => "The pool name",
         :proc => Proc.new { |f| Chef::Config[:knife][:name] = f }
 
       option :start,
-        :short => "-S START",
         :long => "--pool-start STARTRANGE",
         :description => "Start of a pool range <IP, WWPN, WWNN, MAC>",
         :proc => Proc.new { |f| Chef::Config[:knife][:start] = f }
 
       option :end,
-        :short => "-E END",
         :long => "--pool-end ENDRANGE",
         :description => "End of a pool range <IP, WWPN, WWNN, MAC>",
         :proc => Proc.new { |f| Chef::Config[:knife][:end] = f }
 
 
       option :mask,
-        :short => "-M MASK",
         :long => "--subnet-mask SUBNETMASK",
         :description => "The subnet mask for an IP range",
         :proc => Proc.new { |f| Chef::Config[:knife][:mask] = f }
 
       option :gateway,
-        :short => "-G GATEWAY",
         :long => "--gateway IPGATEWAY",
         :description => "The IP Gateway address of a subnet",
         :proc => Proc.new { |f| Chef::Config[:knife][:gateway] = f }
 
       option :org,
-        :short => "-O ORG",
         :long => "--org ORG",
         :description => "The organization",
         :proc => Proc.new { |f| Chef::Config[:knife][:org] = f }      
@@ -80,22 +74,22 @@ class Chef
       def run
         $stdout.sync = true
           
-        pool_type = "#{Chef::Config[:knife][:pool]}"
+        pool_type = "#{Chef::Config[:knife][:pooltype]}".downcase
         case pool_type
         when 'managementip'
           json = { :start_ip => Chef::Config[:knife][:start],   :end_ip => Chef::Config[:knife][:end],
                    :subnet_mask => Chef::Config[:knife][:mask], :gateway => Chef::Config[:knife][:gateway] }.to_json
           
-          xml_response = provisioner.create_management_ip_pool(json)
+          xml_response = provisioner.set_management_ip_pool(json)
           xml_doc = Nokogiri::XML(xml_response)
   
           xml_doc.xpath("configConfMos/outConfigs/pair/ippoolBlock").each do |ippool|
             puts ''
-            puts "Management IP Block from: #{ui.color("#{ippool.attributes['from']}", :magenta)} to: #{ui.color("#{ippool.attributes['to']}", :magenta)}" + 
+            puts "Management IP Block from: #{ui.color("#{ippool.attributes['from']}", :blue)} to: #{ui.color("#{ippool.attributes['to']}", :blue)}" + 
                   " status: #{ui.color("#{ippool.attributes['status']}", :green)}"
           end
 
-          #Ugly...refactor later to parse error with better exception handling. Nokogiri xpath search for elements might be an option
+          
           xml_doc.xpath("configConfMos").each do |ippool|
              puts "#{ippool.attributes['errorCode']} #{ui.color("#{ippool.attributes['errorDescr']}", :red)}"
           end
@@ -104,16 +98,16 @@ class Chef
           json =  { :mac_pool_name => Chef::Config[:knife][:name],  :mac_pool_start => Chef::Config[:knife][:start], 
                     :mac_pool_end => Chef::Config[:knife][:end],    :org => Chef::Config[:knife][:org] }.to_json
           
-          xml_response = provisioner.create_mac_pool(json)
+          xml_response = provisioner.set_mac_pool(json)
           xml_doc = Nokogiri::XML(xml_response)
   
           xml_doc.xpath("configConfMos/outConfigs/pair/macpoolPool").each do |macpool|
             puts ''
-            puts "MAC address pool : #{ui.color("#{macpool.attributes['name']}", :magenta)}" + 
+            puts "MAC address pool : #{ui.color("#{macpool.attributes['name']}", :blue)}" + 
                   " status: #{ui.color("#{macpool.attributes['status']}", :green)}"
           end
 
-          #Ugly...refactor later to parse error with better exception handling. Nokogiri xpath search for elements might be an option
+          
           xml_doc.xpath("configConfMos").each do |macpool|
              puts "#{macpool.attributes['errorCode']} #{ui.color("#{macpool.attributes['errorDescr']}", :red)}"
           end          
@@ -122,16 +116,16 @@ class Chef
           json =  { :wwnn_name => Chef::Config[:knife][:name],  :wwnn_from => Chef::Config[:knife][:start], 
                     :wwnn_to => Chef::Config[:knife][:end],     :org => Chef::Config[:knife][:org] }.to_json
           
-          xml_response = provisioner.create_wwnn_pool(json)
+          xml_response = provisioner.set_wwnn_pool(json)
           xml_doc = Nokogiri::XML(xml_response)
             
           xml_doc.xpath("configConfMos/outConfigs/pair/fcpoolInitiators").each do |wwnn|
             puts ''
-            puts "WWNN pool : #{ui.color("#{wwnn.attributes['name']}", :magenta)}" + 
+            puts "WWNN pool : #{ui.color("#{wwnn.attributes['name']}", :blue)}" + 
                   " status: #{ui.color("#{wwnn.attributes['status']}", :green)}"
           end
           
-          #Ugly...refactor later to parse error with better exception handling. Nokogiri xpath search for elements might be an option
+          
           xml_doc.xpath("configConfMos").each do |wwnn|
              puts "#{wwnn.attributes['errorCode']} #{ui.color("#{wwnn.attributes['errorDescr']}", :red)}"
           end
@@ -141,16 +135,16 @@ class Chef
           json =  { :wwpn_name => Chef::Config[:knife][:name],  :wwpn_from => Chef::Config[:knife][:start], 
                     :wwpn_to => Chef::Config[:knife][:end],     :org => Chef::Config[:knife][:org] }.to_json
         
-          xml_response = provisioner.create_wwpn_pool(json)
+          xml_response = provisioner.set_wwpn_pool(json)
           xml_doc = Nokogiri::XML(xml_response)
           
           xml_doc.xpath("configConfMos/outConfigs/pair/fcpoolInitiators").each do |wwpn|
             puts ''
-            puts "WWPN pool : #{ui.color("#{wwpn.attributes['name']}", :magenta)}" + 
+            puts "WWPN pool : #{ui.color("#{wwpn.attributes['name']}", :blue)}" + 
                   " status: #{ui.color("#{wwpn.attributes['status']}", :green)}"
           end
         
-          #Ugly...refactor later to parse error with better exception handling. Nokogiri xpath search for elements might be an option
+          
           xml_doc.xpath("configConfMos").each do |wwpn|
              puts "#{wwpn.attributes['errorCode']} #{ui.color("#{wwpn.attributes['errorDescr']}", :red)}"
           end
@@ -159,16 +153,16 @@ class Chef
           json =  { :uuid_pool_name => Chef::Config[:knife][:name], :uuid_from => Chef::Config[:knife][:start], 
                     :uuid_to => Chef::Config[:knife][:end],         :org => Chef::Config[:knife][:org] }.to_json
         
-          xml_response = provisioner.create_uuid_pool(json)
+          xml_response = provisioner.set_uuid_pool(json)
           xml_doc = Nokogiri::XML(xml_response)
           
           xml_doc.xpath("configConfMos/outConfigs/pair/uuidpoolPool").each do |uuid|
             puts ''
-            puts "UUID pool : #{ui.color("#{uuid.attributes['name']}", :magenta)}" + 
+            puts "UUID pool : #{ui.color("#{uuid.attributes['name']}", :blue)}" + 
                   " status: #{ui.color("#{uuid.attributes['status']}", :green)}"
           end
         
-          #Ugly...refactor later to parse error with better exception handling. Nokogiri xpath search for elements might be an option
+          
           xml_doc.xpath("configConfMos").each do |uuid|
              puts "#{uuid.attributes['errorCode']} #{ui.color("#{uuid.attributes['errorDescr']}", :red)}"
           end