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

Vetoable version of GraphListener

    XMLWordPrintable

Details

    Description

      GraphListener gives you access to a FlowNode but not a StepContext which sometimes has richer information. It also provides no straightforward way to block the execution of a step before it begins (though you can abort the build soon afterward). For some purposes it would be nice to have something like

      public interface StepListener extends ExtensionPoint {
        void newStep(Step step, StepContext context) throws Exception;
      }
      

      The natural place to call this would be right before Step.start, at which point all the abovementioned information is available, and any Exception can just be handled in onFailure just like a synchronous error from start would.

      Attachments

        Activity

          abayer Andrew Bayer added a comment -

          Super-preliminary PRs up for workflow-api and workflow-cps.

          abayer Andrew Bayer added a comment - Super-preliminary PRs up for workflow-api and workflow-cps.
          kshultz Karl Shultz added a comment -

          This has been done and merged, thanks Andrew!

          Merged PRs:

          kshultz Karl Shultz added a comment - This has been done and merged, thanks Andrew! Merged PRs: workflow-api 96 workflow-cps 299

          People

            abayer Andrew Bayer
            jglick Jesse Glick
            Votes:
            0 Vote for this issue
            Watchers:
            3 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved: