andrewfraley / arris_cable_modem_stats

Retrieves stats from Arris cable modems and sends to InfluxDB
50 stars 33 forks source link

influxdb client to be 2.x #36

Closed cbschuld closed 2 years ago

cbschuld commented 2 years ago

updates / modifications

influxdb client to the latest 2.x python image to 3.10-alpine (unnecessary maintenance) write modes changed to use the updated client calls

breaking changes in settings

influx_host, influx_database, influx_port, influx_username, influx_password and influx_use_ssl are dropped influx_org, influx_url, influx_bucket and influx_token added

also fixes #35

andrewfraley commented 2 years ago

@cbschuld Thank you for this! I was planning on adding support for InfluxDB 2.x as a separate storage provider so folks could continue to use Influx 1.x, myself included as I'm not ready to switch all my other integrations yet. Sometime soon I'll take a look at this and see what we can do. For now I'll leave this PR open so folks can use your fork if they want immediate 2.x support. Thanks!

cbschuld commented 2 years ago

@andrewfraley makes sense; happy to help with that as well

andrewfraley commented 2 years ago

Added support for InfluxDB 2.x in release v1.6.0. @cbschuld thanks for the original PR for this. I implemented it so that we could support both 1.x and 2.x, but used some of your code for arris_stats_influx2.py. Thanks!

cbschuld commented 2 years ago

you are welcome Andrew; thanks for the kudos