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

New build artifact overview is unsorted

    Details

    • Type: Improvement
    • Status: Closed
    • Priority: Major
    • Resolution: Fixed
    • Component/s: core
    • Labels:
      None
    • Similar Issues:
      Show 5 results

      Description

      Most likely introduced in build 1.330 (we upgraded from 1.327 to 1.336)

      The tree view for build artifacts is unsorted. Previous list was alphabetical ordered.
      The list or artifacts we have, starts with a increasing number to benefit from alphabetic ordering

        Attachments

          Activity

          cbos Cees Bos created issue -
          Hide
          mindless Alan Harder added a comment -

          Do you see unsorted list in simple list view (<17 artifacts) or only in tree view (17-40 artifacts). I don't see any code here regarding sorting, before or after the tree view was added in 1.330, so I would expect the tree to be in arbitrary order and the list to be in whatever order the directory listing returned, which is not guaranteed to be alphabetical.
          Can we reclassify this as a request to add sorting for both list and tree displays?

          Show
          mindless Alan Harder added a comment - Do you see unsorted list in simple list view (<17 artifacts) or only in tree view (17-40 artifacts). I don't see any code here regarding sorting, before or after the tree view was added in 1.330, so I would expect the tree to be in arbitrary order and the list to be in whatever order the directory listing returned, which is not guaranteed to be alphabetical. Can we reclassify this as a request to add sorting for both list and tree displays?
          Hide
          cbos Cees Bos added a comment -

          We have also a job with only 5 artifacts, that list is also unordered.
          I did compare it with the list of 40+ (which is the same as the workspace file view). That list is ordered.
          I agree that it is better to have both sorted.

          Another feature we use is the option to download all artifact in a zip. For the jobs we use this feature we have 40+ artifacts, so that is still working
          http://hudsonserver:8080/hudson/job/cm-trunk-wip-w/lastSuccessfulBuild/artifact/cm/build/isvp/*zip*/isvp.zip
          This feature is also provided by the workspace file view and very useful with such a long list.
          We don't use it (yet) for a smaller list, maybe others did. If not it is not required to add it.

          Show
          cbos Cees Bos added a comment - We have also a job with only 5 artifacts, that list is also unordered. I did compare it with the list of 40+ (which is the same as the workspace file view). That list is ordered. I agree that it is better to have both sorted. Another feature we use is the option to download all artifact in a zip. For the jobs we use this feature we have 40+ artifacts, so that is still working http://hudsonserver:8080/hudson/job/cm-trunk-wip-w/lastSuccessfulBuild/artifact/cm/build/isvp/*zip*/isvp.zip This feature is also provided by the workspace file view and very useful with such a long list. We don't use it (yet) for a smaller list, maybe others did. If not it is not required to add it.
          Hide
          mindless Alan Harder added a comment -

          Ok, I plan to add a simple per-directory sort.. I hope that will be sufficient. (the contents of each directory will be sorted, but when shown as a simple list these may not appear sorted if they come from different subdirs in the artifacts tree).

          Regarding zip download, it is always available.. ie, from a build page you can always tack /artifact on the end of the URL to reach the DirectoryBrowser which has the zip download option.. it's just that there is no visible link to this when there are < 40 artifacts.. might be useful to have such a link (but where? hm).

          Show
          mindless Alan Harder added a comment - Ok, I plan to add a simple per-directory sort.. I hope that will be sufficient. (the contents of each directory will be sorted, but when shown as a simple list these may not appear sorted if they come from different subdirs in the artifacts tree). Regarding zip download, it is always available.. ie, from a build page you can always tack /artifact on the end of the URL to reach the DirectoryBrowser which has the zip download option.. it's just that there is no visible link to this when there are < 40 artifacts.. might be useful to have such a link (but where? hm).
          mindless Alan Harder made changes -
          Field Original Value New Value
          Assignee mindless [ mindless ]
          mindless Alan Harder made changes -
          Issue Type Bug [ 1 ] Improvement [ 4 ]
          mindless Alan Harder made changes -
          Status Open [ 1 ] In Progress [ 3 ]
          Hide
          scm_issue_link SCM/JIRA link daemon added a comment -

          Code changed in hudson
          User: : mindless
          Path:
          trunk/hudson/main/core/src/main/java/hudson/model/Run.java
          trunk/hudson/main/core/src/main/resources/lib/hudson/artifactList.jelly
          trunk/www/changelog.html
          http://fisheye4.cenqua.com/changelog/hudson/?cs=24413
          Log:
          [FIXED JENKINS-4976] Add per-directory sorting for artifact list and tree display.
          Also, always show a link to directory browser (as "download as zip" is accessible here)
          instead of only when there are 40+ artifacts.

          Show
          scm_issue_link SCM/JIRA link daemon added a comment - Code changed in hudson User: : mindless Path: trunk/hudson/main/core/src/main/java/hudson/model/Run.java trunk/hudson/main/core/src/main/resources/lib/hudson/artifactList.jelly trunk/www/changelog.html http://fisheye4.cenqua.com/changelog/hudson/?cs=24413 Log: [FIXED JENKINS-4976] Add per-directory sorting for artifact list and tree display. Also, always show a link to directory browser (as "download as zip" is accessible here) instead of only when there are 40+ artifacts.
          scm_issue_link SCM/JIRA link daemon made changes -
          Status In Progress [ 3 ] Resolved [ 5 ]
          Resolution Fixed [ 1 ]
          abayer Andrew Bayer made changes -
          Status Resolved [ 5 ] Closed [ 6 ]

            People

            • Assignee:
              mindless Alan Harder
              Reporter:
              cbos Cees Bos
            • Votes:
              0 Vote for this issue
              Watchers:
              1 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved: