search

sonic-net / sonic-buildimage

Scripts which perform an installable binary image build for SONiC
Other
743 stars 1.43k forks source link

Telemetry is not built by default in the sonic image after the https://github.com/sonic-net/sonic-buildimage/pull/16863 #17252

Open nhe-NV opened 1 year ago

nhe-NV commented 1 year ago

Description

After the https://github.com/sonic-net/sonic-buildimage/pull/16863 is merged to the 202305, the sonic image not include the telemetry by default.

Steps to reproduce the issue:

1.Deploy the latest sonic image, check that the telemetry is not inlcuded. 2. 3.

Describe the results you received:

CONTAINER ID IMAGE COMMAND CREATED STATUS PORTS NAMES 94670c172e6e 63068dfc7ac2 "/usr/bin/start.py" 2 hours ago Exited (255) 2 hours ago doai 54e237d88a36 docker-sflow:latest "/usr/local/bin/supe…" 4 hours ago Exited (137) 58 minutes ago sflow 083df735e46a 0cfb168154c1 "/usr/local/bin/supe…" 10 hours ago Up 4 minutes what-just-happened 75165b1799de docker-snmp:latest "/usr/local/bin/supe…" 10 hours ago Up 4 minutes snmp 8c3de067c65c docker-sonic-mgmt-framework:latest "/usr/local/bin/supe…" 10 hours ago Up 4 minutes mgmt-framework 91514a99b0a8 docker-lldp:latest "/usr/bin/docker-lld…" 10 hours ago Up 4 minutes lldp 43564ac4c971 docker-sonic-gnmi:latest "/usr/local/bin/supe…" 10 hours ago Up 4 minutes gnmi 567811fc0def e9452eec9630 "/usr/bin/docker_ini…" 10 hours ago Up 5 minutes dhcp_relay e50448048ac9 docker-platform-monitor:latest "/usr/bin/docker_ini…" 10 hours ago Up 5 minutes pmon a69afdc3af51 docker-router-advertiser:latest "/usr/bin/docker-ini…" 10 hours ago Up 5 minutes radv 8934b0966478 docker-syncd-mlnx:latest "/usr/local/bin/supe…" 10 hours ago Up 5 minutes syncd 8e12d208cc2d docker-fpm-frr:latest "/usr/bin/docker_ini…" 10 hours ago Up 5 minutes bgp 99ca8af0abec docker-teamd:latest "/usr/local/bin/supe…" 10 hours ago Up 5 minutes teamd e66155d721d0 docker-orchagent:latest "/usr/bin/docker-ini…" 10 hours ago Up 5 minutes swss 7b29ec74df31 docker-eventd:latest "/usr/local/bin/supe…" 10 hours ago Up 5 minutes eventd 6c21c69bde8c docker-database:latest "/usr/local/bin/dock…" 10 hours ago Up 2 hours database

Describe the results you expected:

This is kind of backward compatiblity issue

Output of show version:

(paste your output here)

Output of show techsupport:

(paste your output here or download and attach the file here )

Additional information you deem important (e.g. issue happens only occasionally):

dgsudharsan commented 1 year ago

@ganglyu Please use this bug to add the fix on db_migrator to migrate from telemetry to gnmi.

ganglyu commented 8 months ago

For 202305 branch, telemetry is build by default. For 202311 branch and master branch, we use gnmi to replace telemetry, and we use db_migrator to migrate GNMI configuration.