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

Do not log Stapler warnings when admins are shown admin monitors in LTS

    • Icon: Bug Bug
    • Resolution: Fixed
    • Icon: Minor Minor
    • core
    •  2.277.2 released Apr 7, 2021

      https://github.com/jenkinsci/jenkins/pull/5207 should be backported into LTS, the warnings are annoying. The bug should never even have made it into LTS

      Also, why is there no changelog entry? This is an annoying warning getting logged regularly.

          [JENKINS-65186] Do not log Stapler warnings when admins are shown admin monitors in LTS

          Daniel Beck created issue -
          Daniel Beck made changes -
          Resolution New: Fixed [ 1 ]
          Status Original: Open [ 1 ] New: Resolved [ 5 ]
          Daniel Beck made changes -
          Description Original: [https://github.com/jenkinsci/jenkins/pull/5207] should be backported into LTS, the warnings are annoying. The bug should never even have made it into LTS :(

          Also, why is there no changelog entry?
          New: [https://github.com/jenkinsci/jenkins/pull/5207] should be backported into LTS, the warnings are annoying. The bug should never even have made it into LTS :(

          Also, why is there no changelog entry? This is an annoying warning getting logged regularly.

          Mark Waite added a comment - - edited

          Comments from timja and fqueiruga wondered if a changelog entry was necessary. Willing to add an entry to the weekly changelog if that helps .

          Mark Waite added a comment - - edited Comments from timja and fqueiruga wondered if a changelog entry was necessary. Willing to add an entry to the weekly changelog if that helps .

          Daniel Beck added a comment -

          Yeah I saw that comment after I posted this, and I clearly missed it when I reviewed the fix.

          In theory it's just too trivial, agreed, but now we have log spam in LTS

          Daniel Beck added a comment - Yeah I saw that comment after I posted this, and I clearly missed it when I reviewed the fix. In theory it's just too trivial, agreed, but now we have log spam in LTS

          Sorry if this is late, maybe a changelog entry would refer to the log spam reduction. The change itself is not very interesting, as it is very internal

          Félix Queiruga Balado added a comment - Sorry if this is late, maybe a changelog entry would refer to the log spam reduction. The change itself is not very interesting, as it is very internal
          Mark Waite made changes -
          Labels Original: lts-candidate New: 2.277.2-fixed lts-candidate

          Jesse Glick added a comment -

          danielbeck could you set a meaningful issue title please?

          Jesse Glick added a comment - danielbeck could you set a meaningful issue title please?
          Daniel Beck made changes -
          Summary Original: Backport PR 5207 New: Do not log Stapler warnings when admins are shown admin monitors in LTS
          Mark Waite made changes -
          Status Original: Resolved [ 5 ] New: Closed [ 6 ]

            Unassigned Unassigned
            danielbeck Daniel Beck
            Votes:
            0 Vote for this issue
            Watchers:
            4 Start watching this issue

              Created:
              Updated:
              Resolved: