azure-armrest 0.10.0 → 0.11.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: a58247b13fa15be3fc773cb02e734ae36d3a21db9be243ecf4fa3b66014e3117
-  data.tar.gz: 123caf6a1a8ec857e5da499357b41ef0177346d938fe06f3a0d933b029bddf77
+  metadata.gz: c05fdb104f21ace674e10937a178a0ba1976c11f501d708107e5543976b1f8ae
+  data.tar.gz: 1d78a637421c4166578bf53a56f24dd7b7781dc74323eb02659f0e72ee9c55b0
 SHA512:
-  metadata.gz: e13e25b3086f351299f9e1b17c0be38a3610d055f44f6528883823e692015f6be582fd5281a4113c2a243226510dd3cdee8d5bf753f463f344bb263b5104aa28
-  data.tar.gz: d7b81384f7f93248088536cc366e6e0e17cae8ad2ae9b94b3c287fd0f39b5dd4664dc5e49e55e8d5990643ec9bdc94679699798c23e07b5850d5e9f24d16023f
+  metadata.gz: 4b20b6dcc409488a5559dce6be791448ebdeb78a077e48453c3aa8de143573c508ca92845f3725fc5ca83acb2c0c4970effaf82ed675a9c191b0e281bae3880f
+  data.tar.gz: ebc6a58804b759f3bfed65436e742dd1f7d815df6df252184a6da066fe119725b124287a31133cd49fa5740affe8ff0bbb73f1ecabcf087cb8521d7f118839e2

data/.codeclimate.yml

@@ -0,0 +1,16 @@
+prepare:
+  fetch:
+  - url: https://raw.githubusercontent.com/ManageIQ/manageiq-style/master/.rubocop_base.yml
+    path: ".rubocop_base.yml"
+  - url: https://raw.githubusercontent.com/ManageIQ/manageiq-style/master/.rubocop_cc_base.yml
+    path: ".rubocop_cc_base.yml"
+  - url: https://raw.githubusercontent.com/ManageIQ/manageiq-style/master/styles/base.yml
+    path: styles/base.yml
+  - url: https://raw.githubusercontent.com/ManageIQ/manageiq-style/master/styles/cc_base.yml
+    path: styles/cc_base.yml
+plugins:
+  rubocop:
+    enabled: true
+    config: ".rubocop_cc.yml"
+    channel: rubocop-0-82
+version: '2'

data/.rubocop.yml

@@ -1,4 +1,4 @@
+inherit_gem:
+  manageiq-style: ".rubocop_base.yml"
 inherit_from:
-- https://raw.githubusercontent.com/ManageIQ/guides/master/.rubocop_base.yml
-# put all local rubocop config into .rubocop_local.yml as it will be loaded by .rubocop_cc.yml as well
-- .rubocop_local.yml
+- ".rubocop_local.yml"

data/.rubocop_cc.yml

@@ -0,0 +1,4 @@
+inherit_from:
+- ".rubocop_base.yml"
+- ".rubocop_cc_base.yml"
+- ".rubocop_local.yml"

data/.travis.yml

@@ -1,5 +1,4 @@
 language: ruby
-sudo: false
 cache: bundler
 rvm:
 - "2.5.7"
@@ -11,4 +10,9 @@ matrix:
   - rvm: ruby-head
   - rvm: jruby-head
   fast_finish: true
-after_script: bundle exec codeclimate-test-reporter
+before_script:
+  - curl -L https://codeclimate.com/downloads/test-reporter/test-reporter-latest-linux-amd64 > ./cc-test-reporter
+  - chmod +x ./cc-test-reporter
+  - ./cc-test-reporter before-build
+after_script:
+  - ./cc-test-reporter after-build --exit-code $TRAVIS_TEST_RESULT

data/CHANGES

@@ -1,3 +1,10 @@
+= 0.11.0 - 20-Apr-2021
+* Modify the VirtualMachineService#get method to include instance view
+  information
+* Update default ssl_version to TLSv1_2
+* Fix tenants method
+* Handle application/xml exception content-types
+
 = 0.10.0 - 14-May-2020
 * Updated the rest-client dependency to 2.1.0.
 * Updated the azure-signature dependency to 0.3.0.

data/README.md

@@ -8,7 +8,6 @@ A Ruby interface for Azure using the new REST API.
 [![Build Status](https://travis-ci.org/ManageIQ/azure-armrest.svg)](https://travis-ci.org/ManageIQ/azure-armrest)
 [![Code Climate](https://codeclimate.com/github/ManageIQ/azure-armrest/badges/gpa.svg)](https://codeclimate.com/github/ManageIQ/azure-armrest)
 [![Test Coverage](https://codeclimate.com/github/ManageIQ/azure-armrest/badges/coverage.svg)](https://codeclimate.com/github/ManageIQ/azure-armrest/coverage)
-[![Dependency Status](https://gemnasium.com/ManageIQ/azure-armrest.svg)](https://gemnasium.com/ManageIQ/azure-armrest)
 [![Security](https://hakiri.io/github/ManageIQ/azure-armrest/master.svg)](https://hakiri.io/github/ManageIQ/azure-armrest/master)
 
 ## Synopsis

data/azure-armrest.gemspec

@@ -8,7 +8,7 @@ Gem::Specification.new do |spec|
   spec.email    = ['dberger@redhat.com', 'bsorota@redhat.com', 'gblomqui@redhat.com', 'billwei@redhat.com']
   spec.summary  = 'An interface for ARM/JSON Azure REST API'
   spec.homepage = 'http://github.com/ManageIQ/azure-armrest'
-  spec.license  = 'Apache 2.0'
+  spec.license  = 'Apache-2.0'
   spec.files    = `git ls-files -z`.split("\x0").reject { |f| f.match(%r{^(test|spec|features)/}) }
 
   spec.description = <<-EOF
@@ -17,18 +17,18 @@ different than the current azure gem, which uses the older (XML) interface
 behind the scenes.
   EOF
 
-  spec.add_dependency 'json', '~> 2'
-  spec.add_dependency 'rest-client', '~> 2.1.0'
-  spec.add_dependency 'memoist', '~> 0.15'
-  spec.add_dependency 'azure-signature', '~> 0.3.0'
   spec.add_dependency 'activesupport', '>= 4.2.2'
   spec.add_dependency 'addressable', '~> 2.5.0'
+  spec.add_dependency 'azure-signature', '~> 0.3.0'
+  spec.add_dependency 'json', '~> 2'
+  spec.add_dependency 'memoist', '~> 0.15'
   spec.add_dependency 'parallel', '~> 1.12.0'
+  spec.add_dependency 'rest-client', '~> 2.1.0'
 
   spec.add_development_dependency 'bundler'
+  spec.add_development_dependency 'manageiq-style'
   spec.add_development_dependency 'rake'
   spec.add_development_dependency 'rspec', '~> 3.0'
   spec.add_development_dependency 'simplecov'
-  spec.add_development_dependency 'codeclimate-test-reporter', '~> 1.0.0'
   spec.add_development_dependency 'timecop', '~> 0.7'
 end

data/lib/azure/armrest.rb

@@ -1,6 +1,5 @@
 require 'rest-client'
 require 'json'
-require 'thread'
 require 'addressable'
 require 'parallel'
 require 'memoist'

data/lib/azure/armrest/armrest_service.rb

@@ -1,5 +1,6 @@
 require 'time'
 require 'active_support/core_ext/hash/conversions'
+require 'active_support/core_ext/hash/keys'
 require_relative 'model/base_model'
 
 module Azure
@@ -154,7 +155,7 @@ module Azure
       # Returns a list of tenants that can be accessed.
       #
       def tenants
-        url = url_with_api_version(configuration.api_version, configuration.resource_url, 'tenants')
+        url = url_with_api_version(configuration.api_version, configuration.environment.resource_url, 'tenants')
         resp = rest_get(url)
         JSON.parse(resp.body)['value'].map{ |hash| Azure::Armrest::Tenant.new(hash) }
       end
@@ -253,11 +254,22 @@ module Azure
 
         def raise_api_exception(err)
           begin
-            response = JSON.parse(err.http_body)
-            code     = response['error']['code']
-            message  = response['error']['message']
+            content_type_header = err.response.headers[:content_type]
+
+            response = case content_type_header.match(%r{(application/\w+)})[1]
+                       when "application/json"
+                         JSON.parse(err.http_body)
+                       when "application/xml"
+                         # The XML document that is returned has Error, Code, and Message
+                         # so we need to downcase the keys to stay consistent with the
+                         # parsed-json hash.
+                         Hash.from_xml(err.http_body).deep_transform_keys(&:downcase)
+                       end
+
+            code    = response['error']['code']
+            message = response['error']['message']
           rescue
-            code = err.try(:http_code) || err.try(:code)
+            code    = err.try(:http_code) || err.try(:code)
             message = err.try(:http_body) || err.try(:message)
           end
 

data/lib/azure/armrest/configuration.rb

@@ -107,7 +107,7 @@ module Azure
           :content_type  => 'application/json',
           :grant_type    => 'client_credentials',
           :proxy         => ENV['http_proxy'],
-          :ssl_version   => 'TLSv1',
+          :ssl_version   => 'TLSv1_2',
           :timeout       => 60,
           :max_threads   => 10,
           :max_retries   => 3,

data/lib/azure/armrest/model/storage_account.rb

@@ -288,7 +288,7 @@ module Azure
         url = File.join(properties.primary_endpoints.file, share, file)
         url += "?timeout=#{timeout}" if timeout
 
-        hash = options.transform_keys.each { |okey| 'x-ms-' + okey.to_s.tr('_', '-') }
+        hash = options.transform_keys { |okey| 'x-ms-' + okey.to_s.tr('_', '-') }
 
         hash['verb'] = 'PUT'
 
@@ -380,7 +380,7 @@ module Azure
         url = File.join(properties.primary_endpoints.file, share, file) + "?comp=range"
         url += "&timeout=#{timeout}" if timeout
 
-        hash = options.transform_keys.each { |okey| 'x-ms-' + okey.to_s.tr('_', '-') }
+        hash = options.transform_keys { |okey| 'x-ms-' + okey.to_s.tr('_', '-') }
 
         hash['verb'] = 'PUT'
         hash['x-ms-write'] ||= 'update'

data/lib/azure/armrest/resource_group_based_service.rb

@@ -232,7 +232,7 @@ module Azure
           api_version = configuration.provider_default_api_version(provider_name, full_service_name)
         end
         api_version ||= configuration.provider_default_api_version(provider_name, service_name)
-        api_version ||= configuration.api_version
+        api_version || configuration.api_version
       end
 
       def delete_by_url(url, resource_name = '')
@@ -260,7 +260,7 @@ module Azure
       # arguments provided, and appends it with the api_version.
       #
       def build_url(resource_group = nil, *args)
-        url = File.join(configuration.environment.resource_url, build_id_string(resource_group, *args))
+        File.join(configuration.environment.resource_url, build_id_string(resource_group, *args))
       end
 
       def build_id_string(resource_group = nil, *args)
@@ -279,6 +279,8 @@ module Azure
                 query << "&$top=#{value}"
               elsif key.casecmp('filter').zero?
                 query << "&$filter=#{value}" # Allow raw filter
+              elsif key.casecmp('expand').zero?
+                query << "&$expand=#{value}"
               else
                 if query.include?("$filter")
                   query << " and #{key} eq '#{value}'"

data/lib/azure/armrest/resource_provider_service.rb

@@ -142,7 +142,6 @@ module Azure
       private
 
       def build_url(namespace = nil, *args)
-        id = configuration.subscription_id
         url = File.join(base_url, 'providers')
         url = File.join(url, namespace) if namespace
         url = File.join(url, *args) unless args.empty?

data/lib/azure/armrest/storage/managed_storage_helper.rb

@@ -86,7 +86,7 @@ module Azure::Armrest::Storage::ManagedStorageHelper
     rescue Azure::Armrest::ForbiddenException => err
       log('warn', "ManagedStorageHelper.read: #{err}")
       raise err
-    rescue RestClient::Exception, Azure::Armrest::ForbiddenException => err
+    rescue RestClient::Exception, Azure::Armrest::Exception => err
       raise err unless retries < max_retries
       log('warn', "ManagedStorageHelper.read: #{err} - retry number #{retries}")
       retries += 1

data/lib/azure/armrest/version.rb

@@ -1,6 +1,6 @@
 module Azure
   module Armrest
     # The version of the azure-armrest library.
-    VERSION = '0.10.0'.freeze
+    VERSION = '0.11.0'.freeze
   end
 end

data/lib/azure/armrest/virtual_machine_service.rb

@@ -71,23 +71,48 @@ module Azure
       # Retrieves the settings of the VM named +vmname+ in resource group
       # +group+, which will default to the same as the name of the VM.
       #
-      # By default this method will retrieve the model view. If the +model_view+
-      # parameter is false, it will retrieve an instance view. The difference is
-      # in the details of the information retrieved.
+      # You can also specify any query options. At this time only the
+      # :expand => 'instanceView' option is supported, but others could
+      # be added over time.
       #
-      def get(vmname, group = configuration.resource_group, model_view = true)
-        model_view ? super(vmname, group) : get_instance_view(vmname, group)
+      # For backwards compatibility, the third argument may also be a boolean
+      # which will retrieve the model view by default. Set to false if you only
+      # want the instance view.
+      #
+      # Examples:
+      #
+      #   vms = VirtualMachineService.new(credentials)
+      #
+      #   # Standard call, get just the model view
+      #   vms.get('some_name', 'some_group')
+      #   vms.get('some_name', 'some_group', true) # same
+      #
+      #   # Get the instance view only
+      #   vms.get('some_name', 'some_group', false)
+      #
+      #   # Get the instance view merged with the model view
+      #   vms.get('some_name', 'some_group', :expand => 'instanceView')
+      #
+      def get(vmname, group = configuration.resource_group, options = {})
+        if options.kind_of?(Hash)
+          url = build_url(group, vmname, options)
+          response = rest_get(url)
+          VirtualMachineInstance.new(response)
+        else
+          options ? super(vmname, group) : get_instance_view(vmname, group)
+        end
       end
 
       # Convenient wrapper around the get method that retrieves the model view
-      # for +vmname+ in resource_group +group+.
+      # for +vmname+ in resource_group +group+ without the instance view
+      # information.
       #
       def get_model_view(vmname, group = configuration.resource_group)
-        get(vmname, group, true)
+        get(vmname, group)
       end
 
-      # Convenient wrapper around the get method that retrieves the instance view
-      # for +vmname+ in resource_group +group+.
+      # Convenient wrapper around the get method that retrieves only the
+      # instance view for +vmname+ in resource_group +group+.
       #
       def get_instance_view(vmname, group = configuration.resource_group)
         raise ArgumentError, "must specify resource group" unless group
@@ -333,7 +358,7 @@ module Azure
         raise ArgumentError, "must specify name of the vm" unless vmname
 
         url = build_url(group, vmname, action)
-        response = rest_post(url)
+        response = rest_post(url, options.to_json)
 
         Azure::Armrest::ResponseHeaders.new(response.headers).tap do |headers|
           headers.response_code = response.code

metadata

@@ -1,116 +1,116 @@
 --- !ruby/object:Gem::Specification
 name: azure-armrest
 version: !ruby/object:Gem::Version
-  version: 0.10.0
+  version: 0.11.0
 platform: ruby
 authors:
 - Daniel J. Berger
 - Bronagh Sorota
 - Greg Blomquist
 - Bill Wei
-autorequire: 
+autorequire:
 bindir: bin
 cert_chain: []
-date: 2020-05-14 00:00:00.000000000 Z
+date: 2021-04-20 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
-  name: json
+  name: activesupport
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - "~>"
+    - - ">="
       - !ruby/object:Gem::Version
-        version: '2'
+        version: 4.2.2
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - "~>"
+    - - ">="
       - !ruby/object:Gem::Version
-        version: '2'
+        version: 4.2.2
 - !ruby/object:Gem::Dependency
-  name: rest-client
+  name: addressable
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 2.1.0
+        version: 2.5.0
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 2.1.0
+        version: 2.5.0
 - !ruby/object:Gem::Dependency
-  name: memoist
+  name: azure-signature
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '0.15'
+        version: 0.3.0
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '0.15'
+        version: 0.3.0
 - !ruby/object:Gem::Dependency
-  name: azure-signature
+  name: json
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 0.3.0
+        version: '2'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 0.3.0
+        version: '2'
 - !ruby/object:Gem::Dependency
-  name: activesupport
+  name: memoist
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - ">="
+    - - "~>"
       - !ruby/object:Gem::Version
-        version: 4.2.2
+        version: '0.15'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - ">="
+    - - "~>"
       - !ruby/object:Gem::Version
-        version: 4.2.2
+        version: '0.15'
 - !ruby/object:Gem::Dependency
-  name: addressable
+  name: parallel
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 2.5.0
+        version: 1.12.0
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 2.5.0
+        version: 1.12.0
 - !ruby/object:Gem::Dependency
-  name: parallel
+  name: rest-client
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 1.12.0
+        version: 2.1.0
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 1.12.0
+        version: 2.1.0
 - !ruby/object:Gem::Dependency
   name: bundler
   requirement: !ruby/object:Gem::Requirement
@@ -125,6 +125,20 @@ dependencies:
     - - ">="
       - !ruby/object:Gem::Version
         version: '0'
+- !ruby/object:Gem::Dependency
+  name: manageiq-style
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
 - !ruby/object:Gem::Dependency
   name: rake
   requirement: !ruby/object:Gem::Requirement
@@ -167,20 +181,6 @@ dependencies:
     - - ">="
       - !ruby/object:Gem::Version
         version: '0'
-- !ruby/object:Gem::Dependency
-  name: codeclimate-test-reporter
-  requirement: !ruby/object:Gem::Requirement
-    requirements:
-    - - "~>"
-      - !ruby/object:Gem::Version
-        version: 1.0.0
-  type: :development
-  prerelease: false
-  version_requirements: !ruby/object:Gem::Requirement
-    requirements:
-    - - "~>"
-      - !ruby/object:Gem::Version
-        version: 1.0.0
 - !ruby/object:Gem::Dependency
   name: timecop
   requirement: !ruby/object:Gem::Requirement
@@ -208,9 +208,11 @@ executables: []
 extensions: []
 extra_rdoc_files: []
 files:
+- ".codeclimate.yml"
 - ".gitignore"
 - ".rspec"
 - ".rubocop.yml"
+- ".rubocop_cc.yml"
 - ".rubocop_local.yml"
 - ".travis.yml"
 - CHANGES
@@ -273,9 +275,9 @@ files:
 - lib/azure/armrest/virtual_machine_service.rb
 homepage: http://github.com/ManageIQ/azure-armrest
 licenses:
-- Apache 2.0
+- Apache-2.0
 metadata: {}
-post_install_message: 
+post_install_message:
 rdoc_options: []
 require_paths:
 - lib
@@ -290,8 +292,8 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.0.3
-signing_key: 
+rubygems_version: 3.2.5
+signing_key:
 specification_version: 4
 summary: An interface for ARM/JSON Azure REST API
 test_files: []