-
-
Notifications
You must be signed in to change notification settings - Fork 1.8k
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
GEOIP_PATH_MMDB support #310
Comments
We were planning to add support for this in the on-premise repo but the very recent account/license requirements may make it harder than we want it to. Still, https://dev.maxmind.com/geoip/geoipupdate/#Direct_Downloads may help you get started. What you need to do is download this file, put/copy it into a volume and then mount that volume to Sentry containers. |
Sure, but can I point to that file with an environment variable? |
Ah scratch what I said above, sorry. You'd need to set it in your |
Right, so would you consider adding that as a default so that one doesn't need to customize it? |
Yes, it can be a part of the automation work we intend to do around this. It won't happen very soon though. If you are interested, I can assist you creating a pull request and getting it merged rather than waiting for us. |
@BYK I just tried to create a volume and add it to the installation, but I'm getting this:
I'm attaching the volume with Am I doing something wrong? The permissions seems fine:
|
You just need to add this to |
Relevant: sentry-kubernetes/charts#92 |
How should one configure GEOIP_PATH_MMDB for the docker image?
The text was updated successfully, but these errors were encountered: