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

Expose WebAppMain#DEFAULT_RING_BUFFER_SIZE actual value

    XMLWordPrintable

Details

    • 2.259

    Description

      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.

      Attachments

        Issue Links

          Activity

            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_issue_link 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
            danielbeck Daniel Beck added a comment -

            danielbeck Daniel Beck added a comment -

            People

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

              Dates

                Created:
                Updated:
                Resolved: