Uploaded image for project: 'Jenkins'
  1. Jenkins
  2. JENKINS-52729

Launcher.ProcStarter.stdout(TaskListener) discards remotability

    XMLWordPrintable

Details

    • Bug
    • Status: Resolved (View Workflow)
    • Major
    • Resolution: Fixed
    • core
    • None
    • Jenkins 2.136

    Description

      When Launcher.ProcStarter.stdout(TaskListener) is called, this is merely an alias for stdout(OutputStream): getLogger() is called on the master side. Whereas the TaskListener passed to RemoteLauncher constructor and used for printCommandLine is properly remoted, that used for process stdout (and bundled stderr) is not.

      When the implementation happened to be a StreamTaskListener, RemoteLauncher.launch winds up wrapping the stream in RemoteOutputStream, so content is still sent back to the master. But an implementation which does something else with the content on the remote side will not be honored.

      Attachments

        Issue Links

          Activity

            jglick Jesse Glick created issue -
            jglick Jesse Glick made changes -
            Field Original Value New Value
            Link This issue blocks JENKINS-38381 [ JENKINS-38381 ]
            jglick Jesse Glick made changes -
            Epic Link JENKINS-38313 [ 174485 ]
            jglick Jesse Glick made changes -
            Link This issue relates to JENKINS-31096 [ JENKINS-31096 ]
            jglick Jesse Glick made changes -
            Link This issue relates to JENKINS-48923 [ JENKINS-48923 ]
            jglick Jesse Glick made changes -
            Remote Link This issue links to "PR 3563 (Web Link)" [ 21231 ]
            jglick Jesse Glick made changes -
            Assignee Jesse Glick [ jglick ]
            jglick Jesse Glick made changes -
            Status Open [ 1 ] In Progress [ 3 ]
            jglick Jesse Glick made changes -
            Status In Progress [ 3 ] In Review [ 10005 ]
            oleg_nenashev Oleg Nenashev made changes -
            Released As Jenkins 2.136
            Resolution Fixed [ 1 ]
            Status In Review [ 10005 ] Resolved [ 5 ]
            oleg_nenashev Oleg Nenashev made changes -
            Labels lts-candidate
            olivergondza Oliver Gond┼ża made changes -
            Labels lts-candidate
            jglick Jesse Glick made changes -
            Link This issue relates to JENKINS-54073 [ JENKINS-54073 ]
            jglick Jesse Glick made changes -
            Link This issue relates to JENKINS-54929 [ JENKINS-54929 ]

            People

              jglick Jesse Glick
              jglick Jesse Glick
              Votes:
              1 Vote for this issue
              Watchers:
              2 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved: