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

Job Depot selection always uses default Accurev installation

    XMLWordPrintable

Details

    • Bug
    • Status: Resolved (View Workflow)
    • Minor
    • Resolution: Fixed
    • accurev-plugin
    • None
    • OS: Windows 2008 R2 SP1
      JRE: Oracle JRE 1.8.0_45
      Accurev 5.6 and 6.0.1
      Jenkins: 2.32.3
      Running directly from service setup by Windows installer
      Accurev Plugins: 0.7.14

    Description

      For various reasons, I have to support and run builds from multiple versions of Accurev, specifically 5.6 and 6.0.1. I have both versions of the Accurev client installed to different non-standard locations. We have been getting by with an older version of the Accurev plugin that doesn't use a full path for the executable and adding the location to the PATH environment variable in the jenkins.xml file used by the Jenkins Windows service. To support multiple versions of Accurev, we thus have multiple instances of Jenkins, each with a different Accurev installation added to the PATH.

      In our test environment, we have updated to Jenkins 2.32.3 and Accurev plugin 0.7.14. The refactoring to make Accurev installations a Tool object gave me hope that it would be able to support multiple installations, not just a customized default. Unfortunately, it appears that when configuring Accurev for a Job, the Depot field is no longer a plain textbox, but is now a dropdown dynamically generated from the server. We have 6.0.1 configured as the default installation, and it appears that it is always using the default to populate the Depot dropdown. When configured against a 5.6 Server, the Depot dropdown is blank and I can see a error in the Jenkins log "The AccuRev server you are using is too old."

      Desired behavior to me would be to use the Accurev executable as set two fields down in the configuration when populating the Depot dropdown, or otherwise returning to the behavior where that field is populated manually. Possibly, the Accurev executable field should be moved above the Server field, as the Accurev executable version affects which server you can connect to.

      This may not be the only place where the default installation is chosen instead of using the executable configured, but without being able to set the Depot I cannot move forward in my testing.

      Attachments

        1. Accurev_Job.png
          Accurev_Job.png
          8 kB
        2. Accurev_Tools.png
          Accurev_Tools.png
          15 kB
        3. image-2017-10-10-19-32-43-099.png
          image-2017-10-10-19-32-43-099.png
          23 kB
        4. jenkins.err.log
          17 kB
        5. jenkins.err.log.commentary
          0.4 kB
        6. screenshot-1.png
          screenshot-1.png
          12 kB

        Issue Links

          Activity

            casz Joseph Petersen (old) added a comment - - edited

            Sounds like you picked up the version 1.0 artifact  I been working on.
            My bad the link I gave you was to the wrong branch... For now on full links!

            https://ci.jenkins.io/job/Plugins/job/accurev-plugin/job/PR-57/6/artifact/target/accurev.hpi

            casz Joseph Petersen (old) added a comment - - edited Sounds like you picked up the version 1.0 artifact   I been working on. My bad the link I gave you was to the wrong branch... For now on full links! https://ci.jenkins.io/job/Plugins/job/accurev-plugin/job/PR-57/6/artifact/target/accurev.hpi
            rabdsquirlz James Welch added a comment -

            That works better, created https://issues.jenkins-ci.org/browse/JENKINS-47538 to continue

            rabdsquirlz James Welch added a comment - That works better, created https://issues.jenkins-ci.org/browse/JENKINS-47538 to continue

            Code changed in jenkins
            User: Joseph Petersen
            Path:
            src/main/java/hudson/plugins/accurev/AccurevSCM.java
            src/main/java/hudson/plugins/accurev/cmd/ShowStreams.java
            src/main/resources/hudson/plugins/accurev/AccurevSCM/config.jelly
            http://jenkins-ci.org/commit/accurev-plugin/370cc747f6092757491b04372eaac9f9f1f69ab3
            Log:
            JENKINS-47374 revert fancy combobox

            scm_issue_link SCM/JIRA link daemon added a comment - Code changed in jenkins User: Joseph Petersen Path: src/main/java/hudson/plugins/accurev/AccurevSCM.java src/main/java/hudson/plugins/accurev/cmd/ShowStreams.java src/main/resources/hudson/plugins/accurev/AccurevSCM/config.jelly http://jenkins-ci.org/commit/accurev-plugin/370cc747f6092757491b04372eaac9f9f1f69ab3 Log: JENKINS-47374 revert fancy combobox

            Code changed in jenkins
            User: Joseph Petersen
            Path:
            src/main/java/hudson/plugins/accurev/AccurevSCM.java
            src/main/java/hudson/plugins/accurev/cmd/ShowStreams.java
            src/main/resources/hudson/plugins/accurev/AccurevSCM/config.jelly
            http://jenkins-ci.org/commit/accurev-plugin/5d39b5e075ba661b105fe0629a02f81f0b77434b
            Log:
            JENKINS-47374 revert fancy combobox

            scm_issue_link SCM/JIRA link daemon added a comment - Code changed in jenkins User: Joseph Petersen Path: src/main/java/hudson/plugins/accurev/AccurevSCM.java src/main/java/hudson/plugins/accurev/cmd/ShowStreams.java src/main/resources/hudson/plugins/accurev/AccurevSCM/config.jelly http://jenkins-ci.org/commit/accurev-plugin/5d39b5e075ba661b105fe0629a02f81f0b77434b Log: JENKINS-47374 revert fancy combobox

            Released as 0.7.15

            casz Joseph Petersen (old) added a comment - Released as 0.7.15

            People

              jetersen Joseph Petersen
              rabdsquirlz James Welch
              Votes:
              0 Vote for this issue
              Watchers:
              3 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved: