Skip to content

[otel container]: Create otel container for HFT (#23666)#1930

Merged
r12f merged 5 commits into202412from
dev/jc/otel-image
Jan 23, 2026
Merged

[otel container]: Create otel container for HFT (#23666)#1930
r12f merged 5 commits into202412from
dev/jc/otel-image

Conversation

@Janetxxx
Copy link
Member

@Janetxxx Janetxxx commented Jan 13, 2026

[otel container]: Create otel container for HFT
Resolve conflicts in sonic-net/sonic-buildimage#23666

Why I did it

This PR adds OpenTelemetry (OTEL) collector support to the system by introducing a new docker-sonic-otel container. The OTEL collector enables telemetry data collection and export for High Frequency Telemetry (HFT) support.

Work item tracking
  • Microsoft ADO (number only):

How I did it

  • Introduces new docker-sonic-otel container with configuration and build files
  • Updates service health checker to handle OTEL container conditionally based on image availability
  • Adds build system variables and configuration for conditional OTEL

How to verify it

Run sudo config feature state otel enabled to enable the otel collector feature, and tested it on KVM T0 testbed (ceos neighbor deployed).

Checked otel container:

admin@vlab-01:~$ docker ps -a | grep otel
400fca26a6c4   docker-sonic-otel:latest             "/usr/local/bin/supe…"   21 hours ago   Up 21 hours             otel

root@vlab-01:/# supervisorctl status
dependent-startup                EXITED    Dec 04 03:07 AM
otel                             RUNNING   pid 219, uptime 0:02:31
rsyslogd                         RUNNING   pid 17, uptime 1:42:34
start                            EXITED    Dec 04 03:05 AM
supervisor-proc-exit-listener    RUNNING   pid 8, uptime 1:42:35

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

  • 202205
  • 202211
  • 202305
  • 202311
  • 202405
  • 202411
  • 202505

Tested branch (Please provide the tested image version)

Description for the changelog

Link to config_db schema for YANG module changes

A picture of a cute animal (not mandatory but encouraged)

[otel container]: Create otel container for HFT
@Janetxxx Janetxxx requested review from Pterosaur and r12f and removed request for lguohan, qiluo-msft and xumia January 13, 2026 00:35
@Janetxxx
Copy link
Member Author

/azp run Azure.sonic-buildimage

@azure-pipelines
Copy link

No pipelines are associated with this pull request.

@Pterosaur
Copy link
Contributor

/azp run Azure.sonic-buildimage

@azure-pipelines
Copy link

No pipelines are associated with this pull request.

@r12f
Copy link

r12f commented Jan 13, 2026

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

Why I did it
This PR fixes a service name conflict in the otel container's monit configuration. The monit program check name was incorrectly using container_memory_telemetry instead of container_memory_otel, which would conflict with the telemetry container's monitoring configuration.

Work item tracking
Microsoft ADO (number only):
How I did it
Fixed a service name conflict in the otel container's monit configuration.


Signed-off-by: Janet Cui <janet970527@gmail.com>
Co-authored-by: Ze Gan <ganze718@gmail.com>
@Janetxxx
Copy link
Member Author

/azp run Azure.sonic-buildimage

@azure-pipelines
Copy link

No pipelines are associated with this pull request.

@Janetxxx
Copy link
Member Author

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

Signed-off-by: Janet Cui <janet970527@gmail.com>
@Janetxxx
Copy link
Member Author

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

Signed-off-by: Janet Cui <janet970527@gmail.com>
@Janetxxx
Copy link
Member Author

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

Signed-off-by: Janet Cui <janet970527@gmail.com>
@Janetxxx
Copy link
Member Author

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@r12f
Copy link

r12f commented Jan 23, 2026

CI failing due to unrelated issue. Fix in progress but needs to get PR merged due to timeline.

@r12f r12f merged commit 28ebe6c into 202412 Jan 23, 2026
14 of 17 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.

3 participants