search

grafana / helm-charts

Apache License 2.0
1.55k stars 2.21k forks source link

[tempo-distributed] Cannot create ingester without enabling zone-awareness #3202

Open MicroOps-cn opened 3 days ago

MicroOps-cn commented 3 days ago

@KyriosGN0 In a submission last week, region awareness was added. If region awareness is not enabled, the container for the singer will not be created. The problem is here, It seems that the defaultZone variable is not being used: https://github.com/grafana/helm-charts/blob/main/charts/tempo-distributed/templates/ingester/_helpers-ingester.tpl#L5-L25

KyriosGN0 commented 3 days ago

yes, i have been made aware of this issue, and i opened a PR to fix this https://github.com/grafana/helm-charts/pull/3189

KyriosGN0 commented 3 days ago

@MicroOps-cn fix was merged, tested and it works