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

Feature Request: Adding columns to DB for latency and jitter values #48

Closed
alexdelprete opened this issue Oct 19, 2022 · 2 comments · Fixed by #226
Closed

Feature Request: Adding columns to DB for latency and jitter values #48

alexdelprete opened this issue Oct 19, 2022 · 2 comments · Fixed by #226
Assignees
Labels
🎉 feature New feature or request

Comments

@alexdelprete
Copy link
Contributor

Right now DL/UL/Ping values have specific columns in the results table, could it be possible to add the columns relevant to latency and jitter values for DL/UL/Ping? I know you also record the entire json result in a column, but it would be simpler to have the data in the columns to build graphs. :)

  • ping/latency - ping/jitter
  • download/latency/iqm - download/latency/jitter
  • upload/latency/iqm - upload/latency/jitter

JSON result:

{
    "type": "result",
    "timestamp": "2022-10-19T14:58:06Z",
    "ping": {
        "jitter": 0.669,
        "latency": 9.098,
        "low": 8.265,
        "high": 9.836
    },
    "download": {
        "bandwidth": 101028973,
        "bytes": 731422984,
        "elapsed": 7206,
        "latency": {
            "iqm": 58.306,
            "low": 9.128,
            "high": 325.879,
            "jitter": 47.725
        }
    },
    "upload": {
        "bandwidth": 29942177,
        "bytes": 187101993,
        "elapsed": 6301,
        "latency": {
            "iqm": 10.126,
            "low": 8.357,
            "high": 12.516,
            "jitter": 0.967
        }
    },
    "packetLoss": 0,
    "isp": "Tecno General S.r.l",
    "interface": {
        "internalIp": "10.1.10.23",
        "name": "eth0",
        "macAddr": "XX:XX:XX:XX:XX:XX",
        "isVpn": false,
        "externalIp": "XXX.XXX.XXX.XXX"
    },
    "server": {
        "id": 4302,
        "host": "speedtest.vodafone.it",
        "port": 8080,
        "name": "Vodafone IT",
        "location": "Milan",
        "country": "Italy",
        "ip": "217.171.46.93"
    },
@alexjustesen alexjustesen self-assigned this Oct 19, 2022
@alexdelprete
Copy link
Contributor Author

alexdelprete commented Oct 21, 2022

I configured influxdb2 integration and checked data after the first speedtest. These are the available fields I could choose from:

image

I picked download/upload, but I noticed very low values in influx:

image

so I checked the mariadb data (record #7):

image

So the data was ok. But the reported speed of that test was this:

image

I didn't understand at first...then I thought maybe those download/upload columns in the db represent the amount of data sent/received, and that you get the actual speed values from the json in the data column. Problem is that the data column is not exported to influx.

So, to integrate my OP, I think also the DL/UL speed values should be in their own columns, probably the quantity of data sent/received is not so important, and you could actually reuse those columns. :)

@alexdelprete
Copy link
Contributor Author

alexdelprete commented Oct 21, 2022

@alexjustesen from the json data, download/bytes and upload/bytes is being written in the download/upload db columns: maybe it's a bug, and download/bandwidth upload/bandwidth values should be written there.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🎉 feature New feature or request
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants