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

p4-plugin does not provide accurate polling results in workflow job

    XMLWordPrintable

Details

    Description

      WorkflowJob does not seem to poll correctly for the p4-plugin SCMs in jobs. This might be a p4-plugin issue. The SCMCheckout for a PerforceScm never has a pollingBaseline set.

      From what I can tell, it looks like PerforceScm needs a better definition for "calcRevisionsFromBuild" to correctly provide polling results.

      compareRemoteRevisionWith should also use the baseline to determine the PollingResult state

      Attachments

        Issue Links

          is duplicated by

          Bug - A problem which impairs or prevents the functions of the product. JENKINS-29598 p4 polling doesn't trigger workflow

          • Minor - Minor loss of function, or other problem where easy workaround is present.
          • Closed

          Activity

            Ascending order - Click to sort in descending order
            rpk2msu Rick King added a comment -

            Have added and tested out a fix.
            It is included on this pull request.
            https://github.com/jenkinsci/p4-plugin/pull/22

            rpk2msu Rick King added a comment - Have added and tested out a fix. It is included on this pull request. https://github.com/jenkinsci/p4-plugin/pull/22
            rpk2msu Rick King added a comment -

            This issue is now fixed and merged into master.

            rpk2msu Rick King added a comment - This issue is now fixed and merged into master.
            p4paul Paul Allen added a comment -

            Resolved to Closed

            p4paul Paul Allen added a comment - Resolved to Closed

            People

              rpocase Robby Pocase
              rpocase Robby Pocase
              Votes:
              1 Vote for this issue
              Watchers:
              4 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved: