[Helm] Issue with loki service in singleBinary

[fyannk]

Describe the bug
Fresh install through Helm Chart of single binary is not working when going through deffault service.

Currently we have here:

  selector:
    {{- include "loki.selectorLabels" . | nindent 4 }}

And we should have:

  selector:
    {{- include "loki.singleBinarySelectorLabels" . | nindent 4 }}

Hence, service point to all instances created by Helm Chart, Canary included !

Promtail cannot push logs and Grafana cannot read them

To Reproduce
Fresh install the solution with values:

loki:
  storage:
      type: 'filesystem'
    rulerConfig:
      storage:
        type: local

Expected behavior
Included default service should be working

Environment:

• Infrastructure: Kubernetes
• Deployment tool: helm

Logs

level=error ts=2022-09-27T11:18:17.537024173Z caller=client.go:380 component=client host=loki:3100 msg="final error sending batch" status=404 error="server returned HTTP status 404 Not Found (404): 404 page not found"
level=error ts=2022-09-27T11:18:18.337102338Z caller=client.go:380 component=client host=loki:3100 msg="final error sending batch" status=404 error="server returned HTTP status 404 Not Found (404): 404 page not found"

[jeschkies]

@fyannk thanks. Would you mind creating a PR? You've seemed to nailed down the issue already 🙂

[besteban1989]

I'm running with the same issue, @fyannk your solution works, I updated the kubernetes service manually, also created the PR. @jeschkies can you help us reviewing the PR ?

Thanks.

[besteban1989]

@vlad-diachenko can we have a new helm chart version? If you can add a new version will be very helpful. Thanks.

[vlad-diachenko]

@vlad-diachenko can we have a new helm chart version? If you can add a new version will be very helpful. Thanks.

Hey @besteban1989 . sorry for the delay. yes, I released the new version of helm chart at the moment when I merged the PR... so, you can use the latest chart, the changes are there....