Package updates by rgwilton · Pull Request #274 · netmod-wg/yang-ver-dt

rgwilton · 2026-01-30T10:21:17Z

PR for current round of package updates

Improve text for #120

…ang-ver-dt into remove-pkg-exclusion

Copilot

Pull request overview

This PR updates the YANG package specification and associated documentation to version 0.8.0, incorporating refinements to the package model structure, terminology clarifications, and expanded conformance guidance.

Changes:

Updated package version from 0.7.0 to 0.8.0 with revision date 2026-01-30
Introduced new module-name typedef and restructured import-only-module exclusion mechanism to support version-specific exclusions
Enhanced conformance section with clearer guidance on package schema comparison, deviation usage, and handling of multiple included package versions

Reviewed changes

Copilot reviewed 2 out of 3 changed files in this pull request and generated 3 comments.

File	Description
yang-packages/ietf-yang-package-types.yang	Updated module version, added module-name typedef, restructured import-only-module exclusions to support per-version exclusions, removed replaces-version field, added mandatory constraints, reformatted descriptions
yang-packages/draft-ietf-netmod-yang-packages.xml	Updated terminology definitions, clarified conformance requirements, restructured and expanded conformance section, updated examples and references to use consistent terminology

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Copilot · 2026-01-31T17:42:05Z

yang-packages/draft-ietf-netmod-yang-packages.xml


            <t>A package definition CANNOT remove a package from a mount point,
-            but it MAY replace it a different package version, which SHOULD be
+            but it MAY add a different package version, which SHOULD be


The phrase 'add a different package version' is unclear. Based on the context that this is about replacing a package at a mount point, consider 'replace it with a different package version' for clarity.

Suggested change

but it MAY add a different package version, which SHOULD be

but it MAY replace it with a different package version, which SHOULD be

Copilot · 2026-01-31T17:42:05Z

yang-packages/draft-ietf-netmod-yang-packages.xml

+      <t>Better and easier conformance is a major design goal for YANG Packages.  YANG
+      package conformance is similiar to how YANG <xref target="RFC7950"/>
+      requires that servers either implement a module faithfully, or otherwise
+      use deviations to indicate areas of non-conformance.  Ulitmately, each


Corrected spelling of 'Ulitmately' to 'Ultimately'.

Suggested change

use deviations to indicate areas of non-conformance. Ulitmately, each

use deviations to indicate areas of non-conformance. Ultimately, each

Copilot · 2026-01-31T17:42:05Z

yang-packages/draft-ietf-netmod-yang-packages.xml

+      use deviations to indicate areas of non-conformance.  Ulitmately, each
+      version of a YANG package resolves, as per (<xref target="resolution"/>),
+      to a YANG schema that is defined as a set of implemented modules and
+      import-only modules, devations, features, and mounted schema.  For YANG


Corrected spelling of 'devations' to 'deviations'.

Suggested change

import-only modules, devations, features, and mounted schema. For YANG

import-only modules, deviations, features, and mounted schema. For YANG

reshad-equinix · 2026-01-31T17:43:09Z

yang-packages/draft-ietf-netmod-yang-packages.txt

+   *  resolved package schema: The resolved package schema is the YANG
+      schema defined by a package after package resolution has been
+      performed, as defined in Section 4.  The resolved package schema
+      identifies the implemented modules (with any deviations applied),


So if a module has been removed, that module wouldn't be part of implemented other import-only module list?

reshad-equinix · 2026-01-31T17:44:43Z

yang-packages/draft-ietf-netmod-yang-packages.txt


+   *  resolved package schema: The resolved package schema is the YANG
+      schema defined by a package after package resolution has been


Should we have "YANG schema" as a term introduced in another doc?

reshad-equinix · 2026-01-31T17:46:44Z

yang-packages/draft-ietf-netmod-yang-packages.txt


   *  Tooling should be able to easily work with YANG package
-      definitions to compare package versions and to compare server
+      definitions to compare YANG package versions and to compare server


Drawing a bit of a blank on "expected package definition". So we expect Package X 1.0.0 to have a certain schema and we want to compare whether a server/device actually has that schema? Or are we comparing something else e.g. module versions?

reshad-equinix · 2026-01-31T17:53:32Z

yang-packages/draft-ietf-netmod-yang-packages.txt

+      version of the package.  A package definition MAY also remove
+      modules or mandatory-features.  In all cases, this is achieved by
+      listing the replacement package in the 'mounts/package' list,
+      which MUST import a version of the package to be replaced.


If we don't have it already, it would be good to have an example of that.

reshad-equinix · 2026-01-31T17:57:06Z

yang-packages/draft-ietf-netmod-yang-packages.txt

+      Identities are only available for use by the server for
+      implemented modules.


?
I am not understanding this.

reshad-equinix · 2026-01-31T18:05:30Z

yang-packages/draft-ietf-netmod-yang-packages.xml

+      <t>Better and easier conformance is a major design goal for YANG Packages.  YANG
+      package conformance is similiar to how YANG <xref target="RFC7950"/>
+      requires that servers either implement a module faithfully, or otherwise
+      use deviations to indicate areas of non-conformance.  Ulitmately, each


Suggested change

use deviations to indicate areas of non-conformance. Ulitmately, each

use deviations to indicate areas of non-conformance. Ultimately, each

reshad-equinix · 2026-01-31T18:07:23Z

yang-packages/draft-ietf-netmod-yang-packages.xml

+      datastore schema used by the server is compatible with the client.</t>
+
+      <t>In the case where a device does not completely conform to
+      an standard or industry defined YANG package definition, then there are a


Suggested change

an standard or industry defined YANG package definition, then there are a

a standard or industry defined YANG package definition, then there are a

reshad-equinix · 2026-01-31T18:08:44Z

yang-packages/draft-ietf-netmod-yang-packages.txt


-7.1.  Resolving Conflicting Module Versions in Included Packages
+7.2.  Handling multiple included package versions


Is included packages the same as import-only packages?

reshad-equinix · 2026-01-31T18:09:48Z

yang-packages/draft-ietf-netmod-yang-packages.txt

+       description
+         "Module names are typed as YANG identifiers.";
+     }
+
     typedef version-or-rev-date {


I thought we already had a type for this in yang-server, will check

reshad-equinix · 2026-01-31T18:17:19Z

yang-packages/draft-ietf-netmod-yang-packages.txt

Shouldn't we flag this? I mean we have to choose between 2 NBC module versions, to me that indicates incorrect package.

rgwilton added 8 commits January 12, 2026 15:44

use "mandatory-features" rather than "mandatory features"

0f8e21a

Tweak to improve the text for #120

cd0df11

Improve text for #120

Merge branch 'remove-pkg-exclusion' of https://github.com/netmod-wg/y…

e7c98a3

…ang-ver-dt into remove-pkg-exclusion

Proposed changes for #239

22c7fae

Rework conformance section

723c6b9

Add missing version update.

8b5c88a

Ban circular packge includes

3bad6db

Add text for #264

7e19931

reshad-equinix requested a review from Copilot January 31, 2026 17:41

Copilot AI reviewed Jan 31, 2026

View reviewed changes

reshad-equinix reviewed Jan 31, 2026

View reviewed changes

rgwilton added 8 commits February 3, 2026 17:43

Some text improvements for #256

94b94de

Define a file type for package #246

2fe0d47

Tweaked improvements for #245

47a76ee

Some improved text for #254

fd4b147

Tweaked text improvements for #258

94242d2

Add IANA registry #31

9791bc6

Remove plurals for some lists/leaf-lists #254 #275

8140190

Update security considerations to latest template #275

ca9c2df

	but it MAY add a different package version, which SHOULD be
	but it MAY replace it with a different package version, which SHOULD be

	use deviations to indicate areas of non-conformance. Ulitmately, each
	use deviations to indicate areas of non-conformance. Ultimately, each

	import-only modules, devations, features, and mounted schema. For YANG
	import-only modules, deviations, features, and mounted schema. For YANG


		* resolved package schema: The resolved package schema is the YANG
		schema defined by a package after package resolution has been

		Identities are only available for use by the server for
		implemented modules.

	an standard or industry defined YANG package definition, then there are a
	a standard or industry defined YANG package definition, then there are a


		7.1. Resolving Conflicting Module Versions in Included Packages
		7.2. Handling multiple included package versions

Conversation

rgwilton commented Jan 30, 2026

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Reviewed changes

Uh oh!

Copilot AI Jan 31, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Jan 31, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Jan 31, 2026

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants