Skip to content

ci: scope conan cache key by build type in build_debug.yml#776

Merged
leoparente merged 1 commit into
developfrom
chore/cache-debug-conan-separately
May 18, 2026
Merged

ci: scope conan cache key by build type in build_debug.yml#776
leoparente merged 1 commit into
developfrom
chore/cache-debug-conan-separately

Conversation

@leoparente
Copy link
Copy Markdown
Contributor

Summary

  • The Debug code-coverage and build jobs in build_debug.yml used the same conan cache key as the Release pipeline (conan-${{ runner.os }}-amd64-<hash>).
  • actions/cache@v5 saves only the first cache populated under a given key; the permissive restore-keys: conan-${{ runner.os }}-amd64- fallback then let Debug runs restore a Release-populated build/p/ that lacks debug-built dependencies.
  • Result: every Debug CI run rebuilt third-party deps from source instead of pulling them from cache.
  • Fix: tag the Debug primary key and restore-keys with -debug- so the two pipelines maintain independent caches.

Test plan

  • First post-merge Debug run: expect a cache miss (new key), full conan build.
  • Subsequent Debug runs: expect a cache hit and no source rebuild of conan packages.
  • Release (build-develop.yml) cache behavior unchanged.

The Debug workflow shared a conan cache key with the Release build
(`conan-${{ runner.os }}-amd64-<hash>`). Only one job saves a given key,
and the permissive `restore-keys` fallback let Debug pull in a Release-
populated cache that doesn't contain its debug-built dependencies — so
every Debug run rebuilt deps from source.

Tag the Debug cache key (and matching restore-keys) with `-debug-` so
the two builds maintain independent caches.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
@leoparente
Copy link
Copy Markdown
Contributor Author

@codex review

@leoparente leoparente marked this pull request as ready for review May 18, 2026 18:45
@leoparente leoparente self-assigned this May 18, 2026
@github-actions
Copy link
Copy Markdown

LCOV of commit 7045a0a during Debug Builds #120

  lines......: 81.1% (13546 of 16698 lines)
  functions..: 72.4% (1396 of 1929 functions)
  branches...: no data found

Files changed coverage rate: n/a

Full coverage report

@leoparente leoparente merged commit becd76a into develop May 18, 2026
18 checks passed
@leoparente leoparente deleted the chore/cache-debug-conan-separately branch May 18, 2026 19:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants