Fix missed conversions from OpenStruct to Hash

Patrick Robinson · Patrick Robinson · commit 946358c6bfca · 2020-03-13T09:28:38.000+11:00
diff --git a/app/models/git_repository.rb b/app/models/git_repository.rb
@@ -187,7 +187,6 @@ def capture_stdout(*command, dir: repo_cache_dir)
       *command,
       whitelist_env: ['HOME', 'PATH'],
       timeout: 30.minutes,
-      err: '/dev/null',
       dir: dir
     )
     unless result[:status]
diff --git a/plugins/gcloud/lib/samson_gcloud/tag_resolver.rb b/plugins/gcloud/lib/samson_gcloud/tag_resolver.rb
@@ -17,7 +17,7 @@ def resolve_docker_image_tag(image)
           whitelist_env: ["PATH"]
         )
         raise "GCLOUD ERROR: unable to resolve #{image}\n#{result[:error]}" unless result[:status]
-        digest = JSON.parse(result.output).dig_fetch("image_summary", "digest")
+        digest = JSON.parse(result[:output]).dig_fetch("image_summary", "digest")
 
         base = image.split(":", 2).first
         "#{base}@#{digest}"

Original file line number	Diff line number	Diff line change
`@@ -187,7 +187,6 @@ def capture_stdout(*command, dir: repo_cache_dir)`
`187`	`187`	`*command,`
`188`	`188`	`whitelist_env: ['HOME', 'PATH'],`
`189`	`189`	`timeout: 30.minutes,`
`190`		`- err: '/dev/null',`
`191`	`190`	`dir: dir`
`192`	`191`	`)`
`193`	`192`	`unless result[:status]`
Original file line number	Diff line number	Diff line change
`@@ -17,7 +17,7 @@ def resolve_docker_image_tag(image)`
`17`	`17`	`whitelist_env: ["PATH"]`
`18`	`18`	`)`
`19`	`19`	`raise "GCLOUD ERROR: unable to resolve #{image}\n#{result[:error]}" unless result[:status]`
`20`		`- digest = JSON.parse(result.output).dig_fetch("image_summary", "digest")`
	`20`	`+ digest = JSON.parse(result[:output]).dig_fetch("image_summary", "digest")`
`21`	`21`
`22`	`22`	`base = image.split(":", 2).first`
`23`	`23`	`"#{base}@#{digest}"`