smart_proxy_salt 5.0.1 → 5.1.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 4ba18b31812c408bbbef40601e2f7279ee6f04d55a62b999b7b993f0158aede8
-  data.tar.gz: 82822b47407c7c5e66151627a7935d294566c6e5dc17c4bbf7c2674890653826
+  metadata.gz: ea43480d38aa6f8d3694e61899d0b5dadeacaacc775a82d8ab2a456810bb8bcd
+  data.tar.gz: 6101554d6bf03679d29399f4ad92fbc51c4bfe1250e928433eca682ad015bac7
 SHA512:
-  metadata.gz: d95ddd85199c088db57c6248d99d7a370e36c74191aeda0d74fd4f7fb7d573ee50a0b4a107092d2f8f6c3018c131e6127236cc5818f84c835eecabd0b21ef6bf
-  data.tar.gz: d9866d18feed738e03d0e47784ee75b41921b65bcbbd33e88aec5856cfd5160e4b0b5e5485d5c25c3a6c601d6c519bde984599b734f7f081b7adce9de53c3277
+  metadata.gz: 9de7cddee069d666c6cc4a7400c2df16d647c4e244286a0376e8c1416555c85e1d295975017b28041305be31e2cbe02343c9da5e9688c8666b8141c0d1a8fbb1
+  data.tar.gz: 30464b1b58712d5a022c49b4e104fe29e5b82ad44d356cd50d59fb3b54dca0ce6b8fc7ead55e93f6ecf193c24a04782a924fbc650129c570b4606bf04f5b4408

data/bin/foreman-node

@@ -45,7 +45,7 @@ end
 
 def get_grains(minion)
   grains = {
-    :name  => minion,
+    :name => minion,
     :facts => plain_grains(minion).merge(:_timestamp => Time.now, :_type => 'foreman_salt')
   }
 
@@ -94,9 +94,10 @@ end
 def plainify(hash, prefix = nil)
   result = []
   hash.each_pair do |key, value|
-    if value.is_a?(Hash)
+    case value
+    when Hash
       result.push plainify(value, get_key(key, prefix))
-    elsif value.is_a?(Array)
+    when Array
       result.push plainify(array_to_hash(value), get_key(key, prefix))
     else
       new = {}
@@ -169,12 +170,14 @@ def enc(minion)
   res = http.start { |conn| conn.request(req) }
 
   raise "Error retrieving node #{minion}: #{res.class}\nCheck Foreman's /var/log/foreman/production.log for more information." unless res.code == '200'
+
   res.body
 end
 
 minion = ARGV[0] || raise('Must provide minion as an argument')
 
 raise 'Invalid hostname' unless valid_hostname? minion
+
 begin
   result = ''
 

data/lib/smart_proxy_salt/cli.rb

@@ -56,16 +56,15 @@ module Proxy
 
         def autosign_list
           return [] unless File.exist?(autosign_file)
+
           File.read(autosign_file).split("\n").reject do |v|
             v =~ /^\s*#.*|^$/ ## Remove comments and empty lines
           end.map(&:chomp)
         end
 
         def append_value_to_file(filepath, value)
-          File.open(filepath, File::CREAT|File::RDWR) do |file|
-            unless file.any? { |line| line.chomp == value}
-              file.puts value
-            end
+          File.open(filepath, File::CREAT | File::RDWR) do |file|
+            file.puts value unless file.any? { |line| line.chomp == value }
           end
           logger.info "Added an entry to '#{filepath}' successfully."
           true
@@ -75,7 +74,6 @@ module Proxy
         end
 
         def remove_value_from_file(filepath, value)
-
           return true unless File.exist?(filepath)
 
           found = false
@@ -84,14 +82,14 @@ module Proxy
             if entry == value
               found = true
               nil
-            elsif entry == ""
+            elsif entry == ''
               nil
             else
               line
             end
           end.uniq.compact
           if found
-            File.write(filepath, entries.join())
+            File.write(filepath, entries.join)
             logger.info "Removed an entry from '#{filepath}' successfully."
           end
           true

data/lib/smart_proxy_salt/rest.rb

@@ -23,6 +23,7 @@ module Proxy
 
           files.each do |file|
             next unless file =~ /\.sls\Z/ && file != 'top.sls'
+
             states << file.gsub('.sls', '').gsub('/init', '').chomp('/').tr('/', '.')
           end
 

data/lib/smart_proxy_salt/salt.rb

@@ -12,11 +12,11 @@ module Proxy
     class Plugin < ::Proxy::Plugin
       plugin 'salt', Proxy::Salt::VERSION
 
-      default_settings :autosign_file      => '/etc/salt/autosign.conf',
-                       :autosign_key_file  => '/var/lib/foreman-proxy/salt/grains/autosign_key',
-                       :salt_command_user  => 'root',
-                       :use_api            => false,
-                       :saltfile           => '/etc/foreman-proxy/settings.d/salt.saltfile'
+      default_settings :autosign_file => '/etc/salt/autosign.conf',
+                       :autosign_key_file => '/var/lib/foreman-proxy/salt/grains/autosign_key',
+                       :salt_command_user => 'root',
+                       :use_api => false,
+                       :saltfile => '/etc/foreman-proxy/settings.d/salt.saltfile'
 
       requires :dynflow, '>= 0.5.0'
 

data/lib/smart_proxy_salt/salt_runner.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require 'smart_proxy_dynflow/runner/base'
 require 'smart_proxy_dynflow/runner/command_runner'
 

data/lib/smart_proxy_salt/salt_task_launcher.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require 'smart_proxy_dynflow/task_launcher'
 
 module Proxy

data/lib/smart_proxy_salt/version.rb

@@ -3,6 +3,6 @@
 module Proxy
   # Salt module
   module Salt
-    VERSION = '5.0.1'.freeze
+    VERSION = '5.1.0'
   end
 end

data/sbin/upload-salt-reports

@@ -1,4 +1,4 @@
-#!/usr/bin/salt_python_wrapper
+#!/usr/bin/python3
 # Uploads reports from the Salt job cache to Foreman
 
 from __future__ import print_function

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: smart_proxy_salt
 version: !ruby/object:Gem::Version
-  version: 5.0.1
+  version: 5.1.0
 platform: ruby
 authors:
 - Michael Moll
@@ -9,96 +9,15 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2022-11-28 00:00:00.000000000 Z
+date: 2024-04-17 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
-  name: test-unit
-  requirement: !ruby/object:Gem::Requirement
-    requirements:
-    - - "~>"
-      - !ruby/object:Gem::Version
-        version: '2'
-  type: :development
-  prerelease: false
-  version_requirements: !ruby/object:Gem::Requirement
-    requirements:
-    - - "~>"
-      - !ruby/object:Gem::Version
-        version: '2'
-- !ruby/object:Gem::Dependency
-  name: mocha
-  requirement: !ruby/object:Gem::Requirement
-    requirements:
-    - - "~>"
-      - !ruby/object:Gem::Version
-        version: '1'
-  type: :development
-  prerelease: false
-  version_requirements: !ruby/object:Gem::Requirement
-    requirements:
-    - - "~>"
-      - !ruby/object:Gem::Version
-        version: '1'
-- !ruby/object:Gem::Dependency
-  name: webmock
-  requirement: !ruby/object:Gem::Requirement
-    requirements:
-    - - "~>"
-      - !ruby/object:Gem::Version
-        version: '1'
-  type: :development
-  prerelease: false
-  version_requirements: !ruby/object:Gem::Requirement
-    requirements:
-    - - "~>"
-      - !ruby/object:Gem::Version
-        version: '1'
-- !ruby/object:Gem::Dependency
-  name: rake
-  requirement: !ruby/object:Gem::Requirement
-    requirements:
-    - - "~>"
-      - !ruby/object:Gem::Version
-        version: '13'
-  type: :development
-  prerelease: false
-  version_requirements: !ruby/object:Gem::Requirement
-    requirements:
-    - - "~>"
-      - !ruby/object:Gem::Version
-        version: '13'
-- !ruby/object:Gem::Dependency
-  name: rubocop
-  requirement: !ruby/object:Gem::Requirement
-    requirements:
-    - - '='
-      - !ruby/object:Gem::Version
-        version: 0.50.0
-  type: :development
-  prerelease: false
-  version_requirements: !ruby/object:Gem::Requirement
-    requirements:
-    - - '='
-      - !ruby/object:Gem::Version
-        version: 0.50.0
-- !ruby/object:Gem::Dependency
-  name: rack-test
+  name: smart_proxy_dynflow
   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: smart_proxy_dynflow
-  requirement: !ruby/object:Gem::Requirement
-    requirements:
+        version: '0.5'
     - - ">="
       - !ruby/object:Gem::Version
         version: 0.5.0
@@ -106,14 +25,16 @@ dependencies:
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '0.5'
     - - ">="
       - !ruby/object:Gem::Version
         version: 0.5.0
-description: SaltStack Plug-In for Foreman's Smart Proxy
+description: This plug-in adds support for Salt to Foreman's Smart Proxy
 email: foreman-dev@googlegroups.com
 executables:
 - foreman-node
-- salt_python_wrapper
 extensions: []
 extra_rdoc_files:
 - README.md
@@ -122,7 +43,6 @@ files:
 - LICENSE
 - README.md
 - bin/foreman-node
-- bin/salt_python_wrapper
 - bundler.d/salt.rb
 - etc/foreman.conf.example
 - etc/foreman.yaml.example
@@ -160,15 +80,17 @@ required_ruby_version: !ruby/object:Gem::Requirement
   requirements:
   - - ">="
     - !ruby/object:Gem::Version
-      version: '0'
+      version: '2.7'
+  - - "<"
+    - !ruby/object:Gem::Version
+      version: '4'
 required_rubygems_version: !ruby/object:Gem::Requirement
   requirements:
   - - ">="
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubyforge_project: 
-rubygems_version: 2.7.6
+rubygems_version: 3.3.26
 signing_key: 
 specification_version: 4
 summary: SaltStack Plug-In for Foreman's Smart Proxy

data/bin/salt_python_wrapper

@@ -1,16 +0,0 @@
-#!/bin/sh
-
-set -u
-
-for py in 'python3' 'python'; do
-  exe=$(type -p ${py})
-  if [ -n "${exe}" ]; then
-    if ${exe} -c 'import salt.config'; then
-      ${exe} "$@"
-      exit $?
-    fi
-  fi
-done
-
-echo "No usable python version found, check if python or python3 can import salt.config!" 1>&2
-exit 1