And kudos <@UB9K6R4JH> for configuring the monitor...
# github-workflows-kt
p
And kudos @LeoColman for configuring the monitoring thanks to which we have nice insights into failures and could quantify the problem!
đŸ«¶ 1
pulled and restarted manually
metrics look much better after the latest fixes
👌 1
stable number of threads:
cache items are kept alive until refresh (that's how I understand it)
👆 1
v
Metrics for the metadata cache are missing 🙂
👍 1
p
@Vampire is the consistency check time reasonable now?
v
I very much hope so 😄
p
Copy code
time .github/workflows/build.main.kts
.github/workflows/build.main.kts:9:1: warning: file 'tecolicom:actions-use-homebrew-tools___major:[v1,v2-alpha)' not found
.github/workflows/build.main.kts:9:1: warning: dependencyCollectionException: Failed to collect dependencies at tecolicom:actions-use-homebrew-tools___major:jar:[v1,v2-alpha)
.github/workflows/build.main.kts  5.43s user 0.99s system 46% cpu 13.887 total
some binding fails to be fetched?
(it's in jEdit's repo)
v
Not found? Why not found suddenly?
Besides that it just executed successfully: https://github.com/jEdit-editor/jEdit/actions/runs/14540692135
First run 03:23. Second run, so warm cache, 00:56. Especially with the cache now staying warm also after an hour this is quite some improvement, yes, assuming it stays like that due to my ktor-client fix.
👍 1
But the question is still what this "not found" is you posted
It's just a warning, but I also do not see it on the GHA execution
That's from the first execution: 😄
p
Ah wait, I think I still had the changed repo to the localhost (I was testing yesterday) and at that moment the server wasn't running. Sorry, false alarm
👌 1
However, I don't understand the graph, why so many cache entries in the were added. Like 100?
v
Remember the description of the prefetch PR
That is exactly what the resolution engine for main.kts does for a version range. It requests the POMs for all those versions in the range for whatever reason.
And because of that the prefetch would make sense to prefetch those in parallel, that they are not generated one after the other but in parallel
p
Oh, now I understand. Version ranges are pretty expensive then
v
With a cold cache, yes
👍 1
p
Got it, ok... I'll think about it - the PR, just give me some more time
👌 1
v
We could persist the current cache entries on disk and refill the cache on server startup with the persisted cache entries, that would be the next performance optimization in that regard 🙂
p
Yes, I thought about it as well
👌 1
Logging off for Easter 👋
👋 1