Ingress-nginx: lua module in nginx-slim

Created on 28 Sep 2017 · 5Comments · Source: kubernetes/ingress-nginx

Would you consider putting the lua module back into nginx-slim?

I need to add some custom lua code and I'm wondering if I need to create a custom nginx-slim or if we can add it back here.

Source

WilliamDurin

Most helpful comment

@alonl for now just fork and add back the lua build https://github.com/kubernetes/ingress-nginx/commit/95b04ff9eb5b105ad21e7bbfd89d0a83574c76e2#diff-c1fe8869a51a706788a14475a7c342c1

aledbf on 23 Oct 2017

👍2

Closing. Adding lua to nginx make the image not supported in several platforms.

aledbf on 28 Sep 2017

Too bad :(, we were using it too. @aledbf do you have any suggestion, maybe, of how to add it back, or fork and build our own is the only way?

alonl on 23 Oct 2017

@alonl for now just fork and add back the lua build https://github.com/kubernetes/ingress-nginx/commit/95b04ff9eb5b105ad21e7bbfd89d0a83574c76e2#diff-c1fe8869a51a706788a14475a7c342c1

aledbf on 23 Oct 2017

👍2

@alonl just a question, is your fork somewhere public? I am needing similar stuff now...

All the best, Sebastian

skurfuerst on 22 Nov 2017

@skurfuerst I pushed it to here for now. It's just the build script modification. The steps from there:

Build your own nginx base image (run export REGISTRY=<YOUR_REGISTRY>; make build container push from within images/nginx).
Change BASEIMAGE?=quay.io/kubernetes-ingress-controller/nginx-$(ARCH):0.29 in Makefile to your newly created base image.
Run export REGISTRY=<YOUR_REGISTRY>; make build container push in the root directory to build nginx-ingress-controller image.