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

Plugin dynamic load doesn't init PeriodicWork and AperiodicWork extensions

    Details

    • Type: Bug
    • Status: Resolved (View Workflow)
    • Priority: Minor
    • Resolution: Duplicate
    • Component/s: core
    • Labels:
      None
    • Similar Issues:

      Description

      PluginManager.dynamicLoad(..) doesn't properly load/init extensions of PeriodWork and AperiodicWork. Still trying to figure out why. =)

        Attachments

          Issue Links

            Activity

            Hide
            abayer Andrew Bayer added a comment -

            Ok, it looks like dynamicLoad(..) never ends up calling PeriodicWork.run() - I'm not sure where that's supposed to be called normally.

            Show
            abayer Andrew Bayer added a comment - Ok, it looks like dynamicLoad(..) never ends up calling PeriodicWork.run() - I'm not sure where that's supposed to be called normally.
            Hide
            danielbeck Daniel Beck added a comment -

            Regularly started during Jenkins startup by AperiodicWork.init() (etc.) because it's annotated @Initializer(after=JOB_LOADED)..

            The perfect place to do this would probably be Jenkins.refreshExtensions. In fact, it says:

            // TODO: we need some generalization here so that extension points can be notified when a refresh happens?

            Show
            danielbeck Daniel Beck added a comment - Regularly started during Jenkins startup by AperiodicWork.init() (etc.) because it's annotated @Initializer(after=JOB_LOADED) .. The perfect place to do this would probably be Jenkins.refreshExtensions . In fact, it says: // TODO: we need some generalization here so that extension points can be notified when a refresh happens?

              People

              • Assignee:
                Unassigned
                Reporter:
                abayer Andrew Bayer
              • Votes:
                2 Vote for this issue
                Watchers:
                3 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved: