Skip to content

Commit 91ee098

Browse files
committed
CI: caching: upd doc
1 parent 85b51c4 commit 91ee098

File tree

1 file changed

+2
-2
lines changed

1 file changed

+2
-2
lines changed

.github/workflows/caching.yml

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,8 +1,8 @@
11
name: Caching
22

33
# Docs on the workflow:
4-
# 1. GitHub cache scoping goes: main branch -> PR main branch -> PR. (essentially in a tree fashion). That is why it is useful to build caches on `master` - to generate & keep the main project state & they would be shared to the whole tree.
5-
# 2. GitHub has a 10G default limit of cache pool per repo. HLS is a big project & monorepo of many projects, so to keep cache useful - the main branch state caches should be preserved & their storage preferred to the PRs, since PRs from internal branches - would count into 10G pool, but would be available only inside of the PR scope, which can bork cache fore the whole community. That is short story why `dist-newstyle` (especially full) - is not includded into `master` cache.
4+
# 1. GitHub cache scoping goes: main branch -> PR main branch -> PR. (essentially in a tree fashion). Building & keeping caches on `master` allows to the main project state cache be shared to the whole tree.
5+
# 2. GitHub has a 10G as default limit of cache pool per repo. HLS is a big community project & monorepo of many projects, so cache pool can get exhausted & invaidated easilly. Because of that, to keep cache useful - the main branch state caches should be preserved & kept well below the limit & that preferred to the PR caches, since PRs from internal branches - would count into the repo 10G pool, while PR cache is available only inside of the PR, which would bork cache fore the whole community. That is a short story why `dist-newstyle` (especially full) - is not includded into `master` cache.
66

77
defaults:
88
run:

0 commit comments

Comments
 (0)