-
Notifications
You must be signed in to change notification settings - Fork 1
add support for geolocation #7
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
Conversation
alix-graylog
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This looks good, thanks for adding this!
| apiVersion: v1 | ||
| kind: Secret | ||
| metadata: | ||
| name: {{ $geoSecretName }} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is this for the Maxmind api key?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
yup! Both api key and account id
graylog/values.yaml
Outdated
| protocol: TCP | ||
| plugins: | ||
| myPluginA: | ||
| url: "https://someurl/myplugin.jar" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is this to allow for downloading of 3rd party plugins?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
yes, but this values schema has been updated. See #6 for more details
|
@monrax can you please address Alix's questions above when you get a chance? |
|
I just updated this branch, and rebased on top of This PR introduces support for the Graylog Geolocation Processor by fetching the to perform the upgrade. Another method is also available: given the URL of an .mmdb file available in a remote location, and a SHA256 hash of that file, by specifying both the Both methods have been tested in both microk8s and EKS. |
|
Turning this into a draft PR until #6 is merged. |
alix-graylog
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I was able to test this one. Everything from what I can see is working as expected.
Add support for geolocation through: