Skip to content

[countersyncd]: resolve object names by template id#211

Merged
Pterosaur merged 2 commits intoAzure:202412from
Pterosaur:backport/4316-to-202412
Mar 13, 2026
Merged

[countersyncd]: resolve object names by template id#211
Pterosaur merged 2 commits intoAzure:202412from
Pterosaur:backport/4316-to-202412

Conversation

@Pterosaur
Copy link

What I did

Backported sonic-net/sonic-swss#4316 to the 202412 branch.

This preserves the template-id keyed object-name resolution fix in countersyncd for the 202412 code line.

Why I did it

Without this fix, object-name resolution can pick the wrong template context and produce incorrect name association for HFT/IPFIX processing.

How I verified it

Backport was validated as part of the combined 202412 countersyncd stack verification.

A merged 202412 build containing the backports of public PRs 4316/4317/4318/4319 was built and deployed to DUT str4-sn5640-2, then the full HFT regression was run:

  • high_frequency_telemetry/test_high_frequency_telemetry.py
  • result: tests=14, failures=0, errors=0, skipped=7
  • image: 20241212.54

Details if related

Source public PR: sonic-net/sonic-swss#4316

Use the current IPFIX data set template ID to resolve the correct\nmessage key and object_names instead of picking an arbitrary key from\ntemporary/applied template maps.\n\nThis avoids misattributing counters when multiple templates or sessions\ncoexist in the same actor.\n\nAdd a regression test covering two template IDs with different\nobject_names in the same input stream.\n\nVerified in CI-like container with:\n  cargo test --locked --manifest-path crates/countersyncd/Cargo.toml \\n    test_object_names_follow_template_id -- --nocapture

Signed-off-by: Ze Gan <ganze718@gmail.com>
(cherry picked from commit ad034ddbd69eabbb5049980e397f865741d5d834)
Signed-off-by: Ze Gan <ganze718@gmail.com>
(cherry picked from commit 6d66637355f072f50b7acef4ef32755db5b50b16)
@Pterosaur Pterosaur requested a review from prsunny as a code owner March 13, 2026 01:31
@Pterosaur Pterosaur requested a review from r12f March 13, 2026 01:36
@Pterosaur Pterosaur merged commit 5a95721 into Azure:202412 Mar 13, 2026
5 checks passed
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