actions
/
stale
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Merge pull request #1080 from akv-platform/fix-delete-cache 

Fix delete cache
This commit is contained in:
Marko Zivic 2023-08-24 09:21:05 +02:00 committed by GitHub
parent 184e7afe93 8986f6218b
commit 796531a7b3
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
3 changed files with 16 additions and 6 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
README.md
View File

@ -29,8 +29,8 @@ You can find more information about the required permissions under the correspon
## Statefulness ## Statefulness
If the action ends because of [operations-per-run](#operations-per-run) then the next run will start from the first If the action ends because of [operations-per-run](#operations-per-run) then the next run will start from the first
unprocessed issue skipping the issues proceeded during the previous run(s). The state is reset when all the issues are unprocessed issue skipping the issues processed during the previous run(s). The state is reset when all the issues are
proceeded. This should be considered for scheduling workflow runs. processed. This should be considered for scheduling workflow runs.
The saved state lifetime is the same as the The saved state lifetime is the same as the
[actions cache](https://docs.github.com/en/actions/using-workflows/caching-dependencies-to-speed-up-workflows#usage-limits-and-eviction-policy) [actions cache](https://docs.github.com/en/actions/using-workflows/caching-dependencies-to-speed-up-workflows#usage-limits-and-eviction-policy)

7
dist/index.js vendored
View File

@ -1636,7 +1636,7 @@ const resetCacheWithOctokit = (cacheKey) => __awaiter(void 0, void 0, void 0, fu
} }
catch (error) { catch (error) {
if (error.status) { if (error.status) {
core.debug(`Cache ${cacheKey} does not exist`); core.warning(`Error delete ${cacheKey}: [${error.status}] ${error.message || 'Unknown reason'}`);
} }
else { else {
throw error; throw error;
@ -1650,7 +1650,10 @@ class StateCacheStorage {
const filePath = path_1.default.join(tmpDir, STATE_FILE); const filePath = path_1.default.join(tmpDir, STATE_FILE);
fs_1.default.writeFileSync(filePath, serializedState); fs_1.default.writeFileSync(filePath, serializedState);
try { try {
yield resetCacheWithOctokit(CACHE_KEY); const cacheExists = yield checkIfCacheExists(CACHE_KEY);
if (cacheExists) {
yield resetCacheWithOctokit(CACHE_KEY);
}
const fileSize = fs_1.default.statSync(filePath).size; const fileSize = fs_1.default.statSync(filePath).size;
if (fileSize === 0) { if (fileSize === 0) {
core.info(`the state will be removed`); core.info(`the state will be removed`);

11
src/classes/state/state-cache-storage.ts
View File

@ -54,7 +54,11 @@ const resetCacheWithOctokit = async (cacheKey: string): Promise<void> => {
); );
} catch (error) { } catch (error) {
if (error.status) { if (error.status) {
core.debug(`Cache ${cacheKey} does not exist`); core.warning(
`Error delete ${cacheKey}: [${error.status}] ${
error.message || 'Unknown reason'
}`
);
} else { } else {
throw error; throw error;
} }
@ -67,7 +71,10 @@ export class StateCacheStorage implements IStateStorage {
fs.writeFileSync(filePath, serializedState); fs.writeFileSync(filePath, serializedState);
try { try {
await resetCacheWithOctokit(CACHE_KEY); const cacheExists = await checkIfCacheExists(CACHE_KEY);
if (cacheExists) {
await resetCacheWithOctokit(CACHE_KEY);
}
const fileSize = fs.statSync(filePath).size; const fileSize = fs.statSync(filePath).size;
if (fileSize === 0) { if (fileSize === 0) {