Added both lua-resty-http lua plugin and Crowdsec-Openresty-Bouncer - #7
Conversation
Required for crowdsec-openresty-bouncer
|
Docker Image for build 1 is available on DockerHub as:
|
|
Docker Image for build 2 is available on DockerHub as:
|
Config files and templates are saved in /defaults/crowdsec A script in the main docker image will handle deployment.
LePresidente
changed the title
Uninstall gettext since its not required after the crowdsec install.
|
This is failing because you also need to see the version env var in the Jenkinsfile |
|
I think i got all the environment variables, not sure what i'm missing |
|
The build shows that the version is still not coming through to the install script, but I can't see why not. Very strange, I'm looking into it |
| GREEN='\E[1;32m' | ||
| RESET='\E[0m' | ||
|
|
||
| echo -e "${BLUE}❯ ${CYAN}Installing Crowdsec OpenResty Bouncer ${YELLOW}${CROWDSEC_OPENRESTY_BOUNCER_VERSION}...${RESET}" |
There was a problem hiding this comment.
Use ${CROWDSEC_OPENRESTY_BOUNCER_VERSION:-} instead of ${CROWDSEC_OPENRESTY_BOUNCER_VERSION} so that the variable falls back to empty, it will be checked with my comment below
|
|
||
| bash ./install.sh --NGINX_CONF_DIR=/etc/nginx/conf.d --LIB_PATH=/etc/nginx/lualib --CONFIG_PATH=/defaults/crowdsec --DATA_PATH=/defaults/crowdsec --docker | ||
| sed -i 's|ENABLED=.*|ENABLED=false|' /defaults/crowdsec/crowdsec-openresty-bouncer.conf | ||
| rm /tmp/crowdsec |
There was a problem hiding this comment.
this should be
rm -rf /tmp/crowdsec
|
The CI build failure was due to some restrictive permissions for changes by users outside of this project, that has been fixed. It's getting the variable now, but still failing, see my latest comments on the files. |
|
Thanks for the help trying to debug this, hopefully its sorted now |
|
Yep it's working, the failure is in an extended golang image. Go just released a new version and this is causing problems with some packages. I'll merge this anyway, the main images will push fine regardless. |
|
I had to disable this here as the v2.9.17 release of NginxProxyManager that used this change caused out of control memory leak. It was reported by some users and then when I used that image it froze my server after climbing past 3gb. The v2.9.17 docker tag is still available for debugging. |
|
Ok i'll have a look and get back to you |
|
So either my server doesn't have enough hosts configurator or activity to catch this. I've gone through the code and can't find anything obvious causing the issue. Does the memory leak happen at startup or over time when accessing the server? |
|
At startup. After nginx reloads as part of startup it takes 20secs for my server to freeze. I haven't tested with a clean slate though |
|
Ok I see the spike it does clear itself on my server but something is definitely wrong. Will revert back when i find the issue |
2 participants
Added both lua-resty-http lua plugin and Crowdsec-Openresty-Bouncer