-
Improvement
-
Resolution: Fixed
-
Minor
-
None
-
support-core:2.2
-
-
support-core-2.74
The AdministrativeMonitors component checks if a monitor is activated before checking if it is enabled: https://github.com/jenkinsci/support-core-plugin/blob/support-core-2.72/src/main/java/com/cloudbees/jenkins/support/impl/AdministrativeMonitors.java#L67
This causes unnecessary load. Some monitor implementation of isActivated contains loops and certain complexity.
The component should first check if the monitor is enabled and if it is, then only it should check if it is activated.