-
Notifications
You must be signed in to change notification settings - Fork 74
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
fix: missing env var INFLUXDB_URL #1487
Conversation
@ackintosh thanks for sharing! Could you rebase on the latest master? I'll trigger another CI run and make sure we review this soon. |
The sidecar shows the warning message on console: MESSAGE << sidecar >> InfluxDB unavailable; no metrics will be dispatched: no InfluxDB URL in $INFLUXDB_URL env var
I have rebased this on the latest master branch. 😃 |
I was holding it wrong, patch works :) Got some metrics in :) |
can we merge this please? Move the bytes working group is already relying on this patch |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lgtm, thanks for sharing this fix, I'll merge today.
Merged, thanks for the fix @ackintosh ! And thanks for the heads up @b5 , @Arqu |
This PR fixes #1428.