always log foreman-rake output

this is useful when debugging which migrations ran and how long they took
theforeman · Jun 17, 2024 · f31e9ba · f31e9ba
1 parent 7a820e4
commit f31e9ba
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 5 deletions.
diff --git a/manifests/rake.pp b/manifests/rake.pp
@@ -22,7 +22,7 @@
     command     => "/usr/sbin/foreman-rake ${title}",
     user        => $user,
     environment => sort(join_keys_to_values({ 'HOME' => $app_root } + $environment, '=')),
-    logoutput   => 'on_failure',
+    logoutput   => true,
     refreshonly => $unless =~ Undef,
     timeout     => $timeout,
     unless      => $unless,

diff --git a/spec/defines/foreman_rake_spec.rb b/spec/defines/foreman_rake_spec.rb
@@ -22,7 +22,7 @@
           .with_command('/usr/sbin/foreman-rake db:migrate')
           .with_user('foreman')
           .with_environment(['HOME=/usr/share/foreman'])
-          .with_logoutput('on_failure')
+          .with_logoutput(true)
           .with_refreshonly(true)
           .with_timeout(nil)
           .with_unless(nil)
@@ -37,7 +37,7 @@
           .with_command('/usr/sbin/foreman-rake db:migrate')
           .with_user('foreman')
           .with_environment(['HOME=/usr/share/foreman', 'SEED_USER=admin'])
-          .with_logoutput('on_failure')
+          .with_logoutput(true)
           .with_refreshonly(true)
       end
     end
@@ -51,7 +51,7 @@
           .with_user('foreman')
           .with_environment(['HOME=/usr/share/foreman'])
           .with_timeout(60)
-          .with_logoutput('on_failure')
+          .with_logoutput(true)
           .with_refreshonly(true)
       end
     end
@@ -64,7 +64,7 @@
           .with_command('/usr/sbin/foreman-rake db:migrate')
           .with_user('foreman')
           .with_environment(['HOME=/usr/share/foreman'])
-          .with_logoutput('on_failure')
+          .with_logoutput(true)
           .with_refreshonly(false)
           .with_unless('/usr/bin/true')
       end