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

Accurev changelog displays "Action=Dispatch" transactions

    • Icon: Bug Bug
    • Resolution: Fixed
    • Icon: Major Major
    • accurev-plugin
    • None

      AccuRev synchronization process which runs several times per hour (as the user "accusync").

      While I'm not intimately familiar with our synch process, the AccuRev transaction log indicates every "sync" has 'action=dispatch' and no file modifications.

      Previous versions (v0.6.27) of the AccuRev Jenkins plugin did not display these dispatch transactions in the changelog, but after upgrading to v0.6.30 our change log is flooded with them.

      Documentation available does not indicate that there is a way to manually disable/ignore these changelog entries... so I will assume that in previous versions the plugin was ignoring them and that functionality has been changed.

      •	[John Doe] updated file to include help file release 1.0.10
      •	[Jane Doe] Fixed sorted result set. 
      •	[accusync] 
      •	[accusync] 
      •	[accusync] 
      •	[accusync] 
      •	[accusync] 
      •	[accusync] 
      •	[accusync] 
      •	[accusync] 
      •	[accusync] 
      •	[accusync] 
      •	[accusync] 
      •	[accusync] 
      •	[accusync] 
      •	[accusync] 
      •	[accusync] 
      •	[accusync] 
      •	[accusync] 
      •	[accusync] 
      •	[accusync] 
      •	[accusync] 
      •	[accusync] 
      •	[accusync] 
      •	[accusync] 
      •	[accusync] 
      •	[accusync] 
      •	... about 100 more ...
      •	[accusync] 
      
      

          [JENKINS-24710] Accurev changelog displays "Action=Dispatch" transactions

          So issue is a lot simpler it seems.

          you can do accurev hist -k promote,purge.... and just leave out dispatch filter the output

          Joseph Petersen (old) added a comment - So issue is a lot simpler it seems. you can do accurev hist -k promote,purge.... and just leave out dispatch filter the output

          Joseph Petersen (old) added a comment - - edited

          sadly accurev hist -k only accepts a single parameter and does not accept multiple -k flags...

          But I noticed there was already a method called filterTransactions which is always called after parsing the Transactions list.

          So simple added:

          filteredTransactions.removeIf(t -> t.getAction().equalsIgnoreCase("dispatch"));
          

          Joseph Petersen (old) added a comment - - edited sadly accurev hist -k only accepts a single parameter and does not accept multiple -k flags... But I noticed there was already a method called filterTransactions which is always called after parsing the Transactions list. So simple added: filteredTransactions.removeIf(t -> t.getAction().equalsIgnoreCase( "dispatch" ));

          Code changed in jenkins
          User: Joseph
          Path:
          src/main/java/hudson/plugins/accurev/AccurevLauncher.java
          src/main/java/hudson/plugins/accurev/AccurevPlugin.java
          src/main/java/hudson/plugins/accurev/AccurevSCM.java
          src/main/java/hudson/plugins/accurev/AccurevStream.java
          src/main/java/hudson/plugins/accurev/AccurevTransaction.java
          src/main/java/hudson/plugins/accurev/ParseChangeLog.java
          src/main/java/hudson/plugins/accurev/UUIDUtils.java
          src/main/java/hudson/plugins/accurev/cmd/Login.java
          src/main/java/hudson/plugins/accurev/cmd/PopulateCmd.java
          src/main/java/hudson/plugins/accurev/cmd/ShowStreams.java
          src/main/java/hudson/plugins/accurev/delegates/AbstractModeDelegate.java
          src/main/java/hudson/plugins/accurev/delegates/ReftreeDelegate.java
          src/main/java/hudson/plugins/accurev/delegates/SnapshotDelegate.java
          src/main/java/hudson/plugins/accurev/delegates/WorkspaceDelegate.java
          src/main/java/hudson/plugins/accurev/parsers/xml/ParseShowStreams.java
          src/main/java/hudson/plugins/jetty/security/Password.java
          src/main/resources/hudson/plugins/accurev/AccurevSCM/config.jelly
          src/main/resources/hudson/plugins/accurev/AccurevSCM/global.jelly
          http://jenkins-ci.org/commit/accurev-plugin/b8296bb9c55c924f48c91356fe9ba7d9e35d4298
          Log:
          Merged PR #28 from casz/master

          Poll failed searching in parents when “Show one stream at a time” was
          checked.
          Fixed regression in PR #25

          • Improved logic for Workspace

          Was not working with “One stream at a time” settings.

          • Fixed Change log on Ignore Parent

          getChangesFromStreams did not respect Ignore Stream Parent.

          • Fixed getStreamRules.

          IgnoreStreamParent should be the first to be considered, then One
          Stream at a time on server, then find all streams.

          Introduced unique identifiers, also added a migrator to ensure best
          compatibility moving forward.

          • Fixed backwards compatibility

          Made the Server UUID change fully backwards compatibility :tada:

          I cannot believe this has been open since 2012...

          • StringUtils used
          • Allow empty password even with OBF
          • Javadoc and removed unused migrate field

          These usually originate from accurev sync.

          • PR #24 fix PollOnMaster enabled when build using workspace

          builds that use workspace or reftree definitely requires workspace even
          if PollOnMaster enabled.

          SCM/JIRA link daemon added a comment - Code changed in jenkins User: Joseph Path: src/main/java/hudson/plugins/accurev/AccurevLauncher.java src/main/java/hudson/plugins/accurev/AccurevPlugin.java src/main/java/hudson/plugins/accurev/AccurevSCM.java src/main/java/hudson/plugins/accurev/AccurevStream.java src/main/java/hudson/plugins/accurev/AccurevTransaction.java src/main/java/hudson/plugins/accurev/ParseChangeLog.java src/main/java/hudson/plugins/accurev/UUIDUtils.java src/main/java/hudson/plugins/accurev/cmd/Login.java src/main/java/hudson/plugins/accurev/cmd/PopulateCmd.java src/main/java/hudson/plugins/accurev/cmd/ShowStreams.java src/main/java/hudson/plugins/accurev/delegates/AbstractModeDelegate.java src/main/java/hudson/plugins/accurev/delegates/ReftreeDelegate.java src/main/java/hudson/plugins/accurev/delegates/SnapshotDelegate.java src/main/java/hudson/plugins/accurev/delegates/WorkspaceDelegate.java src/main/java/hudson/plugins/accurev/parsers/xml/ParseShowStreams.java src/main/java/hudson/plugins/jetty/security/Password.java src/main/resources/hudson/plugins/accurev/AccurevSCM/config.jelly src/main/resources/hudson/plugins/accurev/AccurevSCM/global.jelly http://jenkins-ci.org/commit/accurev-plugin/b8296bb9c55c924f48c91356fe9ba7d9e35d4298 Log: Merged PR #28 from casz/master JENKINS-28118 Fixed poll on ancestor Poll failed searching in parents when “Show one stream at a time” was checked. Fixed regression in PR #25 Improved logic for Workspace Was not working with “One stream at a time” settings. Fixed Change log on Ignore Parent getChangesFromStreams did not respect Ignore Stream Parent. Fixed getStreamRules. IgnoreStreamParent should be the first to be considered, then One Stream at a time on server, then find all streams. JENKINS-31316 Fixed server rename Introduced unique identifiers, also added a migrator to ensure best compatibility moving forward. Fixed backwards compatibility Made the Server UUID change fully backwards compatibility :tada: JENKINS-13817 Mask the damn password already I cannot believe this has been open since 2012... StringUtils used Allow empty password even with OBF Javadoc and removed unused migrate field JENKINS-24710 filter out dispatch transactions These usually originate from accurev sync. PR #24 fix PollOnMaster enabled when build using workspace builds that use workspace or reftree definitely requires workspace even if PollOnMaster enabled.

          Skylar can you confirm that they are filtered now?

          Joseph Petersen (old) added a comment - Skylar can you confirm that they are filtered now?

          Skylar Sutton added a comment -

          Thanks. Will try to test today or tomorrow... been bogged down with a few things.

          Skylar Sutton added a comment - Thanks. Will try to test today or tomorrow... been bogged down with a few things.

          Skylar Sutton added a comment -

          Joseph - Is there a snapshot available? I am no longer setup to build this project and the GIT tags are only showing release tags right now.

          Skylar Sutton added a comment - Joseph - Is there a snapshot available? I am no longer setup to build this project and the GIT tags are only showing release tags right now.

          The latest version includes this fix

          Joseph Petersen (old) added a comment - The latest version includes this fix

          If you want any snapshots, they can be found on each build through jenkins ci
          https://jenkins.ci.cloudbees.com/job/plugins/job/accurev-plugin/org.jenkins-ci.plugins$accurev/lastSuccessfulBuild/artifact/
          Then you can just click previous build till you find the one you want.

          Joseph Petersen (old) added a comment - If you want any snapshots, they can be found on each build through jenkins ci https://jenkins.ci.cloudbees.com/job/plugins/job/accurev-plugin/org.jenkins-ci.plugins$accurev/lastSuccessfulBuild/artifact/ Then you can just click previous build till you find the one you want.

          Skylar Sutton added a comment -

          Got it, apologies... did not see it listed in the 0.7.2 release notes.

          I upgraded and will report back after our nightly build.

          Skylar Sutton added a comment - Got it, apologies... did not see it listed in the 0.7.2 release notes . I upgraded and will report back after our nightly build.

          Skylar Sutton added a comment -

          Marking resolved - changelog no longer includes the [accusync] entries. Thanks!

          Skylar Sutton added a comment - Marking resolved - changelog no longer includes the [accusync] entries. Thanks!

            jetersen Joseph Petersen
            skylarsutton Skylar Sutton
            Votes:
            0 Vote for this issue
            Watchers:
            3 Start watching this issue

              Created:
              Updated:
              Resolved: