Uploaded image for project: 'Puppet Server'
  1. Puppet Server
  2. SERVER-1671

generate() stdout is written to puppetserver.log

    Details

    • Template:
    • Team:
      Systems Engineering
    • Story Points:
      2
    • Sprint:
      SE 2017-01-11, SE 2017-01-25
    • Release Notes:
      Bug Fix
    • Release Notes Summary:
      Hide
      SERVER-1571 updated the generate() function to merge stdout and stderr and return the combined value, but this change had an unintended side effect of logging both stdout and stderr to the puppetserver log file. This has been fixed to preserve the behavior of returning both stderr and stdout from the generate() function call but only logging stderr to puppetserver.log. If a command invoked through generate() does not write to stderr then no log will be generated.
      Show
      SERVER-1571 updated the generate() function to merge stdout and stderr and return the combined value, but this change had an unintended side effect of logging both stdout and stderr to the puppetserver log file. This has been fixed to preserve the behavior of returning both stderr and stdout from the generate() function call but only logging stderr to puppetserver.log. If a command invoked through generate() does not write to stderr then no log will be generated.
    • QA Risk Assessment:
      Needs Assessment

      Description

      It seems recent versions of Puppetserver log all stdout and stderr output in lines with

       [c.p.p.ShellUtils] Executed an external process which logged to STDERR:
      

      instead of just doing so for generate() calls that actually produce output on stderr.

      My guess would be that this is related to SERVER-1571 where stdout and stderr seem to be merged now in generate calls (not quite sure what the use case is for having error messages or warnings mixed into your output).

        Attachments

          Issue Links

            Activity

              jsd-sla-details-panel

                People

                • Assignee:
                  jeremy.barlow Jeremy Barlow
                  Reporter:
                  mhoermann Matthias Hörmann
                • Votes:
                  0 Vote for this issue
                  Watchers:
                  5 Start watching this issue

                  Dates

                  • Created:
                    Updated:
                    Resolved: