-
Notifications
You must be signed in to change notification settings - Fork 37
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
lua_code_cache off not working #14
Comments
I reckon this is caused by missing musl libc in alpine linux |
Yeah, Alpine version has problems with That's why I recommended using Debian flavor in the Usage during development section. Probably I need to emphasize it more in the readme. That said, we've had no issues caused by musl libc in production. |
The only real fix I see now is to build OpenResty with glibc in Alpine flavor too, but that would substantially increase the image size. I'm not sure it's worth it, given that you can use Debian version for development. |
Thanks for quick reply @skozin. Making it a bit more clear in the documentation would be nice. But also i did run into issues on production. The logs showed me syntax error but the script was correct though. It even popped up with errors on lines which didn't exist. |
So you're not using |
Sorry for late reply. Yes exactly. Also sometimes I run into syntax errors altough the syntax is correct. If i switch to debian it is working correctly |
I inherited from your alpine version dockerfile and ran into issues.
Setting up a new container works as expected, also it uses my custom nginx.conf
nginx.conf
example.lua
ngx.print('<H1>Hello World.</H1>');
However. Setting lua_code_cache to off won't work on alpine linux. The usual warning
won't come up as expected. Also, if i change the code in example.lua it won't show me the new content but it comes up with an syntax exception in the logs and crashes. Although the syntax i am using is fine.
If i switched to the debian version and it all works as expected. Something seems to be broken with alpine linux version.
The text was updated successfully, but these errors were encountered: