influxdata / community-templates

InfluxDB Community Templates: Quickly collect & analyze time series data from a range of sources: Kubernetes, MySQL, Postgres, AWS, Nginx, Jenkins, and more.
https://www.influxdata.com/products/influxdb-templates/gallery/
Apache License 2.0
352 stars 157 forks source link

Removing unnecessary option and fixing conversion #231

Closed lbeier closed 3 years ago

lbeier commented 3 years ago

First of all, thanks for these excellent templates.

I've applied two small fixes in this PR.

The first one is removing the unnecessary -u Mbps option in the command. As per the docs, this option is only applicable for the human-readable output.

The second one is a fix in the download and upload conversion. The output bandwidth for both download and upload is returned in bytes from the Speedtest cli. In order to have it in megabit per second, which matches the suffix Mb/s used in the template, we need to divide it by 125000 instead of 100000.