Skip to content

Comments

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

Merged
mssonicbld merged 1 commit intosonic-net:202511from
mssonicbld:cherry/202511/4131
Feb 15, 2026
Merged

[action] [PR:4131] [countersyncd]: Add retry between client and otel collector#4220
mssonicbld merged 1 commit intosonic-net:202511from
mssonicbld:cherry/202511/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

<!--
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**
@mssonicbld
Copy link
Collaborator Author

Original PR: #4131

@mssonicbld
Copy link
Collaborator Author

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@mssonicbld mssonicbld merged commit 3c4d3b2 into sonic-net:202511 Feb 15, 2026
15 of 16 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