-
Notifications
You must be signed in to change notification settings - Fork 57
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
create a healthcheck #38
Comments
this has not been updated because the entrypoint exec's the openvpn binary and it rarely if ever crashes. I would consider a readiness check but i'm looking for input from users. |
I made a sidecar with a few lines of Go, not because it helps verify the VPN server is responding, but because the AWS LB health checks otherwise continuously generated a lot of noise in the VPN server logs. |
Sharing is caring :). I'm sure it's pretty simple but i might incorporate it (i do too hate the flooded logs from ELB healthchecks) |
I'll send you a PR over the weekend! Also, just fyi, maybe you're interested in this commit as well? Say you have an RDS database attached to your k8s VPC that isn't publicly accessible, you won't be able to connect to it over VPN unless you also route to the VPC cidr. |
Ah, i was actually in the process of removing specific envs for routes (as you can stick them into |
Ah, ofc, that makes a lot of sense! |
Preferably accessible over HTTP but otherwise
exec
ableThe text was updated successfully, but these errors were encountered: