RFR: 2063: Reduce the health check times of local repo instance [v3]
Zhao Song
zsong at openjdk.org
Thu Oct 12 23:29:36 UTC 2023
> In the method HostedRepositoryPool#materializeClone, when the bot is trying to reuse a local repo instance, the bot will always check whether the repo is still good by processing command "git fsck --connectivity-only".
>
> Sometimes this command would be slow and Erik said that we should believe the bots are doing right things, so we should assume the local repos are good, so the health check is not always needed. But shutdown of the bot could make a local repo instance unhealthy, so we should at least do the health check once for each local repo instance after the bot is restarted.
>
> To solve this issue, Erik suggested to maintain a static map for keeping track of paths of known checked/good repositories.
>
> To leverage this patch to mlbridge bot, I also made some changes to WebrevStorage.
Zhao Song has updated the pull request incrementally with one additional commit since the last revision:
update
-------------
Changes:
- all: https://git.openjdk.org/skara/pull/1569/files
- new: https://git.openjdk.org/skara/pull/1569/files/e58482b0..0722a701
Webrevs:
- full: https://webrevs.openjdk.org/?repo=skara&pr=1569&range=02
- incr: https://webrevs.openjdk.org/?repo=skara&pr=1569&range=01-02
Stats: 3 lines in 1 file changed: 2 ins; 0 del; 1 mod
Patch: https://git.openjdk.org/skara/pull/1569.diff
Fetch: git fetch https://git.openjdk.org/skara.git pull/1569/head:pull/1569
PR: https://git.openjdk.org/skara/pull/1569
More information about the skara-dev
mailing list