Skip to content

Commit

Permalink
[sonic-mgmt] Call docker cmd on appropriate container in generate_tel…
Browse files Browse the repository at this point in the history
…emetry_config() funct (sonic-net#11552)

Description of PR
generate_telemetry_config() function already establishes that the GNMI container is 'telemetry' so call docker supervisorctl status using this container name.

Summary:
Fixes #
aristanetworks/sonic-qual.msft#61
  • Loading branch information
veronica-arista authored and mssonicbld committed May 16, 2024
1 parent 4c671bd commit 329e82f
Showing 1 changed file with 2 additions and 1 deletion.
3 changes: 2 additions & 1 deletion tests/common/helpers/gnmi_utils.py
Original file line number Diff line number Diff line change
Expand Up @@ -52,7 +52,8 @@ def generate_telemetry_config(self, duthost):
self.gnmi_config_table = "TELEMETRY"
self.gnmi_container = "telemetry"
# GNMI program is telemetry or gnmi-native
res = duthost.shell("docker exec telemetry supervisorctl status", module_ignore_errors=True)
res = duthost.shell("docker exec %s supervisorctl status" % self.gnmi_container,
module_ignore_errors=True)
if 'telemetry' in res['stdout']:
self.gnmi_program = "telemetry"
else:
Expand Down

0 comments on commit 329e82f

Please sign in to comment.