[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Notifications: Proposed edits not included and why



Hi

The following are suggested edits that I did not execute and my
reasoning.

1. Misc. changes that got overwritten by subsequent changes or wg
agreement.

2. Chapter 4) Why do we need a streamNameType? It is just a string.

A-> People wanted everything to be a type in order to enable
extensibility.

3. In section 3.2.2 it says:

    The contents of all event streams made available to a NETCONF client
    (i.e., the notification sent by the NETCONF server) must be encoded
    in XML.

"This is a round-about way to state the obvious -- the contents of the
<notification> element must be encoded in XML."

A-> I think the current text is fine.

4. Didn't s/Schema/schema/ since it seems appropriate to capitalize.

5. There was a suggestion that since filtering was explained in the base
protocol specification, we don't need to explain it again in the
notification context. I don't agree. I think saying it is the same and
then confirming what that means in this context will ensure better
interoperability.

6. Didn't move section 5 to an appendix. This has been discussed before,
and I believe there is agreement to keep it where it is.

7. In section 6., para 2 it says:

    The access control framework and the choice of transport will have a
    major impact on the security of the solution.

"What impact would that be exactly?"

A-> I think the current text, while vague is fine for the security
considerations section. Can we access the impact without making
assumptions about the access control framework?

8. @@ -410,12 +410,12 @@
    Negative Response:
 
       An <rpc-error> element is included within the <rpc-reply> if the
-      request cannot be completed for any reason.  Subscription
requests
+      request cannot be completed for some reason.  Subscription
requests
       will fail if a filter with invalid syntax is provided or if the
       name of a non-existent profile or stream is provided.

A-> I believe the original text is better.

9. @@ -473,12 +473,12 @@
 
    Description:
 
-      An event notification is sent to the client who initiated a
-      <create-subscription> command asynchronously when an event of
-      interest (i.e., meeting the specified filtering criteria) to them
-      has occurred.  An event notification is a complete and
well-formed
+      When an event of interest occurs (i.e., matches the specified
+      filtering criteria), an event notification is sent asynchronously

+      to client(s) that initiated a <create-subscription> command
+      for the event.  An event notification is a complete and
well-formed
       XML document.  Note that <notification> is not an RPC method but

A->I believe the original text is better.

10. @@ -490,7 +490,7 @@
 
    Response:
 
-      No response.  Not applicable.
+      This is not applicable as there is no response.

A-> I believe the original text is better. Note that this text is not in
a paragraph.

11. -   XPATH support for the Notification capability is advertised as
part
+   XPATH support for the notification capability is advertised as part

A-> Again, I believe capitalization is appropriate.

12.     A replayComplete notification is sent to indicate that all of
the
    replay notifications have been sent.  If this subscription has a
stop
-   time, then this session becomes a normal NETCONF session again.  In
-   the case of a subscription without a stop time, after the
-   replayComplete notification has been sent, it can be expected that
-   any notifications generated since the start of the subscription
-   creation will be sent followed by notifications as they arise
-   naturally within the system.
+   time, then this session becomes a normal NETCONF session again.  If
+   the subscription has no stop time, it can be expected that any
+   notifications generated since the start of the subscription creation
+   will be sent after the replayComplete notification has been sent,
+   followed by notifications as they arise naturally within the system.

A-> I added the comma, but I believe the original text was better
otherwise.

13. @@ -1012,7 +1011,7 @@
 
    While it may be possible to retrieve information about subscriptions
    via a get operation, subscriptions are not stored configuration.
-   They are non-persistent state information and their lifetime is
+   They are non-persistent state information, and their lifetime is
    defined by their session.

A-> I don't actually think there is suppose to be a comma there, or at
least one is not required.

Sharon Chisholm
Nortel 
Ottawa, Ontario
Canada

--
to unsubscribe send a message to netconf-request@ops.ietf.org with
the word 'unsubscribe' in a single line as the message text body.
archive: <http://ops.ietf.org/lists/netconf/>