Re: [Forces-protocol] Re: draft-ietf-forces-protocol-01.txt

avri@psg.com Mon, 25 October 2004 19:43 UTC

Received: from ietf-mx.ietf.org (ietf-mx.ietf.org [132.151.6.1]) by ietf.org (8.9.1a/8.9.1a) with ESMTP id PAA27507 for <forces-protocol-web-archive@ietf.org>; Mon, 25 Oct 2004 15:43:03 -0400 (EDT)
Received: from megatron.ietf.org ([132.151.6.71]) by ietf-mx.ietf.org with esmtp (Exim 4.33) id 1CMAxj-0005uL-JE for forces-protocol-web-archive@ietf.org; Mon, 25 Oct 2004 15:57:03 -0400
Received: from localhost.localdomain ([127.0.0.1] helo=megatron.ietf.org) by megatron.ietf.org with esmtp (Exim 4.32) id 1CMAhs-0005Aw-B6; Mon, 25 Oct 2004 15:40:40 -0400
Received: from odin.ietf.org ([132.151.1.176] helo=ietf.org) by megatron.ietf.org with esmtp (Exim 4.32) id 1CMAdk-0003T1-Mi for forces-protocol@megatron.ietf.org; Mon, 25 Oct 2004 15:36:24 -0400
Received: from ietf-mx.ietf.org (ietf-mx.ietf.org [132.151.6.1]) by ietf.org (8.9.1a/8.9.1a) with ESMTP id PAA26988 for <forces-protocol@ietf.org>; Mon, 25 Oct 2004 15:36:22 -0400 (EDT)
From: avri@psg.com
Received: from tla.crepundia.net ([194.71.127.149] helo=report.tla-group.com) by ietf-mx.ietf.org with esmtp (Exim 4.33) id 1CMArC-0005lo-Mr for forces-protocol@ietf.org; Mon, 25 Oct 2004 15:50:22 -0400
Received: from [127.0.0.1] (report.tla-group.com [194.71.127.149]) by report.tla-group.com (8.12.4/8.12.4) with ESMTP id i9PJBUep002380; Mon, 25 Oct 2004 21:11:31 +0200
In-Reply-To: <417CE8C7.2050701@zurich.ibm.com>
References: <468F3FDA28AA87429AD807992E22D07E02579228@orsmsx408> <572328E8-265E-11D9-9DB1-000393CC2112@psg.com> <417CE8C7.2050701@zurich.ibm.com>
Mime-Version: 1.0 (Apple Message framework v619)
Content-Type: text/plain; charset=US-ASCII; format=flowed
Message-Id: <1F9AD305-26BD-11D9-9DB1-000393CC2112@psg.com>
Content-Transfer-Encoding: 7bit
Subject: Re: [Forces-protocol] Re: draft-ietf-forces-protocol-01.txt
Date: Mon, 25 Oct 2004 15:36:10 -0400
To: Robert Haas <rha@zurich.ibm.com>
X-Mailer: Apple Mail (2.619)
X-Spam-Score: 1.4 (+)
X-Scan-Signature: 1449ead51a2ff026dcb23465f5379250
Content-Transfer-Encoding: 7bit
Cc: Hormuzd M Khosravi <hormuzd.m.khosravi@intel.com>, "\(\(Ram Gopal \)\)" <ram.gopal@nokia.com>, Ligang Dong <donglg@mail.hzic.edu.cn>, forces-protocol@ietf.org, Jamal Hadi Salim <hadi@znyx.com>, Weiming Wang <wmwang@mail.hzic.edu.cn>
X-BeenThere: forces-protocol@ietf.org
X-Mailman-Version: 2.1.5
Precedence: list
List-Id: forces-protocol <forces-protocol.ietf.org>
List-Unsubscribe: <https://www1.ietf.org/mailman/listinfo/forces-protocol>, <mailto:forces-protocol-request@ietf.org?subject=unsubscribe>
List-Archive: <http://www1.ietf.org/pipermail/forces-protocol>
List-Post: <mailto:forces-protocol@ietf.org>
List-Help: <mailto:forces-protocol-request@ietf.org?subject=help>
List-Subscribe: <https://www1.ietf.org/mailman/listinfo/forces-protocol>, <mailto:forces-protocol-request@ietf.org?subject=subscribe>
Sender: forces-protocol-bounces@ietf.org
Errors-To: forces-protocol-bounces@ietf.org
X-Spam-Score: 1.4 (+)
X-Scan-Signature: 848ed35f2a4fc0638fa89629cb640f48
Content-Transfer-Encoding: 7bit

On 25 okt 2004, at 07.51, Robert Haas wrote:

> Avri,
> Looks good. I have a few corrections, although not essential for now, 
> unless you can't sleep after all the excitment on the list ...

Too late.  I went to sleep after submitting at 4:30 AM.  Not that I 
could have changed anything after submitting.  I submitted before going 
to sleep because I did not want to risk missing the deadline and was 
not sure that i would be able to do any useful editing after 2-3 sleep 
anyway.

so these edits will go in -02

http://psg.com/~avri/forces/draft/draft-ietf-forces-protocol-02-1.txt

>
> - In the Config Response Msg section, the draft says
>
>   Type (16 bits):
>       The operations are same as those defined for Config messages.
>
> whereas the other messages (Query, Event Notif) have now explicitely 
> defined a "*-Response" operation type.  The table should therefore now 
> look like:
>
>   +-------------------+-------+------------+--------+-------------+
>   |     Operation     | Query | Query-Resp | Config | Config-Resp |
>   +-------------------+-------+------------+--------+-------------+
>   |        Set        |       |            |    X   |             |
>         Set-Response                                       X
>   |                   |       |            |        |             |
>   |       Delete      |       |            |    X   |             |
>       Delete-Response                                      X
>   |                   |       |            |        |             |
>   |       Update      |       |            |    X   |            |
>       Update-Response                                      X   |       
>             |       |            |        |             |
>   |        Get        |   X   |            |        |             |
>        Get-Response                  X   |                   |       | 
>            |        |             |
>   |  Event subscribe  |       |            |    X   |             |
>   |  Ev.Sub.-Response |       |            |        |      X      |
>   |                   |       |            |        |             |
>   | Event unsubscribe |       |            |    X   |             |
>   | Ev.Unsub.-Response|       |            |        |      X      |
>   +-------------------+-------+------------+--------+-------------+
>

done

>
>     +-----------+--------------+-------------+------------------+
>     | Operation | Packet-Redir | Event-Notif | Event-Notif-Resp |
>     +-----------+--------------+-------------+------------------+
>     |  Payload  |       X      |             |                  |
>     |           |              |             |                  |
>     |   Report  |              |      X      |                  |
>     |Report-Resp|              |             |         X        |
>     +-----------+--------------+-------------+------------------+
> Note that Operation "Event" was changed to "Report"
>

done

>
> And the text I quoted above in the Config-Resp section 6.3.2 should be 
> changed accordingly to :
>
>   Type (16 bits):
>       The operations include, ADD-Resp, DEL-Resp, UPDATE/REPLACE-Resp, 
> DEL ALL-Resp, EVENT
>       SUBSCRIBE-Resp, EVENT UNSUBSCRIBE-Resp, PACKET SUBSCRIBE, 
> CANCEL-Resp.

done

>
> I would drop in 6.3.1 and 6.3.2 the operation types PACKET *SUBSCRIBE.


I did not understand what you wanted here.

i notice you include PACKET SUBSCRIBE in your change, but not 
UNSUBSCIBE.

While 6.3.1 current has both.  I am not sure what change you are 
suggesting.

I have left them alone for the moment.

> Note that all the messages above don't appear in the summary table, 
> but maybe we should not overload the table with all of them.

don't mind adding them.  what do folks think?
i do need to find a way to adjust the table cell sizes so that it looks 
a little better.

speaking of aesthetics, as the draft gets closer to being finished.  i 
will work on getting pictures to stay on a single page.  not worth the 
effort before then.

that is assuming i stay in the (co) editor role.  otherwise someone 
else can do it.

>
> -  add an Editorial's note after the text below (search for MIID):
>
>      *  MIID table: a list of virtual LFB Instance IDs that map to a
>         list of Instance IDs of LFBs in that FE
>
> Editoral's note: it remains to be decided whether such an MIID table 
> should be  used.

done.

I should have noticed this was still an open issue.  Actually i did 
notice, i just forgot to note it in the draft.

>
> - search and correct typo "GET_RESPOSE"
>

found one instance. fixed.

> - add a newline between Inter-FE topology and Intra-FE topology (sec 
> 6.1.2):
>
>         Note:  The attributes below were previously under Query
>                message.
>      *  Inter-FE topology Intra-FE topology
>

done

>
> - you should appear as "Editor", not "co-editor".

I have never been sure of that.  and therefore listed myself as 
co-editor, since i was pretty sure i was functioning as at least that.

> At least that's my understanding. Also, the ETRI name appears on the 
> first page. I thought you said you didn't want that ? I personally 
> don't care.

i had removed it, but in the exercise to put all the names in the 
header, i reinserted and forgot to remove it. removed it now.

>
> - I like the name "Core ForCES LFBs"

yeah, me too.

>
> Thanks,

thank you.

a.

ps.  i put repositories for -01 on the web site

psg.com/~avri/forces/draft/draft-ietf-forces-protocol-01.[txt, zip, 
tar, tsg]


_______________________________________________
Forces-protocol mailing list
Forces-protocol@ietf.org
https://www1.ietf.org/mailman/listinfo/forces-protocol