-
Notifications
You must be signed in to change notification settings - Fork 102
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
Network Monitor #146
Network Monitor #146
Conversation
add datamodel draft add stub for service datamodel: use uint64 everywhere, since float are not supported by yang and decimal64 are not supported by our codegen toolchain Signed-off-by: Matteo Bertrone <m.bertrone@gmail.com>
to return pseudo random numebers Signed-off-by: Matteo Bertrone <m.bertrone@gmail.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks!
@mbertrone This seems to be a nice feature. Will it be applied to the framework; like all the polycube services benefit from this or only specific services? Thanks! @frisso I recommend keeping this on hold before we merge to master, as this isn't a fully functional feature. Do you share the same thought? |
|
IMHO, We may first have it working in separate branch. Please do let me know If you still think otherwise. Thanks! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@mbertrone can you please point to any test case or result to validate this feature? I don't see it here in any document. Thanks!
This PR adds a transparent service called
Network Monitor
.The purpose of this module is collecting info over a link, that can be used by a remote backend server to detect attacks and anomalies.
Note: right now the implementation is not fully completed; the module is returning pseudo-random metrics.