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

Make ExecutorListener an extension point

    XMLWordPrintable

    Details

    • Type: Improvement
    • Status: Closed (View Workflow)
    • Priority: Minor
    • Resolution: Fixed
    • Component/s: core
    • Labels:
    • Similar Issues:
    • Released As:
      2.318

      Description

      Currently https://javadoc.jenkins.io/hudson/model/ExecutorListener.html can is implemented by Computer and may also be implemented by an associated RetentionStrategy or ComputerLauncher; but there is no way to receive these events from outside of these objects (just some interested plugin).

        Attachments

          Issue Links

            Activity

            Hide
            teilo James Nord added a comment -

            marking as lts-candidate.  it does not fit in the definition - but is needed to fix a thread issue in metrics plugin.

            Show
            teilo James Nord added a comment - marking as lts-candidate.  it does not fit in the definition - but is needed to fix a thread issue in metrics plugin.
            Hide
            jglick Jesse Glick added a comment -

            True, but the fix is not at all simple and deserves “soaking”. I would not be inclined to advertise this for LTS backport.

            Show
            jglick Jesse Glick added a comment - True, but the fix is not at all simple and deserves “soaking”. I would not be inclined to advertise this for LTS backport.
            Hide
            danielbeck Daniel Beck added a comment -

            is needed to fix a thread issue in metrics plugin

            The plugin can go with a weekly core dependency until LTS has picked it up.

            Show
            danielbeck Daniel Beck added a comment - is needed to fix a thread issue in metrics plugin The plugin can go with a weekly core dependency until LTS has picked it up.
            Hide
            danielbeck Daniel Beck added a comment -

            Already in 2.318, so with 2.319 being the baseline, no need to backport.

            Show
            danielbeck Daniel Beck added a comment - Already in 2.318, so with 2.319 being the baseline, no need to backport.

              People

              Assignee:
              jglick Jesse Glick
              Reporter:
              jglick Jesse Glick
              Votes:
              0 Vote for this issue
              Watchers:
              3 Start watching this issue

                Dates

                Created:
                Updated:
                Resolved: