foreman_azure_rm 3.0.1 → 3.0.2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 36b86a8554404e4e3dfe00aff4a9930256cdf9eca82a39a8ebe3061dc40c32fb
-  data.tar.gz: cb98d0fe5118012610fb7161c2605208c1ebb15effb3e72bf556707cc8ff5e51
+  metadata.gz: 670bd7e4c753d63c24b4efc5d5f22104fd8ec5400912411cfe6a283bf9e01044
+  data.tar.gz: 19445ca94e173aad6869b27c6691abbab24995046c459e09f2446098e8c0b60c
 SHA512:
-  metadata.gz: 1cd12744b93cb7154e71912600c81b57faeace8be953e49912484fb0b0e391f653d7f58af2ea711071ee09bbd83bce433863e92538e70c19621d36744305dc68
-  data.tar.gz: 36b34aed0cb621830f8646f24fc71235a38a6cdd63d74af76e875c9e10b5b10c509c964776af62830ece8569167187c342fab5789baaf2aef1c8dba849fad6c0
+  metadata.gz: 81c8664096dae3577cbd45d47e1549d4a753edf63baf6a7b189628bbdc9f24980f89877542afd242ca0a7d31d953b008d309350a529ec52ef594ecb358d34f0b
+  data.tar.gz: 1fb422fabc9b0404c7978d798b511ae85a90ef341415cf30c4f4887ddc47eb908c55cdf57498fbe39d82ec52bce1591518fee50674f2cd8ff7a5c1d7000750cc

data/{lib → app/lib}/foreman_azure_rm/azure_sdk_adapter.rb

@@ -11,7 +11,7 @@ module ForemanAzureRm
     end
 
     def resource_client
-      #resource_manager_endpoint_url
+      # resource_manager_endpoint_url
       @resource_client ||= Resources::Client.new(azure_credentials(@environment_settings.resource_manager_endpoint_url))
     end
 
@@ -33,11 +33,12 @@ module ForemanAzureRm
 
     def azure_credentials(base_url)
       provider = MsRestAzure::ApplicationTokenProvider.new(
-      @tenant,
-      @app_ident,
-      @secret_key,
-      @ad_settings)
-      
+        @tenant,
+        @app_ident,
+        @secret_key,
+        @ad_settings
+      )
+
       credentials = MsRest::TokenCredentials.new(provider)
 
       {
@@ -46,7 +47,7 @@ module ForemanAzureRm
         client_id: @app_ident,
         client_secret: @secret_key,
         subscription_id: @sub_id,
-        base_url: base_url
+        base_url: base_url,
       }
     end
 
@@ -91,7 +92,7 @@ module ForemanAzureRm
     end
 
     def rgs
-      rgs      = resource_client.resource_groups.list
+      rgs = resource_client.resource_groups.list
       rgs.map(&:name)
     end
 
@@ -107,7 +108,7 @@ module ForemanAzureRm
       network_client.public_ipaddresses.get(rg_name, pip_name)
     end
 
-    def vm_nic(rg_name,nic_name)
+    def vm_nic(rg_name, nic_name)
       network_client.network_interfaces.get(rg_name, nic_name)
     end
 
@@ -120,14 +121,15 @@ module ForemanAzureRm
     end
 
     def list_vm_sizes(region)
-      return [] unless region.present?
+      return [] if region.blank?
+
       stripped_region = region.gsub(/\s+/, '').downcase
-      compute_client.virtual_machine_sizes.list(stripped_region).value()
+      compute_client.virtual_machine_sizes.list(stripped_region).value
     end
 
     def list_vms(region)
       # List all VMs in a resource group
-      virtual_machines = compute_client.virtual_machines.list_by_location(region)
+      compute_client.virtual_machines.list_by_location(region)
     end
 
     def get_vm(rg_name, vm_name)
@@ -166,7 +168,7 @@ module ForemanAzureRm
       compute_client.gallery_image_versions.list_by_gallery_image(rg_name, gallery_name, gallery_image_name)
     end
 
-    def get_storage_accts
+    def get_storage_accts # rubocop:disable Naming/AccessorMethodName
       result = storage_client.storage_accounts.list
       result.value
     end
@@ -177,9 +179,9 @@ module ForemanAzureRm
 
     def create_or_update_vm_extensions(rg_name, vm_name, vm_extension_name, extension_params)
       compute_client.virtual_machine_extensions.create_or_update(rg_name,
-                                                          vm_name,
-                                                          vm_extension_name,
-                                                          extension_params) 
+        vm_name,
+        vm_extension_name,
+        extension_params)
     end
 
     def create_or_update_pip(rg_name, pip_name, parameters)
@@ -207,7 +209,7 @@ module ForemanAzureRm
     end
 
     def check_vm_status(rg_name, vm_name)
-      virtual_machine = compute_client.virtual_machines.get(rg_name, vm_name, expand:'instanceView')
+      virtual_machine = compute_client.virtual_machines.get(rg_name, vm_name, expand: 'instanceView')
       get_status(virtual_machine)
     end
 
@@ -215,9 +217,7 @@ module ForemanAzureRm
       vm_statuses = virtual_machine.instance_view.statuses
       vm_status = nil
       vm_statuses.each do |status|
-        if status.code.include? 'PowerState'
-          vm_status = status.code.split('/')[1]
-        end
+        vm_status = status.code.split('/')[1] if status.code.include? 'PowerState'
       end
       vm_status
     end
@@ -238,11 +238,10 @@ module ForemanAzureRm
 
     def actual_gallery_image_id(rg_name, image_id)
       gallery_names = list_galleries.map(&:name)
-      gallery_names.each do |gallery|
-        gallery_image = list_gallery_images(rg_name, gallery).detect { |image| image.name == image_id }
-        return gallery_image&.id
-      end
-      nil
+      return unless (gallery = gallery_names.first)
+
+      gallery_image = list_gallery_images(rg_name, gallery).detect { |image| image.name == image_id }
+      gallery_image&.id
     end
 
     def fetch_gallery_image_id(rg_name, image_id)

data/lib/foreman_azure_rm/engine.rb

@@ -2,17 +2,19 @@ module ForemanAzureRm
   class Engine < ::Rails::Engine
     engine_name 'foreman_azure_rm'
 
-    #autoloading all files inside lib dir
-    config.eager_load_paths += Dir["#{config.root}/lib"]
-    config.eager_load_paths += Dir["#{config.root}/app/models/concerns/"]
-    config.eager_load_paths += Dir["#{config.root}/app/helpers/"]
-
-    initializer 'foreman_azure_rm.register_plugin', :before => :finisher_hook do
-      Foreman::Plugin.register :foreman_azure_rm do
-        requires_foreman '>= 3.13'
-        register_gettext
-        compute_resource ForemanAzureRm::AzureRm
-        parameter_filter ComputeResource, :azure_vm, :tenant, :app_ident, :secret_key, :sub_id, :region, :cloud
+    lib = config.root.join('app', 'lib')
+
+    config.autoload_paths << lib
+    config.eager_load_paths << lib
+
+    initializer 'foreman_azure_rm.register_plugin', :before => :finisher_hook do |app|
+      app.reloader.to_prepare do
+        Foreman::Plugin.register :foreman_azure_rm do
+          requires_foreman '>= 3.13'
+          register_gettext
+          compute_resource ForemanAzureRm::AzureRm
+          parameter_filter ComputeResource, :azure_vm, :tenant, :app_ident, :secret_key, :sub_id, :region, :cloud
+        end
       end
     end
 
@@ -29,14 +31,6 @@ module ForemanAzureRm
       end
     end
 
-    initializer "foreman_azure_rm.zeitwerk" do
-      Rails.autoloaders.each do |loader|
-        loader.ignore(
-          ForemanAzureRm::Engine.root.join('lib/foreman_azure_rm/version.rb')
-        )
-      end
-    end
-
     config.to_prepare do
       require 'azure_mgmt_resources'
       require 'azure_mgmt_network'

data/lib/foreman_azure_rm/version.rb

@@ -1,3 +1,3 @@
 module ForemanAzureRm
-  VERSION = '3.0.1'.freeze
+  VERSION = '3.0.2'.freeze
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: foreman_azure_rm
 version: !ruby/object:Gem::Version
-  version: 3.0.1
+  version: 3.0.2
 platform: ruby
 authors:
 - Aditi Puntambekar
@@ -10,7 +10,7 @@ authors:
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2024-09-20 00:00:00.000000000 Z
+date: 2024-12-11 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: azure_mgmt_resources
@@ -117,6 +117,7 @@ files:
 - app/controllers/foreman_azure_rm/concerns/compute_resources_controller_extensions.rb
 - app/controllers/foreman_azure_rm/concerns/hosts_controller_extensions.rb
 - app/helpers/azure_compute_resource_helper.rb
+- app/lib/foreman_azure_rm/azure_sdk_adapter.rb
 - app/models/concerns/foreman_azure_rm/vm_extensions/managed_vm.rb
 - app/models/foreman_azure_rm/azure_rm.rb
 - app/models/foreman_azure_rm/azure_rm_compute.rb
@@ -139,7 +140,6 @@ files:
 - db/migrate/20200507103900_fix_image_uuid_prefix.rb
 - lib/foreman_azure_rm.rake
 - lib/foreman_azure_rm.rb
-- lib/foreman_azure_rm/azure_sdk_adapter.rb
 - lib/foreman_azure_rm/engine.rb
 - lib/foreman_azure_rm/version.rb
 - locale/Makefile
@@ -202,7 +202,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.2.33
+rubygems_version: 3.5.22
 signing_key:
 specification_version: 4
 summary: Azure Resource Manager as a compute resource for The Foreman