skip full wipe for --binary-path with source repos

[p-datadog]

Summary

Builds on #6560. This is a small follow-up that makes repeated build.sh invocations faster when using --binary-path with a local source repo.

Today, every build.sh call with --binary-path runs clean-binaries first — wiping everything in binaries/ — then copies the source repo back in. For a repo like dd-trace-rb (~85MB), that's a full delete and full re-copy on every invocation, even when nothing in the repo has changed.

This moves clean-binaries into the flat-copy path only (artifact-based tracers like Java, .NET, PHP). For the source-repo path, rsync --delete handles keeping the destination in sync incrementally — it only transfers files that actually changed and removes files that no longer exist in the source. When nothing changed, rsync does nothing.

What changed

Depends on #6560 — please review that first.

-clean-binaries                    # always wipe everything
 if source-repo detected; then
-    rsync ... "$BINARY_PATH/" "$expected_repo_dir/"
+    rsync ... --delete "$BINARY_PATH/" "$expected_repo_dir/"   # incremental sync
 else
+    clean-binaries                 # wipe only for flat artifacts
     cp -r "$BINARY_PATH"/* ./
 fi

This is the same pattern that utils/scripts/watch.sh uses — rsync with delete for incremental sync into binaries/.

What it feels like in practice

First run — same as before, full copy:

$ ./build.sh ruby --binary-path ~/code/dd-trace-rb
Copying source repo into binaries/dd-trace-rb/
# ~85MB transferred

Second run, nothing changed — rsync skips everything:

$ ./build.sh ruby --binary-path ~/code/dd-trace-rb
Copying source repo into binaries/dd-trace-rb/
# nothing transferred, finishes instantly

Second run, one file changed — rsync transfers just that file:

$ ./build.sh ruby --binary-path ~/code/dd-trace-rb
Copying source repo into binaries/dd-trace-rb/
# only the changed file transferred

Artifact-based workflows (Java jars, .NET tarballs, etc.) are unchanged — they still get clean-binaries before the flat copy.

Test plan

• Verify first run with --binary-path ~/code/dd-trace-rb does a full copy
• Verify second run with no changes transfers nothing
• Verify second run after modifying a file only transfers that file
• Verify artifact-based tracers still get clean-binaries before flat copy

Generated with Claude Code

[github-actions]

CODEOWNERS have been resolved as:

utils/build/build.sh                                                    @DataDog/system-tests-core

[datadog-prod-us1-3]

⚠️ Tests

✨ Fix all issues with BitsAI or with Cursor

⚠️ Warnings

🧪 166 Tests failed

tests.appsec.test_asm_standalone.Test_SCAStandalone_Telemetry_V2.test_telemetry_sca_enabled_propagated[akka-http] from system_tests_suite (Datadog) (Fix with Cursor)

AssertionError: Missing telemetry config item for 'appsec_sca_enabled'
assert None is not None

self = <tests.appsec.test_asm_standalone.Test_SCAStandalone_Telemetry_V2 object at 0x7fe43dfc6660>

    def test_telemetry_sca_enabled_propagated(self):
        self.assert_standalone_is_enabled(self.r0, self.r1)
    
        configuration_by_name: dict[str, list[dict]] = {}
        for data in interfaces.library.get_telemetry_data():
...

tests.appsec.test_asm_standalone.Test_SCAStandalone_Telemetry_V2.test_telemetry_sca_enabled_propagated[akka-http] from system_tests_suite (Datadog) (Fix with Cursor)

AssertionError: Missing telemetry config item for 'appsec_sca_enabled'
assert None is not None

self = <tests.appsec.test_asm_standalone.Test_SCAStandalone_Telemetry_V2 object at 0x7f45e08066f0>

    def test_telemetry_sca_enabled_propagated(self):
        self.assert_standalone_is_enabled(self.r0, self.r1)
    
        configuration_by_name: dict[str, list[dict]] = {}
        for data in interfaces.library.get_telemetry_data():
...

tests.appsec.test_asm_standalone.Test_SCAStandalone_Telemetry_V2.test_telemetry_sca_enabled_propagated[jersey-grizzly2] from system_tests_suite (Datadog) (Fix with Cursor)

AssertionError: Missing telemetry config item for 'appsec_sca_enabled'
assert None is not None

self = <tests.appsec.test_asm_standalone.Test_SCAStandalone_Telemetry_V2 object at 0x7f79008213a0>

    def test_telemetry_sca_enabled_propagated(self):
        self.assert_standalone_is_enabled(self.r0, self.r1)
    
        configuration_by_name: dict[str, list[dict]] = {}
        for data in interfaces.library.get_telemetry_data():
...

View all

ℹ️ Info

No other issues found (see more)

❄️ No new flaky tests detected

_{This comment will be updated automatically if new data arrives.
🔗 Commit SHA: ea5c9b5 | Docs | Datadog PR Page | Was this helpful? React with 👍/👎 or give us feedback!}