Skip to content

Commit 560d23a

Browse files
committed
(FM-8969) Emit task output
Emit log results when something goes wrong otherwise there's no record of what happened.
1 parent 7ace00d commit 560d23a

File tree

1 file changed

+28
-0
lines changed

1 file changed

+28
-0
lines changed

task_spec/spec/acceptance/init_spec.rb

Lines changed: 28 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -32,6 +32,12 @@ def target_platform
3232
hosts.first[:platform]
3333
end
3434

35+
def log_output_errors(result)
36+
return if result['status'] == 'success'
37+
out = result.dig('value', '_output') || 'Unknown result output'
38+
puts logger.info(out)
39+
end
40+
3541
it 'works with version and install tasks' do
3642
puppet_6_version = case target_platform
3743
when %r{debian-11}
@@ -94,6 +100,11 @@ def target_platform
94100
'version' => puppet_6_version,
95101
'stop_service' => true })
96102

103+
results.each do |result|
104+
logger.info("Installed puppet-agent on #{result['target']}: #{result['status']}")
105+
log_output_errors(result)
106+
end
107+
97108
expect(results).to all(include('status' => 'success'))
98109

99110
# It installed a version older than latest puppet6
@@ -121,6 +132,11 @@ def target_platform
121132
# Expect nothing to happen and receive a message regarding this
122133
results = run_task('puppet_agent::install', 'target', { 'collection' => puppet_6_collection })
123134

135+
results.each do |result|
136+
logger.info("Ensuring installed puppet-agent on #{result['target']}: #{result['status']}")
137+
log_output_errors(result)
138+
end
139+
124140
results.each do |res|
125141
expect(res).to include('status' => 'success')
126142
expect(res['value']['_output']).to match(%r{Version parameter not defined and agent detected. Nothing to do.})
@@ -144,6 +160,12 @@ def target_platform
144160

145161
# Upgrade to latest puppet6 version
146162
results = run_task('puppet_agent::install', 'target', { 'collection' => 'puppet6', 'version' => 'latest' })
163+
164+
results.each do |result|
165+
logger.info("Upgraded puppet-agent to latest puppet6 on #{result['target']}: #{result['status']}")
166+
log_output_errors(result)
167+
end
168+
147169
expect(results).to all(include('status' => 'success'))
148170

149171
# Verify that it upgraded
@@ -173,6 +195,12 @@ def target_platform
173195

174196
# Succesfully upgrade from puppet6 to puppet7
175197
results = run_task('puppet_agent::install', 'target', { 'collection' => puppet_7_collection, 'version' => 'latest' })
198+
199+
results.each do |result|
200+
logger.info("Upgraded puppet-agent to puppet7 on #{result['target']}: #{result['status']}")
201+
log_output_errors(result)
202+
end
203+
176204
expect(results).to all(include('status' => 'success'))
177205

178206
# Verify that it upgraded

0 commit comments

Comments
 (0)