-
Hello Nigel! Thank you so much for creating this awesome project! I am wondering if it is possible to add more than 5 ping targets in the config.ini file? When I try to add a sixth target it doesn't show up in tail -f /var/log/wiperf_agent.log. I read through the documentation and didn't see anything about it being a fixed number of targets. Is there any way to work around this? Thanks! |
Beta Was this translation helpful? Give feedback.
Replies: 8 comments 7 replies
-
Hi Lance, The number of ping targets is currently fixed at five targets in the code. I would need to update the code to support this. The main reason for implementing it this way was that in the config file, it's quite tricky to know how many targets to check for, as the data structure capabilities of the 'ini' file format are quite limited. I'm pretty sure I can fix this in a future release, I just need to think of the best approach and add it to the poller code. Just thinking about it quickly, I wonder if maybe a
|
Beta Was this translation helpful? Give feedback.
-
As it does them sequentially, it would be as many as you can fit in to your (5m) poll cycle (which would be quite a lot). The only caveat is that you would also need to modify the report templates on Grafana yourself to cater for the extra ping tests that you use. I'll have a look at the code and make sure there are no show-stoppers and get back to you soon. Nigel. |
Beta Was this translation helpful? Give feedback.
-
Update was quite minor to get this working. New version (minor update) coming in the next couple of days with this new feature included (just need to do docs & install script): |
Beta Was this translation helpful? Give feedback.
-
nice improvement :) this new "feature" will also affect the HTTP test? |
Beta Was this translation helpful? Give feedback.
-
Uh-oh....scope creep ;-) ....let me have a look, Lol. |
Beta Was this translation helpful? Give feedback.
-
Yes, I'll remove hard-coded limits on ping, http, dns & smb tests. I have the code running OK, just need to document & create the install script & update docs. |
Beta Was this translation helpful? Give feedback.
-
Thanks @wifinigel :) |
Beta Was this translation helpful? Give feedback.
-
OK guys, here it is: https://wifinigel.github.io/wiperf/whats_new_v2.2/ |
Beta Was this translation helpful? Give feedback.
Hi Lance,
The number of ping targets is currently fixed at five targets in the code.
I would need to update the code to support this.
The main reason for implementing it this way was that in the config file, it's quite tricky to know how many targets to check for, as the data structure capabilities of the 'ini' file format are quite limited.
I'm pretty sure I can fix this in a future release, I just need to think of the best approach and add it to the poller code.
Just thinking about it quickly, I wonder if maybe a
number_of_targets
field would solve the issue: