Skip to content

Comments

[action] [PR:4131] [countersyncd]: Add retry between client and otel collector#197

Merged
mssonicbld merged 1 commit intoAzure:202412from
mssonicbld:cherry/msft-202412/4131
Jan 23, 2026
Merged

[action] [PR:4131] [countersyncd]: Add retry between client and otel collector#197
mssonicbld merged 1 commit intoAzure:202412from
mssonicbld:cherry/msft-202412/4131

Conversation

@mssonicbld
Copy link
Collaborator

What I did
Added a retry mechanism in countersyncd for communication between the OTEL client and the Open Telemetry collector.

Why I did it
To improve the robustness of countersyncd by handling transient connection failures, ensuring metrics are reliably sent even if the otel collector is temporarily unavailable.

How I verified it

Details if related

@mssonicbld
Copy link
Collaborator Author

Original PR: sonic-net/sonic-swss#4131

@mssonicbld
Copy link
Collaborator Author

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@r12f r12f force-pushed the cherry/msft-202412/4131 branch from 50da66f to 72467f6 Compare January 23, 2026 21:15
<!--
Please make sure you have read and understood the contribution guildlines:
https://github.com/Azure/SONiC/blob/gh-pages/CONTRIBUTING.md

1. Make sure your commit includes a signature generted with `git commit -s`
2. Make sure your commit title follows the correct format: [component]: description
3. Make sure your commit message contains enough details about the change and related tests
4. Make sure your pull request adds related reviewers, asignees, labels

Please also provide the following information in this pull request:
-->

**What I did**
Added a retry mechanism in countersyncd for communication between the OTEL client and the Open Telemetry collector.

**Why I did it**
To improve the robustness of countersyncd by handling transient connection failures, ensuring metrics are reliably sent even if the otel collector is temporarily unavailable.

**How I verified it**

**Details if related**
@r12f r12f force-pushed the cherry/msft-202412/4131 branch from 72467f6 to e9bf973 Compare January 23, 2026 21:26
@mssonicbld mssonicbld merged commit 697b453 into Azure:202412 Jan 23, 2026
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant