smart_proxy_ansible 2.1.2 → 3.1.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
-SHA1:
-  metadata.gz: d9d9191afda919cc1c6d8ed95f374b6e6824075e
-  data.tar.gz: d8ba3c8dcd1322349adbeba3cb4a272120e47fb5
+SHA256:
+  metadata.gz: 8c78dd643ea513a8ffc694a01c86f12495aff1e1b14c7e106db9d29ab529d26d
+  data.tar.gz: 8b2fde1a83bf65fb55fc122ce8ff931fd314f5289f9cf34e255d85959bebb653
 SHA512:
-  metadata.gz: 8a4d4039752a5b99984f3af42feb6b7525139134312451e91603bf61f18a2c268217d963ac1984aefaacbd9d276fd86f016dc9e599cafceb3d026dc787c2bb30
-  data.tar.gz: efe3fc55690b2379863cd67a8b474684a4a293e96908782e2ef2229662216967387a072d63763618bf52f9503ed945c8c89204351959bb7cce12c90500883969
+  metadata.gz: 695b0b451cc16439fd4fb76f9567f49e971c609c6d4696c19afa5feb3b90ac6a069e392ce38b177111041cae577662683cb6f594e86adc1096a6bfb5d7ac7ccc
+  data.tar.gz: 94a5bac7bce83785b0cc9067e9ffd20e9084e8befd575c3f6fae7da5ac7e78b202bf137daa36e6e726a8fd54de190180e42ac1af96ff6b2fcb1b9381364f2a59

data/README.md

@@ -2,6 +2,10 @@
 
 Proxy plugin to make [foreman_ansible](https://github.com/theforeman/foreman_ansible) actions run in the proxy
 
+## Compatibility
+
+This plugin requires at least Foreman Proxy 2.3.
+
 ## Installation (in development)
 
 ### Prerequisites

data/lib/smart_proxy_ansible/api.rb

@@ -13,32 +13,38 @@ module Proxy
       get '/roles/variables' do
         variables = {}
         RolesReader.list_roles.each do |role_name|
-          begin
-            variables[role_name] = extract_variables(role_name)[role_name]
-          rescue ReadVariablesException => e
-            # skip what cannot be parsed
-            logger.error e
-          end
+          variables.merge!(extract_variables(role_name))
+        rescue ReadVariablesException => e
+          # skip what cannot be parsed
+          logger.error e
         end
         variables.to_json
       end
 
       get '/roles/:role_name/variables' do |role_name|
-        begin
-          extract_variables(role_name).to_json
-        rescue ReadVariablesException => e
-          logger.error e
-          {}.to_json
-        end
+        extract_variables(role_name).to_json
+      rescue ReadVariablesException => e
+        logger.error e
+        {}.to_json
       end
 
       private
 
       def extract_variables(role_name)
         variables = {}
-        RolesReader.roles_path.split(':').each do |path|
-          variables[role_name] ||= VariablesExtractor
-                                   .extract_variables("#{path}/#{role_name}")
+        role_name_parts = role_name.split('.')
+        if role_name_parts.count == 3
+          RolesReader.collections_paths.split(':').each do |path|
+            variables[role_name] = VariablesExtractor
+                                   .extract_variables("#{path}/ansible_collections/#{role_name_parts[0]}/#{role_name_parts[1]}/roles/#{role_name_parts[2]}") if variables[role_name].nil? || variables[role_name].empty?
+          end
+        else
+          RolesReader.roles_path.split(':').each do |path|
+            role_path = "#{path}/#{role_name}"
+            if File.directory?(role_path)
+              variables[role_name] ||= VariablesExtractor.extract_variables(role_path)
+            end
+          end
         end
         variables
       end

data/lib/smart_proxy_ansible/plugin.rb

@@ -2,11 +2,7 @@ module Proxy
   module Ansible
     # Calls for the smart-proxy API to register the plugin
     class Plugin < Proxy::Plugin
-      http_rackup_path File.expand_path('http_config.ru',
-                                        File.expand_path('../', __FILE__))
-      https_rackup_path File.expand_path('http_config.ru',
-                                         File.expand_path('../', __FILE__))
-
+      rackup_path File.expand_path('http_config.ru', __dir__)
       settings_file 'ansible.yml'
       plugin :ansible, Proxy::Ansible::VERSION
 

data/lib/smart_proxy_ansible/roles_reader.rb

@@ -6,19 +6,32 @@ module Proxy
     class RolesReader
       class << self
         DEFAULT_CONFIG_FILE = '/etc/ansible/ansible.cfg'.freeze
-        DEFAULT_ROLES_PATH = '/etc/ansible/roles'.freeze
+        DEFAULT_ROLES_PATH = '/etc/ansible/roles:/usr/share/ansible/roles'.freeze
+        DEFAULT_COLLECTIONS_PATHS = '/etc/ansible/collections:/usr/share/ansible/collections'.freeze
 
         def list_roles
-          roles_path.split(':').map { |path| read_roles(path) }.flatten
+          roles = roles_path.split(':').map { |path| read_roles(path) }.flatten
+          collection_roles = collections_paths.split(':').map { |path| read_collection_roles(path) }.flatten
+          roles + collection_roles
         end
 
-        def roles_path(roles_line = roles_path_from_config)
-          # Default to /etc/ansible/roles if none found
-          return DEFAULT_ROLES_PATH if roles_line.empty?
-          roles_path_key = roles_line.first.split('=').first.strip
-          # In case of commented roles_path key "#roles_path", return default
-          return DEFAULT_ROLES_PATH unless roles_path_key == 'roles_path'
-          roles_line.first.split('=').last.strip
+        def roles_path
+          config_path(path_from_config('roles_path'), DEFAULT_ROLES_PATH)
+        end
+
+        def collections_paths
+          config_path(path_from_config('collections_paths'), DEFAULT_COLLECTIONS_PATHS)
+        end
+
+        def config_path(config_line, default)
+          # Default to /etc/ansible/roles if config_line is empty
+          return default if config_line.empty?
+
+          config_line_key = config_line.first.split('=').first.strip
+          # In case of commented roles_path key "#roles_path" or #collections_paths, return default
+          return default if ['#roles_path', '#collections_paths'].include?(config_line_key)
+
+          config_line.first.split('=').last.strip
         end
 
         def logger
@@ -34,30 +47,43 @@ module Proxy
         private
 
         def read_roles(roles_path)
-          rescue_and_raise_file_exception ReadRolesException,
-                                          roles_path, 'roles' do
-            Dir.glob("#{roles_path}/*").map do |path|
-              path.split('/').last
-            end
+          glob_path("#{roles_path}/*").map do |path|
+            path.split('/').last
           end
+        rescue Errno::ENOENT, Errno::EACCES => e
+          logger.debug(e.backtrace)
+          message = "Could not read Ansible roles #{roles_path} - #{e.message}"
+          raise ReadRolesException.new(message), message
+        end
+
+        def glob_path(path)
+          Dir.glob path
+
         end
 
-        def roles_path_from_config
-          rescue_and_raise_file_exception ReadConfigFileException,
-                                          DEFAULT_CONFIG_FILE, 'config file' do
-            File.readlines(DEFAULT_CONFIG_FILE).select do |line|
-              line =~ /^\s*roles_path/
-            end
+        def read_collection_roles(collections_path)
+          Dir.glob("#{collections_path}/ansible_collections/*/*/roles/*").map do |path|
+            parts = path.split('/')
+            role = parts.pop
+            parts.pop
+            collection = parts.pop
+            author = parts.pop
+            "#{author}.#{collection}.#{role}"
           end
+        rescue Errno::ENOENT, Errno::EACCES => e
+          logger.debug(e.backtrace)
+          message = "Could not read Ansible roles #{collections_path} - #{e.message}"
+          raise ReadRolesException.new(message), message
         end
 
-        def rescue_and_raise_file_exception(exception, path, type)
-          yield
+        def path_from_config(config_key)
+          File.readlines(DEFAULT_CONFIG_FILE).select do |line|
+            line =~ /^\s*#{config_key}/
+          end
         rescue Errno::ENOENT, Errno::EACCES => e
           logger.debug(e.backtrace)
-          exception_message = "Could not read Ansible #{type} "\
-            "#{path} - #{e.message}"
-          raise exception.new(exception_message), exception_message
+          message = "Could not read Ansible config file #{DEFAULT_CONFIG_FILE} - #{e.message}"
+          raise ReadConfigFileException.new(message), message
         end
       end
     end

data/lib/smart_proxy_ansible/variables_extractor.rb

@@ -8,7 +8,7 @@ module Proxy
         def extract_variables(role_path)
           role_files = Dir.glob("#{role_path}/defaults/**/*.yml") +
                        Dir.glob("#{role_path}/defaults/**/*.yaml")
-          role_files.flat_map do |role_file|
+          role_files.reduce({}) do |memo, role_file|
             loaded_yaml = {}
             begin
               loaded_yaml = YAML.load_file(role_file)
@@ -16,7 +16,7 @@ module Proxy
               raise ReadVariablesException.new "#{role_file} is not YAML file"
             end
             raise ReadVariablesException.new "Could not parse YAML file: #{role_file}" unless loaded_yaml.is_a? Hash
-            loaded_yaml.keys
+            memo.merge loaded_yaml
           end
         end
       end

data/lib/smart_proxy_ansible/version.rb

@@ -2,6 +2,6 @@ module Proxy
   # Version, this allows the proxy and other plugins know
   # what version of the Ansible plugin is running
   module Ansible
-    VERSION = '2.1.2'
+    VERSION = '3.1.1'
   end
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: smart_proxy_ansible
 version: !ruby/object:Gem::Version
-  version: 2.1.2
+  version: 3.1.1
 platform: ruby
 authors:
 - Ivan Nečas
@@ -9,50 +9,22 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2019-03-07 00:00:00.000000000 Z
+date: 2021-08-09 00:00:00.000000000 Z
 dependencies:
-- !ruby/object:Gem::Dependency
-  name: bundler
-  requirement: !ruby/object:Gem::Requirement
-    requirements:
-    - - "~>"
-      - !ruby/object:Gem::Version
-        version: '1.7'
-  type: :development
-  prerelease: false
-  version_requirements: !ruby/object:Gem::Requirement
-    requirements:
-    - - "~>"
-      - !ruby/object:Gem::Version
-        version: '1.7'
 - !ruby/object:Gem::Dependency
   name: rake
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '10.0'
-  type: :development
-  prerelease: false
-  version_requirements: !ruby/object:Gem::Requirement
-    requirements:
-    - - "~>"
-      - !ruby/object:Gem::Version
-        version: '10.0'
-- !ruby/object:Gem::Dependency
-  name: minitest
-  requirement: !ruby/object:Gem::Requirement
-    requirements:
-    - - "~>"
-      - !ruby/object:Gem::Version
-        version: '0'
+        version: '13.0'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '0'
+        version: '13.0'
 - !ruby/object:Gem::Dependency
   name: mocha
   requirement: !ruby/object:Gem::Requirement
@@ -73,14 +45,14 @@ dependencies:
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '1'
+        version: '3'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '1'
+        version: '3'
 - !ruby/object:Gem::Dependency
   name: rack-test
   requirement: !ruby/object:Gem::Requirement
@@ -95,20 +67,6 @@ dependencies:
     - - "~>"
       - !ruby/object:Gem::Version
         version: '0'
-- !ruby/object:Gem::Dependency
-  name: rubocop
-  requirement: !ruby/object:Gem::Requirement
-    requirements:
-    - - '='
-      - !ruby/object:Gem::Version
-        version: 0.32.1
-  type: :development
-  prerelease: false
-  version_requirements: !ruby/object:Gem::Requirement
-    requirements:
-    - - '='
-      - !ruby/object:Gem::Version
-        version: 0.32.1
 - !ruby/object:Gem::Dependency
   name: logger
   requirement: !ruby/object:Gem::Requirement
@@ -163,8 +121,7 @@ extra_rdoc_files:
 files:
 - LICENSE
 - README.md
-- bin/json_inventory.sh
-- bundler.plugins.d/smart_proxy_ansible.rb
+- bundler.d/ansible.rb
 - lib/smart_proxy_ansible.rb
 - lib/smart_proxy_ansible/api.rb
 - lib/smart_proxy_ansible/exception.rb
@@ -186,15 +143,14 @@ required_ruby_version: !ruby/object:Gem::Requirement
   requirements:
   - - ">="
     - !ruby/object:Gem::Version
-      version: '0'
+      version: '2.5'
 required_rubygems_version: !ruby/object:Gem::Requirement
   requirements:
   - - ">="
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubyforge_project: 
-rubygems_version: 2.6.8
+rubygems_version: 3.1.2
 signing_key: 
 specification_version: 4
 summary: Smart-Proxy Ansible plugin

data/bin/json_inventory.sh

@@ -1,10 +0,0 @@
-#!/usr/bin/env bash
-# An inventory script to load the inventory from JSON file.
-#
-
-if [ -z "$JSON_INVENTORY_FILE" ]; then
-   echo "JSON_INVENTORY_FILE not specified"
-   exit 1
-fi
-
-cat $JSON_INVENTORY_FILE