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

AccuRev plugin does not start job on detected changes

    • Icon: Bug Bug
    • Resolution: Fixed
    • Icon: Major Major
    • accurev-plugin
    • Jenkins 2.32.3
      AccuRev plugin 0.7.11

      Windows Server 2012 R2 running Jenkins and AccuRev client
      AccuRev client 6.2.3d

      Linux server running AccuRev server (same version as client)

      AccuRev plugin 0.7.11 detects changes on stream sub-folder polling but states there are "no changes" and does not start Jenkins job.

      Stream layout:

      • BI-HI-UI-SCS-DEV
        • hi-analytics-library
        • hi-analytics-library-web
        • ...

      Job polling found changes and stated there were "no changes":

      AccuRev Polling Log
      
      Started on Mar 20, 2017 9:34:04 AM
      No workspace required.
      Running commands from folder "C:\Program Files (x86)\Jenkins\jobs\hi-analytics-library"
      [hi-analytics-library] $ accurev info -H server.domain.com:5050
      Currently authenticated with Accurev server as 'cmlogin', not logging in again.
      [hi-analytics-library] $ accurev show -H server.domain.com:5050 -fx -p BI -s BI-HI-UI-SCS-DEV streams
      Checking transactions of type chstream, defcomp, mkstream, promote in stream [BI-HI-UI-SCS-DEV]
      [hi-analytics-library] $ accurev hist -H server.domain.com:5050 -fx -p BI -s BI-HI-UI-SCS-DEV -t now.1 -k chstream
      No transactions of type [chstream]
      [hi-analytics-library] $ accurev hist -H server.domain.com:5050 -fx -p BI -s BI-HI-UI-SCS-DEV -t now.1 -k defcomp
      No transactions of type [defcomp]
      [hi-analytics-library] $ accurev hist -H server.domain.com:5050 -fx -p BI -s BI-HI-UI-SCS-DEV -t now.1 -k mkstream
      Last transaction of type [mkstream] is [id=161128, date=Mon Mar 28 16:44:49 EDT 2016, author=SOMEONE, action=mkstream, msg=]
      [hi-analytics-library] $ accurev hist -H server.domain.com:5050 -fx -p BI -s BI-HI-UI-SCS-DEV -t now.1 -k promote
      Last transaction of type [promote] is [id=232299, date=Mon Mar 20 09:18:44 EDT 2017, author=PERSON, action=promote, msg=HCINT-4297
      
      Changes in hi-analytics-library@0.16.8
      - Fix DocViewer bug where documents fail to load behind loadbalancer.
        - Set Response.BufferOutput to false to stream data.
        - Don't render error response from web request.
        - Remove duplicate ASP.Net headers.
      - Fix Logging missing entries and duplicate entries.
        - Moved log request handler to PreSendRequestHeaders event.]
      Done. Took 4.4 sec
      No changes
      

      "Build Now" found same changes polling did:

      Changes
      Summary
      HCINT-4297 Changes in hi-analytics-library@0.16.8 - Fix DocViewer bug where documents fail to load behind loadbalancer. - Set Response.BufferOutput to false to stream data. - Don't render error response from web request. - Remove duplicate ASP.Net headers. - Fix Logging missing entries and duplicate entries. - Moved log request handler to PreSendRequestHeaders event.
      
      AccuWork Issue Number - 60254
      Transaction 232299 by PERSON :
      HCINT-4297<br><br>Changes in hi-analytics-library@0.16.8<br>- Fix DocViewer bug where documents fail to load behind loadbalancer.<br>&nbsp; - Set Response.BufferOutput to false to stream data.<br>&nbsp; - Don&#039;t render error response from web request.<br>&nbsp; - Remove duplicate ASP.Net headers.<br>- Fix Logging missing entries and duplicate entries.<br>&nbsp; - Moved log request handler to PreSendRequestHeaders event.
      The file was modified	hi-analytics-library/AL.Reports/Properties/AssemblyInfo.cs --- Version - 22299/17
      The file was modified	hi-analytics-library/AL.Reports/Texts.Designer.cs --- Version - 22299/12
      The file was modified	hi-analytics-library/AL.Reports/Texts.resx --- Version - 22299/12
      The file was modified	hi-analytics-library/gradle.properties --- Version - 22299/12
      The file was modified	hi-analytics-library/Documentation/Content/VersionHistory/Pre-release.aml --- Version - 22299/10
      The file was modified	hi-analytics-library/AL.Reports/HttpModule/ReportsHttpModule.cs --- Version - 22299/6
      The file was modified	hi-analytics-library/AL.Reports/HttpModule/RequestTrackerProvider.cs --- Version - 22299/3
      The file was modified	hi-analytics-library/AL.Reports.Test/HttpModule/RequestTrackerProviderTest.cs --- Version - 22299/3
      The file was modified	hi-analytics-library/AL.Reports/DocViewer.aspx --- Version - 22299/2
      The file was modified	hi-analytics-library/AL.Reports.Test/DocViewerTest.cs --- Version - 22299/2
      The file was modified	hi-analytics-library/AL.Reports/DocViewer.aspx.cs --- Version - 22299/2
      

          [JENKINS-42939] AccuRev plugin does not start job on detected changes

          Whats your Filter for Poll SCM set to?
          Or what is your subpath set to?

          Joseph Petersen (old) added a comment - Whats your Filter for Poll SCM set to? Or what is your subpath set to?

          Sub-path = gradle.cmd,gradlew,gradlew.bat,gradle,build-config,hi-analytics-library
          Filter for Poll SCM = hi-analytics-library/

          Timothy Williams added a comment - Sub-path = gradle.cmd,gradlew,gradlew.bat,gradle,build-config,hi-analytics-library Filter for Poll SCM = hi-analytics-library/

          Joseph Petersen (old) added a comment - - edited

          mitorez
          If you refer back to JENKINS-41526 I suggested a change.

          You never replied so I went ahead, your new filter should be: hi-analytics-library/*
          sub-path should stay the same

          Joseph Petersen (old) added a comment - - edited mitorez If you refer back to JENKINS-41526  I suggested a change. You never replied so I went ahead, your new filter should be: hi-analytics-library/* sub-path should stay the same

          Oh, is that what this comment meant?

          So all I could do was simple wildcard without implementing too much.

          I thought you were having a conversation with Arno, I didn't realize you were telling me what to change with the next release (0.7.10). I did see a reference to java/* in the 0.7.10 release notes with a reference to JENKINS-41526, but didn't realize what it meant. My bad.

          So I don't need to update the Sub-path, I just need to add {} to my current *Filter for Poll SCM value to make it hi-analytics-library/*?

          Timothy Williams added a comment - Oh, is that what this comment meant? So all I could do was simple wildcard without implementing too much. I thought you were having a conversation with Arno, I didn't realize you were telling me what to change with the next release (0.7.10). I did see a reference to java/* in the 0.7.10 release notes with a reference to JENKINS-41526 , but didn't realize what it meant. My bad. So I don't need to update the Sub-path , I just need to add { } to my current *Filter for Poll SCM value to make it hi-analytics-library/* ?

          Yes just the filter for poll

          Joseph Petersen (old) added a comment - Yes just the filter for poll

          Timothy Williams added a comment - - edited

          I updated my jobs and I'll test with the next promotes.

          Question: With the wildcard changes you made, do you think I''d be able to just remove /* and specify the folder name now, or do you think I'd still hit the same issue with folder names that begin with the same string (e.g. hi-analytics-library and hi-analytics-library-web) kicking off multiple jobs?

          Thanks!

          Timothy Williams added a comment - - edited I updated my jobs and I'll test with the next promotes. Question: With the wildcard changes you made, do you think I''d be able to just remove /* and specify the folder name now, or do you think I'd still hit the same issue with folder names that begin with the same string (e.g. hi-analytics-library and hi-analytics-library-web ) kicking off multiple jobs? Thanks!

          If you want to avoid hi-analytics-library also starting hi-analytics-library-web then you need the /*

          Joseph Petersen (old) added a comment - If you want to avoid hi-analytics-library also starting hi-analytics-library-web then you need the /*

          Thanks Joseph, that's what I thought - just wanted to be sure

          Timothy Williams added a comment - Thanks Joseph, that's what I thought - just wanted to be sure

          Verified appending * to polling folder path resolved this issue.

          Timothy Williams added a comment - Verified appending * to polling folder path resolved this issue.

            jetersen Joseph Petersen
            mitorez Timothy Williams
            Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

              Created:
              Updated:
              Resolved: