Skip to content
This repository has been archived by the owner on Feb 26, 2024. It is now read-only.

Update for Influx v2 #12

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

DragonQ
Copy link

@DragonQ DragonQ commented Mar 10, 2021

This commit adds support for InfluxDB v2 and Flux queries. InfluxDB v1 with InfluxQL queries should still work, although I haven't tested this.

InfluxDB v2 is the default option in the example settings ini file because the included docker-compose example uses that version.

@antwilliams
Copy link

@DragonQ - thanks for doing this, one comment is that you missed adding influxdb-client to the requirements.txt file and it is needed for v2 connectivity.

I think it would also be good if you added the org= field in the example-octograph.ini field.

I don't know if this is likely to be merged in any time soon but I am using your branch directly at the moment along with the flux based dashboard.

@DragonQ
Copy link
Author

DragonQ commented Nov 30, 2021

@DragonQ - thanks for doing this, one comment is that you missed adding influxdb-client to the requirements.txt file and it is needed for v2 connectivity.

I think it would also be good if you added the org= field in the example-octograph.ini field.

Done, thanks for the feedback. I have another branch that adds the "tracker" tariff to the mix but it's based off this branch and I haven't tested it off master.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants