choria-colt 0.3.0 → 0.5.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 17cd0f4a8fd7bc14aed8bd1f702405d257c8ab18b31406eb13eb3b3b3d6623cd
-  data.tar.gz: 9ad55b6d635820f4486476ed9722e23f68f85037f0b94c0ed93d500a64e10e29
+  metadata.gz: e5a3d94cfa899f41ced9fc0405ed5f4fb41924267d4b56f6d31ba28420c9b0e7
+  data.tar.gz: df5bef6793badb31de0ee0185d0f8d3ca44288fedcf64b34084dbb60d1a6de30
 SHA512:
-  metadata.gz: cac0c586629af294b520c29e4967824cd832b257405be6a95ba4e2a552f1a92cba936b22e219ed0342b8b1cfff34b3257a5c3cf5ab6607f66450d864b636ba12
-  data.tar.gz: 6d6cf405dd1973a4e7d813a58de872a8b37514e821ea90c216a42b3242240895ed1fb4adda8dd24d065fbbf926b7ae186fea610b9e54f5c2e5df3a7fa75b94ef
+  metadata.gz: bba0206e6f84679cd6c7ed5fd1831dc8e2beba052b096b265ac272baadeaf20a69585d6165a37ca1f7db2ca48a6fe532eeb77171c75f656bb0aa9f2c50796ccb
+  data.tar.gz: 02d6f79e80a664445793596ea9d96f106a14523ba9824893e7c1d43cc2c3484f63e27a1a20173d30e608e48f4d59872db9878385da2b197021d56374c502defa

data/.rubocop.yml

@@ -10,6 +10,11 @@ AllCops:
 Metrics/AbcSize:
   Max: 20
 
+Metrics/ClassLength:
+  Max: 120
+  Exclude:
+    - 'lib/choria/colt/cli.rb'
+
 Metrics/MethodLength:
   Max: 20
 

data/.rubocop_todo.yml

@@ -1,22 +1,17 @@
 # This configuration was generated by
 # `rubocop --auto-gen-config`
-# on 2022-04-07 15:43:34 UTC using RuboCop version 1.26.0.
+# on 2022-04-28 09:34:37 UTC using RuboCop version 1.26.0.
 # The point is for the user to remove these configuration records
 # one by one as the offenses are removed from the code base.
 # Note that changes in the inspected code, or installation of new
 # versions of RuboCop, may require this file to be generated again.
 
 # Offense count: 1
-# Configuration parameters: CountComments, CountAsOne.
-Metrics/ClassLength:
-  Max: 109
+# Configuration parameters: IgnoredMethods, CountRepeatedAttributes.
+Metrics/AbcSize:
+  Max: 22
 
 # Offense count: 1
 # Configuration parameters: IgnoredMethods.
 Metrics/CyclomaticComplexity:
-  Max: 9
-
-# Offense count: 1
-# Configuration parameters: IgnoredMethods.
-Metrics/PerceivedComplexity:
-  Max: 9
+  Max: 8

data/CHANGELOG.md

@@ -0,0 +1,58 @@
+# Changelog
+
+## [v0.5.0](https://github.com/opus-codium/choria-colt/tree/v0.5.0) (2022-05-02)
+
+[Full Changelog](https://github.com/opus-codium/choria-colt/compare/v0.4.0...v0.5.0)
+
+**Merged pull requests:**
+
+- Process and display RPC errors like execution errors [\#24](https://github.com/opus-codium/choria-colt/pull/24) ([neomilium](https://github.com/neomilium))
+- CLI: Be more green\(washing\) [\#23](https://github.com/opus-codium/choria-colt/pull/23) ([neomilium](https://github.com/neomilium))
+- CLI: Support @file notation on --targets option [\#22](https://github.com/opus-codium/choria-colt/pull/22) ([neomilium](https://github.com/neomilium))
+- Do not pass explicit options to rpcclient [\#21](https://github.com/opus-codium/choria-colt/pull/21) ([smortex](https://github.com/smortex))
+
+## [v0.4.0](https://github.com/opus-codium/choria-colt/tree/v0.4.0) (2022-04-25)
+
+[Full Changelog](https://github.com/opus-codium/choria-colt/compare/v0.3.0...v0.4.0)
+
+**Merged pull requests:**
+
+- Spec: Fix tests after duration introduction [\#20](https://github.com/opus-codium/choria-colt/pull/20) ([neomilium](https://github.com/neomilium))
+- CLI: Add `colt task status` subcommand [\#19](https://github.com/opus-codium/choria-colt/pull/19) ([neomilium](https://github.com/neomilium))
+
+## [v0.3.0](https://github.com/opus-codium/choria-colt/tree/v0.3.0) (2022-04-12)
+
+[Full Changelog](https://github.com/opus-codium/choria-colt/compare/v0.2.0...v0.3.0)
+
+**Merged pull requests:**
+
+- Improve CLI output [\#18](https://github.com/opus-codium/choria-colt/pull/18) ([neomilium](https://github.com/neomilium))
+- CLI: Add an option to choose log level [\#17](https://github.com/opus-codium/choria-colt/pull/17) ([neomilium](https://github.com/neomilium))
+- Improve stability, fix results retrieving [\#16](https://github.com/opus-codium/choria-colt/pull/16) ([neomilium](https://github.com/neomilium))
+- CLI: Add an option to set log level [\#15](https://github.com/opus-codium/choria-colt/pull/15) ([neomilium](https://github.com/neomilium))
+- Setup CI [\#14](https://github.com/opus-codium/choria-colt/pull/14) ([neomilium](https://github.com/neomilium))
+- CLI: Format error [\#13](https://github.com/opus-codium/choria-colt/pull/13) ([neomilium](https://github.com/neomilium))
+- CLI: Display default value if available for parameters [\#12](https://github.com/opus-codium/choria-colt/pull/12) ([neomilium](https://github.com/neomilium))
+- Test CLI formatter using RSpec [\#11](https://github.com/opus-codium/choria-colt/pull/11) ([neomilium](https://github.com/neomilium))
+- Colorize outputs [\#10](https://github.com/opus-codium/choria-colt/pull/10) ([neomilium](https://github.com/neomilium))
+- Fix task input option parsing and task output display [\#9](https://github.com/opus-codium/choria-colt/pull/9) ([neomilium](https://github.com/neomilium))
+- Implement continous result display [\#8](https://github.com/opus-codium/choria-colt/pull/8) ([neomilium](https://github.com/neomilium))
+- Improve robustness on errors [\#7](https://github.com/opus-codium/choria-colt/pull/7) ([neomilium](https://github.com/neomilium))
+- Minor code readability improvements [\#6](https://github.com/opus-codium/choria-colt/pull/6) ([neomilium](https://github.com/neomilium))
+- Improve logging [\#5](https://github.com/opus-codium/choria-colt/pull/5) ([neomilium](https://github.com/neomilium))
+- Force convertion to boolean if parameter is true/false [\#4](https://github.com/opus-codium/choria-colt/pull/4) ([neomilium](https://github.com/neomilium))
+- Tasks: Autofill default values [\#3](https://github.com/opus-codium/choria-colt/pull/3) ([neomilium](https://github.com/neomilium))
+- CLI: Support filter on Puppet classes [\#2](https://github.com/opus-codium/choria-colt/pull/2) ([neomilium](https://github.com/neomilium))
+- Tasks: Structure data before return results [\#1](https://github.com/opus-codium/choria-colt/pull/1) ([neomilium](https://github.com/neomilium))
+
+## [v0.2.0](https://github.com/opus-codium/choria-colt/tree/v0.2.0) (2022-03-03)
+
+[Full Changelog](https://github.com/opus-codium/choria-colt/compare/v0.1.1...v0.2.0)
+
+## [v0.1.1](https://github.com/opus-codium/choria-colt/tree/v0.1.1) (2022-03-03)
+
+[Full Changelog](https://github.com/opus-codium/choria-colt/compare/v0.1.0...v0.1.1)
+
+
+
+\* *This Changelog was automatically generated by [github_changelog_generator](https://github.com/github-changelog-generator/github-changelog-generator)*

data/README.md

@@ -47,7 +47,14 @@ Or install it yourself as:
 
 After checking out the repo, run `bin/setup` to install dependencies. You can also run `bin/console` for an interactive prompt that will allow you to experiment.
 
-To install this gem onto your local machine, run `bundle exec rake install`. To release a new version, update the version number in `version.rb`, and then run `bundle exec rake release`, which will create a git tag for the version, push git commits and the created tag, and push the `.gem` file to [rubygems.org](https://rubygems.org).
+To install this gem onto your local machine, run `bundle exec rake install`.
+
+
+To release a new version:
+  1. update the version number in `version.rb`
+  1. generate CHANGELOG.md with `bundle exec rake changelog`
+  1. commit changes
+  1. run `bundle exec rake release`, which will create a git tag for the version, push git commits and the created tag, and push the `.gem` file to [rubygems.org](https://rubygems.org).
 
 ## Contributing
 

data/Rakefile

@@ -3,6 +3,17 @@
 require 'bundler/gem_tasks'
 require 'rubocop/rake_task'
 
+require 'choria/colt/version'
+
 RuboCop::RakeTask.new
 
+require 'github_changelog_generator/task'
+
+GitHubChangelogGenerator::RakeTask.new :changelog do |config|
+  config.user = 'opus-codium'
+  config.project = 'choria-colt'
+  config.since_tag = 'v0.1.0'
+  config.future_release = "v#{Choria::Colt::VERSION}"
+end
+
 task default: :rubocop

data/choria-colt.gemspec

@@ -37,6 +37,7 @@ Gem::Specification.new do |spec|
   spec.add_dependency 'tty-logger'
 
   # spec.add_development_dependency 'byebug'
+  spec.add_development_dependency 'github_changelog_generator'
   spec.add_development_dependency 'rake'
   spec.add_development_dependency 'rspec'
   spec.add_development_dependency 'rubocop'

data/lib/choria/colt/cli/formatter.rb

@@ -5,6 +5,33 @@ module Choria
   class Colt
     class CLI < Thor
       class Formatter
+        module Result
+          def sender
+            self[:sender]
+          end
+
+          def exitcode
+            dig(:data, :exitcode)
+          end
+
+          def ok?
+            exitcode&.zero?
+          end
+
+          def runtime
+            dig(:data, :runtime)
+          end
+
+          # CLI
+          def output
+            if dig(:result, :_output).nil?
+              JSON.pretty_generate(self[:result]).split("\n")
+            else
+              dig(:result, :_output)
+            end
+          end
+        end
+
         attr_reader :pastel
 
         def initialize(colored:)
@@ -13,27 +40,36 @@ module Choria
         end
 
         def process_result(result)
-          return process_error(result) unless result.dig(:data, :exitcode)&.zero?
+          result.extend Formatter::Result
 
-          process_success(result)
+          case result[:statuscode]
+          when 0
+            # 0 	OK
+            process_success(result)
+          when 1
+            # 1 	OK, failed. All the data parsed ok, we have a action matching the request but the requested action could not be completed. 	RPCAborted
+            process_error(result) # unless result.ok?
+          else
+            # 2 	Unknown action 	UnknownRPCAction
+            # 3 	Missing data 	MissingRPCData
+            # 4 	Invalid data 	InvalidRPCData
+            # 5 	Other error
+            process_rpc_error(result)
+          end
         end
 
         def process_success(result)
-          output_lines = [
-            "#{pastel.host(result[:sender]).ljust(60, ' ')}duration: #{pastel.bright_white result[:data][:runtime]}",
-          ]
-
-          output_lines += if result.dig(:result, :_output).nil?
-                            JSON.pretty_generate(result[:result]).split("\n").map { |line| "  #{line}" }
-                          else
-                            result.dig(:result, :_output).map { |line| "  #{line}" }
-                          end
+          host = format_host(result, "#{pastel.bright_green '√'} ")
+          headline = "#{pastel.on_green ' '} "
 
-          output_lines.join("\n")
+          [
+            host,
+            result.output.map { |line| "#{headline}#{line}" },
+          ].flatten.join("\n")
         end
 
-        def process_error(result) # rubocop:disable Metrics/AbcSize
-          host = "#{pastel.bright_red '⨯'} #{pastel.host(result[:sender]).ljust(60, ' ')}duration: #{pastel.bright_white result[:data][:runtime]}"
+        def process_error(result) # rubocop:disable Metrics/AbcSize, Metrics/MethodLength
+          host = format_host(result, "#{pastel.bright_red '⨯'} ")
           output = result.dig(:result, '_output')
           error_details = JSON.pretty_generate(result.dig(:result, :_error, :details)).split "\n"
           error_description = [
@@ -61,6 +97,26 @@ module Choria
             ].flatten.map { |line| "#{headline}#{line}" },
           ].flatten.join("\n")
         end
+
+        def process_rpc_error(result)
+          host = "#{pastel.bright_red '⨯'} #{pastel.host(result.sender)}"
+          headline = "#{pastel.on_red ' '} "
+
+          [
+            host,
+            "#{headline}#{pastel.bright_red "RPC error (#{result[:statuscode]})"}: #{pastel.bright_white result[:statusmsg]}",
+          ].join("\n")
+        end
+
+        private
+
+        def format_duration(result)
+          result.runtime.nil? ? '' : "duration: #{pastel.bright_white format('%.2fs', result.runtime)}"
+        end
+
+        def format_host(result, headline)
+          "#{headline}#{pastel.host(result.sender).ljust(60, ' ')}#{format_duration(result)}"
+        end
       end
     end
   end

data/lib/choria/colt/cli.rb

@@ -27,6 +27,7 @@ module Choria
                desc: 'Select the targets which have the specified Puppet classes.'
         def run(*args) # rubocop:disable Metrics/AbcSize
           input = extract_task_parameters_from_args(args)
+          targets = extract_targets_from_options
 
           raise Thor::Error, 'Task name is required' if args.empty?
           raise Thor::Error, "Too many arguments: #{args}" unless args.count == 1
@@ -35,8 +36,7 @@ module Choria
 
           task_name = args.shift
 
-          targets = options['targets']&.split(',')
-          targets = nil if options['targets'] == 'all'
+          logger.debug "Targets: #{targets}"
 
           targets_with_classes = options['targets_with_classes']&.split(',')
 
@@ -45,8 +45,8 @@ module Choria
           end
 
           File.write 'last_run.json', JSON.pretty_generate(results)
-        rescue Choria::Orchestrator::Error => e
-          raise Thor::Error, "#{e.class}: #{e}"
+        rescue Choria::Orchestrator::Error
+          # This error is already logged and displayed.
         end
 
         desc 'show [task name] [options]', 'Show available tasks and task documentation'
@@ -78,6 +78,22 @@ module Choria
           end
         end
 
+        desc 'status <task id>', 'Show task results'
+        long_desc <<~DESC
+          Show results from a previously ran task.
+
+          A task ID is required to request Choria services and retrieve results.
+        DESC
+        def status(task_id)
+          results = colt.wait_bolt_task task_id do |result|
+            $stdout.puts formatter.process_result(result)
+          end
+
+          File.write 'last_run.json', JSON.pretty_generate(results)
+        rescue Choria::Orchestrator::Error
+          # This error is already logged and displayed.
+        end
+
         no_commands do # rubocop:disable Metrics/BlockLength
           def colt
             @colt ||= Choria::Colt.new logger: logger
@@ -114,6 +130,16 @@ module Choria
             end.to_h
           end
 
+          def extract_targets_from_options
+            return nil if options['targets'] == 'all'
+
+            targets = options['targets']&.split(',')
+            targets&.map do |t|
+              t = File.read(t.sub(/^@/, '')).lines.map(&:strip) if t.start_with? '@'
+              t
+            end&.flatten
+          end
+
           def show_tasks_summary(tasks)
             tasks.reject! { |_task, metadata| metadata['metadata']['private'] }
 

data/lib/choria/colt/data_structurer.rb

@@ -2,6 +2,8 @@ module Choria
   class Colt
     module DataStructurer
       def self.structure(res) # rubocop:disable Metrics/AbcSize
+        return res unless [0, 1].include? res[:statuscode]
+
         # data.stdout seems to always be JSON, so parse it once.
         res[:result] = JSON.parse res[:data][:stdout]
         res[:data].delete :stdout

data/lib/choria/colt/version.rb

@@ -2,6 +2,6 @@
 
 module Choria
   class Colt
-    VERSION = '0.3.0'
+    VERSION = '0.5.1'
   end
 end

data/lib/choria/colt.rb

@@ -21,7 +21,7 @@ module Choria
 
     def run_bolt_task(task_name, input: {}, targets: nil, targets_with_classes: nil, &block)
       logger.debug "Instantiate task '#{task_name}' and validate input"
-      task = Choria::Orchestrator::Task.new(task_name, input: input, orchestrator: orchestrator)
+      task = Choria::Orchestrator::Task.new(name: task_name, input: input, orchestrator: orchestrator)
 
       task.on_result(&block) if block_given?
 
@@ -33,6 +33,18 @@ module Choria
       raise
     end
 
+    def wait_bolt_task(task_id, &block)
+      task = Choria::Orchestrator::Task.new(id: task_id, orchestrator: orchestrator)
+
+      task.on_result(&block) if block_given?
+
+      task.wait
+      task.results
+    rescue Choria::Orchestrator::Error => e
+      logger.error e.message
+      raise
+    end
+
     def tasks(environment:, cache: nil)
       tasks_names = orchestrator.tasks_support.tasks(environment).map do |task|
         task['name']

data/lib/choria/orchestrator/task/result_set.rb

@@ -0,0 +1,28 @@
+require 'choria/colt/data_structurer'
+
+module Choria
+  class Orchestrator
+    class Task
+      class ResultSet
+        attr_reader :results
+
+        def initialize(on_result:)
+          @results = []
+          @on_result = on_result
+        end
+
+        def integrate_rpc_error(rpc_error)
+          result = rpc_error[:body]
+          result[:sender] = rpc_error[:senderid]
+          integrate_result(result)
+        end
+
+        def integrate_result(result)
+          structured_result = Choria::Colt::DataStructurer.structure(result).with_indifferent_access
+          @results << structured_result
+          @on_result&.call(structured_result)
+        end
+      end
+    end
+  end
+end

data/lib/choria/orchestrator/task.rb

@@ -1,4 +1,4 @@
-require 'choria/colt/data_structurer'
+require_relative 'task/result_set'
 
 require 'active_support'
 require 'active_support/core_ext/hash/indifferent_access'
@@ -7,18 +7,18 @@ module Choria
   class Orchestrator
     class Task
       class Error < Orchestrator::Error; end
+      class NoNodesLeftError < Error; end
 
-      attr_reader :name, :input, :environment, :rpc_results, :results
-      attr_accessor :rpc_responses
+      attr_reader :id, :name, :input, :environment
 
-      def initialize(name, orchestrator:, input: {}, environment: 'production')
+      def initialize(orchestrator:, id: nil, name: nil, input: {}, environment: 'production')
+        @id = id
         @name = name
         @environment = environment
         @orchestrator = orchestrator
-        @input = default_input.merge input
-
-        @results = []
+        return if @name.nil?
 
+        @input = default_input.merge input
         logger.debug "Task inputs: #{input}"
         validate_inputs
       end
@@ -31,69 +31,82 @@ module Choria
         metadata['files'].to_json
       end
 
-      def wait # rubocop:disable Metrics/AbcSize
-        rpc_responses_ok, rpc_responses_error = rpc_responses.partition { |res| (res[:body][:statuscode]).zero? }
-        rpc_responses_error.each do |res|
-          logger.error "Task request failed on '#{res[:senderid]}':\n#{pp res}"
-        end
+      def results
+        result_set.results
+      end
 
-        task_ids = rpc_responses_ok.map { |res| res[:body][:data][:task_id] }.uniq
+      def run
+        raise Error, 'Unable to run a task by ID' if name.nil?
 
-        raise NotImplementedError, "Multiple task IDs: #{task_ids}" unless task_ids.count == 1
+        @pending_targets = rpc_client.discover
+        _download
+        _run_no_wait
+      end
 
-        @id = task_ids.first
+      def wait
+        raise Error, 'Task ID is required!' if @id.nil?
 
-        wait_results
+        logger.info "Waiting task #{@id} results…"
+        @rpc_results = []
+        loop do
+          self.rpc_results = rpc_client.task_status(task_id: @id).map(&:results)
+          break if @pending_targets.empty?
+        end
       end
 
       def on_result(&block)
-        @on_result = lambda { |result|
-          block.call(result)
-        }
+        @on_result = ->(result) { block.call(result) }
       end
 
       private
 
-      def rpc_results=(results)
-        new_result_hosts = (results.map { |res| res[:sender] }) - (@results.map { |res| res[:sender] })
-
-        new_result_hosts.each do |host|
-          result = results.find { |res| res[:sender] == host }
-
-          next unless result[:data][:exitcode] != -1
-
-          logger.debug "New result for task ##{@id}: #{result}"
-          structured_result = Choria::Colt::DataStructurer.structure(result).with_indifferent_access
-
-          @on_result&.call(structured_result)
+      def result_set
+        @result_set ||= ResultSet.new(on_result: @on_result)
+      end
 
-          @results << structured_result
+      def rpc_results=(results)
+        pending_results, completed_results = results.partition { |res| res[:data][:exitcode] == -1 }
+        @pending_targets ||= pending_results.map { |res| res[:sender] }
+
+        new_results = completed_results.select { |res| @pending_targets.include? res[:sender] }
+        new_results.each do |res|
+          logger.debug "New result for task ##{@id}: #{res}"
+          result_set.integrate_result(res)
+          @pending_targets.delete res[:sender]
         end
-
-        @rpc_results = results
       end
 
-      def wait_results
-        raise 'Task ID is required!' if @id.nil?
+      def process_rpc_response(rpc_response)
+        rpc_response.extend Orchestrator::RpcResponse
+        logger.debug "  RPC Response: '#{rpc_response}'"
+        return unless rpc_response.rpc_error?
 
-        logger.info 'Waiting task results…'
-
-        @results = []
-        @rpc_results = []
-
-        loop do
-          self.rpc_results = @orchestrator.rpc_client.task_status(task_id: @id).map(&:results)
+        @pending_targets.delete rpc_response.sender
+        result_set.integrate_rpc_error(rpc_response)
+      end
 
-          break if terminated?
+      def _download
+        logger.info "Downloading task '#{name}' on #{rpc_client.discover.size} nodes…"
+        rpc_client.download(task: name, files: files, verbose: false) do |rpc_response|
+          process_rpc_response(rpc_response)
         end
+
+        raise NoNodesLeftError, "No nodes left to continue after 'download' action" if @pending_targets.empty?
       end
 
-      def terminated?
-        @rpc_results.each do |result|
-          return false if result[:data][:exitcode] == -1
+      def _run_no_wait # rubocop:disable Metrics/AbcSize
+        logger.info "Starting task '#{name}' on #{rpc_client.discover.size} nodes…"
+        task_ids = []
+        rpc_client.run_no_wait(task: name, files: files, input: input.to_json, verbose: false) do |rpc_response|
+          process_rpc_response(rpc_response)
+          task_ids << rpc_response.task_id
         end
+        raise NoNodesLeftError, "No nodes left to continue after 'run_no_wait' action" if @pending_targets.empty?
+
+        task_ids.uniq!
+        raise NotImplementedError, "Multiple task IDs: #{task_ids}" unless task_ids.count == 1
 
-        true
+        @id = task_ids.first
       end
 
       def _metadata
@@ -118,6 +131,10 @@ module Choria
       def logger
         @orchestrator.logger
       end
+
+      def rpc_client
+        @orchestrator.rpc_client
+      end
     end
   end
 end

data/lib/choria/orchestrator.rb

@@ -6,6 +6,24 @@ module Choria
     class Error < StandardError; end
     class DiscoverError < Error; end
 
+    module RpcResponse
+      def sender
+        self[:senderid]
+      end
+
+      def rpc_error?
+        !rpc_success?
+      end
+
+      def rpc_success?
+        [0, 1].include? self[:body][:statuscode]
+      end
+
+      def task_id
+        self[:body][:data][:task_id]
+      end
+    end
+
     include MCollective::RPC
 
     attr_reader :logger
@@ -33,51 +51,13 @@ module Choria
       end
 
       logger.info 'Discovering targets…'
-      raise DiscoverError, 'No request sent, no node discovered' if rpc_client.discover.size.zero?
+      raise DiscoverError, 'No requests sent, no nodes discovered' if rpc_client.discover.size.zero?
 
-      logger.info "Downloading task '#{task.name}' on #{rpc_client.discover.size} nodes…"
-      rpc_client.download(task: task.name, files: task.files, verbose: verbose)
-
-      responses = []
-      logger.info "Starting task '#{task.name}' on #{rpc_client.discover.size} nodes…"
-      rpc_client.run_no_wait(task: task.name, files: task.files, input: task.input.to_json, verbose: verbose) do |response|
-        logger.debug "  Response: '#{response}'"
-        responses << response
-      end
-
-      # TODO: Include stats in logs when logger will be available (see MCollective::RPC#printrpcstats)
-
-      task.rpc_responses = responses
-    end
-
-    def validate_rpc_result(result)
-      raise Error, "The RPC agent returned an error: #{result[:statusmsg]}" unless (result[:statuscode]).zero?
+      task.run
     end
 
     def rpc_client
-      @rpc_client ||= rpcclient('bolt_tasks', options: rpc_options)
-    end
-
-    private
-
-    def rpc_options
-      {
-        verbose: false,
-        disctimeout: nil,
-        timeout: 5,
-        config: '/etc/choria/client.conf',
-        collective: 'mcollective',
-        discovery_method: nil,
-        discovery_options: [],
-        filter: {
-          'fact' => [], 'cf_class' => [], 'agent' => [], 'identity' => [], 'compound' => []
-        },
-        progress_bar: false,
-        mcollective_limit_targets: false,
-        batch_size: nil,
-        batch_sleep_time: 1,
-        output_format: :json,
-      }
+      @rpc_client ||= rpcclient('bolt_tasks', options: {})
     end
   end
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: choria-colt
 version: !ruby/object:Gem::Version
-  version: 0.3.0
+  version: 0.5.1
 platform: ruby
 authors:
 - Romuald Conty
 autorequire: 
 bindir: exe
 cert_chain: []
-date: 2022-04-12 00:00:00.000000000 Z
+date: 2022-05-02 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: activesupport
@@ -108,6 +108,20 @@ dependencies:
     - - ">="
       - !ruby/object:Gem::Version
         version: '0'
+- !ruby/object:Gem::Dependency
+  name: github_changelog_generator
+  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
@@ -189,6 +203,7 @@ files:
 - ".rspec"
 - ".rubocop.yml"
 - ".rubocop_todo.yml"
+- CHANGELOG.md
 - Gemfile
 - README.md
 - Rakefile
@@ -203,6 +218,7 @@ files:
 - lib/choria/colt/version.rb
 - lib/choria/orchestrator.rb
 - lib/choria/orchestrator/task.rb
+- lib/choria/orchestrator/task/result_set.rb
 - sig/choria/colt.rbs
 homepage: https://github.com/opus-codium/choria-colt
 licenses: []