search

honeycombio / helm-charts

Helm repository and charts for Honeycomb
Apache License 2.0
30 stars 39 forks source link

fix: update honeycomb templates to use release namespace #315

Closed fchikwekwe closed 11 months ago

fchikwekwe commented 11 months ago

Which problem is this PR solving?

This updates the honeycomb helm chart to use release namespace just like the collector helm chart did.

Tested changes locally to ensure that namespace changes work as intended. 

➜  kubectl config set-context --current --namespace=default    
Context "kind-kind" modified.
➜  helm template ./charts/opentelemetry-collector | grep -i namespace
WARNING: This chart is deprecated
  namespace: default
  namespace: default
  namespace: default
  namespace: default
  namespace: default
➜ kubectl config set-context --current --namespace=kube-public
Context "kind-kind" modified.
➜  helm template ./charts/opentelemetry-collector | grep -i namespace
WARNING: This chart is deprecated
  namespace: kube-public
  namespace: kube-public
  namespace: kube-public
  namespace: kube-public
  namespace: kube-public