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

Zip artifact downloads missing a top-level directory

    XMLWordPrintable

    Details

    • Similar Issues:
    • Released As:
      2.276 and 2.263.3

      Description

      Upgrading from 2.263.1 to 2.263.2 has this undocumented change.
      Zip files of artifacts no longer have a top-level directory, but all the contents at the root.

      Examples:

      • $BUILD_URL/artifact/target/*zip*/target.zip
      • $BUILD_URL/artifact/dist/Release/*zip*/Release.zip
      • $BUILD_URL/Documentation/*zip*/Documentation.zip

      Possible regression of JENKINS-19947

        Attachments

          Issue Links

            Activity

            Hide
            markewaite Mark Waite added a comment - - edited

            James Howe I built a local copy of the Jenkins master branch with pull request 5187 and confirmed that it passes the job that was consistently failing on Jenkins 2.275 and Jenkins 2.263.2. You're welcome to use my build to test or you can use the ci.jenkins.io build that will be generated as part of the pull request.

            Show
            markewaite Mark Waite added a comment - - edited James Howe I built a local copy of the Jenkins master branch with pull request 5187 and confirmed that it passes the job that was consistently failing on Jenkins 2.275 and Jenkins 2.263.2. You're welcome to use my build to test or you can use the ci.jenkins.io build that will be generated as part of the pull request.
            Hide
            grayaii Alex Gray added a comment -

            Oh wow. Thank you Mark Waite for the quick turnaround! This is a show stopper for us (we have custom scripts that look on these artifacts).  Can't wait for this to be in a TLS (We are only allowed to use TLS versions). (And thank you Oleg for verifying the PR works!)

            Show
            grayaii Alex Gray added a comment - Oh wow. Thank you Mark Waite  for the quick turnaround! This is a show stopper for us (we have custom scripts that look on these artifacts).  Can't wait for this to be in a TLS (We are only allowed to use TLS versions). (And thank you Oleg for verifying the PR works!)
            Hide
            njesper Jesper Andersson added a comment -

            Alex Gray, if you are doing automatic downloading of artifacts you should probably have a look at JENKINS-64669 that I just filed. Normal "occasional" downloads might not be enough to trigger problems, but high frequency automated downloads was what triggered it for us. (And zip-on-the-fly downloads might not be affected - I've not investigated that.)

            Show
            njesper Jesper Andersson added a comment - Alex Gray , if you are doing automatic downloading of artifacts you should probably have a look at  JENKINS-64669 that I just filed. Normal "occasional" downloads might not be enough to trigger problems, but high frequency automated downloads was what triggered it for us. (And zip-on-the-fly downloads might not be affected - I've not investigated that.)
            Hide
            grayaii Alex Gray added a comment -

            Ouchie. Yeah, that a big one.  We use datadog to monitor open file handles, but I didn't even get that far due to this ticket.
            Wow.  Thank you so much for pointing that out.  I just voted for that ticket too. You just saved us a ton of headahe.

            Show
            grayaii Alex Gray added a comment - Ouchie. Yeah, that a big one.  We use datadog to monitor open file handles, but I didn't even get that far due to this ticket. Wow.  Thank you so much for pointing that out.  I just voted for that ticket too. You just saved us a ton of headahe.
            Hide
            mwils2424 Matt Wilson added a comment -

            Thanks Jesper Andersson we've run into that issue the past couple of nights after upgrading this weekend.  Upped our limits last night, but good to know we need to keep an eye on this. 

            Show
            mwils2424 Matt Wilson added a comment - Thanks Jesper Andersson  we've run into that issue the past couple of nights after upgrading this weekend.  Upped our limits last night, but good to know we need to keep an eye on this. 

              People

              Assignee:
              wfollonier Wadeck Follonier
              Reporter:
              jameshowe James Howe
              Votes:
              2 Vote for this issue
              Watchers:
              9 Start watching this issue

                Dates

                Created:
                Updated:
                Resolved: