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

influxdb metrics client - update to 2.0 #8633

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

drahnr
Copy link
Contributor

@drahnr drahnr commented Nov 28, 2022

Updates influxdb to 2.0 so it works with influx cloud again

Closing #8632

@drahnr drahnr requested a review from a team as a code owner November 28, 2022 16:41
@drahnr drahnr changed the title influx db influxdb metrics client - update to 2.0 Nov 28, 2022
@drahnr drahnr force-pushed the bernhard-influx-db branch 9 times, most recently from c18aefe to ee68912 Compare December 7, 2022 14:43
Signed-off-by: Bernhard Schuster <bernhard@ahoi.io>
Signed-off-by: Bernhard Schuster <bernhard@ahoi.io>
Signed-off-by: Bernhard Schuster <bernhard@ahoi.io>
@drahnr drahnr force-pushed the bernhard-influx-db branch from ee68912 to 6cff5df Compare December 7, 2022 14:43
@taylorsilva
Copy link
Member

@drahnr Very old, but if you're still interested in getting this in it looks like more needs to be updated to accommodate this fix. Seeing this error when trying to run the unit tests:

Failed to compile atccmd:


# github.com/concourse/concourse/atc/metric/emitter

atc/metric/emitter/influxdb_client.go:37:13: undefined: Options

If you don't have time to work on this then feel free to close the PR. Based on the lack of engagement with this PR and the related issue, I have a feeling not a lot of people out there are using Concourse with Influxdb.

@taylorsilva
Copy link
Member

If someone else wants to pick up this PR, feel free to open a new PR with your changes!

@drahnr
Copy link
Contributor Author

drahnr commented Dec 5, 2024

I won't have capacity in the foreseeable future to complete the work.

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.

2 participants