Skip to content

CI Bug: wrong cache key for no-std test coverage #2060

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
JulianGCalderon opened this issue Apr 7, 2025 · 0 comments · Fixed by #2093
Closed

CI Bug: wrong cache key for no-std test coverage #2060

JulianGCalderon opened this issue Apr 7, 2025 · 0 comments · Fixed by #2093
Labels
bug Something isn't working pipelines This PR/issue is exclusively about improving our CI

Comments

@JulianGCalderon
Copy link
Contributor

When trying to fetch the test coverage cache for no-std tests, the wrong key is used, which causes the step to fail.

Error: Failed to restore cache entry. Exiting as fail-on-cache-miss is set. Input key: codecov-cache-test-no_std--...

This happens because the tests are partitioned into:

  • codecov-cache-test-no_std#1
  • codecov-cache-test-no_std#2
  • codecov-cache-test-no_std#3
  • codecov-cache-test-no_std#4

But the upload coverage workflow tries to restore it as if it were a single partition:

  • codecov-cache-test-no_std

This applies to:

  • codecov-cache-test-no_std-extensive_hints
  • codecov-cache-test-no_std-mod_builtin

We should change the upload workflow to take it into account and restore it as separate partitions.

links:

@JulianGCalderon JulianGCalderon added bug Something isn't working pipelines This PR/issue is exclusively about improving our CI labels Apr 7, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working pipelines This PR/issue is exclusively about improving our CI
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant