Skip to content

[Fix]: Allow empty object lists in high-frequency-telemetry YANG model#1593

Merged
Pterosaur merged 1 commit intoAzure:202412from
Pterosaur:hft_yang
Sep 10, 2025
Merged

[Fix]: Allow empty object lists in high-frequency-telemetry YANG model#1593
Pterosaur merged 1 commit intoAzure:202412from
Pterosaur:hft_yang

Conversation

@Pterosaur
Copy link
Contributor

Origin PR: sonic-net/sonic-buildimage#23917

Why I did it

We must have a dummy HFT table to support dynamically configuring HFT. This Yang model change is to support the dummy table.

Work item tracking
  • Microsoft ADO (number only):

How I did it

  • Remove mandatory constraint for object_names list
  • Add test cases for empty lists and validation scenarios
  • Include high-frequency telemetry model

How to verify it

Check Azp status. This PR includes its test.

Which release branch to backport (provide reason below if selected)

  • 202412

Tested branch (Please provide the tested image version)

master

…l (#23917)

Why I did it
We must have a dummy HFT table to support dynamically configuring HFT. This Yang model change is to support the dummy table.

How I did it
Remove mandatory constraint for object_names list
Add test cases for empty lists and validation scenarios
Include high-frequency telemetry model
How to verify it
Check Azp status. This PR includes its test.
@Pterosaur Pterosaur changed the base branch from master to 202412 September 10, 2025 00:51
@Pterosaur Pterosaur merged commit 0681b74 into Azure:202412 Sep 10, 2025
3 checks passed
r12f pushed a commit that referenced this pull request Oct 26, 2025
#1593)

Origin PR: sonic-net/sonic-buildimage#23917

#### Why I did it
We must have a dummy HFT table to support dynamically configuring HFT.
This Yang model change is to support the dummy table.

##### Work item tracking
- Microsoft ADO **(number only)**:

#### How I did it
- Remove mandatory constraint for object_names list
- Add test cases for empty lists and validation scenarios
- Include high-frequency telemetry model
- 
#### How to verify it

Check Azp status. This PR includes its test.

#### Which release branch to backport (provide reason below if selected)

- [x] 202412

#### Tested branch (Please provide the tested image version)

master
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.

1 participant