Skip to main content

Early Review of draft-ietf-rats-network-device-subscription-02

Request Review of draft-ietf-rats-network-device-subscription-01
Requested revision 01 (document currently at 04)
Type Early Review
Team YANG Doctors (yangdoctors)
Deadline 2022-08-29
Requested 2022-08-02
Requested by Nancy Cam-Winget
Authors Henk Birkholz , Eric Voit , Wei Pan
I-D last updated 2022-10-27
Completed reviews Yangdoctors Early review of -02 by Jürgen Schönwälder (diff)
The document has matured enough that an early YANG review by the Yang doctors is beneficial
as we get more reviews before a WGLC.
Assignment Reviewer Jürgen Schönwälder
State Completed
Request Early review on draft-ietf-rats-network-device-subscription by YANG Doctors Assigned
Posted at
Reviewed revision 02 (document currently at 04)
Result On the Right Track
Completed 2022-10-27
* Abstract

  s/other Conceptual Message/other Conceptual Messages/

* Introduction

  s/there are can be additional/there can be additional/

  For my taste, this is a somewhat difficult to understand
  introduction, as the text immediately gets lost in details.

* Sequence Diagram

  What does "Figure 1 below is a sequence diagram which updates Figure
  5" mean? Is this a formal update of the figure? Also "This sequence
  diagram replaces the [..]" makes me wonder. Perhaps you mean extend
  or adapt?

  I have not tried to understand all the RATS details.

* Configuring the <attestation> Event Stream

  s/current configuration of stream/current configuration of a stream/

* YANG Module

  Did the module not change since 2021-05-11? Otherwise, update the

  Update the copyright year to 2022.

  There is no reference for ietf-tcg-algs, did not know where to find
  it. I figured out by searching that it is also in

  What are "conceptual YANG specifications"??

     identity pcr-unsubscribable {
       base sn:establish-subscription-error;
         "Requested PCR is subscribable by the Attester.";

  So is it subscribable or unsubscribable?

  Add a units statement to tpm20-subscription-heartbeat.

  I did not spot any YANG specific issues, whether the definitions
  make sense from the RATS perspective, I can't judge. This requires
  reviews by people who understand RATS technology.

  Security considerations are missing.

  IANA considerations are missing.

  I did not compile the YANG module since getting all YANG modules it
  depends on and their dependents into place is, well, non-trivial, in
  particular with intermitted Internet access.