Skip to content
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

[promtail] support multiple clients via clients #1425

Merged
merged 1 commit into from
Jun 1, 2022
Merged

[promtail] support multiple clients via clients #1425

merged 1 commit into from
Jun 1, 2022

Conversation

dungdm93
Copy link
Contributor

@dungdm93 dungdm93 commented May 30, 2022

@dungdm93 dungdm93 changed the title [charts/promtail] support multiple clients via clientConfigs [promtail] support multiple clients via clientConfigs May 30, 2022
@dungdm93 dungdm93 changed the title [promtail] support multiple clients via clientConfigs [promtail] support multiple clients via clients May 30, 2022
Signed-off-by: Đặng Minh Dũng <dungdm93@live.com>
Copy link
Contributor

@trevorwhitney trevorwhitney left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks, I like this approach! LGTM

@damyan
Copy link

damyan commented Jun 3, 2022

I guess https://github.com/grafana/helm-charts/blob/main/charts/promtail/README.md.gotmpl#L155-L156 should have been adjusted as well, shouldn't they?

bors-openebs-mayastor bot pushed a commit to openebs/mayastor-extensions that referenced this pull request Dec 27, 2023
398: fix(chart): promtail subchart changes for chart v6.13.1 r=niladrih a=niladrih

The `lokiAddress` key is no longer useful with the new loki-stack helm chart. Clients are not input into the `clients[]` array.
Ref: grafana/helm-charts#1425

Co-authored-by: Niladri Halder <niladri.halder26@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants