[NETCONF-56] create-notification-stream can not be used in helium Sr3 and lithium. Created: 08/Jul/15  Updated: 15/Mar/19  Resolved: 29/Jul/16

Status: Resolved
Project: netconf
Component/s: restconf-nb
Affects Version/s: None
Fix Version/s: None

Type: Improvement
Reporter: wuzhan Assignee: Jakub Toth
Resolution: Done Votes: 0
Labels: None
Remaining Estimate: Not Specified
Time Spent: Not Specified
Original Estimate: Not Specified
Environment:

Operating System: All
Platform: All

Issue Links:
Duplicate
is duplicated by NETCONF-161 Netconf-device notifications in ODL Resolved
is duplicated by NETCONF-183 Websocket server only serves XML Resolved
is duplicated by NETCONF-194 documentation on stream creation for ... Resolved

 Description   

I tested it as below. It failed in helium Sr3 and lithium, but successed in hydrogen version.

<?xml version="1.0" encoding="UTF-8"?><input xmlns="urn:opendaylight:params:xml:ns:yang:controller:md:sal:remote">
<notifications>org.opendaylight.yang.gen.v1.urn.opendaylight.inventory.rev130819.NodeUpdated</notifications>
</input>
[post]http://10.46.45.36:8181/restconf/operations/sal-remote:create-notification-stream
1653 ms
state:
501
content:
<?xml version="1.0" encoding="UTF-8" standalone="no"?>
<errors xmlns="urn:ietf:params:xml:ns:yang:ietf-restconf">
<error>
<error-type>rpc</error-type>
<error-tag>operation-not-supported</error-tag>
<error-message>No implementation for this operation is available.</error-message>
</error>
</errors>

 Comments   
Comment by wuzhan [ 10/Jul/15 ]

Is there any way else reg notification?

Comment by wuzhan [ 10/Jul/15 ]

(In reply to wuzhan from comment #1)
> Is there any way else reg notification?

I finally found there was no implementation.

Comment by Tony Tkacik [ 13/Jul/15 ]

THis functionality was never officially supported in Hydrogen
and was ommited in Helium.

If you want to listen on inventory updates please follow:

https://wiki.opendaylight.org/view/OpenDaylight_Controller:MD-SAL:Restconf:Change_event_notification_subscription

and

https://github.com/opendaylight/docs/blob/master/manuals/developer-guide/src/main/asciidoc/controller/websocket-notifications.adoc

Which documents how to use RESTCONF and WebSockets notifications.

Comment by wuzhan [ 14/Jul/15 ]

(In reply to Tony Tkacik from comment #3)
> THis functionality was never officially supported in Hydrogen
> and was ommited in Helium.
>
> If you want to listen on inventory updates please follow:
>
> https://wiki.opendaylight.org/view/OpenDaylight_Controller:MD-SAL:Restconf:
> Change_event_notification_subscription
>
> and
>
> https://github.com/opendaylight/docs/blob/master/manuals/developer-guide/src/
> main/asciidoc/controller/websocket-notifications.adoc
>
> Which documents how to use RESTCONF and WebSockets notifications.

That you for reply.
But I want to send alarm to rest client. I think this is basic feature for controller actively sending message to App.
Why not to implement RESTCONF notification? or there are other ways?

Comment by wuzhan [ 14/Jul/15 ]

(In reply to wuzhan from comment #4)
Maybe I do not want to write in-memory DB, but send some message to client.

Comment by Tony Tkacik [ 14/Jul/15 ]

In that case I would rather reclasify this as new feature request - as I mentioned before behaviour you were observing were never part of officially released functionality for Hydrogen, Helium and Lithium.

Your alarm use-case makes a lot of sense, and I would like to cooperate to create common alarm functionality in OpenDaylight.

Comment by wuzhan [ 14/Jul/15 ]

(In reply to Tony Tkacik from comment #6)
> In that case I would rather reclasify this as new feature request - as I
> mentioned before behaviour you were observing were never part of officially
> released functionality for Hydrogen, Helium and Lithium.
>
> Your alarm use-case makes a lot of sense, and I would like to cooperate to
> create common alarm functionality in OpenDaylight.

Thank you. I wish ODL can support RESTCONF notification first in Lithium next version.

Comment by Robert Varga [ 13/Nov/15 ]

Move to NETCONF project

Comment by Jakub Toth [ 12/Jul/16 ]

https://git.opendaylight.org/gerrit/#/c/41639/

Comment by Ivan Hrasko [ 21/Jul/16 ]

cherry pick for beryllium SR3:
https://git.opendaylight.org/gerrit/#/c/42242

Comment by Ivan Hrasko [ 26/Jul/16 ]

(In reply to Ivan Hrasko from comment #12)
> cherry pick for beryllium SR3:
> https://git.opendaylight.org/gerrit/#/c/42242

Correct link to merged cherry-pick to Beryllium SR3:
https://git.opendaylight.org/gerrit/#/c/42159/

Comment by Ivan Hrasko [ 26/Jul/16 ]

https://git.opendaylight.org/gerrit/#/c/41639/ successfully merged to master.

Generated at Wed Feb 07 20:14:03 UTC 2024 using Jira 8.20.10#820010-sha1:ace47f9899e9ee25d7157d59aa17ab06aee30d3d.