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

Active Monitoring not working on Debian Bullseye distributed package #8386

Open
alexolivan opened this issue May 9, 2024 · 3 comments
Open
Assignees
Labels
Bug Ready to Test a feedback is needed on a proposal or implementation

Comments

@alexolivan
Copy link

alexolivan commented May 9, 2024

Environment:

  • OS name: Debian
  • OS version: Bullseye
  • Architecture: amd64
  • ntopng version/revision: ntopng Community v.6.1.240509 (Debian GNU/Linux 11 (bullseye))

What happened:
AM does nothing.
Log states /usr/share/ntopng/scripts/lua/modules/am_utils.lua failure: round() function is not defined

Workaround:
Added at the top of the script the folling:

require "lua_utils"
local format_utils = require "format_utils"

How did you reproduce it?
Just try to setup AM.
It never gets any value/data

Debug Information:

@MatteoBiscosi
Copy link
Member

hi @alexolivan which kind of active monitoring type did you configure?

@alexolivan
Copy link
Author

Hi.

Initially, as I was struggling to have any AM results, I just simply setup a ICMP ping to 8.8.8.8

However, although certainly not sure, I feel that the kind of monitoring type shouldn't matter in this case, since the issue is that the round() function, used by the code on the script, is not 'included' from the the appropriate 'utils' lib.

Initially I just searched for the round() function, and I found that it sat at some 'utils' script (don't recall which one). Then took a look at other similar scripts calling round(), and I confirmed they were 'requiring' some of those util/libs scripts.
So I simply 'copy/pasted' the 'requiring' stuff somewhere at am_utils.lua , and voila! ...that was it.
If round() at am_utils.lua is to be found somehow other way, then, that way is not working,

Cheers.

@MatteoBiscosi
Copy link
Member

Hi @alexolivan sorry for the late response, i should have fixed the issue (tested and fixed on deb11), so in a couple of hours you should be able to update and the issue should be gone, please let me know.

@MatteoBiscosi MatteoBiscosi added Ready to Test a feedback is needed on a proposal or implementation and removed In Progress labels Jun 5, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug Ready to Test a feedback is needed on a proposal or implementation
Projects
None yet
Development

No branches or pull requests

2 participants