Skip to content

lua entry thread aborted: runtime error #96

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
Zoey2936 opened this issue Feb 24, 2025 · 7 comments · Fixed by #97
Closed

lua entry thread aborted: runtime error #96

Zoey2936 opened this issue Feb 24, 2025 · 7 comments · Fixed by #97

Comments

@Zoey2936
Copy link

With the latest NPMplus release from today:

[error] 4622#4622: *17 lua entry thread aborted: runtime error: /usr/local/nginx/lib/lua/crowdsec.lua:173: Failed to create the timer: process exiting

stack traceback:

coroutine 0:

	[C]: in function 'error'

	/usr/local/nginx/lib/lua/crowdsec.lua:173: in function 'Setup_metrics_timer'

	/usr/local/nginx/lib/lua/crowdsec.lua:183: in function </usr/local/nginx/lib/lua/crowdsec.lua:169>, context: ngx.timer, client: <ip>, server: <ip>:443

There seems to be an error in the main branch here. NPMplus is built from the last tagged release if the nginx bouncer repo, but that repo uses the latest commit from here

@Giuseppe-Sciarra
Copy link

the problem is in nmp repo then? so we have to wait for the main repo is updated/fixed?

@blotus
Copy link
Member

blotus commented Feb 24, 2025

Hello,

How did the error happen exactly ?

What is during a normal start of nginx ? Or during a reload/shutdown ?

The error reason (process exiting) seems to suggest nginx was shutting down while we tried to create/recreate the timer.

Also @Zoey2936, we have started to tag versions of the lua lib, so in the future (starting with the next release), new bouncer versions will use a specific version of this repo.

In the meantime, if you are using the Makefile from the nginx bouncer repo to build it, you can set LUA_BOUNCER_BRANCH to 175231e163d3770fdefdaaa355d96996b11ada8f, it should then use the last commit before we added support for metrics in the bouncer.

@Zoey2936
Copy link
Author

NPMplus reloads nginx shortly after starting nginx, then the error is shown. Appsec and crowdsec at least logged that they are initialized. Will move to the commit tomorrow to fix it temporary

@Zoey2936
Copy link
Author

https://github.com/ZoeyVid/NPMplus/actions/runs/13516142052/job/37765101999 sadly the git -b option only allows branches/tags as input, not commits shas, can you maybe create a tag for this commit, please?

@blotus
Copy link
Member

blotus commented Feb 25, 2025

I've just created a before-metrics tag.

@Zoey2936
Copy link
Author

Thanks!

@Zoey2936

This comment has been minimized.

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 a pull request may close this issue.

3 participants