fog-azure-rm 0.3.7 → 0.3.8

data/lib/fog/azurerm/docs/traffic_manager.md

@@ -1,6 +1,6 @@
 # Traffic Manager
 
-This document explains how to get started using Azure Traffic Manager Service with Fog. With this gem you can create/update/list/delete Traffic Manager Profiles and End Points.
+This document explains how to get started using Azure Traffic Manager Service with Fog. With this gem you can create, update, list or delete Traffic Manager Profiles and End Points.
 
 ## Usage
 
@@ -14,19 +14,19 @@ require 'fog/azurerm'
 Next, create a connection to the Traffic Manager Service:
 
 ```ruby
-    azure_traffic_manager_service = Fog::TrafficManager::AzureRM.new(
-      tenant_id:        '<Tenantid>',                                                       # Tenant id of Azure Active Directory Application
-      client_id:        '<Clientid>',                                                       # Client id of Azure Active Directory Application
-      client_secret:    '<ClientSecret>',                                                   # Client Secret of Azure Active Directory Application
-      subscription_id:  '<Subscriptionid>',                                                 # Subscription id of an Azure Account
-      environment:      '<AzureCloud/AzureChinaCloud/AzureUSGovernment/AzureGermanCloud>'   # Azure cloud environment. Default is AzureCloud.
+fog_traffic_manager_service = Fog::TrafficManager::AzureRM.new(
+      tenant_id:        '<Tenant Id>',                                                          # Tenant Id of Azure Active Directory Application
+      client_id:        '<Client Id>',                                                          # Client Id of Azure Active Directory Application
+      client_secret:    '<Client Secret>',                                                      # Client Secret of Azure Active Directory Application
+      subscription_id:  '<Subscription Id>',                                                    # Subscription Id of an Azure Account
+      environment:      '<AzureCloud/AzureChinaCloud/AzureUSGovernment/AzureGermanCloud>'       # Azure cloud environment. Default is AzureCloud.
 )
 ```
 
 ## Check Traffic Manager Profile Existence
 
 ```ruby
-azure_traffic_manager_service.traffic_manager_profiles.check_traffic_manager_profile_exists(<Resource Group Name>, <Profile Name>)
+fog_traffic_manager_service.traffic_manager_profiles.check_traffic_manager_profile_exists('<Resource Group Name>', '<Profile Name>')
 ```
 
 ## Create Traffic Manager Profile
@@ -34,17 +34,17 @@ azure_traffic_manager_service.traffic_manager_profiles.check_traffic_manager_pro
 Create a new Traffic Manager Profile. The parameter 'traffic_routing_method' can be 'Performance', 'Weighted' or 'Priority'.
 
 ```ruby
-      profile = azure_traffic_manager_service.traffic_manager_profiles.create(
+profile = fog_traffic_manager_service.traffic_manager_profiles.create(
         name: '<Profile Name>',
         resource_group: '<Resource Group Name>',
-        traffic_routing_method: 'Performance',
+        traffic_routing_method: '<Routing Method Name>',
         relative_name: '<Profile Relative Name>',
-        ttl: '30',
-        protocol: 'http',
-        port: '80',
-        path: '/monitorpage.aspx',
-        tags: { key1: "value1", key2: "value2", keyN: "valueN" }                          # [Optional]
-      )
+        ttl: '<TTL>',
+        protocol: '<Protocol Name>',
+        port: '<Port Number>',
+        path: '<Path>',
+        tags: { key1: 'value1', key2: 'value2', keyN: 'valueN' }                          # [Optional]
+)
 ```
 
 ## List Traffic Manager Profiles
@@ -52,10 +52,10 @@ Create a new Traffic Manager Profile. The parameter 'traffic_routing_method' can
 List Traffic Manager Profiles in a resource group
 
 ```ruby
-    profiles  = azure_traffic_manager_service.traffic_manager_profiles(resource_group: '<Resource Group name>')
-    profiles.each do |profile|
-      puts "#{profile.name}"
-    end
+profiles  = fog_traffic_manager_service.traffic_manager_profiles(resource_group: '<Resource Group Name>')
+profiles.each do |profile|
+    puts "#{profile.name}"
+end
 ```
 
 ## Retrieve a single Traffic Manager Profile
@@ -63,10 +63,10 @@ List Traffic Manager Profiles in a resource group
 Get a single record of Traffic Manager Profile
 
 ```ruby
-     profile = azure_traffic_manager_service
-               .traffic_manager_profiles
-                .get('<Resource Group name>', '<Profile name>')
-     puts "#{profile.name}"
+profile = fog_traffic_manager_service
+             .traffic_manager_profiles
+             .get('<Resource Group Name>', '<Profile Name>')
+puts "#{profile.name}"
 ```
 
 ## Update a Traffic Manager Profile
@@ -74,13 +74,13 @@ Get a single record of Traffic Manager Profile
 Get a Traffic Manager Profile object from the get method and then update that Traffic Manager Profile. You can update the Traffic Manager Profile by passing the modifiable attributes in the form of a hash.
 
 ```ruby
-    profile.update(
-              traffic_routing_method: 'Weighted',
-              ttl: '35',
-              protocol: 'https',
-              port: '90',
-              path: '/monitorpage1.aspx'
-    )
+profile.update(
+              traffic_routing_method: '<Routing Method Name>',
+              ttl: '<TTL>',
+              protocol: '<Protocol Name>',
+              port: '<Port Number>',
+              path: '<Path>'
+)
 ```
 
 ## Destroy a single Traffic Manager Profile
@@ -88,18 +88,18 @@ Get a Traffic Manager Profile object from the get method and then update that Tr
 Get a Traffic Manager Profile object from the get method and then destroy that Traffic Manager Profile.
 
 ```ruby
-     profile.destroy
+profile.destroy
 ```
 
 ## Check Traffic Manager Endpoint Existence
 
 ```ruby
- azure_network_service.traffic_manager_end_points.check_traffic_manager_endpoint_exists(
-   <Resource Group Name>,
-   <Profile Name>,
-   <Endpoint Name>,
-   <Type(externalEndpoints)>
-   )
+azure_network_service.traffic_manager_end_points.check_traffic_manager_endpoint_exists(
+        '<Resource Group Name>',
+        '<Profile Name>',
+        '<Endpoint Name>',
+        '<Type(<Endpoint Type>)>'
+)
 ```
 
 ## Create Traffic Manager Endpoint
@@ -107,14 +107,14 @@ Get a Traffic Manager Profile object from the get method and then destroy that T
 Traffic Manager Profile is pre-requisite of Traffic Manager Endpoint. Create a new Traffic Manager Endpoint. The parameter 'type' can be 'externalEndpoints, 'azureEndpoints' or 'nestedEndpoints'.
 
 ```ruby
-      endpoint = azure_network_service.traffic_manager_end_points.create(
+endpoint = azure_network_service.traffic_manager_end_points.create(
         name: '<Endpoint Name>',
         traffic_manager_profile_name: '<Profile Name>',
         resource_group: '<Resource Group Name>',
-        type: 'externalEndpoints',
-        target: 'test.com',
-        endpoint_location: 'West US'
-      )
+        type: '<Endpoint Type>',
+        target: '<Target URL>',
+        endpoint_location: '<Location>'
+)
 ```
 
 ## List Traffic Manager Endpoints
@@ -122,10 +122,10 @@ Traffic Manager Profile is pre-requisite of Traffic Manager Endpoint. Create a n
 List Traffic Manager Endpoints in a resource group.
 
 ```ruby
-    endpoints  = azure_traffic_manager_service.traffic_manager_end_points(resource_group: '<Resource Group name>', traffic_manager_profile_name: '<Profile Name>')
-    endpoints.each do |endpoint|
-      puts "#{endpoint.name}"
-    end
+endpoints  = fog_traffic_manager_service.traffic_manager_end_points(resource_group: '<Resource Group Name>', traffic_manager_profile_name: '<Profile Name>')
+endpoints.each do |endpoint|
+   puts "#{endpoint.name}"
+end
 ```
 
 ## Retrieve a single Traffic Manager Endpoint
@@ -133,19 +133,20 @@ List Traffic Manager Endpoints in a resource group.
 Get a single Traffic Manager Endpoint.
 
 ```ruby
-      endpoint = azure_traffic_manager_service
+endpoint = fog_traffic_manager_service
                  .traffic_manager_end_points
-                 .get('<Resource Group name>', '<Profile Name>', '<Endpoint name>', '<Endpoint type>')
-      puts "#{endpoint.name}"
+                 .get('<Resource Group Name>', '<Profile Name>', '<Endpoint name>', '<Endpoint Type>')
+puts "#{endpoint.name}"
 ```
 ## Update a Traffic Manager Endpoint
 
 Get a Traffic Manager Endpoint object from the get method and then update that Traffic Manager Endpoint. You can update the Traffic Manager Endpoint by passing the modifiable attributes in the form of a hash.
 
 ```ruby
-      endpoint.update(type: 'externalEndpoints',
-                      target: 'test1.com',
-                      endpoint_location: 'Central US')
+endpoint.update(type: '<Endpoint Type>',
+                target: '<Target URL>',
+                endpoint_location: '<Location>'
+)
 ```
 
 ## Destroy a single Traffic Manager Endpoint
@@ -153,7 +154,7 @@ Get a Traffic Manager Endpoint object from the get method and then update that T
 Get a Traffic Manager Endpoint object from the get method and then destroy that Traffic Manager Endpoint.
 
 ```ruby
-     endpoint.destroy
+endpoint.destroy
 ```
 
 ## Support and Feedback

data/lib/fog/azurerm/models/compute/server.rb

@@ -108,6 +108,7 @@ module Fog
             service.create_virtual_machine(virtual_machine_params(ssh_key_path), true)
           else
             vm = service.create_virtual_machine(virtual_machine_params(ssh_key_path))
+            vm = service.get_virtual_machine(resource_group, name, false)
             merge_attributes(Fog::Compute::AzureRM::Server.parse(vm))
           end
         end
@@ -183,6 +184,11 @@ module Fog
           end
         end
 
+        def update_attributes
+          vm = service.get_virtual_machine(resource_group, name, false)
+          merge_attributes(Fog::Compute::AzureRM::Server.parse(vm))
+        end
+
         private
 
         def platform_is_linux?(platform)

data/lib/fog/azurerm/models/compute/servers.rb

@@ -19,7 +19,7 @@ module Fog
         def create_async(attributes = {})
           server = new(attributes)
           promise = server.save(true)
-          Fog::AzureRM::AsyncResponse.new(server, promise, true)
+          Fog::AzureRM::AsyncResponse.new(server, promise, true, 'update_attributes')
         end
 
         def get(resource_group_name, virtual_machine_name, async = false)

data/lib/fog/azurerm/requests/compute/get_virtual_machine.rb

@@ -1,3 +1,5 @@
+INSTANCE_VIEW = 'instanceView'.freeze
+
 module Fog
   module Compute
     class AzureRM
@@ -8,9 +10,9 @@ module Fog
           Fog::Logger.debug msg
           begin
             if async
-              response = @compute_mgmt_client.virtual_machines.get_async(resource_group, name)
+              response = @compute_mgmt_client.virtual_machines.get_async(resource_group, name, INSTANCE_VIEW)
             else
-              response = @compute_mgmt_client.virtual_machines.get(resource_group, name)
+              response = @compute_mgmt_client.virtual_machines.get(resource_group, name, INSTANCE_VIEW)
             end
           rescue MsRestAzure::AzureOperationError => e
             raise_azure_exception(e, msg)

data/lib/fog/azurerm/version.rb

@@ -1,5 +1,5 @@
 module Fog
   module AzureRM
-    VERSION = '0.3.7'.freeze
+    VERSION = '0.3.8'.freeze
   end
 end

data/rake-script.sh

@@ -1,4 +1,10 @@
 #!/bin/bash
 rake cc_coverage
+R=$?
 export CODECLIMATE_REPO_TOKEN="b1401494baa004d90402414cb33a7fc6420fd3693e60c677a120ddefd7d84cfd"
-codeclimate-test-reporter
+codeclimate-test-reporter
+if [ $R == 1 ]; then
+  exit 1
+else
+  exit 0
+fi

data/rakefile

@@ -26,4 +26,4 @@ end
 desc 'Generates a coverage report for integration tests'
 task :integration_tests do
   Dir.glob('test/smoke_tests/**/test_*.rb').each { |file| require File.expand_path file, __dir__ }
-end
+end

data/test/api_stub/models/compute/server.rb

@@ -49,6 +49,10 @@ module ApiStub
                     'id' => '/subscriptions/########-####-####-####-############/resourceGroups/fog-test-rg/providers/Microsoft.Network/networkInterfaces/fogtestnetworkinterface'
                   }
                 ]
+              },
+              'instanceView' => {
+                'platformUpgradeDomain' => 5,
+                'platformFaultDomain' => 2
               }
             }
           }
@@ -100,6 +104,10 @@ module ApiStub
                     'id' => '/subscriptions/########-####-####-####-############/resourceGroups/fog-test-rg/providers/Microsoft.Network/networkInterfaces/fogtestnetworkinterface'
                   }
                 ]
+              },
+              'instanceView' => {
+                'platformUpgradeDomain' => 5,
+                'platformFaultDomain' => 2
               }
             }
           }
@@ -179,6 +187,10 @@ module ApiStub
                     'id' => '/subscriptions/########-####-####-####-############/resourceGroups/fog-test-rg/providers/Microsoft.Network/networkInterfaces/fogtestnetworkinterface'
                   }
                 ]
+              },
+              'instanceView' => {
+                'platformUpgradeDomain' => 5,
+                'platformFaultDomain' => 2
               }
             }
           }
@@ -240,6 +252,10 @@ module ApiStub
                     'id' => '/subscriptions/########-####-####-####-############/resourceGroups/fog-test-rg/providers/Microsoft.Network/networkInterfaces/fogtestnetworkinterface'
                   }
                 ]
+              },
+              'instanceView' => {
+                'platformUpgradeDomain' => 5,
+                'platformFaultDomain' => 2
               }
             }
           }

data/test/models/compute/test_server.rb

@@ -69,7 +69,9 @@ class TestServer < Minitest::Test
   def test_save_method_response_for_linux_vm
     response = ApiStub::Models::Compute::Server.create_linux_virtual_machine_response(@compute_client)
     @service.stub :create_virtual_machine, response do
-      assert_instance_of Fog::Compute::AzureRM::Server, @server.save
+      @service.stub :get_virtual_machine, response do
+        assert_instance_of Fog::Compute::AzureRM::Server, @server.save
+      end
     end
 
     # Async
@@ -82,8 +84,10 @@ class TestServer < Minitest::Test
   def test_save_method_response_for_windows_vm
     response = ApiStub::Models::Compute::Server.create_windows_virtual_machine_response(@compute_client)
     @service.stub :create_virtual_machine, response do
-      assert_instance_of Fog::Compute::AzureRM::Server, @server.save
-      refute @server.save.disable_password_authentication
+      @service.stub :get_virtual_machine, response do
+        assert_instance_of Fog::Compute::AzureRM::Server, @server.save
+        refute @server.save.disable_password_authentication
+      end
     end
   end
 

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: fog-azure-rm
 version: !ruby/object:Gem::Version
-  version: 0.3.7
+  version: 0.3.8
 platform: ruby
 authors:
 - Shaffan Chaudhry
@@ -18,7 +18,7 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2017-10-25 00:00:00.000000000 Z
+date: 2017-11-17 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rake
@@ -1041,7 +1041,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
       version: '0'
 requirements: []
 rubyforge_project: 
-rubygems_version: 2.6.14
+rubygems_version: 2.7.2
 signing_key: 
 specification_version: 4
 summary: Module for the 'fog' gem to support Azure Resource Manager cloud services.