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

      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

        Activity

        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).
        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.

          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: