Add IETF 126 feature comparison feedback notes - #46
Closed
lllyfeng wants to merge 10 commits into
Closed
Conversation
Record feedback points for the YANG Libraries Feature Comparison presentation (IETF 126 NETMOD WG). Key point: the anydata validation "Partially Supported" rating is debatable — yangkit's explicit schema registration requirement aligns with RFC 7950 Section 7.10 semantics, while treating unschema'd anydata as opaque data (libyang's approach) is preservation, not validation. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01UzakZ2mfgufYyjbfwBCday
YANG XPath (when/must/leafref/instance-identifier) operates on the data tree by definition, skipping choice/case schema-only nodes. Using XPath to query schema nodes (libyang's lys_find_xpath) is an extra convenience, not a YANG spec requirement. yangkit-xpath fully implements RFC 7950 XPath semantics. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01UzakZ2mfgufYyjbfwBCday
yangkit-model supports dynamic add/remove/modify of schema nodes at design time (model editing/compilation). yangkit-data requires an immutable schema for validation, which is correct — mutating schema during validation is unacceptable. The test conflates model editing (design phase) with data validation (runtime phase). Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01UzakZ2mfgufYyjbfwBCday
Yangkit supports updating data nodes via setValue() followed by validate(). Decoupling mutation from validation is a design choice that enables efficient batch updates. The slides' own annotation "can easily be changed in code source" contradicts the Not Supported rating. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01UzakZ2mfgufYyjbfwBCday
yang-comparator performs schema-tree-level semantic comparison, not text-level diff. It was the reason yangkit was chosen by Swisscom and the author joined the YANG Module Versioning Design Team. Rating it as "Implementable" misrepresents an already mature, production-proven capability. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01UzakZ2mfgufYyjbfwBCday
Draft email to the presenters (INSA Lyon) with CC to netmod@ietf.org, raising constructive feedback on 5 of 6 yangkit evaluation ratings. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01UzakZ2mfgufYyjbfwBCday
Key findings added: - updateValueTest uses addChild() instead of setValue() (API misuse) - Multiple anydata tests have unresolved TODOs from testers themselves - XPath "failure" is only anydata payload traversal (same schema issue) - Testers wrote their own schema comparator using yangkit APIs but rated the capability as "Implementable" Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01UzakZ2mfgufYyjbfwBCday
…less of schema RFC 7950 Section 7.10 defines anydata as an indivisible node with no addressable children. XPath cannot traverse into anydata internals whether or not a schema is registered — this is the data model's design, not an implementation gap. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01UzakZ2mfgufYyjbfwBCday
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01UzakZ2mfgufYyjbfwBCday
…aboration Remove references to testers' uncertainties and personal credentials. Reframe as capability clarification + proposal for community test spec. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01UzakZ2mfgufYyjbfwBCday
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Record feedback points for the YANG Libraries Feature Comparison
presentation (IETF 126 NETMOD WG). Key point: the anydata validation
"Partially Supported" rating is debatable — yangkit's explicit schema
registration requirement aligns with RFC 7950 Section 7.10 semantics,
while treating unschema'd anydata as opaque data (libyang's approach)
is preservation, not validation.
Co-Authored-By: Claude Opus 4.6 noreply@anthropic.com
Claude-Session: https://claude.ai/code/session_01UzakZ2mfgufYyjbfwBCday