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

Build History order not sorted properly

    XMLWordPrintable

    Details

    • Type: Bug
    • Status: Resolved (View Workflow)
    • Priority: Major
    • Resolution: Fixed
    • Component/s: core
    • Labels:
    • Environment:
      regression from 1.630 -> 1.633
    • Similar Issues:

      Description

      as seen in the screenshots below, there is one good (the old version), and one bad (the new version). The build history table now shows an improper sorting, at least for jobs that had existing history with them during the upgrade process.

      Instead of showing the most recent builds at the top, oldest at the bottom, it now has a rather random looking sort to it

      this seems to be caused by the following PR: https://github.com/jenkinsci/jenkins/pull/1641

        Attachments

          Issue Links

            Activity

            Hide
            scm_issue_link SCM/JIRA link daemon added a comment -

            Code changed in jenkins
            User: Daniel Beck
            Path:
            core/src/main/java/jenkins/widgets/HistoryPageEntry.java
            http://jenkins-ci.org/commit/jenkins/7410a593b30d635010c6344a812889074e68ac98
            Log:
            [FIX JENKINS-30899] Do not rely on Run.queueId for builds widget

            Show
            scm_issue_link SCM/JIRA link daemon added a comment - Code changed in jenkins User: Daniel Beck Path: core/src/main/java/jenkins/widgets/HistoryPageEntry.java http://jenkins-ci.org/commit/jenkins/7410a593b30d635010c6344a812889074e68ac98 Log: [FIX JENKINS-30899] Do not rely on Run.queueId for builds widget
            Hide
            scm_issue_link SCM/JIRA link daemon added a comment -

            Code changed in jenkins
            User: Daniel Beck
            Path:
            core/src/main/java/jenkins/widgets/HistoryPageEntry.java
            core/src/main/java/jenkins/widgets/HistoryPageFilter.java
            core/src/test/java/jenkins/widgets/HistoryPageFilterTest.java
            war/images/go-down.svg
            war/images/go-top.svg
            war/images/go-up.svg
            war/src/main/webapp/css/style.css
            war/src/main/webapp/images/16x16/bullseye.png
            war/src/main/webapp/images/16x16/go_down.png
            war/src/main/webapp/images/16x16/go_top.png
            war/src/main/webapp/images/16x16/go_up.png
            war/src/main/webapp/scripts/hudson-behavior.js
            http://jenkins-ci.org/commit/jenkins/2dd658d6fc9a085ea5bd1e8a4e7c178cd95e2f86
            Log:
            Merge pull request #1872 from tfennelly/bh-pagination-daniel

            [FIX JENKINS-30899] Build History Pagination/Search fixes

            Compare: https://github.com/jenkinsci/jenkins/compare/6aba76dcb00f...2dd658d6fc9a

            Show
            scm_issue_link SCM/JIRA link daemon added a comment - Code changed in jenkins User: Daniel Beck Path: core/src/main/java/jenkins/widgets/HistoryPageEntry.java core/src/main/java/jenkins/widgets/HistoryPageFilter.java core/src/test/java/jenkins/widgets/HistoryPageFilterTest.java war/images/go-down.svg war/images/go-top.svg war/images/go-up.svg war/src/main/webapp/css/style.css war/src/main/webapp/images/16x16/bullseye.png war/src/main/webapp/images/16x16/go_down.png war/src/main/webapp/images/16x16/go_top.png war/src/main/webapp/images/16x16/go_up.png war/src/main/webapp/scripts/hudson-behavior.js http://jenkins-ci.org/commit/jenkins/2dd658d6fc9a085ea5bd1e8a4e7c178cd95e2f86 Log: Merge pull request #1872 from tfennelly/bh-pagination-daniel [FIX JENKINS-30899] Build History Pagination/Search fixes Compare: https://github.com/jenkinsci/jenkins/compare/6aba76dcb00f...2dd658d6fc9a
            Hide
            danielbeck Daniel Beck added a comment -

            Merged towards 1.634 which is scheduled for release early next week.

            (Note that 1.633 introduced another issue whose fixes have not yet been merged – JENKINS-30820 and all the duplicates linked there)

            Show
            danielbeck Daniel Beck added a comment - Merged towards 1.634 which is scheduled for release early next week. (Note that 1.633 introduced another issue whose fixes have not yet been merged – JENKINS-30820 and all the duplicates linked there)
            Hide
            dogfood dogfood added a comment -

            Integrated in jenkins_main_trunk #4323
            [FIX JENKINS-30899] Do not rely on Run.queueId for builds widget (Revision 7410a593b30d635010c6344a812889074e68ac98)

            Result = SUCCESS
            tom.fennelly : 7410a593b30d635010c6344a812889074e68ac98
            Files :

            • core/src/main/java/jenkins/widgets/HistoryPageEntry.java
            Show
            dogfood dogfood added a comment - Integrated in jenkins_main_trunk #4323 [FIX JENKINS-30899] Do not rely on Run.queueId for builds widget (Revision 7410a593b30d635010c6344a812889074e68ac98) Result = SUCCESS tom.fennelly : 7410a593b30d635010c6344a812889074e68ac98 Files : core/src/main/java/jenkins/widgets/HistoryPageEntry.java
            Hide
            gsxjay Jay Beeckman added a comment -

            This fix has worked for us.

            Show
            gsxjay Jay Beeckman added a comment - This fix has worked for us.

              People

              Assignee:
              tfennelly Tom FENNELLY
              Reporter:
              sreich Shaun Reich
              Votes:
              29 Vote for this issue
              Watchers:
              35 Start watching this issue

                Dates

                Created:
                Updated:
                Resolved: