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

Attempting to create a root/unfiltered log recorder with level 'ALL' should display a warning

    XMLWordPrintable

    Details

    • Type: Improvement
    • Status: Open (View Workflow)
    • Priority: Major
    • Resolution: Unresolved
    • Component/s: core
    • Labels:
      None
    • Similar Issues:

      Description

      As I learned the hard way, the Jenkins log recorder UI makes it easy for you to accidentally create an unfiltered recorder with a log level of 'ALL' (because that is the default level), and this will easily overwhelm and take down Jenkins if it is under normal load. In my case the only way to recover was to find the log recorder config XML file on the filesystem, move it aside, and restart Jenkins. This caused an unpleasant 25 minute production outage for us.

      I would propose that if the user attempts to create an empty recorder at level ALL we should throw a warning dialog, explaining that this configuration can be dangerous, and give them the option to back out.

      Alternatively, maybe the default level should be SEVERE rather than ALL, so that if you accidentally create an unfiltered recorder it won't log much.

      Daniel Beck do you want to comment on this? I can make an attempt at implementation if you think the logic is sound.

        Attachments

          Issue Links

            Activity

            Hide
            danielbeck Daniel Beck added a comment -

            Should probably just discard root loggers with ALL/FINEST/FINER/FINE level, or force their level to a higher (i.e. less verbose) one.

            Show
            danielbeck Daniel Beck added a comment - Should probably just discard root loggers with ALL/FINEST/FINER/FINE level, or force their level to a higher (i.e. less verbose) one.

              People

              Assignee:
              Unassigned Unassigned
              Reporter:
              owenmehegan Owen Mehegan
              Votes:
              0 Vote for this issue
              Watchers:
              3 Start watching this issue

                Dates

                Created:
                Updated: