splunk / splunk-connect-for-snmp

Splunk connect for SNMP

Home Page:https://splunk.github.io/splunk-connect-for-snmp/

Geek Repo:Geek Repo

Github PK Tool:Github PK Tool

helm chart - traps service should have support for annotations

minhnhupham opened this issue · comments

The Traps service which is created by the helm chart should support annotations so various settings can be configured on different k8s platforms. (e.g. GCP GKE, Azure AKS, AWS EKS, etc...)
Right now there's no way to tweak various settings using annotations for the Trap service that gets created.
https://github.com/splunk/splunk-connect-for-snmp/blob/main/charts/splunk-connect-for-snmp/templates/traps/service.yaml

e.g. If SC4SNMP was deployed on a GKE cluster the traps service would be configured with an external loadbalancer with an external public internet IP address, this may/may not be a desired configuration, but right now there's no way to tweak this with annotations.
An example of this:
https://cloud.google.com/kubernetes-engine/docs/how-to/internal-load-balancing#create

Annotation support actually should be added to all other services in the helm chart as well, since this appears to be missing.

commented

Hello, the change you requested was released in 1.8.2-beta.1. We added two ways of configuring annotations:

  1. under the service
  2. as a common annotation, adds annotation to every service

You can test it using upgrading to specific version instruction. Please let me know if it solves the issue, then it will be included in the main release.
Thanks!

Great thanks.
We've this for the traps service and seems to work correctly.