openBackhaul / NotificationProxy

Consolidates notifications into specific streams
Apache License 2.0
1 stars 1 forks source link

All mounted devices capability is pushed into both ODL streams (config +oper) when user perform edit on ODL attributes #45

Closed Ziabatcha closed 9 months ago

Ziabatcha commented 9 months ago

Steps followed:

  1. ODL Controller is up and running and ODL streaming is not subscribed
  2. Notification proxy is deployed and running
  3. Invoked API: /v1/listen-to-controllers
  4. Performed PUT request on any ODL attribute for only one node(exampl: NODEA) and observed events for all mounted devices from the controller are pushed into both ODL streams (config + oper)

Expected: Only NODEA related events which is associated for PUT operation should get streamed

FelixFleckensteinMentopolis commented 9 months ago

We do not know why this behaviour should be due to the NotificationProxy. Sending notifications is part of the SDN controller. We only forward them

Ziabatcha commented 9 months ago

The url which used for ODL subscription is incomplete.

Actual url: /rests/data/ietf-restconf-monitoring:restconf-state/streams/stream/data-change-event-subscription/network-topology:network-topology/datastore=CONFIGURATION/scope=SUBTREE/JSON

Expected url: /rests/data/ietf-restconf-monitoring:restconf-state/streams/stream/data-change-event-subscription/network-topology:network-topology/datastore=CONFIGURATION/scope=SUBTREE/JSON?changed-leaf-nodes-only=true

Because of actual url, output notification is not correct

paulBMentopolis commented 9 months ago

Hello Zia,

we added the parameter. Please retest with tag "v1.0.0_test-3".