search

sparky8512 / starlink-grpc-tools

Random scripts and other bits for interacting with the SpaceX Starlink user terminal hardware
The Unlicense
482 stars 64 forks source link

Status works but write to influx returns "No host specified" #49

Closed SkyMoCo closed 2 years ago

SkyMoCo commented 2 years ago

python3 dish_grpc_text.py -s 2 status works returns correctly with current data, but dish_grpc_influx2 returns an error.

I don't see any option to specify a host and the url works just perfect.

python3 dish_grpc_influx2.py -v -u "localhost:8086" -T "" -B StarLink -O BVRanch -s 2 status Data points queued: 1 Data points queued: 1 Data points written: 1 ERROR: The batch item wasn't processed successfully because: No host specified.

SkyMoCo commented 2 years ago

My error. I was looking at the influx 1.0 examples and did not put a proper url in the url option. don't forget the "http://..."

sparky8512 commented 2 years ago

Hmm... Does it work if you specify an actual URL (http://localhost:8086)? I'm not sure why I thought that accepted either a URL or a host/port, I may have been looking at old documentation.

sparky8512 commented 2 years ago

Lol. Should have refreshed before I commented. This is still a bug, as the help text does indicate it will take a host/port, and the default should be changed to a proper URL.

SkyMoCo commented 2 years ago

What about also allowing at config file? gstate.influx_client = InfluxDBClient.from_config_file("influx2.ini")

Great program BTW...

sparky8512 commented 2 years ago

What about also allowing at config file? gstate.influx_client = InfluxDBClient.from_config_file("influx2.ini")

If you just want to read options from a file, you can do so with an @FILNAME arg (see very end of help text). That may not be as pretty as a config file, but it works across all the scripts.

I could still be convinced to add it if you really want it, though, as not all config file options are covered by the command line options.