knife-azure 1.8.7 → 1.9.0

data/lib/azure/service_management/disk.rb

@@ -1,6 +1,6 @@
 #
 # Author:: Barry Davis (barryd@jetstreamsoftware.com)
-# Copyright:: Copyright (c) 2010-2011 Opscode, Inc.
+# Copyright:: Copyright 2010-2018 Chef Software, Inc.
 # License:: Apache License, Version 2.0
 #
 # Licensed under the Apache License, Version 2.0 (the "License");
@@ -19,33 +19,37 @@
 module Azure
   class Disks
     def initialize(connection)
-      @connection=connection
+      @connection = connection
     end
+
     def all
       disks = Array.new
-      response = @connection.query_azure('disks')
-      founddisks = response.css('Disk')
+      response = @connection.query_azure("disks")
+      founddisks = response.css("Disk")
       founddisks.each do |disk|
         item = Disk.new(disk)
         disks << item
       end
       disks
     end
+
     def find(name)
       founddisk = nil
-      self.all.each do |disk|
+      all.each do |disk|
         next unless disk.name == name
         founddisk = disk
       end
       founddisk
     end
+
     def exists(name)
-      find(name) != nil
+      !find(name).nil?
     end
+
     def clear_unattached
-      self.all.each do |disk|
+      all.each do |disk|
         next unless disk.attached == false
-        @connection.query_azure('disks/' + disk.name, 'delete')
+        @connection.query_azure("disks/" + disk.name, "delete")
       end
     end
   end
@@ -55,8 +59,8 @@ module Azure
   class Disk
     attr_accessor :name, :attached
     def initialize(disk)
-      @name = disk.at_css('Name').content
-      @attached = disk.at_css('AttachedTo') != nil
+      @name = disk.at_css("Name").content
+      @attached = !disk.at_css("AttachedTo").nil?
     end
   end
 end

data/lib/azure/service_management/host.rb

@@ -1,6 +1,6 @@
 #
 # Author:: Barry Davis (barryd@jetstreamsoftware.com)
-# Copyright:: Copyright (c) 2010-2011 Opscode, Inc.
+# Copyright:: Copyright 2010-2018 Chef Software, Inc.
 # License:: Apache License, Version 2.0
 #
 # Licensed under the Apache License, Version 2.0 (the "License");
@@ -20,7 +20,7 @@ module Azure
   class Hosts
     include AzureUtility
     def initialize(connection)
-      @connection=connection
+      @connection = connection
     end
 
     # force_load should be true when there is something in local cache and we want to reload
@@ -29,8 +29,8 @@ module Azure
       if not @hosted_services || force_load
         @hosted_services = begin
           hosted_services = Hash.new
-          responseXML = @connection.query_azure('hostedservices')
-          servicesXML = responseXML.css('HostedServices HostedService')
+          responseXML = @connection.query_azure("hostedservices")
+          servicesXML = responseXML.css("HostedServices HostedService")
           servicesXML.each do |serviceXML|
             host = Host.new(@connection).parse(serviceXML)
             hosted_services[host.name] = host
@@ -42,13 +42,13 @@ module Azure
     end
 
     def all
-      self.load.values
+      load.values
     end
 
     # first look up local cache if we have already loaded list.
     def exists?(name)
       return @hosted_services.key?(name) if @hosted_services
-      self.exists_on_cloud?(name)
+      exists_on_cloud?(name)
     end
 
     # Look up on cloud and not local cache
@@ -56,7 +56,7 @@ module Azure
       ret_val = @connection.query_azure("hostedservices/#{name}")
       error_code, error_message = error_from_response_xml(ret_val) if ret_val
       if ret_val.nil? || error_code.length > 0
-        Chef::Log.debug('Unable to find hosted(cloud) service:' + error_code + ' : ' + error_message) if ret_val
+        Chef::Log.debug("Unable to find hosted(cloud) service:" + error_code + " : " + error_message) if ret_val
         false
       else
         true
@@ -66,7 +66,7 @@ module Azure
     # first look up local cache if we have already loaded list.
     def find(name)
       return @hosted_services[name] if @hosted_services && @hosted_services.key?(name)
-      self.fetch_from_cloud(name)
+      fetch_from_cloud(name)
     end
 
     # Look up hosted service on cloud and not local cache
@@ -74,7 +74,7 @@ module Azure
       ret_val = @connection.query_azure("hostedservices/#{name}")
       error_code, error_message = error_from_response_xml(ret_val) if ret_val
       if ret_val.nil? || error_code.length > 0
-        Chef::Log.warn('Unable to find hosted(cloud) service:' + error_code + ' : ' + error_message) if ret_val
+        Chef::Log.warn("Unable to find hosted(cloud) service:" + error_code + " : " + error_message) if ret_val
         nil
       else
         Host.new(@connection).parse(ret_val)
@@ -85,10 +85,11 @@ module Azure
       host = Host.new(@connection)
       host.create(params)
     end
+
     def delete(name)
-      if self.exists?(name)
-          servicecall = "hostedservices/" + name
-        @connection.query_azure(servicecall, "delete") 
+      if exists?(name)
+        servicecall = "hostedservices/" + name
+        @connection.query_azure(servicecall, "delete")
       end
     end
   end
@@ -106,35 +107,38 @@ module Azure
       @deploys_loaded = false
       @deploys = Hash.new
     end
+
     def parse(serviceXML)
-      @name = xml_content(serviceXML, 'ServiceName')
-      @url = xml_content(serviceXML, 'Url')
-      @label = xml_content(serviceXML, 'HostedServiceProperties Label')
-      @dateCreated = xml_content(serviceXML, 'HostedServiceProperties DateCreated')
-      @description = xml_content(serviceXML, 'HostedServiceProperties Description')
-      @location = xml_content(serviceXML, 'HostedServiceProperties Location')
-      @dateModified = xml_content(serviceXML, 'HostedServiceProperties DateLastModified')
-      @status = xml_content(serviceXML, 'HostedServiceProperties Status')
+      @name = xml_content(serviceXML, "ServiceName")
+      @url = xml_content(serviceXML, "Url")
+      @label = xml_content(serviceXML, "HostedServiceProperties Label")
+      @dateCreated = xml_content(serviceXML, "HostedServiceProperties DateCreated")
+      @description = xml_content(serviceXML, "HostedServiceProperties Description")
+      @location = xml_content(serviceXML, "HostedServiceProperties Location")
+      @dateModified = xml_content(serviceXML, "HostedServiceProperties DateLastModified")
+      @status = xml_content(serviceXML, "HostedServiceProperties Status")
       self
     end
+
     def create(params)
       builder = Nokogiri::XML::Builder.new do |xml|
-        xml.CreateHostedService('xmlns'=>'http://schemas.microsoft.com/windowsazure') {
+        xml.CreateHostedService("xmlns" => "http://schemas.microsoft.com/windowsazure") do
           xml.ServiceName params[:azure_dns_name]
           xml.Label Base64.encode64(params[:azure_dns_name])
-          xml.Description 'Explicitly created hosted service'
+          xml.Description "Explicitly created hosted service"
           unless params[:azure_service_location].nil?
             xml.Location params[:azure_service_location]
           end
           unless params[:azure_affinity_group].nil?
             xml.AffinityGroup params[:azure_affinity_group]
           end
-        }
+        end
       end
       @connection.query_azure("hostedservices", "post", builder.to_xml)
     end
+
     def details
-      response = @connection.query_azure('hostedservices/' + @name + '?embed-detail=true')
+      response = @connection.query_azure("hostedservices/" + @name + "?embed-detail=true")
     end
 
     # Deployments within this hostedservice

data/lib/azure/service_management/image.rb

@@ -1,6 +1,6 @@
 #
 # Author:: Barry Davis (barryd@jetstreamsoftware.com)
-# Copyright:: Copyright (c) 2010-2011 Opscode, Inc.
+# Copyright:: Copyright 2010-2018 Chef Software, Inc.
 # License:: Apache License, Version 2.0
 #
 # Licensed under the Apache License, Version 2.0 (the "License");
@@ -19,29 +19,30 @@
 module Azure
   class Images
     def initialize(connection)
-      @connection=connection
+      @connection = connection
     end
+
     def load
       @images ||= begin
-        osimages = self.get_images("OSImage")   #get OSImages
-        vmimages = self.get_images("VMImage")   #get VMImages
+        osimages = get_images("OSImage")   #get OSImages
+        vmimages = get_images("VMImage")   #get VMImages
 
         all_images = osimages.merge(vmimages)
       end
     end
 
     def all
-      self.load.values
+      load.values
     end
 
     # img_type = OSImages or VMImage
     def get_images(img_type)
       images = Hash.new
 
-      if(img_type == "OSImage")
-        response = @connection.query_azure('images')
-      elsif(img_type == "VMImage")
-        response = @connection.query_azure('vmimages')
+      if img_type == "OSImage"
+        response = @connection.query_azure("images")
+      elsif img_type == "VMImage"
+        response = @connection.query_azure("vmimages")
       end
 
       unless response.to_s.empty?
@@ -57,21 +58,21 @@ module Azure
     end
 
     def is_os_image(image_name)
-      os_images = self.get_images("OSImage").values
-      os_images.detect {|img| img.name == image_name} ? true : false
+      os_images = get_images("OSImage").values
+      os_images.detect { |img| img.name == image_name } ? true : false
     end
 
     def is_vm_image(image_name)
-      vm_images = self.get_images("VMImage").values
-      vm_images.detect {|img| img.name == image_name} ? true : false
+      vm_images = get_images("VMImage").values
+      vm_images.detect { |img| img.name == image_name } ? true : false
     end
 
     def exists?(name)
-      self.all.detect {|img| img.name == name} ? true : false
+      all.detect { |img| img.name == name } ? true : false
     end
 
     def find(name)
-      self.load[name]
+      load[name]
     end
   end
 end
@@ -81,13 +82,13 @@ module Azure
     attr_accessor :category, :label
     attr_accessor :name, :os, :eula, :description, :location
     def initialize(image)
-      @category = image.at_css('Category').content
-      @label = image.at_css('Label').content
-      @name = image.at_css('Name').content
-      @os = image.at_css('OS').content
-      @location = image.at_css('Location').content.gsub(";", ", ") if image.at_css('Location')
-      @eula = image.at_css('Eula').content if image.at_css('Eula')
-      @description = image.at_css('Description').content if image.at_css('Description')
+      @category = image.at_css("Category").content
+      @label = image.at_css("Label").content
+      @name = image.at_css("Name").content
+      @os = image.at_css("OS").content
+      @location = image.at_css("Location").content.gsub(";", ", ") if image.at_css("Location")
+      @eula = image.at_css("Eula").content if image.at_css("Eula")
+      @description = image.at_css("Description").content if image.at_css("Description")
     end
   end
 end

data/lib/azure/service_management/loadbalancer.rb

@@ -1,6 +1,6 @@
 #
 # Author:: Aiman Alsari (aiman.alsari@gmail.com)
-# Copyright:: Copyright (c) 2013 Opscode, Inc.
+# Copyright:: Copyright 2013-2018 Chef Software, Inc.
 # License:: Apache License, Version 2.0
 #
 # Licensed under the Apache License, Version 2.0 (the "License");
@@ -48,27 +48,27 @@ module Azure
     end
 
     def parse(lbXML, hostedservicename)
-      @name = xml_content(lbXML, 'Name')
-      ip_configXML = lbXML.css('FrontendIpConfiguration')
-      @subnet = xml_content(ip_configXML, 'SubnetName')
-      @vip = xml_content(ip_configXML, 'StaticVirtualNetworkIPAddress')
+      @name = xml_content(lbXML, "Name")
+      ip_configXML = lbXML.css("FrontendIpConfiguration")
+      @subnet = xml_content(ip_configXML, "SubnetName")
+      @vip = xml_content(ip_configXML, "StaticVirtualNetworkIPAddress")
       @service = hostedservicename
       self
     end
 
     def create(params)
       if params[:azure_lb_static_vip] && !params[:azure_subnet_name]
-        Chef::Log.fatal 'Unable to create Loadbalancer, :azure_subnet_name needs to be set if :azure_lb_static_vip is set'
+        Chef::Log.fatal "Unable to create Loadbalancer, :azure_subnet_name needs to be set if :azure_lb_static_vip is set"
       end
-      builder = Nokogiri::XML::Builder.new(encoding: 'utf-8') do |xml|
-        xml.LoadBalancer(xmlns: 'http://schemas.microsoft.com/windowsazure') {
+      builder = Nokogiri::XML::Builder.new(encoding: "utf-8") do |xml|
+        xml.LoadBalancer(xmlns: "http://schemas.microsoft.com/windowsazure") do
           xml.Name params[:azure_load_balancer]
-          xml.FrontendIpConfiguration {
-            xml.Type 'Private'
+          xml.FrontendIpConfiguration do
+            xml.Type "Private"
             xml.SubnetName params[:azure_subnet_name] if params[:azure_subnet_name]
             xml.StaticVirtualNetworkIPAddress params[:azure_lb_static_vip] if params[:azure_lb_static_vip]
-          }
-        }
+          end
+        end
       end
       deploy_name = @connection.deploys.get_deploy_name_for_hostedservice(params[:azure_dns_name])
       servicecall = "hostedservices/#{params[:azure_dns_name]}/deployments/#{deploy_name}/loadbalancers"

data/lib/azure/service_management/rest.rb

@@ -1,6 +1,6 @@
 #
 # Author:: Barry Davis (barryd@jetstreamsoftware.com)
-# Copyright:: Copyright (c) 2010-2011 Opscode, Inc.
+# Copyright:: Copyright 2010-2018 Chef Software, Inc.
 # License:: Apache License, Version 2.0
 #
 # Licensed under the Apache License, Version 2.0 (the "License");
@@ -30,22 +30,22 @@ module AzureAPI
     end
 
     def query_azure(service_name,
-                    verb = 'get',
-                    body = '',
-                    params = '',
+                    verb = "get",
+                    body = "",
+                    params = "",
                     services = true,
                     content_type = nil)
-      svc_str = services ? '/services' : ''
+      svc_str = services ? "/services" : ""
       uri = URI.parse("#{@host_name}/#{@subscription_id}#{svc_str}/#{service_name}")
       scheme = !uri.scheme ? "https://" : ""
       request_url = "#{scheme}#{@host_name}/#{@subscription_id}#{svc_str}/#{service_name}"
-      print '.'
+      print "."
       response = http_query(request_url, verb, body, params, content_type)
       if response.code.to_i == 307
         Chef::Log.debug "Redirect to #{response['Location']}"
-        response = http_query(response['Location'], verb, body, params, content_type)
+        response = http_query(response["Location"], verb, body, params, content_type)
       end
-      @last_request_id = response['x-ms-request-id']
+      @last_request_id = response["x-ms-request-id"]
       response
     end
 
@@ -55,18 +55,18 @@ module AzureAPI
       http = http_setup(uri)
       request = request_setup(uri, verb, body, content_type)
       response = http.request(request)
-      @last_request_id = response['x-ms-request-id']
+      @last_request_id = response["x-ms-request-id"]
       response
     end
 
-    def query_for_completion()
+    def query_for_completion
       uri = URI.parse("#{@host_name}/#{@subscription_id}/operations/#{@last_request_id}")
       scheme = !uri.scheme ? "https://" : ""
       request_url = "#{scheme}#{@host_name}/#{@subscription_id}/operations/#{@last_request_id}"
-      response = http_query(request_url, 'get', '', '')
+      response = http_query(request_url, "get", "", "")
       if response.code.to_i == 307
         Chef::Log.debug "Redirect to #{response['Location']}"
-        response = http_query(response['Location'], 'get', '', '')
+        response = http_query(response["Location"], "get", "", "")
       end
       response
     end
@@ -87,20 +87,21 @@ module AzureAPI
       rescue OpenSSL::X509::CertificateError => err
         raise "Invalid Azure Certificate pem file. Error: #{err}"
       end
-        http.key = OpenSSL::PKey::RSA.new(@pem_file)
+      http.key = OpenSSL::PKey::RSA.new(@pem_file)
       http
     end
+
     def request_setup(uri, verb, body, content_type)
-      if verb == 'get'
+      if verb == "get"
         request = Net::HTTP::Get.new(uri.request_uri)
-      elsif verb == 'post'
+      elsif verb == "post"
         request = Net::HTTP::Post.new(uri.request_uri)
-      elsif verb == 'delete'
+      elsif verb == "delete"
         request = Net::HTTP::Delete.new(uri.request_uri)
-      elsif verb == 'put'
+      elsif verb == "put"
         request = Net::HTTP::Put.new(uri.request_uri)
       end
-      text = verb == 'put' && content_type.nil?
+      text = verb == "put" && content_type.nil?
       request["x-ms-version"] = "2014-05-01"
       request["content-type"] = text ? "text/plain" : "application/xml"
       request["accept"] = "application/xml"
@@ -117,7 +118,7 @@ module AzureAPI
       puts "=== response.inspect ==="
       puts response.inspect
       puts "=== all of the headers ==="
-      puts response.each_header { |h, j| puts h.inspect + ' : ' + j.inspect}
+      puts response.each_header { |h, j| puts h.inspect + " : " + j.inspect }
     end
   end
 end

data/lib/azure/service_management/role.rb

@@ -1,6 +1,6 @@
 #
 # Author:: Barry Davis (barryd@jetstreamsoftware.com)
-# Copyright:: Copyright (c) 2010-2011 Opscode, Inc.
+# Copyright:: Copyright 2010-2018 Chef Software, Inc.
 # License:: Apache License, Version 2.0
 #
 # Licensed under the Apache License, Version 2.0 (the "License");
@@ -15,8 +15,8 @@
 # See the License for the specific language governing permissions and
 # limitations under the License.
 #
-require 'securerandom'
-require 'azure/service_management/utility'
+require "securerandom"
+require "azure/service_management/utility"
 
 module Azure
   class Roles
@@ -26,6 +26,7 @@ module Azure
       @connection = connection
       @roles = nil
     end
+
     # do not use this unless you want a list of all roles(vms) in your subscription
     def all
       @roles = Array.new
@@ -48,16 +49,16 @@ module Azure
       host.find_role(role_name)
     end
 
-    def find(role_name, params= nil)
+    def find(role_name, params = nil)
       if params && params[:azure_dns_name]
         return find_in_hosted_service(role_name, params[:azure_dns_name])
       end
 
-      all if @roles == nil
+      all if @roles.nil?
 
       # TODO - optimize this lookup
       @roles.each do |role|
-        if(role.name == role_name)
+        if role.name == role_name
           return role
         end
       end
@@ -69,7 +70,7 @@ module Azure
       if roles && roles.length > 1
         return false
       end
-      return true
+      true
     end
 
     def exists?(name)
@@ -78,19 +79,19 @@ module Azure
 
     def delete(params)
       role = find(params[:name])
-      if role != nil
+      if !role.nil?
         roleXML = nil
         roleXML = @connection.query_azure("hostedservices/#{role.hostedservicename}", "get", "", "embed-detail=true")
-        osdisk = roleXML.css(roleXML, 'OSVirtualHardDisk')
-        disk_name = xml_content(osdisk, 'DiskName')
-        storage_account_name = xml_content(osdisk, 'MediaLink').gsub("http://", "").gsub(/.blob(.*)$/, "")
+        osdisk = roleXML.css(roleXML, "OSVirtualHardDisk")
+        disk_name = xml_content(osdisk, "DiskName")
+        storage_account_name = xml_content(osdisk, "MediaLink").gsub("http://", "").gsub(/.blob(.*)$/, "")
 
         if !params[:preserve_azure_os_disk] && !params[:preserve_azure_vhd] && !params[:wait]
           # default compmedia = true. So, it deletes role and associated resources
           check_and_delete_role_and_resources(params, role)
         else
           # compmedia = false. So, it deletes only role and not associated resources
-          check_and_delete_role_and_resources(params, role, compmedia=false)
+          check_and_delete_role_and_resources(params, role, compmedia = false)
           check_and_delete_disks(params, disk_name)
           check_and_delete_service(params)
         end
@@ -98,7 +99,7 @@ module Azure
       end
     end
 
-    def check_and_delete_role_and_resources(params, role, compmedia=true)
+    def check_and_delete_role_and_resources(params, role, compmedia = true)
       if alone_on_hostedservice(role)
         if !params[:preserve_azure_dns_name] && compmedia
           servicecall = "hostedservices/#{role.hostedservicename}"
@@ -107,10 +108,10 @@ module Azure
         end
       else
         servicecall = "hostedservices/#{role.hostedservicename}/deployments" +
-        "/#{role.deployname}/roles/#{role.name}"
+          "/#{role.deployname}/roles/#{role.name}"
       end
       if compmedia
-        @connection.query_azure(servicecall, "delete", "", "comp=media", wait=params[:wait])
+        @connection.query_azure(servicecall, "delete", "", "comp=media", wait = params[:wait])
       else
         @connection.query_azure(servicecall, "delete")
       end
@@ -127,13 +128,13 @@ module Azure
         # So Iteratively check for disk detachment from the VM while waiting for 5 minutes ,
         # exit otherwise after 12 attempts.
         for attempt in 0..12
-           break if @connection.query_azure(servicecall, "get").search("AttachedTo").text == ""
-           if attempt == 12 then puts "The associated disk could not be deleted due to time out." else sleep 25 end
+          break if @connection.query_azure(servicecall, "get").search("AttachedTo").text == ""
+          attempt == 12 ? (puts "The associated disk could not be deleted due to time out.") : (sleep 25)
         end
         unless params[:preserve_azure_vhd]
-          @connection.query_azure(servicecall, 'delete', '', 'comp=media', wait=params[:wait])
+          @connection.query_azure(servicecall, "delete", "", "comp=media", wait = params[:wait])
         else
-          @connection.query_azure(servicecall, 'delete')
+          @connection.query_azure(servicecall, "delete")
         end
       end
     end
@@ -151,11 +152,11 @@ module Azure
     end
 
     def check_and_delete_storage(params, disk_name, storage_account_name)
-     if params[:delete_azure_storage_account]
-        # Iteratively check for disk deletion
+      if params[:delete_azure_storage_account]
+         # Iteratively check for disk deletion
         for attempt in 0..12
-           break unless @connection.query_azure("disks").search("Name").text.include?(disk_name)
-           if attempt == 12 then puts "The associated disk could not be deleted due to time out." else sleep 25 end
+          break unless @connection.query_azure("disks").search("Name").text.include?(disk_name)
+          attempt == 12 ? (puts "The associated disk could not be deleted due to time out.") : (sleep 25)
         end
         begin
           @connection.query_azure("storageservices/#{storage_account_name}", "delete")
@@ -163,7 +164,7 @@ module Azure
           ui.warn("#{ex.message}")
           ui.warn("#{ex.backtrace.join("\n")}")
         end
-      end
+       end
     end
 
     def update(name, params)
@@ -184,55 +185,55 @@ module Azure
     attr_accessor :hostname, :tcpports, :udpports
     attr_accessor :role_xml, :os_type, :os_version
 
-      TCP_ENDPOINTS_MAPPING = { '3389' => 'Remote Desktop',
-                              '5986' => 'PowerShell',
-                              '22' => 'SSH',
-                              '21' => 'FTP',
-                              '25' => 'SMTP',
-                              '53' => 'DNS',
-                              '80' => 'HTTP',
-                              '110' => 'POP3',
-                              '143' => 'IMAP',
-                              '389' => 'LDAP',
-                              '443' => 'HTTPs',
-                              '587' => 'SMTPS',
-                              '995' => 'POP3S',
-                              '993' => 'IMAPS',
-                              '1433' => 'MSSQL',
-                              '3306' => 'MySQL'
-                              }
+    TCP_ENDPOINTS_MAPPING = { "3389" => "Remote Desktop",
+                              "5986" => "PowerShell",
+                              "22" => "SSH",
+                              "21" => "FTP",
+                              "25" => "SMTP",
+                              "53" => "DNS",
+                              "80" => "HTTP",
+                              "110" => "POP3",
+                              "143" => "IMAP",
+                              "389" => "LDAP",
+                              "443" => "HTTPs",
+                              "587" => "SMTPS",
+                              "995" => "POP3S",
+                              "993" => "IMAPS",
+                              "1433" => "MSSQL",
+                              "3306" => "MySQL"
+                            }
 
     def initialize(connection)
       @connection = connection
     end
 
     def parse(roleXML, hostedservicename, deployname)
-      @name = xml_content(roleXML, 'RoleName')
-      @status = xml_content(roleXML, 'InstanceStatus')
-      @size = xml_content(roleXML, 'InstanceSize')
-      @ipaddress = xml_content(roleXML, 'IpAddress')
-      @hostname = xml_content(roleXML, 'HostName')
+      @name = xml_content(roleXML, "RoleName")
+      @status = xml_content(roleXML, "InstanceStatus")
+      @size = xml_content(roleXML, "InstanceSize")
+      @ipaddress = xml_content(roleXML, "IpAddress")
+      @hostname = xml_content(roleXML, "HostName")
       @hostedservicename = hostedservicename
       @deployname = deployname
       @thumbprint = fetch_thumbprint
       @tcpports = Array.new
       @udpports = Array.new
 
-      endpoints = roleXML.css('InstanceEndpoint')
-      @publicipaddress = xml_content(endpoints[0], 'Vip') if !endpoints.empty?
+      endpoints = roleXML.css("InstanceEndpoint")
+      @publicipaddress = xml_content(endpoints[0], "Vip") if !endpoints.empty?
       endpoints.each do |endpoint|
-        if xml_content(endpoint, 'Name').downcase == 'ssh'
-          @sshport = xml_content(endpoint, 'PublicPort')
-        elsif xml_content(endpoint, 'Name').downcase == 'winrm'
-          @winrmport = xml_content(endpoint, 'PublicPort')
+        if xml_content(endpoint, "Name").casecmp("ssh").zero?
+          @sshport = xml_content(endpoint, "PublicPort")
+        elsif xml_content(endpoint, "Name").casecmp("winrm").zero?
+          @winrmport = xml_content(endpoint, "PublicPort")
         else
           hash = Hash.new
-          hash['Name'] = xml_content(endpoint, 'Name')
-          hash['Vip'] = xml_content(endpoint, 'Vip')
-          hash['PublicPort'] = xml_content(endpoint, 'PublicPort')
-          hash['LocalPort'] = xml_content(endpoint, 'LocalPort')
+          hash["Name"] = xml_content(endpoint, "Name")
+          hash["Vip"] = xml_content(endpoint, "Vip")
+          hash["PublicPort"] = xml_content(endpoint, "PublicPort")
+          hash["LocalPort"] = xml_content(endpoint, "LocalPort")
 
-          if xml_content(endpoint, 'Protocol') == 'tcp'
+          if xml_content(endpoint, "Protocol") == "tcp"
             @tcpports << hash
           else # == 'udp'
             @udpports << hash
@@ -243,31 +244,31 @@ module Azure
 
     def parse_role_list_xml(roleListXML)
       @role_xml = roleListXML
-      os_disk_xml = roleListXML.css('OSVirtualHardDisk')
-      @os_type = xml_content(os_disk_xml, 'OS')
-      @os_version = xml_content(os_disk_xml, 'SourceImageName')
+      os_disk_xml = roleListXML.css("OSVirtualHardDisk")
+      @os_type = xml_content(os_disk_xml, "OS")
+      @os_version = xml_content(os_disk_xml, "SourceImageName")
     end
 
     # Expects endpoint_param_string to be in the form {localport}:{publicport}:{lb_set_name}:{lb_probe_path}
     # Only localport is mandatory.
     def parse_endpoint_from_params(protocol, azure_vm_name, endpoint_param_string)
-      fields = endpoint_param_string.split(':').map(&:strip)
+      fields = endpoint_param_string.split(":").map(&:strip)
       hash = {}
-      hash['LocalPort'] = fields[0]
-      hash['Port'] = fields[1] || fields[0]
-      hash['LoadBalancerName'] = fields[2] if fields[2] != 'EXTERNAL' # TODO: hackity hack.. Shouldn't use magic words.
-      hash['LoadBalancedEndpointSetName'] = fields[3]
-      hash['Protocol'] = protocol
-      if TCP_ENDPOINTS_MAPPING.include?(hash['Port']) && protocol == 'TCP'
-        hash['Name'] = TCP_ENDPOINTS_MAPPING[hash['Port']]
+      hash["LocalPort"] = fields[0]
+      hash["Port"] = fields[1] || fields[0]
+      hash["LoadBalancerName"] = fields[2] if fields[2] != "EXTERNAL" # TODO: hackity hack.. Shouldn't use magic words.
+      hash["LoadBalancedEndpointSetName"] = fields[3]
+      hash["Protocol"] = protocol
+      if TCP_ENDPOINTS_MAPPING.include?(hash["Port"]) && protocol == "TCP"
+        hash["Name"] = TCP_ENDPOINTS_MAPPING[hash["Port"]]
       else
-        hash['Name'] = "#{protocol}Endpoint_chef_#{fields[0]}"
+        hash["Name"] = "#{protocol}Endpoint_chef_#{fields[0]}"
       end
       if fields[2]
-        hash['LoadBalancerProbe'] = {}
-        hash['LoadBalancerProbe']['Path'] = fields[4]
-        hash['LoadBalancerProbe']['Port'] = fields[0]
-        hash['LoadBalancerProbe']['Protocol'] = fields[4] ? 'HTTP' : protocol
+        hash["LoadBalancerProbe"] = {}
+        hash["LoadBalancerProbe"]["Path"] = fields[4]
+        hash["LoadBalancerProbe"]["Port"] = fields[0]
+        hash["LoadBalancerProbe"]["Protocol"] = fields[4] ? "HTTP" : protocol
       end
       hash
     end
@@ -283,252 +284,252 @@ module Azure
 
         if existing_endpoints
           existing_endpoints.each do |eep|
-            ep = eep if eep['LoadBalancedEndpointSetName'] && ep['LoadBalancedEndpointSetName'] && ( eep['LoadBalancedEndpointSetName'] == ep['LoadBalancedEndpointSetName'] )
+            ep = eep if eep["LoadBalancedEndpointSetName"] && ep["LoadBalancedEndpointSetName"] && ( eep["LoadBalancedEndpointSetName"] == ep["LoadBalancedEndpointSetName"] )
           end
         end
 
-        if ep['Port'] == params[:port] && ep['Protocol'].downcase == 'tcp'
+        if ep["Port"] == params[:port] && ep["Protocol"].casecmp("tcp").zero?
           puts("Skipping tcp-endpoints: #{ep['LocalPort']} because this port is already in use by ssh/winrm endpoint in current VM.")
           next
         end
 
-        xml.InputEndpoint {
-          xml.LoadBalancedEndpointSetName ep['LoadBalancedEndpointSetName'] if ep['LoadBalancedEndpointSetName']
-          xml.LocalPort ep['LocalPort']
-          xml.Name ep['Name']
-          xml.Port ep['Port']
-          if ep['LoadBalancerProbe']
-            xml.LoadBalancerProbe {
-              xml.Path ep['LoadBalancerProbe']['Path'] if ep['LoadBalancerProbe']['Path']
-              xml.Port ep['LoadBalancerProbe']['Port']
-              xml.Protocol ep['LoadBalancerProbe']['Protocol']
-              xml.IntervalInSeconds ep['LoadBalancerProbe']['IntervalInSeconds'] if ep['LoadBalancerProbe']['IntervalInSeconds']
-              xml.TimeoutInSeconds ep['LoadBalancerProbe']['TimeoutInSeconds'] if ep['LoadBalancerProbe']['TimeoutInSeconds']
-            }
+        xml.InputEndpoint do
+          xml.LoadBalancedEndpointSetName ep["LoadBalancedEndpointSetName"] if ep["LoadBalancedEndpointSetName"]
+          xml.LocalPort ep["LocalPort"]
+          xml.Name ep["Name"]
+          xml.Port ep["Port"]
+          if ep["LoadBalancerProbe"]
+            xml.LoadBalancerProbe do
+              xml.Path ep["LoadBalancerProbe"]["Path"] if ep["LoadBalancerProbe"]["Path"]
+              xml.Port ep["LoadBalancerProbe"]["Port"]
+              xml.Protocol ep["LoadBalancerProbe"]["Protocol"]
+              xml.IntervalInSeconds ep["LoadBalancerProbe"]["IntervalInSeconds"] if ep["LoadBalancerProbe"]["IntervalInSeconds"]
+              xml.TimeoutInSeconds ep["LoadBalancerProbe"]["TimeoutInSeconds"] if ep["LoadBalancerProbe"]["TimeoutInSeconds"]
+            end
           end
-          xml.Protocol ep['Protocol']
-          xml.EnableDirectServerReturn ep['EnableDirectServerReturn'] if ep['EnableDirectServerReturn']
-          xml.LoadBalancerName ep['LoadBalancerName'] if ep['LoadBalancerName']
-          xml.IdleTimeoutInMinutes ep['IdleTimeoutInMinutes'] if ep['IdleTimeoutInMinutes']
-        }
+          xml.Protocol ep["Protocol"]
+          xml.EnableDirectServerReturn ep["EnableDirectServerReturn"] if ep["EnableDirectServerReturn"]
+          xml.LoadBalancerName ep["LoadBalancerName"] if ep["LoadBalancerName"]
+          xml.IdleTimeoutInMinutes ep["IdleTimeoutInMinutes"] if ep["IdleTimeoutInMinutes"]
+        end
       end
     end
 
     def fetch_thumbprint
       query_result = connection.query_azure("hostedservices/#{@hostedservicename}/deployments/#{@hostedservicename}/roles/#{@name}")
-      query_result.at_css("DefaultWinRmCertificateThumbprint").nil? ? '' : query_result.at_css("DefaultWinRmCertificateThumbprint").text
+      query_result.at_css("DefaultWinRmCertificateThumbprint").nil? ? "" : query_result.at_css("DefaultWinRmCertificateThumbprint").text
     end
 
     def setup(params)
       azure_user_domain_name = params[:azure_user_domain_name] || params[:azure_domain_name]
       builder = Nokogiri::XML::Builder.new do |xml|
         xml.PersistentVMRole(
-          'xmlns'=>'http://schemas.microsoft.com/windowsazure',
-          'xmlns:i'=>'http://www.w3.org/2001/XMLSchema-instance'
-        ) {
-          xml.RoleName {xml.text params[:azure_vm_name]}
-          xml.OsVersion('i:nil' => 'true')
-          xml.RoleType 'PersistentVMRole'
-
-          xml.ConfigurationSets {
-            if params[:os_type] == 'Linux'
-              xml.ConfigurationSet('i:type' => 'LinuxProvisioningConfigurationSet') {
-                xml.ConfigurationSetType 'LinuxProvisioningConfiguration'
+          "xmlns" => "http://schemas.microsoft.com/windowsazure",
+          "xmlns:i" => "http://www.w3.org/2001/XMLSchema-instance"
+        ) do
+          xml.RoleName { xml.text params[:azure_vm_name] }
+          xml.OsVersion("i:nil" => "true")
+          xml.RoleType "PersistentVMRole"
+
+          xml.ConfigurationSets do
+            if params[:os_type] == "Linux"
+              xml.ConfigurationSet("i:type" => "LinuxProvisioningConfigurationSet") do
+                xml.ConfigurationSetType "LinuxProvisioningConfiguration"
                 xml.HostName params[:azure_vm_name]
                 xml.UserName params[:ssh_user]
                 unless params[:identity_file].nil?
-                  xml.DisableSshPasswordAuthentication 'true'
-                  xml.SSH {
-                     xml.PublicKeys {
-                       xml.PublicKey {
-                         xml.Fingerprint params[:fingerprint].to_s.upcase
-                         xml.Path '/home/' + params[:ssh_user] + '/.ssh/authorized_keys'
-                       }
-                     }
-                  }
+                  xml.DisableSshPasswordAuthentication "true"
+                  xml.SSH do
+                    xml.PublicKeys do
+                      xml.PublicKey do
+                        xml.Fingerprint params[:fingerprint].to_s.upcase
+                        xml.Path "/home/" + params[:ssh_user] + "/.ssh/authorized_keys"
+                      end
+                    end
+                  end
                 else
                   xml.UserPassword params[:ssh_password]
-                  xml.DisableSshPasswordAuthentication 'false'
+                  xml.DisableSshPasswordAuthentication "false"
                 end
-              }
-            elsif params[:os_type] == 'Windows'
-              xml.ConfigurationSet('i:type' => 'WindowsProvisioningConfigurationSet') {
-                xml.ConfigurationSetType 'WindowsProvisioningConfiguration'
+              end
+            elsif params[:os_type] == "Windows"
+              xml.ConfigurationSet("i:type" => "WindowsProvisioningConfigurationSet") do
+                xml.ConfigurationSetType "WindowsProvisioningConfiguration"
                 xml.ComputerName params[:azure_vm_name]
                 xml.AdminPassword params[:admin_password]
-                xml.ResetPasswordOnFirstLogon 'false'
-                xml.EnableAutomaticUpdates 'false'
+                xml.ResetPasswordOnFirstLogon "false"
+                xml.EnableAutomaticUpdates "false"
                 if params[:azure_domain_name]
-                  xml.DomainJoin {
-                    xml.Credentials {
+                  xml.DomainJoin do
+                    xml.Credentials do
                       xml.Domain azure_user_domain_name
-                     xml.Username params[:azure_domain_user]
-                    xml.Password params[:azure_domain_passwd]
-                   }
-                   xml.JoinDomain params[:azure_domain_name]
-                   xml.MachineObjectOU params[:azure_domain_ou_dn] if params[:azure_domain_ou_dn]
-                  }
+                      xml.Username params[:azure_domain_user]
+                      xml.Password params[:azure_domain_passwd]
+                    end
+                    xml.JoinDomain params[:azure_domain_name]
+                    xml.MachineObjectOU params[:azure_domain_ou_dn] if params[:azure_domain_ou_dn]
+                  end
                 end
-                  if params[:bootstrap_proto].downcase == 'winrm'
-                    if params[:ssl_cert_fingerprint]
-                      xml.StoredCertificateSettings {
-                        xml.CertificateSetting {
-                          xml.StoreLocation "LocalMachine"
-                          xml.StoreName "My"
-                          xml.Thumbprint params[:ssl_cert_fingerprint]
-                        }
-                      }
+                if params[:bootstrap_proto].casecmp("winrm").zero?
+                  if params[:ssl_cert_fingerprint]
+                    xml.StoredCertificateSettings do
+                      xml.CertificateSetting do
+                        xml.StoreLocation "LocalMachine"
+                        xml.StoreName "My"
+                        xml.Thumbprint params[:ssl_cert_fingerprint]
+                      end
                     end
-                    xml.WinRM {
-                      xml.Listeners {
-                       if params[:winrm_transport] == "ssl" || params[:ssl_cert_fingerprint]
-                        xml.Listener {
+                  end
+                  xml.WinRM do
+                    xml.Listeners do
+                      if params[:winrm_transport] == "ssl" || params[:ssl_cert_fingerprint]
+                        xml.Listener do
                           xml.CertificateThumbprint params[:ssl_cert_fingerprint] if params[:ssl_cert_fingerprint]
-                          xml.Protocol 'Https'
-                        }
-                        else
-                        xml.Listener {
-                          xml.Protocol 'Http'
-                        }
+                          xml.Protocol "Https"
                         end
-                      }
-                    }
+                      else
+                        xml.Listener do
+                          xml.Protocol "Http"
+                        end
+                       end
+                    end
                   end
+                end
                 xml.AdminUsername params[:winrm_user]
-                if params[:bootstrap_proto].downcase == 'winrm' && (params[:winrm_max_timeout] || params[:winrm_max_memoryPerShell])
-                  xml.AdditionalUnattendContent {
-                    xml.Passes {
-                      xml.UnattendPass {
-                        xml.PassName 'oobeSystem'
-                        xml.Components {
-                          xml.UnattendComponent {
-                            xml.ComponentName 'Microsoft-Windows-Shell-Setup'
-                            xml.ComponentSettings {
-                              xml.ComponentSetting {
-                                xml.SettingName 'AutoLogon'
+                if params[:bootstrap_proto].casecmp("winrm").zero? && (params[:winrm_max_timeout] || params[:winrm_max_memoryPerShell])
+                  xml.AdditionalUnattendContent do
+                    xml.Passes do
+                      xml.UnattendPass do
+                        xml.PassName "oobeSystem"
+                        xml.Components do
+                          xml.UnattendComponent do
+                            xml.ComponentName "Microsoft-Windows-Shell-Setup"
+                            xml.ComponentSettings do
+                              xml.ComponentSetting do
+                                xml.SettingName "AutoLogon"
                                 xml.Content Base64.encode64(
                                   Nokogiri::XML::Builder.new do |auto_logon_xml|
-                                    auto_logon_xml.AutoLogon {
+                                    auto_logon_xml.AutoLogon do
                                       auto_logon_xml.Username params[:winrm_user]
-                                      auto_logon_xml.Password {
+                                      auto_logon_xml.Password do
                                         auto_logon_xml.Value params[:admin_password]
                                         auto_logon_xml.PlainText true
-                                      }
+                                      end
                                       auto_logon_xml.LogonCount 1
                                       auto_logon_xml.Enabled true
-                                    }
+                                    end
                                   end.to_xml(save_with: Nokogiri::XML::Node::SaveOptions::NO_DECLARATION)
                                 ).strip
-                              }
-                              xml.ComponentSetting {
-                                xml.SettingName 'FirstLogonCommands'
+                              end
+                              xml.ComponentSetting do
+                                xml.SettingName "FirstLogonCommands"
                                 xml.Content Base64.encode64(
                                   Nokogiri::XML::Builder.new do |first_logon_xml|
-                                    first_logon_xml.FirstLogonCommands {
+                                    first_logon_xml.FirstLogonCommands do
                                       if params[:winrm_max_timeout]
-                                        first_logon_xml.SynchronousCommand('wcm:action' => 'add') {
+                                        first_logon_xml.SynchronousCommand("wcm:action" => "add") do
                                           first_logon_xml.Order 1
                                           first_logon_xml.CommandLine "cmd.exe /c winrm set winrm/config @{MaxTimeoutms=\"#{params[:winrm_max_timeout]}\"}"
                                           first_logon_xml.Description "Bump WinRM max timeout to #{params[:winrm_max_timeout]} milliseconds"
-                                        }
+                                        end
                                       end
 
                                       if params[:winrm_max_memoryPerShell]
-                                        first_logon_xml.SynchronousCommand('wcm:action' => 'add') {
+                                        first_logon_xml.SynchronousCommand("wcm:action" => "add") do
                                           first_logon_xml.Order 2
                                           first_logon_xml.CommandLine "cmd.exe /c winrm set winrm/config/winrs @{MaxMemoryPerShellMB=\"#{params[:winrm_max_memoryPerShell]}\"}"
                                           first_logon_xml.Description "Bump WinRM max memory per shell to #{params[:winrm_max_memoryPerShell]} MB"
-                                        }
+                                        end
                                       end
-                                    }
+                                    end
                                   end.to_xml(save_with: Nokogiri::XML::Node::SaveOptions::NO_DECLARATION)
                                 ).strip
-                              }
-                            }
-                          }
-                        }
-                      }
-                    }
-                  }
+                              end
+                            end
+                          end
+                        end
+                      end
+                    end
+                  end
                 end
-              }
+              end
             end
 
-            xml.ConfigurationSet('i:type' => 'NetworkConfigurationSet') {
-              xml.ConfigurationSetType 'NetworkConfiguration'
-              xml.InputEndpoints {
+            xml.ConfigurationSet("i:type" => "NetworkConfigurationSet") do
+              xml.ConfigurationSetType "NetworkConfiguration"
+              xml.InputEndpoints do
 
                 #1. bootstrap_proto = 'winrm' for windows => Set winrm port
                 #2. bootstrap_proto = 'ssh' for windows and linux => Set ssh port
                 #3. bootstrap_proto = 'cloud-api' for windows and linux => Set no port
-                if params[:os_type] == 'Windows' and params[:bootstrap_proto].downcase == 'winrm'
-                  xml.InputEndpoint {
-                  if params[:winrm_transport] == "ssl"
-                    xml.LocalPort '5986'
-                  else
-                    xml.LocalPort '5985'
+                if (params[:os_type] == "Windows") && (params[:bootstrap_proto].casecmp("winrm").zero?)
+                  xml.InputEndpoint do
+                    if params[:winrm_transport] == "ssl"
+                      xml.LocalPort "5986"
+                    else
+                      xml.LocalPort "5985"
+                    end
+                    xml.Name "WinRM"
+                    xml.Port params[:port]
+                    xml.Protocol "TCP"
+                  end
+                elsif params[:bootstrap_proto].casecmp("ssh").zero?
+                  xml.InputEndpoint do
+                    xml.LocalPort "22"
+                    xml.Name "SSH"
+                    xml.Port params[:port]
+                    xml.Protocol "TCP"
                   end
-                  xml.Name 'WinRM'
-                  xml.Port params[:port]
-                  xml.Protocol 'TCP'
-                }
-                elsif(params[:bootstrap_proto].downcase == 'ssh')
-                  xml.InputEndpoint {
-                  xml.LocalPort '22'
-                  xml.Name 'SSH'
-                  xml.Port params[:port]
-                  xml.Protocol 'TCP'
-                }
                 end
                 all_endpoints = Array.new
 
                 if params[:tcp_endpoints]
-                  params[:tcp_endpoints].split(',').map(&:strip).each do |endpoint|
-                    all_endpoints << parse_endpoint_from_params('TCP', params[:azure_vm_name], endpoint)
+                  params[:tcp_endpoints].split(",").map(&:strip).each do |endpoint|
+                    all_endpoints << parse_endpoint_from_params("TCP", params[:azure_vm_name], endpoint)
                   end
                 end
                 if params[:udp_endpoints]
-                  params[:udp_endpoints].split(',').map(&:strip).each do |endpoint|
-                    all_endpoints << parse_endpoint_from_params('UDP', params[:azure_vm_name], endpoint)
+                  params[:udp_endpoints].split(",").map(&:strip).each do |endpoint|
+                    all_endpoints << parse_endpoint_from_params("UDP", params[:azure_vm_name], endpoint)
                   end
                 end
                 add_endpoints_to_xml(xml, all_endpoints, params) if all_endpoints.any?
-              }
+              end
               if params[:azure_subnet_name]
-                xml.SubnetNames {
+                xml.SubnetNames do
                   xml.SubnetName params[:azure_subnet_name]
-                }
+                end
               end
-            }
-          }
+            end
+          end
 
           # Azure resource extension support
-          if params[:bootstrap_proto] == 'cloud-api'
-            xml.ResourceExtensionReferences {
-              xml.ResourceExtensionReference {
+          if params[:bootstrap_proto] == "cloud-api"
+            xml.ResourceExtensionReferences do
+              xml.ResourceExtensionReference do
                 xml.ReferenceName params[:chef_extension]
                 xml.Publisher params[:chef_extension_publisher]
                 xml.Name params[:chef_extension]
                 xml.Version params[:chef_extension_version]
-                xml.ResourceExtensionParameterValues {
+                xml.ResourceExtensionParameterValues do
                   if params[:chef_extension_public_param]
-                    xml.ResourceExtensionParameterValue {
+                    xml.ResourceExtensionParameterValue do
                       xml.Key "PublicParams"
                       xml.Value Base64.encode64(params[:chef_extension_public_param].to_json)
                       xml.Type "Public"
-                    }
+                    end
                   end
                   if params[:chef_extension_private_param]
-                    xml.ResourceExtensionParameterValue {
+                    xml.ResourceExtensionParameterValue do
                       xml.Key "PrivateParams"
                       xml.Value Base64.encode64(params[:chef_extension_private_param].to_json)
                       xml.Type "Private"
-                    }
+                    end
                   end
-                }
+                end
                 xml.State "Enable"
-              }
-            }
+              end
+            end
           end
 
           if params[:azure_availability_set]
@@ -540,26 +541,26 @@ module Azure
           xml.Label Base64.encode64(params[:azure_vm_name]).strip
 
           #OSVirtualHardDisk not required in case azure_source_image is a VMImage
-          unless(params[:is_vm_image])
-            xml.OSVirtualHardDisk {
+          unless params[:is_vm_image]
+            xml.OSVirtualHardDisk do
               disk_name = params[:azure_os_disk_name] || "disk_" + SecureRandom.uuid
               xml.DiskName disk_name
-              domain_suffix = params[:azure_api_host_name] ? params[:azure_api_host_name].scan(/core.*/)[0] : ''
-              xml.MediaLink 'http://' + params[:azure_storage_account] + '.blob.' + domain_suffix + '/vhds/' + disk_name + '.vhd'
+              domain_suffix = params[:azure_api_host_name] ? params[:azure_api_host_name].scan(/core.*/)[0] : ""
+              xml.MediaLink "http://" + params[:azure_storage_account] + ".blob." + domain_suffix + "/vhds/" + disk_name + ".vhd"
               xml.SourceImageName params[:azure_source_image]
-            }
+            end
           end
 
           xml.RoleSize params[:azure_vm_size]
-          xml.ProvisionGuestAgent true if params[:bootstrap_proto] == 'cloud-api'
-        }
+          xml.ProvisionGuestAgent true if params[:bootstrap_proto] == "cloud-api"
+        end
       end
       builder.doc
     end
 
     def create(params, roleXML)
       servicecall = "hostedservices/#{params[:azure_dns_name]}/deployments" +
-      "/#{params['deploy_name']}/roles"
+        "/#{params['deploy_name']}/roles"
       @connection.query_azure(servicecall, "post", roleXML.to_xml)
     end
 
@@ -576,31 +577,31 @@ module Azure
       ## using the required values of the updated role_xml
       builder = Nokogiri::XML::Builder.new do |xml|
         xml.PersistentVMRole(
-          'xmlns' => 'http://schemas.microsoft.com/windowsazure',
-          'xmlns:i' => 'http://www.w3.org/2001/XMLSchema-instance'
-        ) {
-            xml.ConfigurationSets role_xml.at_css('ConfigurationSets').children if !role_xml.at_css('ConfigurationSets').nil?
-            xml.ResourceExtensionReferences role_xml.at_css('ResourceExtensionReferences').children if !role_xml.at_css('ResourceExtensionReferences').nil?
-            xml.AvailabilitySetName role_xml.at_css('AvailabilitySetName').children if !role_xml.at_css('AvailabilitySetName').nil?
-            xml.DataVirtualHardDisks role_xml.at_css('DataVirtualHardDisks').children if !role_xml.at_css('DataVirtualHardDisks').nil?
-            xml.OSVirtualHardDisk role_xml.at_css('OSVirtualHardDisk').children if !role_xml.at_css('OSVirtualHardDisk').nil?
-            xml.RoleSize role_xml.at_css('RoleSize').children if !role_xml.at_css('RoleSize').nil?
-            xml.ProvisionGuestAgent role_xml.at_css('ProvisionGuestAgent').children if !role_xml.at_css('ProvisionGuestAgent').nil?
-        }
+          "xmlns" => "http://schemas.microsoft.com/windowsazure",
+          "xmlns:i" => "http://www.w3.org/2001/XMLSchema-instance"
+        ) do
+          xml.ConfigurationSets role_xml.at_css("ConfigurationSets").children if !role_xml.at_css("ConfigurationSets").nil?
+          xml.ResourceExtensionReferences role_xml.at_css("ResourceExtensionReferences").children if !role_xml.at_css("ResourceExtensionReferences").nil?
+          xml.AvailabilitySetName role_xml.at_css("AvailabilitySetName").children if !role_xml.at_css("AvailabilitySetName").nil?
+          xml.DataVirtualHardDisks role_xml.at_css("DataVirtualHardDisks").children if !role_xml.at_css("DataVirtualHardDisks").nil?
+          xml.OSVirtualHardDisk role_xml.at_css("OSVirtualHardDisk").children if !role_xml.at_css("OSVirtualHardDisk").nil?
+          xml.RoleSize role_xml.at_css("RoleSize").children if !role_xml.at_css("RoleSize").nil?
+          xml.ProvisionGuestAgent role_xml.at_css("ProvisionGuestAgent").children if !role_xml.at_css("ProvisionGuestAgent").nil?
+        end
       end
 
-      builder.doc.to_xml.gsub("&lt\;","<").gsub("&gt\;",">")
+      builder.doc.to_xml.gsub("&lt\;", "<").gsub("&gt\;", ">")
     end
 
     def update_role_xml_for_extension(roleXML, params)
       ## check if 'ResourceExtensionReferences' node already exist in the XML,
       ## if no add it, else retrieve the object of the existing node
-      add_resource_extension_references = roleXML.at_css('ResourceExtensionReferences').nil?
+      add_resource_extension_references = roleXML.at_css("ResourceExtensionReferences").nil?
 
       if add_resource_extension_references
-        resource_extension_references = Nokogiri::XML::Node.new('ResourceExtensionReferences', roleXML)
+        resource_extension_references = Nokogiri::XML::Node.new("ResourceExtensionReferences", roleXML)
       else
-        resource_extension_references = roleXML.css('ResourceExtensionReferences')
+        resource_extension_references = roleXML.css("ResourceExtensionReferences")
       end
 
       ## check if Azure Chef Extension is already installed on the given server,
@@ -608,8 +609,8 @@ module Azure
       ## already installed
       ext = nil
       if !add_resource_extension_references
-        if !resource_extension_references.at_css('ReferenceName').nil?
-          resource_extension_references.css('ReferenceName').each { |node| ext = node if node.content == params[:chef_extension] }
+        if !resource_extension_references.at_css("ReferenceName").nil?
+          resource_extension_references.css("ReferenceName").each { |node| ext = node if node.content == params[:chef_extension] }
         end
       end
 
@@ -617,56 +618,56 @@ module Azure
 
       ## create Azure Chef Extension config and add it in the role_xml
       if add_resource_extension_reference
-        resource_extension_reference = Nokogiri::XML::Node.new('ResourceExtensionReference', roleXML)
+        resource_extension_reference = Nokogiri::XML::Node.new("ResourceExtensionReference", roleXML)
 
-        reference_name = Nokogiri::XML::Node.new('ReferenceName', roleXML)
+        reference_name = Nokogiri::XML::Node.new("ReferenceName", roleXML)
         reference_name.content = params[:chef_extension]
         resource_extension_reference.add_child(reference_name)
 
-        publisher = Nokogiri::XML::Node.new('Publisher', roleXML)
+        publisher = Nokogiri::XML::Node.new("Publisher", roleXML)
         publisher.content = params[:chef_extension_publisher]
         resource_extension_reference.add_child(publisher)
 
-        name = Nokogiri::XML::Node.new('Name', roleXML)
+        name = Nokogiri::XML::Node.new("Name", roleXML)
         name.content = params[:chef_extension]
         resource_extension_reference.add_child(name)
 
-        version = Nokogiri::XML::Node.new('Version', roleXML)
+        version = Nokogiri::XML::Node.new("Version", roleXML)
         version.content = params[:chef_extension_version]
         resource_extension_reference.add_child(version)
 
-        resource_extension_parameter_values = Nokogiri::XML::Node.new('ResourceExtensionParameterValues', roleXML)
+        resource_extension_parameter_values = Nokogiri::XML::Node.new("ResourceExtensionParameterValues", roleXML)
         if params[:chef_extension_public_param]
-          resource_extension_parameter_value = Nokogiri::XML::Node.new('ResourceExtensionParameterValue', roleXML)
+          resource_extension_parameter_value = Nokogiri::XML::Node.new("ResourceExtensionParameterValue", roleXML)
 
-          key = Nokogiri::XML::Node.new('Key', roleXML)
-          key.content = 'PublicParams'
+          key = Nokogiri::XML::Node.new("Key", roleXML)
+          key.content = "PublicParams"
           resource_extension_parameter_value.add_child(key)
 
-          value = Nokogiri::XML::Node.new('Value', roleXML)
+          value = Nokogiri::XML::Node.new("Value", roleXML)
           value.content = Base64.encode64(params[:chef_extension_public_param].to_json)
           resource_extension_parameter_value.add_child(value)
 
-          type = Nokogiri::XML::Node.new('Type', roleXML)
-          type.content = 'Public'
+          type = Nokogiri::XML::Node.new("Type", roleXML)
+          type.content = "Public"
           resource_extension_parameter_value.add_child(type)
 
           resource_extension_parameter_values.add_child(resource_extension_parameter_value)
         end
 
         if params[:chef_extension_private_param]
-          resource_extension_parameter_value = Nokogiri::XML::Node.new('ResourceExtensionParameterValue', roleXML)
+          resource_extension_parameter_value = Nokogiri::XML::Node.new("ResourceExtensionParameterValue", roleXML)
 
-          key = Nokogiri::XML::Node.new('Key', roleXML)
-          key.content = 'PrivateParams'
+          key = Nokogiri::XML::Node.new("Key", roleXML)
+          key.content = "PrivateParams"
           resource_extension_parameter_value.add_child(key)
 
-          value = Nokogiri::XML::Node.new('Value', roleXML)
+          value = Nokogiri::XML::Node.new("Value", roleXML)
           value.content = Base64.encode64(params[:chef_extension_private_param].to_json)
           resource_extension_parameter_value.add_child(value)
 
-          type = Nokogiri::XML::Node.new('Type', roleXML)
-          type.content = 'Private'
+          type = Nokogiri::XML::Node.new("Type", roleXML)
+          type.content = "Private"
           resource_extension_parameter_value.add_child(type)
 
           resource_extension_parameter_values.add_child(resource_extension_parameter_value)
@@ -674,8 +675,8 @@ module Azure
 
         resource_extension_reference.add_child(resource_extension_parameter_values)
 
-        state = Nokogiri::XML::Node.new('State', roleXML)
-        state.content = 'enable'
+        state = Nokogiri::XML::Node.new("State", roleXML)
+        state.content = "enable"
         resource_extension_reference.add_child(state)
 
         if add_resource_extension_references
@@ -686,13 +687,13 @@ module Azure
 
         roleXML.add_child(resource_extension_references) if add_resource_extension_references
 
-        add_provision_guest_agent = roleXML.at_css('ProvisionGuestAgent').nil?
+        add_provision_guest_agent = roleXML.at_css("ProvisionGuestAgent").nil?
 
         if add_provision_guest_agent
-          provision_guest_agent = Nokogiri::XML::Node.new('ProvisionGuestAgent', roleXML)
+          provision_guest_agent = Nokogiri::XML::Node.new("ProvisionGuestAgent", roleXML)
           provision_guest_agent.content = true
         else
-          provision_guest_agent = roleXML.css('ProvisionGuestAgent')
+          provision_guest_agent = roleXML.css("ProvisionGuestAgent")
           provision_guest_agent.first.content = true
         end
 
@@ -707,12 +708,12 @@ module Azure
     def update(name, params, roleXML)
       puts "Updating server role..."
       servicecall = "hostedservices/#{params[:azure_dns_name]}" +
-      "/deployments/#{params[:deploy_name]}/roles/#{name}"
-      ret_val = @connection.query_azure(servicecall, 'put', roleXML, '', true, true, 'application/xml')
+        "/deployments/#{params[:deploy_name]}/roles/#{name}"
+      ret_val = @connection.query_azure(servicecall, "put", roleXML, "", true, true, "application/xml")
       error_code, error_message = error_from_response_xml(ret_val)
       if error_code.length > 0
         Chef::Log.debug(ret_val.to_s)
-        raise 'Unable to update role:' + error_code + ' : ' + error_message
+        raise "Unable to update role:" + error_code + " : " + error_message
       end
     end
   end