Pin Drivers Evergreen Tools to commit

XMLWordPrintableJSON

    • Type: Task
    • Resolution: Done
    • Priority: Unknown
    • None
    • Affects Version/s: None
    • Component/s: Evergreen, Evergreen Tools
    • None
    • None
    • Fully Compatible
    • Go Drivers
    • Not Needed
    • Hide

      1. What would you like to communicate to the user about this feature?
      2. Would you like the user to see examples of the syntax and/or executable code and its output?
      3. Which versions of the driver/connector does this apply to?

      Show
      1. What would you like to communicate to the user about this feature? 2. Would you like the user to see examples of the syntax and/or executable code and its output? 3. Which versions of the driver/connector does this apply to?
    • None
    • None
    • None
    • None
    • None
    • None

      Context

      Cloning DET from master results in unexpected CI failures that are quite difficult to diagnose. E.g., DRIVERS-3550 and DRIVERS-355.

      Definition of done

      Add drivers-evergreen-tools as a git submodule pinned to a commit, and update setup-system.sh to source $DRIVERS_TOOLS from the submodule rather than an unpinned git clone of master. Configure dependabot to open bump PRs, so DET-introduced regressions surface in dependabot PR instead of breaking unrelated driver PRs.

      Note: DET is used as a runtime working dir, so confirm the submodule tolerates the runtime artifacts written into $DRIVERS_TOOLS. If this doesn't work, we can pin a SHA file for the dependabot bump, instead of using git submodules.

      Pitfalls

      A pinned DET means new-server support requires an explicit bump. We'll set the bump cadence in dependabot.yml (e.g., daily) and allow on-demand bumps when testing a new server version.

      Additionally, we will have to make code changes to resolve issues with external dependencies: e.g., https://jira.mongodb.org/browse/DRIVERS-3565

            Assignee:
            Matt Dale
            Reporter:
            Preston Vasquez
            Qingyang Hu
            Votes:
            0 Vote for this issue
            Watchers:
            1 Start watching this issue

              Created:
              Updated:
              Resolved: