Skip to content

Conversation

Copy link

Copilot AI commented Jan 1, 2026

The benchmark job fails when luarocks cannot load its manifest due to size constraints (65536+ constants error). The test job already handles this with retry logic, but the benchmark job was missing it.

Changes:

  • Added until/do/done retry loops with 1s delays to all luarocks install commands in the benchmark job
  • Matches existing pattern from test job (lines 36-70) for consistency
# Before
luarocks install lualogging

# After  
until luarocks install lualogging
do
  sleep 1
done

Applies to: lualogging, lrexlib-pcre2, luaposix, luasocket, busted, busted-htest

Original prompt

This section details on the original issue you should resolve

<issue_title>[bug] fix CI error</issue_title>
<issue_description>see: https://github.com/casbin/lua-casbin/actions/runs/20631650211/job/59268525165

Run luarocks install lualogging
Warning: Failed searching manifest: Failed loading manifest for [https://luarocks.org:](https://luarocks.org/) Error loading file: [string "/home/runner/.cache/luarocks/https___luarocks..."]:223565: main function has more than 65536 constants
Warning: Failed searching manifest: Failed loading manifest for https://raw.githubusercontent.com/rocks-moonscript-org/moonrocks-mirror/master/: Error loading file: [string "/home/runner/.cache/luarocks/https___raw.gith..."]:223565: main function has more than 65536 constants
Warning: Failed searching manifest: Failed downloading http://luafr.org/moonrocks - Failed downloading http://luafr.org/moonrocks/manifest-5.1 - /home/runner/.cache/luarocks/http___luafr.org_moonrocks/manifest-5.1
Warning: Failed searching manifest: Failed downloading http://luarocks.logiceditor.com/rocks - Failed downloading http://luarocks.logiceditor.com/rocks/manifest-5.1 - /home/runner/.cache/luarocks/http___luarocks.logiceditor.com_rocks/manifest-5.1

Error: No results matching query were found for Lua 5.1.
To check if it is available for other Lua versions, use --check-lua-versions.
Error: Process completed with exit code 1.
```</issue_description>

## Comments on the Issue (you are @copilot in this section)

<comments>
</comments>


✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.

@CLAassistant
Copy link

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.
You have signed the CLA already but the status is still pending? Let us recheck it.

@hsluoyz hsluoyz marked this pull request as ready for review January 1, 2026 13:23
The luarocks manifest can fail to load with "main function has more than 65536 constants" error. This adds the same retry logic to the benchmark job that already exists in the test job.

Co-authored-by: hsluoyz <[email protected]>
Copilot AI changed the title [WIP] Fix CI error related to luarocks installation Fix CI error: add retry logic to benchmark job dependency installation Jan 1, 2026
Copilot AI requested a review from hsluoyz January 1, 2026 13:25
@hsluoyz hsluoyz closed this Jan 1, 2026
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.

[bug] fix CI error

3 participants