p4lang / p4runtime

Specification documents for the P4Runtime control-plane API
Apache License 2.0
144 stars 88 forks source link

Improve CI build times through smarter caching #335

Closed smolkaj closed 3 years ago

smolkaj commented 3 years ago

GitHub's caching mechanism never overwrites cache entries for existing keys. Since we were using a static key, this caused our cache to never be refreshed. With this PR, a new cache entry is generated whenever any Bazel files (including dependencies) change.

antoninbas commented 3 years ago

@smolkaj could you rebase the PR so that the CI Bazel build can pass?

smolkaj commented 3 years ago

I rebased, now it builds. Ready to be merged.