forgejo/models/git/TestIterateRepositoryIDsWithLFSMetaObjects/lfs_meta_object.yaml at 99bd29f02f7ae3f8f6c99299a2aea2f83aeb86d4 - fedi/forgejo - git.minimally.online

fedi/forgejo

mirror of https://codeberg.org/forgejo/forgejo.git synced 2024-12-17 22:57:44 +00:00

Gusted aa7346c007 Refactor LFS GC functions

- Remove options that currently aren't set
on `GarbageCollectLFSMetaObjectsOptions` and
`IterateLFSMetaObjectsForRepoOptions`.
- Simplify `IterateRepositoryIDsWithLFSMetaObjects` and
`IterateLFSMetaObjectsForRepo`.
- `IterateLFSMetaObjectsForRepo` was previously able to get in a
loop (`gc-lfs` doctor check was able to reproduce this) because the code
expected that the records would be updated to not match the SQL query,
but that wasn't the case. Simply enforce that only records higher than
the latest `id` from the previous iteration are allowed.
- For `gc-lfs` doctor check this was because `UpdatedLessRecentlyThan`
option was not set, which caused that records just marked as active in
the iteration weren't being filtered.
- Add unit tests
- Most likely a regression from 2cc3a6381c.
- The bug with `gc-lfs` was found on Codeberg.

(cherry picked from commit 7ffa7f5bce)

2024-04-06 07:41:40 +00:00

8 lines

143 B

YAML

Raw Blame History

 -
   id: 1000
   oid: 9d172e5c64b4f0024b9901ec6afe9ea052f3c9b6ff9f4b07956d8c48c86fca82
   size: 25
   repository_id: 1
   created_unix: 1712309123