Skip to content

Nokia collection - functional POC (#1) #39

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

Merged
merged 3 commits into from
May 8, 2025

Conversation

fedibessrour-bce
Copy link
Collaborator

Samples collected from our pmacct running with this code:

Nokia:
{ "event_type": "log", "seq": 4517924, "timestamp": "1746461487.750287", "telemetry_node": "[mock_ipaddress]", "telemetry_port": [mock_port], "telemetry_data": { "/ipv4-address": "\"[mock_ipaddress]\"", "/mac-address": "\"cc:d8:c1:cb:7d:15\"", "/oper-state": "\"up\"", "/timer": "12181", "/type": "\"dynamic\"", "collected_path": "/state/router/interface/ipv4/neighbor-discovery/neighbor", "notification_timestamp": 1072667109579888347, "sensor_path": [ { "name": "state" }, { "filters": { "router-name": "Base" }, "name": "router" }, { "filters": { "interface-name": "interface1" }, "name": "interface" }, { "name": "ipv4" }, { "name": "neighbor-discovery" }, { "filters": { "ipv4-address": "[mock_ipaddress]" }, "name": "neighbor" } ] }, "serialization": "json", "writer_id": "pmtelemetryd-grpc/1" }

Juniper:
{ "event_type": "log", "seq": 4516904, "timestamp": "1746461397.658852", "telemetry_node": "[mock_ipaddress]", "telemetry_port": [mock_port], "telemetry_data": { "/ipv6/state/dup-addr-detect-transmits": "3", "/ipv6/state/enabled": "1", "/ipv6/state/mtu": "4294967295", "/ipv6/unnumbered/interface-ref/state/subinterface": "0", "/ipv6/unnumbered/state/enabled": "0", "extension": "{\"systemId\":\"sys_id\",\"componentId\":65535,\"sensorName\":\"sensor_1003\",\"subscribedPath\":\"/interfaces/\",\"streamedPath\":\"/interfaces/\",\"component\":\"re0/mib2d\",\"sequenceNumber\":\"1020562\",\"payloadGetTimestamp\":\"1746461687009\",\"streamCreationTimestamp\":\"1746461686590\",\"exportTimestamp\":\"1746461687012\"}", "notification_timestamp": 1746461687009175122, "sensor_path": [ { "name": "interfaces" }, { "filters": { "name": "interface1" }, "name": "interface" }, { "name": "subinterfaces" }, { "filters": { "index": "0" }, "name": "subinterface" } ], "system_id": "sys_id" }, "serialization": "json", "writer_id": "pmtelemetryd-grpc/1" }

@fedibessrour-bce fedibessrour-bce requested a review from scuzzilla May 5, 2025 17:33
@fedibessrour-bce fedibessrour-bce self-assigned this May 5, 2025
@fedibessrour-bce fedibessrour-bce changed the title Nokia collection, functional POC (#1) Nokia collection - functional POC (#1) May 5, 2025
@scuzzilla
Copy link
Collaborator

Hi @fedibessrour-bce,

Thanks for your contribution!

The standalone build is currently failing due to missing header dependencies. Could you please review the CMakeLists.txt to ensure these files are auto-generated?

Also, I noticed a minor issue here — please include the fix in your PR:
https://github.com/fedibessrour-bce/mdt-dialout-collector/blob/51edeecdd9e35cd325e8c42c666ebec4acdfc0f8/src/core/mdt_dialout_core.cc#L442

Regards,
Salvatore.

@fedibessrour-bce
Copy link
Collaborator Author

Hi @fedibessrour-bce,

Thanks for your contribution!

The standalone build is currently failing due to missing header dependencies. Could you please review the CMakeLists.txt to ensure these files are auto-generated?

Also, I noticed a minor issue here — please include the fix in your PR: https://github.com/fedibessrour-bce/mdt-dialout-collector/blob/51edeecdd9e35cd325e8c42c666ebec4acdfc0f8/src/core/mdt_dialout_core.cc#L442

Regards, Salvatore.

Hey Salvatore, thanks for the review!
I'm looking at the CI and it looks like the build is working but the failure happens on the "Push image" stage.

Seems like it's an authorization issue, I'm not allowed to push that image. Can you help with that?
denied: installation not allowed to Write organization package

Thanks again!

@scuzzilla scuzzilla merged commit 7829ae6 into network-analytics:main May 8, 2025
0 of 6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants