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

transient file handle leak in LargeText.GzipAwareSession.isGzipStream(File)

    XMLWordPrintable

Details

    Description

      One of the "reproduction" scenarios if JENKINS-45057 identified a separate transient file handle leak in LargeText.GzipAwareSession.isGzipStream(File)

      Attachments

        Issue Links

          is related to

          Bug - A problem which impairs or prevents the functions of the product. JENKINS-45057 "too many files open": file handles leak, job output file not closed

          • Blocker - Blocks development and/or testing work, production could not run.
          • Resolved

          Improvement - An improvement or enhancement to an existing feature or task. JENKINS-42934 Avoid using new FileInputStream / new FileOutputStream

          • Minor - Minor loss of function, or other problem where easy workaround is present.
          • Closed
          links to

          Web Link stapler#124

          Activity

            Ascending order - Click to sort in descending order
            View 17 older comments
            olivergondza Oliver Gondža added a comment -

            danielbeck, I do not oppose if this is serious enough.

            olivergondza Oliver Gondža added a comment - danielbeck , I do not oppose if this is serious enough.
            danielbeck Daniel Beck added a comment -

            Restoring lts-candidate then. Popularity indicates this is relevant enough for a backport. (Even if it's unclear everyone watching would be helped by this fix.)

            oleg_nenashev olivergondza Do we need KK to release Stapler, or could someone else do this?

            danielbeck Daniel Beck added a comment - Restoring  lts-candidate then. Popularity indicates this is relevant enough for a backport. (Even if it's unclear everyone watching would be helped by this fix.) oleg_nenashev olivergondza Do we need KK to release Stapler, or could someone else do this?
            jglick Jesse Glick added a comment -

            I at least am able to release Stapler if this is under consideration for 2.73.3.

            jglick Jesse Glick added a comment - I at least am able to release Stapler if this is under consideration for 2.73.3.
            danielbeck Daniel Beck added a comment -

            olivergondza See above – we can add it if you think the backport to Stapler 1.250 is reasonable for 2.73.3 inclusion.

            danielbeck Daniel Beck added a comment - olivergondza See above – we can add it if you think the backport to Stapler 1.250 is reasonable for 2.73.3 inclusion.
            1334292 Hubert Okon added a comment -

            Instead of using build.properties (which leaks FD) it is possible to use build..getActions() which does not leak FD.

            1334292 Hubert Okon added a comment - Instead of using  build.properties  (which leaks FD) it is possible to use  build. .getActions()  which does not leak FD.

            People

              stephenconnolly Stephen Connolly
              stephenconnolly Stephen Connolly
              Votes:
              2 Vote for this issue
              Watchers:
              9 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved: