Details

    • Type: Task
    • Status: In Progress (View Workflow)
    • Priority: Critical
    • Resolution: Unresolved
    • Component/s: ci
    • Labels:
      None
    • Similar Issues:

      Description

      I complained about this on Apr 26 but was told by R. Tyler Croy and Daniel Beck it was not really an issue. It is an issue. My plugin PR build has now been sitting for 47m waiting for an executor, the build queue has 111 items (https://ci.jenkins.io/load-statistics?type=min says it was up to >210 at one point), and all of our executors are rebuilding core PRs, one of which I see has not been touched since 2012. This is totally unreasonable. We really need to apply this setting at least to core builds, if not everywhere.

        Attachments

        1. 2019-03-23_22-37.png
          2019-03-23_22-37.png
          249 kB
        2. graph.png
          graph.png
          27 kB
        3. screenshot1.png
          screenshot1.png
          26 kB
        4. screenshot2.png
          screenshot2.png
          65 kB

          Issue Links

            Activity

            Hide
            jglick Jesse Glick added a comment -

            Was just looking at a plugin PR build which was stalled, and saw that the server was loaded down rebuilding acceptance-test-harness, including for example a two-year-old PR. Again, most easily solved by a server-wide policy switch to not rebuild all PRs after every base branch update.

            Show
            jglick Jesse Glick added a comment - Was just looking at a plugin PR build which was stalled, and saw that the server was loaded down rebuilding acceptance-test-harness , including for example a two-year-old PR. Again, most easily solved by a server-wide policy switch to not rebuild all PRs after every base branch update.
            Hide
            jglick Jesse Glick added a comment -

            Looked at the queue today and it had 308 items.

            Show
            jglick Jesse Glick added a comment - Looked at the queue today and it had 308 items .
            Hide
            jglick Jesse Glick added a comment -

            W.r.t. the idea of only allowing PRs to be merged (by non-admins) when up to date with the base branch:

            waiting a minimum of 4-5 hours between individual PR merges unless I go to the command line and merge there

            Or just manually simulate Tide in cases where there is a batch of PRs waiting: file a new PR that just merges all the proposed content PRs, and if its build passes, merge that.

            Show
            jglick Jesse Glick added a comment - W.r.t. the idea of only allowing PRs to be merged (by non-admins) when up to date with the base branch: waiting a minimum of 4-5 hours between individual PR merges unless I go to the command line and merge there Or just manually simulate Tide in cases where there is a batch of PRs waiting: file a new PR that just merges all the proposed content PRs, and if its build passes, merge that.
            Hide
            danielbeck Daniel Beck added a comment -

            We have enough difficulty with consistently getting reviews, and maintainers to click Merge, that adding more difficulty on top seems counterproductive.

            Show
            danielbeck Daniel Beck added a comment - We have enough difficulty with consistently getting reviews, and maintainers to click Merge, that adding more difficulty on top seems counterproductive.
            Hide
            oleg_nenashev Oleg Nenashev added a comment -

            I would like to start fromĀ JENKINS-58939 in order to reduce number of rebuilds when they are not desired

            Show
            oleg_nenashev Oleg Nenashev added a comment - I would like to start fromĀ  JENKINS-58939 in order to reduce number of rebuilds when they are not desired

              People

              • Assignee:
                oleg_nenashev Oleg Nenashev
                Reporter:
                jglick Jesse Glick
              • Votes:
                2 Vote for this issue
                Watchers:
                7 Start watching this issue

                Dates

                • Created:
                  Updated: