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

Disable "Require branches to be up to date before merging" on Evergreen repo

    XMLWordPrintable

    Details

    • Type: Task
    • Status: Resolved (View Workflow)
    • Priority: Minor
    • Resolution: Fixed
    • Component/s: evergreen
    • Labels:
      None
    • Sprint:
      Evergreen - Milestone 2
    • Similar Issues:

      Description

      Problem statement

      We enabled some time ago the fact that reviews were required.
      In the go, we also enabled the fact that the branch *must* be up-to-date.

      This makes sense in general, but is mostly a waste of time in our time, given we use the merge strategy, i.e. Jenkins will first merge the PR against master, then build it (showing up as pr-merge on the PR status check).

      I assume this GitHub settings makes sense for other CI systems or when Jenkins is only configured to build the so-called pr-head.

      Expected

      We should only require reviews.
      We can allow not being up to date with master branch given we build pr-merge.

      to clarify, we need to uncheck the bolded line:

        Attachments

          Activity

          Hide
          batmat Baptiste Mathus added a comment -

          I will proceed about this later today or tomorrow I think. Cc R. Tyler Croy in case you think this should be preserved for a good reason and you possibly actually configured it this way on purpose.

          Thanks

          Show
          batmat Baptiste Mathus added a comment - I will proceed about this later today or tomorrow I think. Cc R. Tyler Croy in case you think this should be preserved for a good reason and you possibly actually configured it this way on purpose. Thanks
          Hide
          batmat Baptiste Mathus added a comment -

          Done.

          New config:

          Show
          batmat Baptiste Mathus added a comment - Done. New config:

            People

            • Assignee:
              batmat Baptiste Mathus
              Reporter:
              batmat Baptiste Mathus
            • Votes:
              0 Vote for this issue
              Watchers:
              1 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved: