/
archive.json
2777 lines (2777 loc) · 145 KB
/
archive.json
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
108
109
110
111
112
113
114
115
116
117
118
119
120
121
122
123
124
125
126
127
128
129
130
131
132
133
134
135
136
137
138
139
140
141
142
143
144
145
146
147
148
149
150
151
152
153
154
155
156
157
158
159
160
161
162
163
164
165
166
167
168
169
170
171
172
173
174
175
176
177
178
179
180
181
182
183
184
185
186
187
188
189
190
191
192
193
194
195
196
197
198
199
200
201
202
203
204
205
206
207
208
209
210
211
212
213
214
215
216
217
218
219
220
221
222
223
224
225
226
227
228
229
230
231
232
233
234
235
236
237
238
239
240
241
242
243
244
245
246
247
248
249
250
251
252
253
254
255
256
257
258
259
260
261
262
263
264
265
266
267
268
269
270
271
272
273
274
275
276
277
278
279
280
281
282
283
284
285
286
287
288
289
290
291
292
293
294
295
296
297
298
299
300
301
302
303
304
305
306
307
308
309
310
311
312
313
314
315
316
317
318
319
320
321
322
323
324
325
326
327
328
329
330
331
332
333
334
335
336
337
338
339
340
341
342
343
344
345
346
347
348
349
350
351
352
353
354
355
356
357
358
359
360
361
362
363
364
365
366
367
368
369
370
371
372
373
374
375
376
377
378
379
380
381
382
383
384
385
386
387
388
389
390
391
392
393
394
395
396
397
398
399
400
401
402
403
404
405
406
407
408
409
410
411
412
413
414
415
416
417
418
419
420
421
422
423
424
425
426
427
428
429
430
431
432
433
434
435
436
437
438
439
440
441
442
443
444
445
446
447
448
449
450
451
452
453
454
455
456
457
458
459
460
461
462
463
464
465
466
467
468
469
470
471
472
473
474
475
476
477
478
479
480
481
482
483
484
485
486
487
488
489
490
491
492
493
494
495
496
497
498
499
500
501
502
503
504
505
506
507
508
509
510
511
512
513
514
515
516
517
518
519
520
521
522
523
524
525
526
527
528
529
530
531
532
533
534
535
536
537
538
539
540
541
542
543
544
545
546
547
548
549
550
551
552
553
554
555
556
557
558
559
560
561
562
563
564
565
566
567
568
569
570
571
572
573
574
575
576
577
578
579
580
581
582
583
584
585
586
587
588
589
590
591
592
593
594
595
596
597
598
599
600
601
602
603
604
605
606
607
608
609
610
611
612
613
614
615
616
617
618
619
620
621
622
623
624
625
626
627
628
629
630
631
632
633
634
635
636
637
638
639
640
641
642
643
644
645
646
647
648
649
650
651
652
653
654
655
656
657
658
659
660
661
662
663
664
665
666
667
668
669
670
671
672
673
674
675
676
677
678
679
680
681
682
683
684
685
686
687
688
689
690
691
692
693
694
695
696
697
698
699
700
701
702
703
704
705
706
707
708
709
710
711
712
713
714
715
716
717
718
719
720
721
722
723
724
725
726
727
728
729
730
731
732
733
734
735
736
737
738
739
740
741
742
743
744
745
746
747
748
749
750
751
752
753
754
755
756
757
758
759
760
761
762
763
764
765
766
767
768
769
770
771
772
773
774
775
776
777
778
779
780
781
782
783
784
785
786
787
788
789
790
791
792
793
794
795
796
797
798
799
800
801
802
803
804
805
806
807
808
809
810
811
812
813
814
815
816
817
818
819
820
821
822
823
824
825
826
827
828
829
830
831
832
833
834
835
836
837
838
839
840
841
842
843
844
845
846
847
848
849
850
851
852
853
854
855
856
857
858
859
860
861
862
863
864
865
866
867
868
869
870
871
872
873
874
875
876
877
878
879
880
881
882
883
884
885
886
887
888
889
890
891
892
893
894
895
896
897
898
899
900
901
902
903
904
905
906
907
908
909
910
911
912
913
914
915
916
917
918
919
920
921
922
923
924
925
926
927
928
929
930
931
932
933
934
935
936
937
938
939
940
941
942
943
944
945
946
947
948
949
950
951
952
953
954
955
956
957
958
959
960
961
962
963
964
965
966
967
968
969
970
971
972
973
974
975
976
977
978
979
980
981
982
983
984
985
986
987
988
989
990
991
992
993
994
995
996
997
998
999
1000
{
"magic": "E!vIA5L86J2I",
"timestamp": "2024-03-28T00:57:13.600859+00:00",
"repo": "ietf-ccamp-wg/draft-ietf-ccamp-flexigrid-yang",
"labels": [
{
"name": "bug",
"description": "Something isn't working",
"color": "d73a4a"
},
{
"name": "documentation",
"description": "Improvements or additions to documentation",
"color": "0075ca"
},
{
"name": "duplicate",
"description": "This issue or pull request already exists",
"color": "cfd3d7"
},
{
"name": "enhancement",
"description": "New feature or request",
"color": "a2eeef"
},
{
"name": "good first issue",
"description": "Good for newcomers",
"color": "7057ff"
},
{
"name": "help wanted",
"description": "Extra attention is needed",
"color": "008672"
},
{
"name": "invalid",
"description": "This doesn't seem right",
"color": "e4e669"
},
{
"name": "question",
"description": "Further information is requested",
"color": "d876e3"
},
{
"name": "wontfix",
"description": "This will not be worked on",
"color": "ffffff"
},
{
"name": "agreed",
"description": "",
"color": "0E8A16"
},
{
"name": "discussion required",
"description": "",
"color": "FBCA04"
}
],
"issues": [
{
"number": 3,
"id": "MDU6SXNzdWU3OTU5MTU1NTA=",
"title": "Introduction Update",
"url": "https://github.com/ietf-ccamp-wg/draft-ietf-ccamp-flexigrid-yang/issues/3",
"state": "CLOSED",
"author": "danielkinguk",
"authorAssociation": "CONTRIBUTOR",
"assignees": [],
"labels": [],
"body": "OLD\r\n [RFC7698] provides a framework GMPLS-Based control\r\n\r\nNEW\r\n [RFC7698] provides a framework for GMPLS-Based control",
"createdAt": "2021-01-28T11:21:56Z",
"updatedAt": "2021-02-04T12:03:28Z",
"closedAt": "2021-02-04T12:03:28Z",
"comments": []
},
{
"number": 4,
"id": "MDU6SXNzdWU3OTU5MTYwNjI=",
"title": "Introduction Update ",
"url": "https://github.com/ietf-ccamp-wg/draft-ietf-ccamp-flexigrid-yang/issues/4",
"state": "CLOSED",
"author": "danielkinguk",
"authorAssociation": "CONTRIBUTOR",
"assignees": [],
"labels": [],
"body": "OLD\r\n This document identifies the flexi-grid components, parameters and\r\n their values, characterizes the features and the performances of the\r\n flexi-grid elements.\r\n\r\nNEW\r\n This document identifies the flexi-grid components, parameters and\r\n their values, and it characterizes the features and the performance\r\n of the flexi-grid elements.",
"createdAt": "2021-01-28T11:22:38Z",
"updatedAt": "2021-02-04T12:03:09Z",
"closedAt": "2021-02-04T12:03:09Z",
"comments": []
},
{
"number": 5,
"id": "MDU6SXNzdWU3OTU5MTY0NDA=",
"title": "Introduction Update",
"url": "https://github.com/ietf-ccamp-wg/draft-ietf-ccamp-flexigrid-yang/issues/5",
"state": "CLOSED",
"author": "danielkinguk",
"authorAssociation": "CONTRIBUTOR",
"assignees": [],
"labels": [],
"body": "OLD\r\n An application example is provided towards the end of the document to\r\n better understand their utility.\r\nOLD\r\n\r\nNEW\r\n An application example is provided towards the end of the document to\r\n better understand the utility of this YANG model.\r\nEND",
"createdAt": "2021-01-28T11:23:12Z",
"updatedAt": "2021-02-04T12:02:37Z",
"closedAt": "2021-02-04T12:02:37Z",
"comments": []
},
{
"number": 6,
"id": "MDU6SXNzdWU3OTU5MTY4MDM=",
"title": "Reordering of sections",
"url": "https://github.com/ietf-ccamp-wg/draft-ietf-ccamp-flexigrid-yang/issues/6",
"state": "CLOSED",
"author": "danielkinguk",
"authorAssociation": "CONTRIBUTOR",
"assignees": [],
"labels": [],
"body": "Sections 3 and 3.1 look a bit odd how they are. How about moving them to 2.1 and 2.2 respectively.",
"createdAt": "2021-01-28T11:23:44Z",
"updatedAt": "2021-02-04T12:02:13Z",
"closedAt": "2021-02-04T12:02:13Z",
"comments": []
},
{
"number": 7,
"id": "MDU6SXNzdWU3OTU5MTcxNDY=",
"title": "Request for a diagram showing relationship between YANG modules used.",
"url": "https://github.com/ietf-ccamp-wg/draft-ietf-ccamp-flexigrid-yang/issues/7",
"state": "CLOSED",
"author": "danielkinguk",
"authorAssociation": "CONTRIBUTOR",
"assignees": [
"danielkinguk"
],
"labels": [],
"body": "It would also be good to add a diagram showing the relationship between the various YANG modules.",
"createdAt": "2021-01-28T11:24:16Z",
"updatedAt": "2021-02-22T07:59:07Z",
"closedAt": "2021-02-22T07:59:07Z",
"comments": []
},
{
"number": 8,
"id": "MDU6SXNzdWU3OTU5MTg0MTc=",
"title": "Figure 2 Update",
"url": "https://github.com/ietf-ccamp-wg/draft-ietf-ccamp-flexigrid-yang/issues/8",
"state": "CLOSED",
"author": "danielkinguk",
"authorAssociation": "CONTRIBUTOR",
"assignees": [],
"labels": [],
"body": "I think Figure 2 could be cleaned up a little to show that the transponders are in optical edge nodes. Possibly...\r\n\r\n[https://mailarchive.ietf.org/arch/msg/ccamp/foUbVnfpO0BsPZoBMDdT48-kURw/](url)",
"createdAt": "2021-01-28T11:25:58Z",
"updatedAt": "2021-02-04T12:02:03Z",
"closedAt": "2021-02-04T12:02:03Z",
"comments": []
},
{
"number": 9,
"id": "MDU6SXNzdWU3OTU5MTg3NzA=",
"title": "Section 4 - TED usage ",
"url": "https://github.com/ietf-ccamp-wg/draft-ietf-ccamp-flexigrid-yang/issues/9",
"state": "CLOSED",
"author": "danielkinguk",
"authorAssociation": "CONTRIBUTOR",
"assignees": [],
"labels": [],
"body": "Could you please expand \"TED\" on first use.\r\n",
"createdAt": "2021-01-28T11:26:26Z",
"updatedAt": "2021-02-04T12:01:58Z",
"closedAt": "2021-02-04T12:01:58Z",
"comments": []
},
{
"number": 10,
"id": "MDU6SXNzdWU3OTU5MTg5MzU=",
"title": "Section 4 typo",
"url": "https://github.com/ietf-ccamp-wg/draft-ietf-ccamp-flexigrid-yang/issues/10",
"state": "CLOSED",
"author": "danielkinguk",
"authorAssociation": "CONTRIBUTOR",
"assignees": [],
"labels": [],
"body": "s/provided on each situation/provided in each situation/\r\n",
"createdAt": "2021-01-28T11:26:40Z",
"updatedAt": "2021-02-04T12:00:41Z",
"closedAt": "2021-02-04T12:00:41Z",
"comments": []
},
{
"number": 11,
"id": "MDU6SXNzdWU3OTU5MTkzOTM=",
"title": "Section 4 text change",
"url": "https://github.com/ietf-ccamp-wg/draft-ietf-ccamp-flexigrid-yang/issues/11",
"state": "CLOSED",
"author": "danielkinguk",
"authorAssociation": "CONTRIBUTOR",
"assignees": [],
"labels": [],
"body": "Section 4\r\n We refer to [I-D.ietf-ccamp-flexigrid-media-channel-yang] to complete this\r\n example.\r\n\r\nMaybe...\r\n Details of how to configure a flexi-grid media channel can be found in [I-D.ietf-ccamp-flexigrid-media-channel-yang].\r\n",
"createdAt": "2021-01-28T11:27:19Z",
"updatedAt": "2021-02-04T12:00:12Z",
"closedAt": "2021-02-04T12:00:12Z",
"comments": []
},
{
"number": 12,
"id": "MDU6SXNzdWU3OTU5MTk3MDY=",
"title": "Section 5 Typo",
"url": "https://github.com/ietf-ccamp-wg/draft-ietf-ccamp-flexigrid-yang/issues/12",
"state": "CLOSED",
"author": "danielkinguk",
"authorAssociation": "CONTRIBUTOR",
"assignees": [],
"labels": [],
"body": "OLD\r\n5. YANG Model (Tree Structure) for Flexi-grid topology\r\n\r\nNEW\r\n5. YANG Model (Tree Structure) for Flexi-Grid Topology\r\n",
"createdAt": "2021-01-28T11:27:45Z",
"updatedAt": "2021-02-04T11:59:01Z",
"closedAt": "2021-02-04T11:59:01Z",
"comments": []
},
{
"number": 13,
"id": "MDU6SXNzdWU3OTU5MjAwMjc=",
"title": "Change in Normative References",
"url": "https://github.com/ietf-ccamp-wg/draft-ietf-ccamp-flexigrid-yang/issues/13",
"state": "CLOSED",
"author": "danielkinguk",
"authorAssociation": "CONTRIBUTOR",
"assignees": [],
"labels": [],
"body": "I think three of your informative references should be moved to normative: \r\n\r\n- [I-D.ietf-ccamp-flexigrid-media-channel-yang]\r\n- [I-D.ietf-ccamp-layer0-types]\r\n- [I-D.ietf-ccamp-optical-impairment-topology-yang]",
"createdAt": "2021-01-28T11:28:11Z",
"updatedAt": "2021-02-04T12:04:29Z",
"closedAt": "2021-02-04T12:04:29Z",
"comments": [
{
"author": "danielkinguk",
"authorAssociation": "CONTRIBUTOR",
"body": "For Italo...",
"createdAt": "2021-01-28T13:25:25Z",
"updatedAt": "2021-01-28T13:25:25Z"
}
]
},
{
"number": 14,
"id": "MDU6SXNzdWU3OTU5Nzg0MDc=",
"title": "Request Document Shepherd from Chairs after submission of new version. ",
"url": "https://github.com/ietf-ccamp-wg/draft-ietf-ccamp-flexigrid-yang/issues/14",
"state": "CLOSED",
"author": "danielkinguk",
"authorAssociation": "CONTRIBUTOR",
"assignees": [],
"labels": [],
"body": "",
"createdAt": "2021-01-28T12:53:15Z",
"updatedAt": "2021-02-11T13:05:33Z",
"closedAt": "2021-02-11T13:05:32Z",
"comments": [
{
"author": "danielkinguk",
"authorAssociation": "CONTRIBUTOR",
"body": "Adrian Farrel will be Shepherd.",
"createdAt": "2021-02-11T13:05:32Z",
"updatedAt": "2021-02-11T13:05:32Z"
}
]
},
{
"number": 15,
"id": "MDU6SXNzdWU3OTU5OTY1NjU=",
"title": " Figure 2 could be cleaned up a little to show that the transponders are in optical edge nodes.",
"url": "https://github.com/ietf-ccamp-wg/draft-ietf-ccamp-flexigrid-yang/issues/15",
"state": "CLOSED",
"author": "danielkinguk",
"authorAssociation": "CONTRIBUTOR",
"assignees": [],
"labels": [],
"body": "",
"createdAt": "2021-01-28T13:18:12Z",
"updatedAt": "2021-02-04T11:57:59Z",
"closedAt": "2021-02-04T11:57:59Z",
"comments": [
{
"author": "danielkinguk",
"authorAssociation": "CONTRIBUTOR",
"body": "Figure 2 currently https://tools.ietf.org/html/draft-ietf-ccamp-flexigrid-yang-08#page-4\r\n\r\nTo be updated with: https://mailarchive.ietf.org/arch/msg/ccamp/foUbVnfpO0BsPZoBMDdT48-kURw/",
"createdAt": "2021-01-28T13:18:40Z",
"updatedAt": "2021-01-28T13:18:40Z"
}
]
},
{
"number": 16,
"id": "MDU6SXNzdWU3OTYwMjEzOTM=",
"title": "Prefix may need to follow common rule that is applied across all CCAMP I-Ds",
"url": "https://github.com/ietf-ccamp-wg/draft-ietf-ccamp-flexigrid-yang/issues/16",
"state": "CLOSED",
"author": "danielkinguk",
"authorAssociation": "CONTRIBUTOR",
"assignees": [
"haomianzheng"
],
"labels": [],
"body": "",
"createdAt": "2021-01-28T13:49:52Z",
"updatedAt": "2021-02-22T06:37:47Z",
"closedAt": "2021-02-22T06:37:47Z",
"comments": [
{
"author": "danielkinguk",
"authorAssociation": "CONTRIBUTOR",
"body": "Next steps, we should:\r\n\r\n1. Make just one suggestion for everyone to agree to\r\n2. Reference the YANG models by the model name (not some random descriptive text string)\r\n3. Also include the draft names",
"createdAt": "2021-02-04T13:07:55Z",
"updatedAt": "2021-02-04T13:07:55Z"
},
{
"author": "danielkinguk",
"authorAssociation": "CONTRIBUTOR",
"body": "Ongoing issue, will not stop us submitting but aware that we may need update.",
"createdAt": "2021-02-11T13:08:57Z",
"updatedAt": "2021-02-11T13:08:57Z"
}
]
},
{
"number": 17,
"id": "MDU6SXNzdWU4MDEyNzE3ODU=",
"title": "Final review of I-D before submission (of version 9)",
"url": "https://github.com/ietf-ccamp-wg/draft-ietf-ccamp-flexigrid-yang/issues/17",
"state": "CLOSED",
"author": "danielkinguk",
"authorAssociation": "CONTRIBUTOR",
"assignees": [
"italobusi"
],
"labels": [],
"body": "By Weds 10th Feb",
"createdAt": "2021-02-04T13:13:23Z",
"updatedAt": "2021-02-22T07:59:25Z",
"closedAt": "2021-02-22T07:59:25Z",
"comments": [
{
"author": "dperdices",
"authorAssociation": "CONTRIBUTOR",
"body": "I leave here my review. Most of the things are typos, inconsistencies and minor issues. As you see, it is divided in the same sections of the document so that it is easier to find the issues.\r\n\r\nAnything I get wrong, please let me know\r\n\r\n\r\n-----------------------------------------------------\r\n## 1 Introduction\r\n* ITU recommendations should be cite in a consistent way, i.e. instead of `G.694.1 [G.694.1] and G.872 [G.872]`, it should be `[G.694.1] and [G.872]`, the same way we cite RFCs or drafts.\r\n\r\n* Typo `flexi-grid elements..`\r\n\r\n## 3 Tree diagram\r\n\r\n### 3.1 Prefixes in Data Node Names\r\n\r\n* Should caption of the table be `Table 1:...` instead of `Figure 1:...`?\r\n\r\n## 4 Example of use\r\n\r\n* We should only keep the second version of the figure instead of both of them. \r\n\r\n* Reference in text should be `Figure 2` in this version, but as I pointed out in 3.1, maybe Figure 1 should be Table and this closes this issue.\r\n\r\n## 6 Yang code \r\n\r\n* In contact leaf, Dan's email is not the same of section Authors' Addresses. (Not an issue but a piece of information for Dan in case he didn't notice).\r\n\r\n* I think that should change the reference and description and use `RFC YYYY` instead of `RFC XXXX`. We have in section 3.1 this note \r\n\r\n```\r\nRFC Editor Note: Please replace XXXX with the RFC numbers assigned to\r\ndraft-ietf-ccamp-layer0-types. Please replace YYYY with the RFC\r\nnumber assigned to this document. Please remove this note.\r\n```\r\n\r\nand in the yang code this note\r\n\r\n```\r\nThis version of this YANG module is part of RFC XXXX; see\r\nthe RFC itself for full legal notices.\";\r\n\r\nrevision 2020-10-21 {\r\ndescription\r\n \"Initial Version\";\r\nreference\r\n \"RFC XXXX: A YANG Data Model for Flexi-Grid Optical Networks\";\r\n// RFC Ed.: replace XXXX with actual RFC number, update date\r\n// information and remove this note\r\n}\r\n```\r\n\r\nIt's not a big issue but something kind to the RFC Editors so that they can replace XXXX with RFC number for layer0-types and YYYY with our document's number :)\r\n\r\n* Only validation warning is the one I pointed out in last meeting (solved in layer-0-types)\r\n\r\n## 7 Security considerations\r\n\r\n* `Secure Shell (SSH)` is used there three times. Should we just expand the first time? Maybe keep the ones that are titles of RFCs? Also, should we expand TLS or is already well-known?\r\n\r\n## 9 Contributors\r\n\r\n* Daniel Michaud's corporative email should be `daniel.michaud@estudiante.uam.es` but it is probably already disabled. Jorge should confirm this.\r\n\r\n* Maybe, for consistency, we should ask Victor and Oscar to have also their second surname as Jorge and I do. Or remove our second surname. It's not such a big deal in any case and should be left to each one's discretion.\r\n\r\n* Italo's affiliation (Huawei) is not the same as Haomian's (Huawei Technologies). (Maybe they are different companies and I am messing this up, my apologies in advance in that case :) )\r\n\r\n## 11 References\r\n\r\n### 11.1 Normative references\r\n\r\n* `Madrid, U.` should be `Lopez de Vergara, J. E.`\r\n\r\n* `Lopezalvarez, V` should be `Lopez, V.` (or `Lopez-Alvarez, V.`)\r\n\r\n* `Dios, O.` should be `Gonzalez de Dios, O.`\r\n\r\n### 11.2 Informative references\r\n\r\n* `ITUT` should be `ITU-T`\r\n-------------------------------------",
"createdAt": "2021-02-10T10:37:04Z",
"updatedAt": "2021-02-10T10:38:17Z"
},
{
"author": "italobusi",
"authorAssociation": "MEMBER",
"body": "Please find my review comments in embedded word comments: \r\n[draft-ietf-ccamp-flexigrid-yang-09-ib@2021-02-10.docx](https://github.com/ietf-ccamp-wg/draft-ietf-ccamp-flexigrid-yang/files/5959589/draft-ietf-ccamp-flexigrid-yang-09-ib%402021-02-10.docx)\r\n\r\nProposed text changes, marked with revision bars, are also provided.",
"createdAt": "2021-02-10T16:12:13Z",
"updatedAt": "2021-02-10T16:12:13Z"
},
{
"author": "jorgelopezdevergara",
"authorAssociation": "NONE",
"body": "Given that Daniel Perdices is now working at Universidad Aut\u00f3noma de Madrid, maybe we can switch these affiliations (as I am also at Naudit), to keep this information updated.\r\n\r\nDaniel Perdices and I need to add acknowledgements to a national project (probably, instead of Trafica project that is currently there), and to a research scholar for Daniel Perdices. The new acks would be the \"This work is also partially funded by the Spanish State Research Agency under the project AgileMon (AEI PID2019-104451RB-C21) and by the Spanish Ministry of Science, Innovation and Universities under the program for the training of university lecturers (Grant number: FPU19/05678).\"\r\n\r\nWith respect to Daniel Michaud e-mail, I have his personal e-mail. I'm not sure if he needs to be contacted by e-mail, as he worked on this with me at the beginning of the draft. If the e-mail is needed, we can put his UAM e-mail (daniel.michaud@estudiante.uam.es). \r\n\r\nAlso need to update affiliation in YANG code.",
"createdAt": "2021-02-11T10:20:39Z",
"updatedAt": "2021-02-11T13:19:40Z"
},
{
"author": "sergiobelotti",
"authorAssociation": "CONTRIBUTOR",
"body": "I have some comments about the version 9 that I collected in this attached word version.\r\n[draft-ietf-ccamp-flexigrid-yang-09-comment-SB.docx](https://github.com/ietf-ccamp-wg/draft-ietf-ccamp-flexigrid-yang/files/5965004/draft-ietf-ccamp-flexigrid-yang-09-comment-SB.docx)\r\n",
"createdAt": "2021-02-11T11:55:17Z",
"updatedAt": "2021-02-11T11:55:17Z"
},
{
"author": "danielkinguk",
"authorAssociation": "CONTRIBUTOR",
"body": "Add some short text stating that media channel resource model is discussion in draft-ietf-ccamp-flex-media-foo and not in scope of this document. ",
"createdAt": "2021-02-11T13:16:10Z",
"updatedAt": "2021-02-11T13:16:10Z"
}
]
},
{
"number": 18,
"id": "MDU6SXNzdWU4MDYzOTUwNTM=",
"title": "Request WG presentation slot at 110 where we will present both topo (one-slide) and media channel I-D. ",
"url": "https://github.com/ietf-ccamp-wg/draft-ietf-ccamp-flexigrid-yang/issues/18",
"state": "CLOSED",
"author": "danielkinguk",
"authorAssociation": "CONTRIBUTOR",
"assignees": [],
"labels": [],
"body": "",
"createdAt": "2021-02-11T13:34:25Z",
"updatedAt": "2021-03-10T19:53:51Z",
"closedAt": "2021-03-10T19:53:51Z",
"comments": []
},
{
"number": 19,
"id": "MDU6SXNzdWU4NTMzMjU0MzY=",
"title": "Send response to Yangdoctor Last Call Review",
"url": "https://github.com/ietf-ccamp-wg/draft-ietf-ccamp-flexigrid-yang/issues/19",
"state": "CLOSED",
"author": "danielkinguk",
"authorAssociation": "CONTRIBUTOR",
"assignees": [
"danielkinguk"
],
"labels": [],
"body": "https://mailarchive.ietf.org/arch/msg/ccamp/cRUciNQPjBXfjjgz9FwmUbxbsZI/",
"createdAt": "2021-04-08T10:33:36Z",
"updatedAt": "2021-04-22T11:09:05Z",
"closedAt": "2021-04-22T11:09:05Z",
"comments": []
},
{
"number": 20,
"id": "MDU6SXNzdWU4NTMzMjYwMzI=",
"title": "Section 1 - Introduction NITs",
"url": "https://github.com/ietf-ccamp-wg/draft-ietf-ccamp-flexigrid-yang/issues/20",
"state": "CLOSED",
"author": "danielkinguk",
"authorAssociation": "CONTRIBUTOR",
"assignees": [
"danielkinguk"
],
"labels": [],
"body": "s/[G.694.1] and G.872 [G.872] provides/[G.694.1] and G.872 [G.872] provide/\r\n\r\ns/flexi-grid elements../flexi-grid elements/\r\n",
"createdAt": "2021-04-08T10:34:17Z",
"updatedAt": "2021-04-22T11:11:32Z",
"closedAt": "2021-04-22T11:11:32Z",
"comments": [
{
"author": "danielkinguk",
"authorAssociation": "CONTRIBUTOR",
"body": "Fixed in latest version.",
"createdAt": "2021-04-22T11:11:32Z",
"updatedAt": "2021-04-22T11:11:32Z"
}
]
},
{
"number": 21,
"id": "MDU6SXNzdWU4NTMzMjc0OTQ=",
"title": "Section 4 - Example of Use",
"url": "https://github.com/ietf-ccamp-wg/draft-ietf-ccamp-flexigrid-yang/issues/21",
"state": "CLOSED",
"author": "danielkinguk",
"authorAssociation": "CONTRIBUTOR",
"assignees": [
"danielkinguk"
],
"labels": [],
"body": "\"Thank you first of all for putting together an example to help folks understand the different terminologies used in the document. However, I am confused whose perspective does the example present. The paragraph starts by saying \"In order to configure a network media channel ...\", which to me sounds like a client *configuring* the network media channel. How does configuring of nodes A and E provide connectivity towards the node interface? Or did you mean to say that the act of configuring nodes A and E will somehow bring up the connection towards the node interface?\"\r\n\r\nI (Dan) plan to update as follows: \r\n\r\n>> In order to configure a path between A and E, the operator. \r\n>> Remove \u201cwe\u201d \r\n>> \u201cTransponders A & E are the path termination points\u201d",
"createdAt": "2021-04-08T10:36:05Z",
"updatedAt": "2021-04-22T11:12:10Z",
"closedAt": "2021-04-22T11:12:09Z",
"comments": [
{
"author": "danielkinguk",
"authorAssociation": "CONTRIBUTOR",
"body": "Fixed in latest version. ",
"createdAt": "2021-04-22T11:12:09Z",
"updatedAt": "2021-04-22T11:12:09Z"
}
]
},
{
"number": 22,
"id": "MDU6SXNzdWU4NTMzMjkwMTE=",
"title": "Review I-D and replace YANG model \"use\" with \"configure\" and \"configuration\" ",
"url": "https://github.com/ietf-ccamp-wg/draft-ietf-ccamp-flexigrid-yang/issues/22",
"state": "CLOSED",
"author": "danielkinguk",
"authorAssociation": "CONTRIBUTOR",
"assignees": [
"danielkinguk"
],
"labels": [],
"body": "As we use the YANG model to configure.",
"createdAt": "2021-04-08T10:37:56Z",
"updatedAt": "2021-04-22T11:12:20Z",
"closedAt": "2021-04-22T11:12:20Z",
"comments": [
{
"author": "danielkinguk",
"authorAssociation": "CONTRIBUTOR",
"body": "Fixed in latest version.",
"createdAt": "2021-04-22T11:12:20Z",
"updatedAt": "2021-04-22T11:12:20Z"
}
]
},
{
"number": 23,
"id": "MDU6SXNzdWU4NTMzMjk2NzQ=",
"title": "Check I-D and replace first-person text, such as \"we\". ",
"url": "https://github.com/ietf-ccamp-wg/draft-ietf-ccamp-flexigrid-yang/issues/23",
"state": "CLOSED",
"author": "danielkinguk",
"authorAssociation": "CONTRIBUTOR",
"assignees": [
"danielkinguk"
],
"labels": [],
"body": "OLD:\r\n Then, we also define the links 1 to 5 that interconnect nodes,\r\n indicating which flexi-grid labels are available.\r\n\r\nNEW:\r\n This example configures links 1 to 5 that interconnect nodes,\r\n with flexi-grid labels that are available.",
"createdAt": "2021-04-08T10:38:48Z",
"updatedAt": "2021-04-22T11:11:11Z",
"closedAt": "2021-04-22T11:11:11Z",
"comments": []
},
{
"number": 24,
"id": "MDU6SXNzdWU4NTMzMzA2NDg=",
"title": "Use of XXXX and YYYY",
"url": "https://github.com/ietf-ccamp-wg/draft-ietf-ccamp-flexigrid-yang/issues/24",
"state": "CLOSED",
"author": "danielkinguk",
"authorAssociation": "CONTRIBUTOR",
"assignees": [
"danielkinguk"
],
"labels": [],
"body": "Use of XXXX is assigned to draft-ietf-ccamp-layer0-types:\r\n\r\n import ietf-layer0-types {\r\n prefix \"l0-types\";\r\n reference\r\n \"RFC XXXX: A YANG Data Model for Layer 0 Types\";\r\n }\r\n\r\nyet the YANG model says:\r\n\r\n This version of this YANG module is part of RFC XXXX; see\r\n the RFC itself for full legal notices.\";\r\n\r\nand\r\n \"RFC XXXX: A YANG Data Model for Flexi-Grid Optical Networks\";\r\n // RFC Ed.: replace XXXX with actual RFC number, update date\r\n // information and remove this note\r\n\r\nI (Dan) will update, for instance Page 24 and Page 25 YYYY instead of XXXX\r\n",
"createdAt": "2021-04-08T10:40:04Z",
"updatedAt": "2021-04-22T11:12:33Z",
"closedAt": "2021-04-22T11:12:33Z",
"comments": [
{
"author": "danielkinguk",
"authorAssociation": "CONTRIBUTOR",
"body": "We duplicate IANA should replace XXXX/YYYY text. Need to clean up.",
"createdAt": "2021-04-08T12:10:40Z",
"updatedAt": "2021-04-08T12:10:40Z"
},
{
"author": "danielkinguk",
"authorAssociation": "CONTRIBUTOR",
"body": "Fixed in latest version.",
"createdAt": "2021-04-22T11:12:33Z",
"updatedAt": "2021-04-22T11:12:33Z"
}
]
},
{
"number": 25,
"id": "MDU6SXNzdWU4NTMzMzEwMDQ=",
"title": "Check us of substitutions ",
"url": "https://github.com/ietf-ccamp-wg/draft-ietf-ccamp-flexigrid-yang/issues/25",
"state": "CLOSED",
"author": "danielkinguk",
"authorAssociation": "CONTRIBUTOR",
"assignees": [
"danielkinguk"
],
"labels": [],
"body": "RFC editors prefer that all instructions for substitution are provided in one location instead of having them sprinkled all over the document.\r\n",
"createdAt": "2021-04-08T10:40:32Z",
"updatedAt": "2022-02-03T12:28:25Z",
"closedAt": "2022-02-03T12:28:25Z",
"comments": [
{
"author": "danielkinguk",
"authorAssociation": "CONTRIBUTOR",
"body": "Legitimate use to be highlighted on list when we submit version 11. ",
"createdAt": "2021-10-07T11:25:25Z",
"updatedAt": "2021-10-07T11:25:25Z"
}
]
},
{
"number": 26,
"id": "MDU6SXNzdWU4NTMzMzE4MDM=",
"title": "Section 6 \"YANG model\" revision statement data is incorrect",
"url": "https://github.com/ietf-ccamp-wg/draft-ietf-ccamp-flexigrid-yang/issues/26",
"state": "CLOSED",
"author": "danielkinguk",
"authorAssociation": "CONTRIBUTOR",
"assignees": [
"danielkinguk"
],
"labels": [],
"body": "ietf-flexi-grid-topology@2020-02-22.yang:1: warning: unexpected latest revision \"2020-10-21\" in ietf-flexi-grid-topology@2020-02-22.yang, should be \"2020-02-22\"",
"createdAt": "2021-04-08T10:41:31Z",
"updatedAt": "2021-07-12T16:20:32Z",
"closedAt": "2021-07-12T16:20:32Z",
"comments": []
},
{
"number": 27,
"id": "MDU6SXNzdWU4NTMzMzI4MzM=",
"title": "Application Example",
"url": "https://github.com/ietf-ccamp-wg/draft-ietf-ccamp-flexigrid-yang/issues/27",
"state": "CLOSED",
"author": "danielkinguk",
"authorAssociation": "CONTRIBUTOR",
"assignees": [
"oscargdd",
"danielkinguk"
],
"labels": [],
"body": "The document says - \"An application example is provided towards the end of the document to better understand the utility of this YANG model.\" However, no such example exists in the document. With no example, how is an operator supposed to know how to use the module. Besides, there is no way to know that this model is even valid.\r\n\r\nThe example is/will (be) provided in the media channel. Will updated text and point to reference in media channel I-D.\r\n",
"createdAt": "2021-04-08T10:42:43Z",
"updatedAt": "2021-07-12T16:19:22Z",
"closedAt": "2021-07-12T16:19:22Z",
"comments": [
{
"author": "danielkinguk",
"authorAssociation": "CONTRIBUTOR",
"body": "Dan and Oscar to have skype call and discuss. ",
"createdAt": "2021-04-08T12:30:25Z",
"updatedAt": "2021-04-08T12:30:25Z"
}
]
},
{
"number": 28,
"id": "MDU6SXNzdWU4NTMzMzQ1NDA=",
"title": "Container and When Statements",
"url": "https://github.com/ietf-ccamp-wg/draft-ietf-ccamp-flexigrid-yang/issues/28",
"state": "CLOSED",
"author": "danielkinguk",
"authorAssociation": "CONTRIBUTOR",
"assignees": [
"agva123",
"danielkinguk",
"italobusi",
"sergiobelotti"
],
"labels": [],
"body": "\"What is not clear with the model is the first container and the when statement. As another YANG doctor explained, the when-stmt applies to *instances* of a model, not to the schema. Therefore a when-stmt that points back to a node that is part of schema and is a presence container does not achieve what I think the authors intended, i.e. a when condition.\"\r\n\r\nHaving looked at other examples, several CCAMP documents make use of \"when\" statements, but non-CCAMP I-Ds do not. Do we really need the \"when\" statement? To me \u201cwhen\u201d might apply to a data object, such as \u201ctrue\u201d \u201cfalse\u201d \u201c3\u201d, etc. ",
"createdAt": "2021-04-08T10:44:53Z",
"updatedAt": "2022-06-30T12:26:29Z",
"closedAt": "2022-06-30T12:26:29Z",
"comments": [
{
"author": "danielkinguk",
"authorAssociation": "CONTRIBUTOR",
"body": "We should also check usage for WSON I-D. \r\n\r\nWSON YANG Dr Review - https://datatracker.ietf.org/doc/review-ietf-ccamp-wson-yang-15-yangdoctors-lc-lindem-2018-11-13/\r\nWSON https://datatracker.ietf.org/doc/html/draft-ietf-ccamp-wson-yang-15 in version 15 they commented out. \r\n",
"createdAt": "2021-04-08T10:45:23Z",
"updatedAt": "2021-04-08T10:45:23Z"
},
{
"author": "danielkinguk",
"authorAssociation": "CONTRIBUTOR",
"body": "Dan to check with NETMOD, but it looks like a legitimate use of the \"when\" statement as applied other TE documents. ",
"createdAt": "2021-04-08T12:22:29Z",
"updatedAt": "2021-04-08T12:22:29Z"
},
{
"author": "italobusi",
"authorAssociation": "MEMBER",
"body": "Aihua and I have made some checks on:\r\n\r\nhttps://stackoverflow.com/questions/5689966/how-to-check-if-an-element-exists-in-the-xml-using-xpath/5690522\r\n\r\nhttps://www.w3.org/TR/1999/REC-xpath-19991116/#booleans\r\n\r\nIt seems to us that the current when statements are correct since the boolean () XPath function is automatically applied and it returns true when the node exists\r\n\r\nHowever, it is still good to double check with NETMOD",
"createdAt": "2021-10-01T09:13:56Z",
"updatedAt": "2021-10-01T09:13:56Z"
},
{
"author": "danielkinguk",
"authorAssociation": "CONTRIBUTOR",
"body": "Legitimate use to be highlighted on list when we submit version 11. ",
"createdAt": "2021-10-07T11:25:09Z",
"updatedAt": "2021-10-07T11:25:09Z"
},
{
"author": "danielkinguk",
"authorAssociation": "CONTRIBUTOR",
"body": "<tp>\r\nI commented before that I struggle with the complexity of the 'when' statements in many CCAMP and TEAS I-D such as this one.\r\n\r\nIs it\r\n when \"../../../../../../../../../../\"\r\n + \"nw:network-types/tet:te-topology/\"\r\n or\r\n when \"../../../../../../../../../\"\r\n + \"nw:network-types/tet:te-topology/\"\r\n or\r\n when \"../../../../../../../nw:network-types/tet:te-topology/\"\r\n\r\n? answer all of them depending!\r\n\r\nI speculated that the absolute form of the path statement would be much simpler but it was pointed out to me by Martin that that would not work as 'nw:network is a list and so would return true if 'flexi-grid-topology' appeared anywhere in the list which in this case it is almost bound to; but there is the XPath 'ancestor' function which returns true only if the value appears in an immediate ancestor of the context node. This is almost what I want the caveat being that it returns true if any of the predicates - e.g. primary-path, route-object-exclude-object, etc - has the value in question but I believe that that will never be the case.\r\n\r\nIn which case,\r\n when \"ancestor::node()/nw:network-types/tet:te-topology\"\r\nas proposed by Lada will do the job for almost all the 'when' statements (and I can understand it ).\r\n\r\nThe thread is on the netmod WG list from 29dec21 under the subject\r\nYANG 'when' with absolute path\r\n\r\nTom Petch",
"createdAt": "2022-01-13T12:58:17Z",
"updatedAt": "2022-01-13T12:58:17Z"
},
{
"author": "danielkinguk",
"authorAssociation": "CONTRIBUTOR",
"body": "Lets investigate use of xpath ancestor (parent/grandparent) see: \r\n\r\nhttps://www.educba.com/xpath-ancestor/\r\n\r\nHow widespread is this function in IETF YANG modules? \r\n\r\nSergio and Italo will inspect further and we will also need to check with other similar implementations. \r\n",
"createdAt": "2022-01-13T13:26:55Z",
"updatedAt": "2022-01-13T13:26:55Z"
},
{
"author": "italobusi",
"authorAssociation": "MEMBER",
"body": "YANG model updated but we have found some issues with yanglint compilation:\r\n\r\nhttps://mailarchive.ietf.org/arch/msg/netmod/SvYTvbqNEeBT5IL74xtDQYwfy_A/\r\n",
"createdAt": "2022-01-17T16:08:27Z",
"updatedAt": "2022-01-17T16:08:27Z"
},
{
"author": "danielkinguk",
"authorAssociation": "CONTRIBUTOR",
"body": "Pyang compilation does not return any error/warnings, but when we use yanglint we see errors as it does recognize ancestor statements. \r\n\r\nResponse from Michal Va\u0161ko <mvasko@cesnet.cz> Tue, 18 January 2022 07:07 UTCShow header\r\n\r\n>>\r\nHi,\r\n\r\nyanglint uses its own XPath implementation, which unfortunately does not support axes, hence the error.\r\n\r\nRegards,\r\nMichal\r\n<<\r\n",
"createdAt": "2022-01-27T13:22:27Z",
"updatedAt": "2022-02-03T12:29:20Z"
},
{
"author": "danielkinguk",
"authorAssociation": "CONTRIBUTOR",
"body": "YANG experts have confirmed the YANG xpath axes ancestor structure and function is reasonable, but not widely used in other IETF modules, and we can ignore the libyang compilation errors due to a lack of support for axes. \r\n\r\nWe will summerise the situation with WG and ask other implementers for feedback. \r\n\r\n- Dan to email CCAMP WG for flexi-grid module feedback\r\n- All, decide if we should present an update on the issue to NETMOD and/or CCAMP at IETF 113 (March)\r\n",
"createdAt": "2022-02-03T13:33:08Z",
"updatedAt": "2022-02-03T13:33:08Z"
},
{
"author": "danielkinguk",
"authorAssociation": "CONTRIBUTOR",
"body": "An ongoing discussion with NETMOD and tools team Re: XPATH ancestor function. Although the NETMOD team is not negative towards using ancestor function in our model(s), the tools are not able to cope with the function, and we may need to revert ro a more conservative approach. ",
"createdAt": "2022-02-24T13:07:46Z",
"updatedAt": "2022-02-24T13:07:46Z"
},
{
"author": "danielkinguk",
"authorAssociation": "CONTRIBUTOR",
"body": "Reverted to YANG code that uses absolute path, in version 12. ",
"createdAt": "2022-06-30T12:26:29Z",
"updatedAt": "2022-06-30T12:26:29Z"
}
]
},
{
"number": 29,
"id": "MDU6SXNzdWU4NTMzMzU5NjM=",
"title": "Augmentation Statements",
"url": "https://github.com/ietf-ccamp-wg/draft-ietf-ccamp-flexigrid-yang/issues/29",
"state": "CLOSED",
"author": "danielkinguk",
"authorAssociation": "CONTRIBUTOR",
"assignees": [
"danielkinguk",
"italobusi",
"sergiobelotti"
],
"labels": [],
"body": "\"The second general comments has to do with the 80 augment statements. As an operator, I would find it very tedious to configure this model, what with 80 different XPaths to follow. Would it not be better to gather all the augmenting (flexi-grid) parameters together and use leafrefs to link these data structures to the ietf-network module?\"\r\n\r\nThis does seem like a valid comment? When wandering down the branches of the tree, why not add a whole new branch augmentation with the new objects in a single branch augmentation, instead of lots of augmentations? ",
"createdAt": "2021-04-08T10:46:31Z",
"updatedAt": "2022-01-27T13:18:21Z",
"closedAt": "2022-01-27T13:18:21Z",
"comments": [
{
"author": "dperdices",
"authorAssociation": "CONTRIBUTOR",
"body": "To provide more information, the actual version follows the example provided in RFC8795, see https://datatracker.ietf.org/doc/html/rfc8795#appendix-C.",
"createdAt": "2021-04-08T11:59:10Z",
"updatedAt": "2021-04-08T11:59:10Z"
},
{
"author": "italobusi",
"authorAssociation": "MEMBER",
"body": "This issue was discussed upfront with Netmod WG:\r\n\r\nhttps://mailarchive.ietf.org/arch/msg/netmod/-6L8KYVVhFiS8KpmH2fUpBmdSy8/\r\n\r\nAppendix C of RFC8795 is based on the feedbacks received from Netmod WG.",
"createdAt": "2021-04-08T13:00:02Z",
"updatedAt": "2021-04-08T13:00:02Z"
},
{
"author": "danielkinguk",
"authorAssociation": "CONTRIBUTOR",
"body": "Legitimate used to be highlighted on list when we submit version 11. ",
"createdAt": "2021-10-07T11:25:43Z",
"updatedAt": "2021-10-07T11:25:43Z"
},
{
"author": "danielkinguk",
"authorAssociation": "CONTRIBUTOR",
"body": "Continued in issue #28 ",
"createdAt": "2022-01-27T13:18:21Z",
"updatedAt": "2022-01-27T13:18:21Z"
}
]
},
{
"number": 30,
"id": "MDU6SXNzdWU4NTMzMzczNTc=",
"title": "Tree Structure ",
"url": "https://github.com/ietf-ccamp-wg/draft-ietf-ccamp-flexigrid-yang/issues/30",
"state": "CLOSED",
"author": "danielkinguk",
"authorAssociation": "CONTRIBUTOR",
"assignees": [
"danielkinguk",
"italobusi"
],
"labels": [],
"body": "Having 17 pages of a tree diagram without any explanation for the different sections of the tree diagram is hardly helpful. On the other hand an abridged version of the tree diagram explaining the different sections would be more helpful normative text. Suggest that --tree-depth and --tree-path options be used in pyang to reduce the size of the tree and to break it up into smaller\r\npieces that can then be explained. The full tree diagram can be added as non-normative text in the Appendix.\r\n\r\nThis is a valid comment, having looked at other YANG models I found https://datatracker.ietf.org/doc/html/draft-ietf-opsawg-l3sm-l3nm-07#section-7 as a good example. ",
"createdAt": "2021-04-08T10:48:13Z",
"updatedAt": "2021-07-12T16:18:54Z",
"closedAt": "2021-07-12T16:18:54Z",
"comments": [
{
"author": "italobusi",
"authorAssociation": "MEMBER",
"body": "Please find some proposed text for clause 5 (similar to the text we have provided for OTN topology) to address this issue\r\n\r\n[draft-ietf-ccamp-flexigrid-yang-09-ib@2021-04-26.docx](https://github.com/ietf-ccamp-wg/draft-ietf-ccamp-flexigrid-yang/files/6390242/draft-ietf-ccamp-flexigrid-yang-09-ib%402021-04-26.docx)\r\n\r\nPlease note that this document contains also some review comments to other sections of the draft (for further discussion)\r\n",
"createdAt": "2021-04-28T08:24:47Z",
"updatedAt": "2021-04-28T08:24:47Z"
},
{
"author": "dperdices",
"authorAssociation": "CONTRIBUTOR",
"body": "Thank you, Italo! In my opinion, this new section 5 is really explanatory and makes the model easier to understand so readers will surely appreciate it.\r\n\r\nI leave here some minor issues to check before merging. (English-related things should be double-checked by @danielkinguk, just in case I report things that are in fact correct, my apologies in advance).\r\n- \"Therefore the YANG\" => \"Therefore, the YANG\"\r\n- \"an Flexi-Grid\" => \"a flexi-grid\"\r\n- In the rest of the document, we use flexi-grid (with this capitalization, instead of Flexi-Grid)\r\n- Section 3.2 should not be 3.2 but 5.X. In general, when merging this, we should revise the section numbering.\r\n- \"in the topology as follow.\" => \"in the topology as follows:\"\r\n- \"Therefore the model does not define\" => \"Therefore, the model does not define\"\r\n",
"createdAt": "2021-04-29T08:42:11Z",
"updatedAt": "2021-04-29T08:42:11Z"
},
{
"author": "danielkinguk",
"authorAssociation": "CONTRIBUTOR",
"body": "Dan to integrate into XML version v10.",
"createdAt": "2021-04-29T12:09:28Z",
"updatedAt": "2021-04-29T12:09:28Z"
}
]
},
{
"number": 31,
"id": "MDU6SXNzdWU4NjQ4NzE5NzA=",
"title": "Change affiliation of Victor to Nokia",
"url": "https://github.com/ietf-ccamp-wg/draft-ietf-ccamp-flexigrid-yang/issues/31",
"state": "CLOSED",
"author": "danielkinguk",
"authorAssociation": "CONTRIBUTOR",
"assignees": [],
"labels": [],
"body": "",
"createdAt": "2021-04-22T12:08:11Z",
"updatedAt": "2021-07-12T15:03:09Z",
"closedAt": "2021-07-12T15:03:09Z",
"comments": []
},
{
"number": 32,
"id": "MDU6SXNzdWU4NjQ5MDkxNzQ=",
"title": "Definition of media-channel and superchannel",
"url": "https://github.com/ietf-ccamp-wg/draft-ietf-ccamp-flexigrid-yang/issues/32",
"state": "CLOSED",
"author": "danielkinguk",
"authorAssociation": "CONTRIBUTOR",
"assignees": [
"danielkinguk"
],
"labels": [],
"body": "Definitions will be clarified and described in impairments or layer-0 types document, so we can reference there.",
"createdAt": "2021-04-22T12:47:22Z",
"updatedAt": "2021-07-12T15:01:43Z",
"closedAt": "2021-07-12T15:01:43Z",
"comments": []
},
{
"number": 33,
"id": "MDU6SXNzdWU4NzA5NTA4NTQ=",
"title": "All authors - Confirm prefix agreement with Tom/CCAMP list ",
"url": "https://github.com/ietf-ccamp-wg/draft-ietf-ccamp-flexigrid-yang/issues/33",
"state": "CLOSED",
"author": "danielkinguk",
"authorAssociation": "CONTRIBUTOR",
"assignees": [
"danielkinguk"
],
"labels": [],
"body": "Dan to update V10 I-D to use \"flexgt\". \r\n",
"createdAt": "2021-04-29T12:14:53Z",
"updatedAt": "2021-07-12T14:57:31Z",
"closedAt": "2021-07-12T14:57:31Z",
"comments": []
},
{
"number": 34,
"id": "MDU6SXNzdWU4NzY2ODE3NjU=",
"title": "Issue compiling XML file",
"url": "https://github.com/ietf-ccamp-wg/draft-ietf-ccamp-flexigrid-yang/issues/34",
"state": "CLOSED",
"author": "italobusi",
"authorAssociation": "MEMBER",
"assignees": [
"danielkinguk"
],
"labels": [],
"body": "I am having issues compiling the latest XML file: 8db86ff14251f07d648dc14df95fe59605434c94\r\n\r\nI have used the on-line tool: https://xml2rfc.tools.ietf.org/\r\n\r\nThis is the error message that I got:\r\n\r\n> Error: Unable to parse the XML document: INPUT\r\n> INPUT: Line 150: Opening and ending tag mismatch: t line 132 and section\r\n> INPUT: Line 150: Opening and ending tag mismatch: t line 132 and section\r\n> INPUT: Line 2966: Opening and ending tag mismatch: section line 132 and middle\r\n> INPUT: Line 3027: Opening and ending tag mismatch: middle line 132 and rfc\r\n> INPUT: Line 3027: EndTag: '</' not found\r\n> ",
"createdAt": "2021-05-05T17:30:09Z",
"updatedAt": "2021-07-12T14:49:39Z",
"closedAt": "2021-07-12T14:49:39Z",
"comments": []
},
{
"number": 36,
"id": "MDU6SXNzdWU5NDk0ODIzODE=",
"title": "Editorial comment to section 5.4 of revision 10",
"url": "https://github.com/ietf-ccamp-wg/draft-ietf-ccamp-flexigrid-yang/issues/36",
"state": "CLOSED",
"author": "italobusi",
"authorAssociation": "MEMBER",
"assignees": [
"danielkinguk"
],
"labels": [],
"body": "This paragraph duplicates the content of section 5.3 (looks like an unintended cut&paste):\r\n\r\nhttps://github.com/ietf-ccamp-wg/draft-ietf-ccamp-flexigrid-yang/blob/37aa8c588cd504a859579948cf1289fa54d1cf4d/draft-ietf-ccamp-flexigrid-yang-10.xml#L396-L399\r\n\r\nI think it should be removed\r\n\r\nSee: https://datatracker.ietf.org/doc/html/draft-ietf-ccamp-flexigrid-yang#section-5.4",
"createdAt": "2021-07-21T08:54:34Z",
"updatedAt": "2021-10-20T15:13:17Z",
"closedAt": "2021-10-20T15:13:17Z",
"comments": [
{
"author": "danielkinguk",
"authorAssociation": "CONTRIBUTOR",
"body": "Agree.\r\n",
"createdAt": "2021-07-21T09:03:08Z",
"updatedAt": "2021-07-21T09:03:08Z"
},
{
"author": "danielkinguk",
"authorAssociation": "CONTRIBUTOR",
"body": "Fixed. ",
"createdAt": "2021-10-07T11:24:10Z",
"updatedAt": "2021-10-07T11:24:10Z"
}
]
},
{
"number": 38,
"id": "I_kwDOEaoOG848yuph",
"title": "All - Check version 11 of the document, then submit. ",
"url": "https://github.com/ietf-ccamp-wg/draft-ietf-ccamp-flexigrid-yang/issues/38",
"state": "CLOSED",
"author": "danielkinguk",
"authorAssociation": "CONTRIBUTOR",
"assignees": [
"danielkinguk"
],
"labels": [],
"body": "",
"createdAt": "2021-10-07T11:26:05Z",
"updatedAt": "2021-11-11T15:39:11Z",
"closedAt": "2021-11-11T15:39:11Z",
"comments": [
{
"author": "danielkinguk",
"authorAssociation": "CONTRIBUTOR",
"body": "Please let me have comments by Tuesday 19, 2021. ",
"createdAt": "2021-10-14T12:07:43Z",
"updatedAt": "2021-10-14T12:07:43Z"
},
{
"author": "italobusi",
"authorAssociation": "MEMBER",
"body": "I think the -11 version is ready for submission. I have few comments that could be addressed either before -11 submission or afterwards (before or during WG LC):\r\n\r\n1. an English review would be helpful.\r\n For example, \u201cthe following example is provided\u201d instead of \u201cwe provide the following example\u201d)\r\n2. In the introduction it would be useful to clarify that this model does not require GMPLS implementation:\r\n **OLD**\r\n This document presents a YANG data model [RFC7950] for flexi-grid\r\n objects in the dynamic optical network, including nodes, transponders\r\n and links, as well as how such links interconnect nodes. \r\n **NEW**\r\n This document presents a YANG data model [RFC7950] for flexi-grid\r\n objects in the dynamic optical network, including nodes, transponders\r\n and links, as well as how such links interconnect nodes. This model is\r\n independent of control plane protocols.\r\n3. The use of term \"media channel\" thought the document needs to be reviewed, in line with https://github.com/ietf-ccamp-wg/draft-ietf-ccamp-flexigrid-media-channel-yang/issues/33\r\n4. The layer0-types I-D has now been published as RFC9093",
"createdAt": "2021-10-20T15:23:14Z",
"updatedAt": "2021-10-20T15:23:14Z"
},
{
"author": "danielkinguk",
"authorAssociation": "CONTRIBUTOR",
"body": "We wont be able to refer to the new draft-ietf-ccamp-flexigrid-tunnel-yang I-D until the Chairs approve the new 00 version (uploaded earlier today). I will queue the I-D for submission when the tool opens again at the start of IETF week. ",
"createdAt": "2021-10-25T18:51:32Z",
"updatedAt": "2021-10-25T18:51:32Z"
},
{
"author": "danielkinguk",
"authorAssociation": "CONTRIBUTOR",
"body": "Comments 2, 3 & 4 are addressed in the latest version of the I-D. ",
"createdAt": "2021-10-25T18:55:40Z",
"updatedAt": "2021-10-25T18:55:40Z"
}
]
},
{
"number": 39,
"id": "I_kwDOEaoOG849LF_b",
"title": "Move new YANG code into version 11 of the I-D",
"url": "https://github.com/ietf-ccamp-wg/draft-ietf-ccamp-flexigrid-yang/issues/39",
"state": "CLOSED",
"author": "danielkinguk",
"authorAssociation": "CONTRIBUTOR",
"assignees": [
"danielkinguk"
],
"labels": [],
"body": "To be done before submission of I-D. ",
"createdAt": "2021-10-14T12:08:27Z",
"updatedAt": "2021-10-25T18:47:04Z",
"closedAt": "2021-10-25T18:47:04Z",
"comments": []
},
{
"number": 40,
"id": "I_kwDOEaoOG85ApqAa",
"title": "LC Review from Tom",
"url": "https://github.com/ietf-ccamp-wg/draft-ietf-ccamp-flexigrid-yang/issues/40",
"state": "CLOSED",
"author": "danielkinguk",
"authorAssociation": "CONTRIBUTOR",
"assignees": [
"danielkinguk"
],
"labels": [],
"body": "<tp>\r\nwrong YANG prefix (s.9)\r\n\r\nAfter all the discussions on CCAMP and TEAS WG lists about what would make for a good prefix, this does not bode well - sigh\r\n\r\n<tp2>\r\nAnd while I am on the admin ...\r\n\r\nThe dates ib the module are wrong but differently wrong in different places - if it were all the same wrong I would live with it.\r\n\r\nSecurity Considerations make no mention of any nodes in this I-D - I doubt if the AD will countenance that.\r\n\r\nRFC Editor Note Please replace XXXX with the numbers assigned to RFC9093 Really? Look in the YANG module revision reference and you may think again\r\n\r\nTLP is out of date\r\n\r\nCopyright out of date - perhaps 2022!\r\n\r\nURL lack security (https:) in several places\r\n\r\nURL to an obsolete web page (tool)\r\n\r\n[G.694.1] .... G-872\r\nReally?\r\n\r\nAs I have commented before, it is a pig to review because TE got its YANG structure wrong in the first place so whether or not I will look at the semantics of the model , I do not know - perhaps if Boxing Day gets very boring:-)",
"createdAt": "2021-12-20T11:12:44Z",
"updatedAt": "2022-06-30T13:05:52Z",
"closedAt": "2022-06-30T13:05:51Z",
"comments": [
{
"author": "danielkinguk",
"authorAssociation": "CONTRIBUTOR",
"body": "1. Date used in file name and revision needs to be consistent. (DK)\r\n\r\n2. For \"RFC XXXX\" and \"RFC YYYY\" need to check usage within I-D for consistency (DK)\r\n\r\n3. IANA Section referred to old prefix. (DK)\r\n\r\nOLD\r\n IANA is requested to assign a new YANG module name in the \"YANG\r\n Module Names\" registry [RFC6020] as follows:\r\n\r\n Name: ietf-flexi-grid-topology\r\n Namespace: urn:ietf:params:xml:ns:yang:ietf-flexi-grid-topology\r\n Prefix: flexi-grid-topology\r\n Reference: [This.I-D]\r\nNEW\r\n IANA is requested to assign a new YANG module name in the \"YANG\r\n Module Names\" registry [RFC6020] as follows:\r\n\r\n Name: ietf-flexi-grid-topology\r\n Namespace: urn:ietf:params:xml:ns:yang:ietf-flexi-grid-topology\r\n Prefix: flexgt\r\n Reference: RFC XXXX\r\n\r\n4. Need to check how we reference an ITU-T amendment. (see Issue #41)\r\n\r\n5. Security issues in code, check WSON TOPO (9094) and TE TOPO [RFC8795] (DK)\r\n\r\n6. URL usage should be \"https\" - Check uses of \"http\" and use \"https\" wherever possible (DK)\r\n\r\n7. Time to move from \"Informational\" to \"Standards Track\"? Italo to raise on list. - We should change to \"Standards\" (DK)\r\n\r\nAlso waiting on additional comments from LC. ",
"createdAt": "2021-12-23T13:09:24Z",
"updatedAt": "2022-01-13T13:19:32Z"
},
{
"author": "italobusi",