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

Very slow activity/pipeline screen load (often when logged in)

    Details

    • Similar Issues:
    • Epic Link:
    • Sprint:
      Blue Ocean 1.2-beta1, Blue Ocean 1.2-beta2, Blue Ocean 1.4 - beta 3, Blue Ocean 1.5 - beta 1

      Description

      (to investigate)

       

      • Some people are finding dashboard/pipeline screens slow when logged in vs not logged in (see appropriate comments below) 
        • See comments and support bundles below for this - eg via Brian Saville. In some cases it is related to number of runs. 
      • Some users see activity screen as very slow when there are a large number of runs (this seems to be the more common and serious case...)

       

       

       

      — ORIGINAL TICKET —

      I've noticed that the dashboard loads quickly when I'm not authenticated.

      (classic loads normally)

       

      Deleting the config history like suggested in https://issues.jenkins-ci.org/browse/JENKINS-43208 did not work.

      I have sent an HAR file via email to James Dumay.

      Jenkins version 2.46.3, BlueOcean 1.1.2

        Attachments

        1. haranalysis.png
          haranalysis.png
          118 kB
        2. plugins.txt
          9 kB
        3. support_2017-11-17_08.03.54.zip
          743 kB
        4. support_2017-11-17_08.03.56.zip
          743 kB
        5. support_2017-11-17_08.03.59.zip
          743 kB
        6. support_2017-11-17_08.04.02.zip
          743 kB
        7. support_2017-11-17_08.04.13.zip
          743 kB
        8. support_2017-11-17_08.04.18.zip
          742 kB
        9. support_2017-11-17_11.26.08.zip
          401 kB
        10. support_2017-11-20_23.21.08.zip
          269 kB

          Issue Links

            Activity

            Hide
            vivek Vivek Pandey added a comment - - edited

            Bug identified and fixed. PR opened https://github.com/jenkinsci/blueocean-plugin/pull/1632.

             

            Details on what was causing and fix:

            Analyzing har file showed, FavoriteStatePreloader was returning large number of favorite jobs resulting in most time taken responding to loading dashboard. Bug was in `FavoriteContainer.iterator()` not paginating. Fix adds pagination by default.

            Show
            vivek Vivek Pandey added a comment - - edited Bug identified and fixed. PR opened https://github.com/jenkinsci/blueocean-plugin/pull/1632.   Details on what was causing and fix: Analyzing har file showed, FavoriteStatePreloader was returning large number of favorite jobs resulting in most time taken responding to loading dashboard. Bug was in `FavoriteContainer.iterator()` not paginating. Fix adds pagination by default.
            Hide
            kzantow Keith Zantow added a comment -

            I don't see a problem with that PR, but doesn't the page display all favorites? So I'm not sure how paging would fix that.

            Show
            kzantow Keith Zantow added a comment - I don't see a problem with that PR, but doesn't the page display all favorites? So I'm not sure how paging would fix that.
            Hide
            vivek Vivek Pandey added a comment -

            Keith Zantow Currently yes, it fetches all possible favorites. container iterators are supposed to return default page size. If you call favorite API you get default page size so it makes it consistent.

            Favorites are expensive and displaying 100s of favorite is rather pain to user as bunch of them are auto-favorited. I think a new ticket should be opened to add UI pagination support ('Show more') for favorite as well. Current page size is 100, maybe it should be 26 like other things shown in UI?

             

             

            Show
            vivek Vivek Pandey added a comment - Keith Zantow  Currently yes, it fetches all possible favorites. container iterators are supposed to return default page size. If you call favorite API you get default page size so it makes it consistent. Favorites are expensive and displaying 100s of favorite is rather pain to user as bunch of them are auto-favorited. I think a new ticket should be opened to add UI pagination support ('Show more') for favorite as well. Current page size is 100, maybe it should be 26 like other things shown in UI?    
            Hide
            vivek Vivek Pandey added a comment -

            Keith Zantow over to you as discussed. Listing the issues we discussed to fix as part of this improvement:

            • Paginate favorite list on dashboard (my fix fixes that, that is default list of favorite is default page size of 100, possibly it should be 26 like other objects)
            • Evaluate if top level pipeline object can include 'favorite' as boolean value without much impact
            • Minimize 'item' object properties to include only what favorite UI needs (name of pipeline, favorited or not, commitId...)
            • Fix bug in frontend where it calls favorites API even though it already got pre-loaded list of favorites
            Show
            vivek Vivek Pandey added a comment - Keith Zantow over to you as discussed. Listing the issues we discussed to fix as part of this improvement: Paginate favorite list on dashboard (my fix fixes that, that is default list of favorite is default page size of 100, possibly it should be 26 like other objects) Evaluate if top level pipeline object can include 'favorite' as boolean value without much impact Minimize 'item' object properties to include only what favorite UI needs (name of pipeline, favorited or not, commitId...) Fix bug in frontend where it calls favorites API even though it already got pre-loaded list of favorites
            Hide
            michaelneale Michael Neale added a comment -

            Josh McDonald as mentioned - worth taking a look at (I will take a look at test failures too)

            Show
            michaelneale Michael Neale added a comment - Josh McDonald as mentioned - worth taking a look at (I will take a look at test failures too)

              People

              • Assignee:
                nicu Nicolae Pascu
                Reporter:
                schulzha Hans Schulz
              • Votes:
                7 Vote for this issue
                Watchers:
                21 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved: