Skip to content

feat(#504): enable SONiC device profile (parallel NOS stack) - #507

Open
steiler wants to merge 2 commits into
device-profile-basefrom
sonic-device-profile
Open

steiler wants to merge 2 commits into
device-profile-basefrom
sonic-device-profile

Conversation

@steiler

@steiler steiler commented Sep 24, 2026

Copy link
Copy Markdown
Collaborator

Summary

Parallel SONiC NOS stack off #504 device-profile base: enables sonic only, JSON_IETF validation, parent-bound Set encoding, materialize dispatch, and Get shaping for translib.

Supersedes SONiC-specific scope from legacy #480 (not stacked on Cisco).

Parent: #504

Preconditions

Test plan

  • Config, server mapping, materialize routing, encoder unit tests

Lift base-layer rejection for sonic only, add JSON_IETF validation, parent-bound
Set encoding and translib Get shaping, and materialize dispatch to the sonic encoder.
Cisco IOS-XR remains fail-closed; netconf+sonic is rejected at config load.
@codecov

codecov Bot commented Sep 24, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 80.59701% with 26 lines in your changes missing coverage. Please review.

Files with missing lines Patch % Lines
pkg/datastore/target/gnmi/sonic/encode.go 80.67% 12 Missing and 11 partials ⚠️
pkg/datastore/target/materialize/materialize.go 66.66% 1 Missing and 1 partial ⚠️
pkg/datastore/target/gnmi/gnmi.go 0.00% 1 Missing ⚠️

📢 Thoughts on this report? Let us know!

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

Clear errcheck/ineffassign/staticcheck findings so Go Checks lint passes.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants