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

Expose WebAppMain#DEFAULT_RING_BUFFER_SIZE actual value

    • 2.259

      For Evergreen, we want to detect if already too many logs were generated before the Evergreen Jenkins Plugin even had time to start up.

      The rationale being: if there are already 256 logs when plugin startup state is attained, then there is something (very) wrong per-se.

      The goal is to be able to do something like:

      if (Jenkins.logRecords.size() >= WebAppMain.getDefaultRingBufferSize()) {
          LOGGER.log(Level.SEVERE,
                     "The Jenkins logs buffer is already full. " +
                             "Some logs might have been missed, and so many logs shows something probably very wrong! (max={0})",
                     maxNumberOfLogs);
      }
      

      Currently, as an intermediate shim, I'm using reflection. In addition to being dirty, this is likely to become even more an issue to support Java 9 when using such hacks is being made harder and harder.

          [JENKINS-50669] Expose WebAppMain#DEFAULT_RING_BUFFER_SIZE actual value

          Code changed in jenkins
          User: Baptiste Mathus
          Path:
          src/main/java/io/jenkins/plugins/essentials/logging/EssentialsLoggingConfigurer.java
          http://jenkins-ci.org/commit/essentials-plugin/5f42dc4e3cca1981dc7c5636b4139731c0d16ec4
          Log:
          JENKINS-50669 reflective access shim

          SCM/JIRA link daemon added a comment - Code changed in jenkins User: Baptiste Mathus Path: src/main/java/io/jenkins/plugins/essentials/logging/EssentialsLoggingConfigurer.java http://jenkins-ci.org/commit/essentials-plugin/5f42dc4e3cca1981dc7c5636b4139731c0d16ec4 Log: JENKINS-50669 reflective access shim

          Daniel Beck added a comment -

          Daniel Beck added a comment -

            andreineagu Andrei Neagu
            batmat Baptiste Mathus
            Votes:
            0 Vote for this issue
            Watchers:
            4 Start watching this issue

              Created:
              Updated:
              Resolved: