search

mdegat01 / addon-promtail

Promtail for Home Assistant
MIT License
15 stars 18 forks source link

Multi-tenant Loki does not work #220

Closed Cougar closed 1 year ago

Cougar commented 1 year ago

Is your feature request related to a problem? Please describe. When multi-tenancy is configure in Loki, its name should be also configured in Promtail

Describe the solution you'd like Add user configurable variable for tenant_id. This should go to the end of base_config.yaml after url parameter.

Describe alternatives you've considered

docker exec -ti addon_39bd2704_promtail bash
echo "    tenant_id: tenantname" >> /etc/promtail/base_config.yaml
exit
docker restart addon_39bd2704_promtail

Additional context I guess that it should be added to the template. Just adding one more line at the end of base_config.yaml could break old Loki configurations. However, I haven't tested if this is a problem with non-tenant Loki server if there will be additional X-Scope-OrgID: header in the HTTP request. Maybe it doesn't matter and the change could be a trivial one line plus one new env variable.

github-actions[bot] commented 1 year ago

There hasn't been any activity on this issue recently, so we clean up some of the older and inactive issues. Please make sure to update to the latest version and check if that solves the issue. Let us know if that works for you by leaving a comment 👍 This issue has now been marked as stale and will be closed if no further activity occurs. Thanks!