hybrid_platforms_conductor 33.1.0 → 33.1.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: b6645a2b01eb498947d9d5e39803a664a18b263f5074d75b2f0191989df050c8
-  data.tar.gz: 4191d477ce05a66619be72968cbbc67d2e906e33f2a9cd797fb18ca3122712de
+  metadata.gz: 8698bc386b9a607d4dda6b8a21ee754860b8027a26f851be1887ec8bbe3aaa96
+  data.tar.gz: ddb3c31da1f23113755fcfc07d97785b0ddc03611922568401043405e50014e8
 SHA512:
-  metadata.gz: 332a36210a05466e95172a3b2eececb081deea896fc19252286c1220fc18ceed70e29b29674127cb7ff51a094ff9358afc1c151f55d0d2135fa987a5d4534002
-  data.tar.gz: 2f7b69283769da47dd742f10b3546cf6625422c16fcdeed9882ade4c6f6a5eef2e1e795b10bc5dd2fa93a8d0bc6fcd0e33fab57fa5c4707f6a2a5afb1ee4df36
+  metadata.gz: 8076f914cd5675de3b990e78191569ba96c368c840644183c5a52f75feaa67f2b93172465595f8d6619c921c694ea16fb05eed18cc559b560f8dae7c5584fdf0
+  data.tar.gz: 42b83e310c9f03473e50c4b9694f3320126ebcbe30a4777845eecda87da4ccad78ffb7bcd99f4fb6b2e8b6f98f131052ae667389961602bb525e7e888fb71243

data/CHANGELOG.md

@@ -1,3 +1,9 @@
+# [v33.1.1](https://github.com/sweet-delights/hybrid-platforms-conductor/compare/v33.1.0...v33.1.1) (2021-06-18 13:19:39)
+
+### Patches
+
+* [[Fix] Corrected bugs on log remote_fs plugin not working in dry-run and topographer](https://github.com/sweet-delights/hybrid-platforms-conductor/commit/90dcde919b6fc99b8f79fc642fef8b4150a5a9d6)
+
 # [v33.1.0](https://github.com/sweet-delights/hybrid-platforms-conductor/compare/v33.0.4...v33.1.0) (2021-06-18 11:37:28)
 
 ## Global changes

data/lib/hybrid_platforms_conductor/deployer.rb

@@ -146,6 +146,7 @@ module HybridPlatformsConductor
             logger: @logger,
             logger_stderr: @logger_stderr,
             config: @config,
+            cmd_runner: @cmd_runner,
             nodes_handler: @nodes_handler,
             actions_executor: @actions_executor
           )

data/lib/hybrid_platforms_conductor/hpc_plugins/log/remote_fs.rb

@@ -121,7 +121,9 @@ module HybridPlatformsConductor
           # ...
           # ===== STDERR =====
           # ...
-          if exit_status.is_a?(Symbol)
+          if @cmd_runner.dry_run
+            { error: 'No log to show in dry-run mode' }
+          elsif exit_status.is_a?(Symbol)
             { error: "Error: #{exit_status}\n#{stderr}" }
           else
             stdout_lines = stdout.split("\n")

data/lib/hybrid_platforms_conductor/log.rb

@@ -12,16 +12,19 @@ module HybridPlatformsConductor
     # * *logger* (Logger): Logger to be used [default: Logger.new(STDOUT)]
     # * *logger_stderr* (Logger): Logger to be used for stderr [default: Logger.new(STDERR)]
     # * *config* (Config): Config to be used. [default: Config.new]
+    # * *cmd_runner* (CmdRunner): CmdRunner to be used [default: CmdRunner.new]
     # * *nodes_handler* (NodesHandler): Nodes handler to be used. [default: NodesHandler.new]
     # * *actions_executor* (ActionsExecutor): Actions executor to be used. [default: ActionsExecutor.new]
     def initialize(
       logger: Logger.new($stdout),
       logger_stderr: Logger.new($stderr),
       config: Config.new,
+      cmd_runner: CmdRunner.new,
       nodes_handler: NodesHandler.new,
       actions_executor: ActionsExecutor.new
     )
       super(logger: logger, logger_stderr: logger_stderr, config: config)
+      @cmd_runner = cmd_runner
       @nodes_handler = nodes_handler
       @actions_executor = actions_executor
     end

data/lib/hybrid_platforms_conductor/topographer.rb

@@ -304,7 +304,8 @@ module HybridPlatformsConductor
 
     # Define clusters of ips with 24 bits ranges.
     def define_clusters_ip_24
-      @nodes_graph.each_key do |node_name|
+      # Clone keys as we modify the hash in the loop
+      @nodes_graph.keys.clone.each do |node_name|
         next unless @nodes_graph[node_name][:type] == :node && !@node_metadata[node_name][:private_ips].nil? && !@node_metadata[node_name][:private_ips].empty?
 
         ip_24 = "#{@node_metadata[node_name][:private_ips].first.split('.')[0..2].join('.')}.0/24"

data/lib/hybrid_platforms_conductor/version.rb

@@ -1,5 +1,5 @@
 module HybridPlatformsConductor
 
-  VERSION = '33.1.0'
+  VERSION = '33.1.1'
 
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: hybrid_platforms_conductor
 version: !ruby/object:Gem::Version
-  version: 33.1.0
+  version: 33.1.1
 platform: ruby
 authors:
 - Muriel Salvan