• Icon: Improvement Improvement
    • Resolution: Fixed
    • Icon: Minor Minor
    • ssh-steps-plugin
    • None

      Calling sshCommand forces the allocation of PTY, however running commands on servers where there is no proper PTY support (win32-openssh) can generate failures such as garbage output, ...

      Being able to disable/enable PTY allocation would help for these environments

      EDIT: 

      There is PTY setting at the remote level, but actually not sure why I had to make it true at the command level, I think it needs to be defaulted false as there is no impact for LINUX boxes either way, I have tested it. Made a PR.

      And this PTY is also applicable for other steps, so instead it is good to have it at the remote level.
       

          [JENKINS-56989] Default pty to false for sshCommand

          Jesús Malo Poyatos created issue -
          Jesús Malo Poyatos made changes -
          Remote Link New: This issue links to "Pull request (Web Link)" [ 22590 ]
          Naresh Rayapati made changes -
          Status Original: Open [ 1 ] New: In Progress [ 3 ]
          Naresh Rayapati made changes -
          Status Original: In Progress [ 3 ] New: In Review [ 10005 ]
          Naresh Rayapati made changes -
          Summary Original: Allow explicit enabling/disabling of PTY with sshCommand New: Default pty to false for sshCommand
          Naresh Rayapati made changes -
          Description Original: Calling sshCommand forces the allocation of PTY, however running commands on servers where there is no proper PTY support (win32-openssh) can generate failures such as garbage output, ...

          Being able to disable/enable PTY allocation would help for these environments

           
          New: Calling sshCommand forces the allocation of PTY, however running commands on servers where there is no proper PTY support (win32-openssh) can generate failures such as garbage output, ...

          Being able to disable/enable PTY allocation would help for these environments

          EDIT: 

          There is PTY setting at the remote level, but actually not sure why I had to make it true at the command level, so feel like it needs to be defaulted to `false`. Made a PR: 

           
          Naresh Rayapati made changes -
          Description Original: Calling sshCommand forces the allocation of PTY, however running commands on servers where there is no proper PTY support (win32-openssh) can generate failures such as garbage output, ...

          Being able to disable/enable PTY allocation would help for these environments

          EDIT: 

          There is PTY setting at the remote level, but actually not sure why I had to make it true at the command level, so feel like it needs to be defaulted to `false`. Made a PR: 

           
          New: Calling sshCommand forces the allocation of PTY, however running commands on servers where there is no proper PTY support (win32-openssh) can generate failures such as garbage output, ...

          Being able to disable/enable PTY allocation would help for these environments

          EDIT: 

          There is PTY setting at the remote level, but actually not sure why I had to make it true at the command level, so it needs to be defaulted to `false`. Made a PR: 

           
          Naresh Rayapati made changes -
          Description Original: Calling sshCommand forces the allocation of PTY, however running commands on servers where there is no proper PTY support (win32-openssh) can generate failures such as garbage output, ...

          Being able to disable/enable PTY allocation would help for these environments

          EDIT: 

          There is PTY setting at the remote level, but actually not sure why I had to make it true at the command level, so it needs to be defaulted to `false`. Made a PR: 

           
          New: Calling sshCommand forces the allocation of PTY, however running commands on servers where there is no proper PTY support (win32-openssh) can generate failures such as garbage output, ...

          Being able to disable/enable PTY allocation would help for these environments

          EDIT: 

          There is PTY setting at the remote level, but actually not sure why I had to make it true at the command level, I think it needs to be defaulted false as there is no impact for LINUX boxes either way, I have tested it. Made a PR: 

           
          Naresh Rayapati made changes -
          Description Original: Calling sshCommand forces the allocation of PTY, however running commands on servers where there is no proper PTY support (win32-openssh) can generate failures such as garbage output, ...

          Being able to disable/enable PTY allocation would help for these environments

          EDIT: 

          There is PTY setting at the remote level, but actually not sure why I had to make it true at the command level, I think it needs to be defaulted false as there is no impact for LINUX boxes either way, I have tested it. Made a PR: 

           
          New: Calling sshCommand forces the allocation of PTY, however running commands on servers where there is no proper PTY support (win32-openssh) can generate failures such as garbage output, ...

          Being able to disable/enable PTY allocation would help for these environments

          EDIT: 

          There is PTY setting at the remote level, but actually not sure why I had to make it true at the command level, I think it needs to be defaulted false as there is no impact for LINUX boxes either way, I have tested it. Made a PR.

          And this PTY is also applicable for other steps, so instead it is good to have it at the remote level.
           
          Naresh Rayapati made changes -
          Resolution New: Fixed [ 1 ]
          Status Original: In Review [ 10005 ] New: Resolved [ 5 ]
          Naresh Rayapati made changes -
          Status Original: Resolved [ 5 ] New: Fixed but Unreleased [ 10203 ]

            nrayapati Naresh Rayapati
            jmalopoy Jesús Malo Poyatos
            Votes:
            0 Vote for this issue
            Watchers:
            3 Start watching this issue

              Created:
              Updated:
              Resolved: