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

P4 Plugin: Downstream job not using correct changelist

    XMLWordPrintable

Details

    • Bug
    • Status: Closed (View Workflow)
    • Major
    • Resolution: Fixed
    • p4-plugin
    • Jenkins ver. 1.613 | P4 Plugin 1.2.4 | Build Flow 0.17 | Java 1.7.0_25 (32-bit) | Win8.1 (64-bit)

    Description

      I have a build flow that gets triggered by an SCMTRIGGER (ie "Polling per change"). It creates a downstream build job where the expectation is that it would use the SCMTRIGGER changelist to sync on. This is not the case, however, if a checkin has occured after the initial job was triggered. It will pick up all changes and sync to the latest for the downstream build.

      When I look at the env vars for the downstream job, I see both a CHANGELIST and a P4_CHANGELIST. The first one is the correct CL, the other is the wrong CL for this job. An easy fix would be to use the CHANGELIST instead of the P4_CHANGELIST to sync on. (Can I override the P4_CHANGELIST as a build parameter being passed in??)

      Here is a breakdown of the initial SCMTRIGGER postcheckin job and the subsequent DOWNSTREAM build job it spawned (Perforce changes followed by pertinent env vars for each):

      ////////////////////////////////////////////////////////////////////////////////////////////
      // Build #809 (SCMTRIGGER)
      ////////////////////////////////////////////////////////////////////////////////////////////

      POLLING LOG:
      ------------

      View as plain text
      This page captures the polling log that triggered this build.

      Started on Jul 7, 2015 6:09:47 PM
      Polling SCM changes on master
      P4: Polling on: master with:jenkins_XXX_postcheckin
      ... p4 client -o jenkins_XXX_postcheckin
      +

      P4 Task: establishing connection.
      ... server: perforce.XXX.com:3666
      ... node: XXX001
      ... p4 cstat //jenkins_XXX_postcheckin/...
      +
      P4: Polling with label/change: 441291,now
      ... p4 changes -m100 //jenkins_XXX_postcheckin/...@441291,now
      +
      ... p4 change -o 441300
      +
      ... p4 describe -s 441300
      +
      ... found change: 441300
      next change: 441300
      Done. Took 1.6 sec
      Changes found

      ENVIRONMENT VARIABLES:
      ----------------------

      ...
      BUILD_CAUSE SCMTRIGGER,SCMTRIGGER
      BUILD_CAUSE_SCMTRIGGER true
      BUILD_DISPLAY_NAME #809
      BUILD_ID 809
      BUILD_NUMBER 809
      ...
      EXECUTOR_NUMBER -1
      ...
      JOB_NAME postcheckin_XXX
      ...
      NODE_LABELS master
      ...
      ROOT_BUILD_CAUSE SCMTRIGGER
      ROOT_BUILD_CAUSE_SCMTRIGGER true
      ...

      ////////////////////////////////////////////////////////////////////////////////////////////
      // Build #5649 (UPSTREAMTRIGGER)
      ////////////////////////////////////////////////////////////////////////////////////////////

      CHANGES:
      --------

      441304 by XXX on Jul 7, 2015 6:23:17 AM
      ...
      441300 by XXX on Jul 7, 2015 6:08:44 AM

      ENVIRONMENT VARIABLES:
      ----------------------
      ...
      BUILD_CAUSE UPSTREAMTRIGGER
      BUILD_CAUSE_UPSTREAMTRIGGER true
      BUILD_DISPLAY_NAME #5649
      BUILD_ID 5649
      BUILD_NUMBER 5649
      ...
      CHANGELIST 441300 <=============== This is the CORRECT upstream CL
      ...
      EXECUTOR_NUMBER 0
      ...
      JOB_NAME build_XXX
      ...
      NODE_LABELS build_pool
      ...
      P4_CHANGELIST 441304 <=============== This is the INCORRECT latest CL
      P4_CLIENT jenkins_XXX_build
      ...
      ROOT_BUILD_CAUSE SCMTRIGGER
      ROOT_BUILD_CAUSE_SCMTRIGGER true
      ...

      Attachments

        Activity

          mhulth Martin Hulth created issue -
          p4paul Paul Allen made changes -
          Field Original Value New Value
          Resolution Fixed [ 1 ]
          Status Open [ 1 ] Resolved [ 5 ]
          p4paul Paul Allen made changes -
          Status Resolved [ 5 ] Closed [ 6 ]
          rtyler R. Tyler Croy made changes -
          Workflow JNJira [ 164169 ] JNJira + In-Review [ 208967 ]

          People

            p4paul Paul Allen
            mhulth Martin Hulth
            Votes:
            0 Vote for this issue
            Watchers:
            3 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved: