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

CliGitAPIImpl is hard-coded to use LocalLauncher

      We have a custom Jenkins plugin that executes git commands using GitAPI, for example launchCommand("pull", myRemoteRepo, myBranch).
      When a build is launched on a slave the build fails because the git commands attempt to run on the Jenkins master instead of the slave.

      Our workaround was to replace the LocalLauncher created by CliGitAPIImpl with the Launcher provided by Jenkins during setup (using reflection), so that builds on slaves have a RemoteLauncher. It would be preferable to have a constructor in GitAPI that takes an instance of Launcher so that we do not have to code against CliGitAPIImpl directly.

          [JENKINS-21342] CliGitAPIImpl is hard-coded to use LocalLauncher

          Rusty Gerard created issue -
          Mark Waite made changes -
          Issue Type Original: Bug [ 1 ] New: New Feature [ 2 ]
          Nicolas De Loof made changes -
          Resolution New: Won't Fix [ 2 ]
          Status Original: Open [ 1 ] New: Resolved [ 5 ]
          Mark Waite made changes -
          Status Original: Resolved [ 5 ] New: Closed [ 6 ]
          Doug Jacobsen made changes -
          Resolution Original: Won't Fix [ 2 ]
          Status Original: Closed [ 6 ] New: Reopened [ 4 ]
          Mark Waite made changes -
          Component/s Original: git [ 15543 ]
          Rusty Gerard made changes -
          Assignee Original: Nicolas De Loof [ ndeloof ] New: Rusty Gerard [ rgerard ]
          Rusty Gerard made changes -
          Status Original: Reopened [ 4 ] New: Open [ 1 ]
          Rusty Gerard made changes -
          Assignee Original: Rusty Gerard [ rgerard ]
          Rusty Gerard made changes -
          Link New: This issue is duplicated by JENKINS-30600 [ JENKINS-30600 ]
          R. Tyler Croy made changes -
          Workflow Original: JNJira [ 153206 ] New: JNJira + In-Review [ 178448 ]

            Unassigned Unassigned
            rgerard Rusty Gerard
            Votes:
            2 Vote for this issue
            Watchers:
            6 Start watching this issue

              Created:
              Updated:
              Resolved: