fix(.github/workflows): reuse cached .git repository for branch HEAD commit #3893

darccio · 2025-08-19T16:35:55Z

What does this PR do?

Introduces repository caching by saving .git and materializing the files on checking out when running each job on the matrix.

Motivation

Faster CI times even if more tests are planned to run after swapping pull-request.yml with main-branch-tests.yml in #3887.

Reviewer's Checklist

Changed code has unit tests for its functionality at or near 100% coverage.
System-Tests covering this feature have been added and enabled with the va.b.c-dev version tag.
There is a benchmark for any new code, or changes to existing code.
If this interacts with the agent in a new way, a system test has been added.
New code is free of linting errors. You can check this by running ./scripts/lint.sh locally.
Add an appropriate team label so this PR gets put in the right place for the release notes.
Non-trivial go.mod changes, e.g. adding new modules, are reviewed by @DataDog/dd-trace-go-guild.

Unsure? Have a question? Request a review!

…e soon-to-be merged PR as they ran on it

…flakiness is more probable while testing the PR

… test outcomes in PRs

pr-commenter · 2025-08-19T16:37:04Z

Benchmarks

Benchmark execution time: 2025-08-21 16:15:38

Comparing candidate commit f9763cc in PR branch dario.castane/ktlo/cache-checkouts-on-matrices with baseline commit 283db37 in branch dario.castane/ktlo/disable-main-branch-ci.

Found 0 performance improvements and 0 performance regressions! Performance is the same for 24 metrics, 0 unstable metrics.

…commit

datadog-official · 2025-08-19T16:49:24Z

✅ Tests

🎉 All green!

❄️ No new flaky tests detected
🧪 All tests passed

_{This comment will be updated automatically if new data arrives.

🔗 Commit SHA: 4f673ed | Docs | Was this helpful? Give us feedback!}

darccio · 2025-08-19T16:51:14Z

.github/workflows/main-branch-tests.yml

-  workflow_call: # allows to reuse this workflow
-    inputs:
-      ref:
-        description: 'The branch to run the workflow on'
-        required: true
-        type: string


This is no longer needed because this workflow is never used from another one.

darccio · 2025-08-19T16:51:32Z

.github/workflows/multios-unit-tests.yml

  workflow_dispatch: # manually
    inputs:
      go-version:
+        description: The Go version to use


Added descriptions because a linter was complaining about them.

darccio · 2025-08-19T16:52:34Z

.github/workflows/multios-unit-tests.yml

      - name: Checkout
        uses: actions/checkout@11bd71901bbe5b1630ceea73d27597364c9af683 # v2.7.0
        with:
-          ref: ${{ inputs.ref || github.ref }}
+          ref: ${{ github.sha }}
+          clean: false


If by any chance the previous step doesn't find the cached repo, checkout will just work as usual by downloading the repository. clean: false is only needed because we'll usually restore from the cached .git directory.

darccio · 2025-08-19T16:54:06Z

.github/workflows/unit-integration-tests.yml

        uses: actions/checkout@11bd71901bbe5b1630ceea73d27597364c9af683 # v4.2.2
        with:
-          ref: ${{ inputs.ref || github.ref }}
-          fetch-depth: $(( ${{ github.event_name == 'pull_request' && github.event.pull_request.commits || 0 }} + 1 ))


By default fetch-depth is 1. This expression seems to end with 1 or 2, which I don't think it's required.

…ios-unit-tests

…astane/ktlo/cache-checkouts-on-matrices

…th git commands to materialize from `.git`

…commit (#3893)

darccio added 3 commits August 19, 2025 10:14

chore(ci): ensure merge queue runs the same CI checks when testing th…

b742c82

…e soon-to-be merged PR as they ran on it

feat(.github/workflows): swap pull-request and main-branch-tests, so …

60729ed

…flakiness is more probable while testing the PR

feat(.github/workflows): add pull-request-tests-done job for handling…

d39ddcb

… test outcomes in PRs

darccio requested a review from a team as a code owner August 19, 2025 16:35

darccio force-pushed the dario.castane/ktlo/cache-checkouts-on-matrices branch from 4832f3a to 7b06f33 Compare August 19, 2025 16:38

fix(.github/workflows): reuse cached .git repository for branch HEAD …

c2e5a78

…commit

darccio closed this Aug 19, 2025

darccio force-pushed the dario.castane/ktlo/cache-checkouts-on-matrices branch from 7b06f33 to d39ddcb Compare August 19, 2025 16:42

darccio reopened this Aug 19, 2025

darccio commented Aug 19, 2025

View reviewed changes

fix(.github/workflows): remove left over from a previous life in mult…

6671f89

…ios-unit-tests

darccio force-pushed the dario.castane/ktlo/disable-main-branch-ci branch from d39ddcb to d742705 Compare August 21, 2025 10:05

darccio requested review from a team as code owners August 21, 2025 10:05

darccio force-pushed the dario.castane/ktlo/disable-main-branch-ci branch from d742705 to 849e00d Compare August 21, 2025 10:07

Merge branch 'dario.castane/ktlo/disable-main-branch-ci' into dario.c…

49ba5a8

…astane/ktlo/cache-checkouts-on-matrices

darccio force-pushed the dario.castane/ktlo/cache-checkouts-on-matrices branch 5 times, most recently from 71ac868 to 4ba46f6 Compare August 21, 2025 14:56

feat(.github/workflows): migrate system-tests and replace checkout wi…

ca7461b

…th git commands to materialize from `.git`

darccio force-pushed the dario.castane/ktlo/cache-checkouts-on-matrices branch 2 times, most recently from 7cead25 to b8ed9e4 Compare August 21, 2025 15:36

fix(.github/workflows): pull from the PR's HEAD

4f673ed

darccio force-pushed the dario.castane/ktlo/cache-checkouts-on-matrices branch 2 times, most recently from f9763cc to 4f673ed Compare August 21, 2025 16:16

darccio merged commit 8635ff7 into dario.castane/ktlo/disable-main-branch-ci Aug 21, 2025
209 of 408 checks passed

darccio deleted the dario.castane/ktlo/cache-checkouts-on-matrices branch August 21, 2025 16:17

darccio added a commit that referenced this pull request Aug 22, 2025

fix(.github/workflows): reuse cached .git repository for branch HEAD …

58b7fa2

…commit (#3893)

darccio added a commit that referenced this pull request Aug 27, 2025

fix(.github/workflows): reuse cached .git repository for branch HEAD …

d11299f

…commit (#3893)

darccio added a commit that referenced this pull request Aug 27, 2025

fix(.github/workflows): reuse cached .git repository for branch HEAD …

f9521df

…commit (#3893)

darccio added a commit that referenced this pull request Aug 28, 2025

fix(.github/workflows): reuse cached .git repository for branch HEAD …

bd8a68a

…commit (#3893)

darccio added a commit that referenced this pull request Sep 3, 2025

fix(.github/workflows): reuse cached .git repository for branch HEAD …

966c56a

…commit (#3893)

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

fix(.github/workflows): reuse cached .git repository for branch HEAD commit #3893

fix(.github/workflows): reuse cached .git repository for branch HEAD commit #3893

Uh oh!

darccio commented Aug 19, 2025

Uh oh!

pr-commenter bot commented Aug 19, 2025 •

edited

Loading

Uh oh!

datadog-official bot commented Aug 19, 2025 •

edited

Loading

Uh oh!

darccio Aug 19, 2025

Uh oh!

darccio Aug 19, 2025

Uh oh!

darccio Aug 19, 2025 •

edited

Loading

Uh oh!

darccio Aug 19, 2025

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

fix(.github/workflows): reuse cached .git repository for branch HEAD commit #3893

fix(.github/workflows): reuse cached .git repository for branch HEAD commit #3893

Uh oh!

Conversation

darccio commented Aug 19, 2025

What does this PR do?

Motivation

Reviewer's Checklist

Uh oh!

pr-commenter bot commented Aug 19, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Benchmarks

Uh oh!

datadog-official bot commented Aug 19, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

darccio Aug 19, 2025

Choose a reason for hiding this comment

Uh oh!

darccio Aug 19, 2025

Choose a reason for hiding this comment

Uh oh!

darccio Aug 19, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

darccio Aug 19, 2025

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

pr-commenter bot commented Aug 19, 2025 •

edited

Loading

datadog-official bot commented Aug 19, 2025 •

edited

Loading

darccio Aug 19, 2025 •

edited

Loading