Uploaded image for project: 'Evergreen'
  1. Evergreen
  2. EVG-6121

set-module doesn't change the manifest's commit

    XMLWordPrintable

    Details

    • Type: Bug
    • Status: Closed
    • Priority: Major - P3
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: v2019.08.01
    • Component/s: cli
    • Labels:
      None

      Description

      The evergreen set-module command does not update the manifest/pinned commit for modules. It seems that EVG-5560 is broken.

      Steps to reproduce:

      git clone git@github.com:mongodb/mongo.git
      cd mongo
      git checkout 7e700f666
      evergreen patch -p sys-perf
      # => 5cc3164661837d56fccd68e0
       
      cd ..
      git clone git@github.com:mongodb/genny.git
      cd genny
      git checkout 9ae96a
      evergreen set-module -m genny -i <patch-id>
      # evergreen set-module -m genny -i 5cc3164661837d56fccd68e0
       
      # schedule Linux Standalone / genny_self_tests
       
      cd ..
      mkdir fetch
      cd fetch
      evergreen fetch --source -t <task_id>
      

      Expected: The genny module is checked out at revision 9ae96a.
      Actual: The genny module is checked out at master (31373926 as of 2019-04-26).

      This isn't a problem with fetch, it's just an easy way to demonstrate it. It's a problem with set-module. It seems the set-module command does not update the manifest.

      This makes it hard to see if different combinations of projects and modules reproduce errors.

      It was suggested on slack that the solution is to hard-code the revision that's checked out in the evergreen yaml. This is tedious and requires submitting a whole new patch when you may wan to re-run the same patch with a different module revision.

        Attachments

          Issue Links

            Activity

              People

              • Assignee:
                backlog-server-evg Backlog - Evergreen Team
                Reporter:
                ryan.timmons Ryan Timmons
              • Votes:
                0 Vote for this issue
                Watchers:
                5 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved: