Skip to content

Commit 09a1d9e

Browse files
committed
Add note about cache eviction
1 parent f64a449 commit 09a1d9e

1 file changed

Lines changed: 7 additions & 0 deletions

File tree

src/overlay/caching.ts

Lines changed: 7 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -439,6 +439,13 @@ async function getCacheKeyPrefixBase(
439439
* Searches the GitHub Actions cache for overlay-base databases matching the given languages, and
440440
* returns all stable CodeQL versions found across matching cache entries.
441441
*
442+
* Note that we do not guarantee that the cache entry for these versions of CodeQL will still be
443+
* present by the time we attempt to restore the cache. We could achieve that with a download retry
444+
* loop, but we expect that if there is sufficient Actions cache contention that an overlay-base
445+
* cache entry for a particular CodeQL version is evicted before we can use it, then it is likely
446+
* that the same thing will happen to other overlay-base cache entries, and therefore we will not be
447+
* able to use overlay.
448+
*
442449
* @returns Unique stable CodeQL versions found in cached overlay-base databases, sorted from latest to
443450
* earliest, or undefined if one of the languages is not a built-in language.
444451
*/

0 commit comments

Comments
 (0)