[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: Issue 13.6: Discard changes
At 03:16 PM 2/9/2004, Rob Enns wrote:
>> 13.16) <discard-changes>
>>
>> 13.16.1) Clarifications [PROT]
>>
>> Says content 'automatic' is allowed for the <discard-changes>
>> operation. This is not actually documented.
>>
>> Closed: accepted, protocol document needs to be updated
>
>Here is the source of the confusion: there is a <discard-changes>
>operation which tells the device to discard any changes made to
>the candidate the configuration.
>
>There is _also_ a <discard-changes> argument specified as part of the
>lock operation which can indicate that the device should automatically
>discard any changes to the configuration when the lock is released
>(either intentionally using <unlock/>, or implicitly, by session failure).
>
>The confusion can be fixed by renaming one of these elements.
>We could rename the <discard-changes> argument
>to <rollback>, so that a lock request could look like:
>
><lock>
> <target><candidate/></target>
> <rollback>automatic</rollback>
></lock>
>
>and this element could be conditional on the (tbd) #rollback capability.
I really don't like this approach at all.
Rollback should not be coupled to the candidate config.
This is a separate feature that is not dependent on whether
the device supports writes to <candidate> or to <running>.
Why does the candidate config need its own extensions
to the <lock> command? This seems like really bad design.
I think we should avoid special versions of the protocol
operations unless absolutely required.
>thanks,
> Rob
Andy
>--
>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/>
--
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/>