Feature: Allow using mounted MaxMind GeoIP databases

[mihau]

Currently nginx ingress controller requires providing both maxmind-license-key and maxmind-edition-ids in order for GeoIP databases to be enabled (using the default config template). According to the docs one should be able to just provide the database files (by mounting, or embedding into the image) to use it, however this is not the case currently.

Proposed solution

This could be solved by moving this call to GeoLite2DBExists.
This should allow for providing only maxmind-edition-ids and have the ingress controller load the necessary files if they are available.

/kind feature

[fejta-bot]

Issues go stale after 90d of inactivity.
Mark the issue as fresh with /remove-lifecycle stale.
Stale issues rot after an additional 30d of inactivity and eventually close.

If this issue is safe to close now please do so with /close.

Send feedback to sig-testing, kubernetes/test-infra and/or fejta.
/lifecycle stale

[fejta-bot]

Stale issues rot after 30d of inactivity.
Mark the issue as fresh with /remove-lifecycle rotten.
Rotten issues close after an additional 30d of inactivity.

If this issue is safe to close now please do so with /close.

Send feedback to sig-testing, kubernetes/test-infra and/or fejta.
/lifecycle rotten

[fejta-bot]

Rotten issues close after 30d of inactivity.
Reopen the issue with /reopen.
Mark the issue as fresh with /remove-lifecycle rotten.

Send feedback to sig-testing, kubernetes/test-infra and/or fejta.
/close

[k8s-ci-robot]

@fejta-bot: Closing this issue.

In response to this:

Rotten issues close after 30d of inactivity.
Reopen the issue with /reopen.
Mark the issue as fresh with /remove-lifecycle rotten.

Send feedback to sig-testing, kubernetes/test-infra and/or fejta.
/close

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

[kd7lxl]

Since the docs indicate this is supported, it should be marked /kind bug, at least until removed from the docs.