Skip to content

Update Geronimo to 4.0.0#205

Merged
jmfinelli merged 2 commits intomainfrom
tomjenkinson-patch-8
May 7, 2026
Merged

Update Geronimo to 4.0.0#205
jmfinelli merged 2 commits intomainfrom
tomjenkinson-patch-8

Conversation

@tomjenkinson
Copy link
Copy Markdown
Member

@tomjenkinson tomjenkinson commented Apr 27, 2026

No description provided.

@marcosgopen
Copy link
Copy Markdown
Member

Re-running the job as the previous execution timed-out without saving the artifacts

@jmfinelli
Copy link
Copy Markdown
Contributor

the previous execution timed out because of the race condition in HOWL. You'll get the same result with the action currently running

@marcosgopen
Copy link
Copy Markdown
Member

the previous execution timed out because of the race condition in HOWL. You'll get the same result with the action currently running

Oh, I see. So the update of Geronimo is not fixing the issue I supposed.

@marcosgopen
Copy link
Copy Markdown
Member

Maybe we can test this PR with a custom setting of THREAD_COUNTS (e.g. 1,12,24) to verify the new version of Geronimo works correctly and then address the HOWL issue in a follow-up PR?

@jmfinelli
Copy link
Copy Markdown
Contributor

I would hold further modifications to this PR while we try multiple parameters here. If we're out of options, then limiting the benchmark to 24 threads could be a good workaround

@jmfinelli jmfinelli force-pushed the tomjenkinson-patch-8 branch from ea56c20 to ea569dd Compare May 5, 2026 13:36
@jmfinelli jmfinelli merged commit 1dd3093 into main May 7, 2026
7 of 8 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants