Make s_all a prerequisite for evergreen PR testing to save resources and reduce wasted effort.

XMLWordPrintableJSON

    • Type: Improvement
    • Resolution: Unresolved
    • Priority: Major - P3
    • None
    • Affects Version/s: None
    • Component/s: None
    • None
    • Storage Engines, Storage Engines - Foundations
    • SE Foundations - 2025-11-07
    • 2

      Description
      Currently, the evergreen runs the full test suite on a pull request even if the code formatting check (s_all) fails. This leads to wasted resources and time, as a PR with formatting issues cannot be merged anyway.

      For example, when a developer forgets to run s_all locally, they have to push a new commit to fix the formatting, which then triggers the entire test suite to run again.

      Proposed Solution
      Modify the evergreen PR testing pipeline to make the s_all test a prerequisite. The evergreen should stop and fail immediately if s_all fails. This would prevent the more resource-intensive tests from running until all code formatting is corrected.

      Acceptance Criteria

      • The evergreen PR pipeline checks for s_all test status first.
      • If the s_all test fails, the evergreen terminates without running subsequent tests.

            Assignee:
            [DO NOT USE] Backlog - Storage Engines Team
            Reporter:
            Ravi Giri
            Votes:
            0 Vote for this issue
            Watchers:
            1 Start watching this issue

              Created:
              Updated: