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

Jenkins doesn't detect CVS polling on Pipeline (Workflow) Job

    Details

    • Type: Bug
    • Status: Open (View Workflow)
    • Priority: Blocker
    • Resolution: Unresolved
    • Component/s: cvs-plugin
    • Labels:
    • Environment:
      Jenkins 1.642.1
      Pipeline (workflow-plugin) 1.13
    • Similar Issues:

      Description

      Polling error on a Pipeline Job:

      java.lang.AbstractMethodError: you must override the new overload of compareRemoteRevisionWith
      	at hudson.scm.SCM.compareRemoteRevisionWith(SCM.java:375)
      	at org.jenkinsci.plugins.workflow.job.WorkflowJob.poll(WorkflowJob.java:507)
      	at hudson.triggers.SCMTrigger$Runner.runPolling(SCMTrigger.java:526)
      	at hudson.triggers.SCMTrigger$Runner.run(SCMTrigger.java:555)
      	at hudson.util.SequentialExecutionQueue$QueueEntry.run(SequentialExecutionQueue.java:119)
      	at java.util.concurrent.Executors$RunnableAdapter.call(Executors.java:511)
      	at java.util.concurrent.FutureTask.run(FutureTask.java:266)
      	at java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1142)
      	at java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:617)
      	at java.lang.Thread.run(Thread.java:745)
      

        Attachments

          Issue Links

            Activity

            Hide
            valones Flávio Augusto Valones added a comment -

            I submit a pull request (https://github.com/jenkinsci/cvs-plugin/pull/41) to fix this.

            Show
            valones Flávio Augusto Valones added a comment - I submit a pull request ( https://github.com/jenkinsci/cvs-plugin/pull/41 ) to fix this.
            Hide
            jglick Jesse Glick added a comment -

            Generally you should check for deprecation warnings in SCM subclasses.

            Show
            jglick Jesse Glick added a comment - Generally you should check for deprecation warnings in SCM subclasses.
            Hide
            mc1arke Michael Clarke added a comment -

            Sorry Jesse Glick, who is that comment aimed at? Is there an issue with the pull request, or are you implying that plugin maintainers should be checking the compatibility of their plugin with each Jenkins version?

            Show
            mc1arke Michael Clarke added a comment - Sorry Jesse Glick , who is that comment aimed at? Is there an issue with the pull request, or are you implying that plugin maintainers should be checking the compatibility of their plugin with each Jenkins version?
            Hide
            mkbucc Mark Bucciarelli added a comment -

            What needs to happen so I can use CVS polling from a pipeline?

            As a workaround, I can use cron and trigger a build via HTTP if changes found, but that is a hack.

            P.S. The link to the "compatibility guide" on https://github.com/jenkinsci/workflow-scm-step-plugin/blob/master/README.md returns a 404.

            Show
            mkbucc Mark Bucciarelli added a comment - What needs to happen so I can use CVS polling from a pipeline? As a workaround, I can use cron and trigger a build via HTTP if changes found, but that is a hack. P.S. The link to the "compatibility guide" on https://github.com/jenkinsci/workflow-scm-step-plugin/blob/master/README.md returns a 404.
            Hide
            valones Flávio Augusto Valones added a comment - - edited

            I'm using the plugin modified by the pull request above for five months now in production and the CVS polling works very well.

            Show
            valones Flávio Augusto Valones added a comment - - edited I'm using the plugin modified by the pull request above for five months now in production and the CVS polling works very well.
            Hide
            mkbucc Mark Bucciarelli added a comment -

            Flávio Augusto Valones It looks like the CVS plugin maintainer will not create a new major version with your patch. Would you be willing to publish your fork to Jenkins plugin repo under a new name?

            Show
            mkbucc Mark Bucciarelli added a comment - Flávio Augusto Valones It looks like the CVS plugin maintainer will not create a new major version with your patch. Would you be willing to publish your fork to Jenkins plugin repo under a new name?
            Hide
            mc1arke Michael Clarke added a comment -

            Mark Bucciarelli it's not that I'm unprepared to accept the pull request or perform a release; it's that I no longer use CVS so have no particular way of checking the changes or testing releases. If you'd like to have this change incorporated or released then feel free to take on ownership of the CVS plugin!

            Show
            mc1arke Michael Clarke added a comment - Mark Bucciarelli it's not that I'm unprepared to accept the pull request or perform a release; it's that I no longer use CVS so have no particular way of checking the changes or testing releases. If you'd like to have this change incorporated or released then feel free to take on ownership of the CVS plugin!
            Hide
            valones Flávio Augusto Valones added a comment -

            Sorry Mark Bucciarelli, I have no time now to maintain a plugin. My company is moving from CVS to Git and its usage is becoming ever smaller, probably I will not go make further changes in plugin.

            Show
            valones Flávio Augusto Valones added a comment - Sorry Mark Bucciarelli , I have no time now to maintain a plugin. My company is moving from CVS to Git and its usage is becoming ever smaller, probably I will not go make further changes in plugin.

              People

              • Assignee:
                Unassigned
                Reporter:
                valones Flávio Augusto Valones
              • Votes:
                1 Vote for this issue
                Watchers:
                4 Start watching this issue

                Dates

                • Created:
                  Updated: