[dhcwg] Intdir early review of draft-ietf-dhc-relay-port-05

Ralf Weber <ralf.weber@nominum.com> Thu, 20 July 2017 12:08 UTC

Return-Path: <ralf.weber@nominum.com>
X-Original-To: dhcwg@ietf.org
Delivered-To: dhcwg@ietfa.amsl.com
Received: from ietfa.amsl.com (localhost [IPv6:::1]) by ietfa.amsl.com (Postfix) with ESMTP id D3A92131C1A; Thu, 20 Jul 2017 05:08:12 -0700 (PDT)
MIME-Version: 1.0
Content-Type: text/plain; charset="utf-8"
Content-Transfer-Encoding: 7bit
From: Ralf Weber <ralf.weber@nominum.com>
To: <int-dir@ietf.org>
Cc: dhcwg@ietf.org, ietf@ietf.org, draft-ietf-dhc-relay-port.all@ietf.org
X-Test-IDTracker: no
X-IETF-IDTracker: 6.57.0
Auto-Submitted: auto-generated
Precedence: bulk
Message-ID: <150055249283.9589.17482424577250077220@ietfa.amsl.com>
Date: Thu, 20 Jul 2017 05:08:12 -0700
Archived-At: <https://mailarchive.ietf.org/arch/msg/dhcwg/3BmA4UgC2HoJlfW9cXItZaBgzAg>
Subject: [dhcwg] Intdir early review of draft-ietf-dhc-relay-port-05
X-BeenThere: dhcwg@ietf.org
X-Mailman-Version: 2.1.22
List-Id: <dhcwg.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/dhcwg>, <mailto:dhcwg-request@ietf.org?subject=unsubscribe>
List-Archive: <https://mailarchive.ietf.org/arch/browse/dhcwg/>
List-Post: <mailto:dhcwg@ietf.org>
List-Help: <mailto:dhcwg-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/dhcwg>, <mailto:dhcwg-request@ietf.org?subject=subscribe>
X-List-Received-Date: Thu, 20 Jul 2017 12:08:13 -0000

Reviewer: Ralf Weber
Review result: Ready with Nits

Moin!

I reviewed this document and I'm good with it. The only minor purely cosmetic
note is that it took me some time to understand why you record the port for the
IPv6 relay chain. According to a discussion I had with Bernie that is to be
able to have the agent stateless. Maybe add that to the motivation why you
store it in the packet at the end of section 4 (, to allow stateless operation
in a relay chain)

So long
-Ralf