Skip to content

Add the snmp tool to the sonic-mgmt docker#4110

Merged
lguohan merged 1 commit intosonic-net:masterfrom
wangxin:sonic-mgmt-snmp
Feb 5, 2020
Merged

Add the snmp tool to the sonic-mgmt docker#4110
lguohan merged 1 commit intosonic-net:masterfrom
wangxin:sonic-mgmt-snmp

Conversation

@wangxin
Copy link
Contributor

@wangxin wangxin commented Feb 5, 2020

- What I did
The snmp tool is required for interacting with certain type of PDU hosts in platform PSU/power related testing. This change is to have the snmp tool pre-built in the sonic-mgmt docker image.

- How I did it
Updated the Dockerfile.j2 of docker-sonic-mgmt. Added "snmp" package to the "apt-get install" command.

- How to verify it

  1. Build sonic-mgmt docker image.
  2. Start a docker container from the image.
  3. Run command "snmpget" or "snmpwalk" to verify that the snmp tool is installed.

- Description for the changelog

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

The snmp tool is required for interacting with certain type
of PDU hosts in platform PSU/power related testing. This
change is to have the snmp tool pre-built in the sonic-mgmt
docker image.

Signed-off-by: Xin Wang <xinw@mellanox.com>
@lguohan lguohan merged commit 3c12b32 into sonic-net:master Feb 5, 2020
rlhui pushed a commit that referenced this pull request Jul 7, 2020
The snmp tool is required for interacting with certain type
of PDU hosts in platform PSU/power related testing. This
change is to have the snmp tool pre-built in the sonic-mgmt
docker image.

Signed-off-by: Xin Wang <xinw@mellanox.com>
praveen-li pushed a commit to praveen-li/sonic-buildimage that referenced this pull request May 28, 2021
…c-net#4110)

The snmp tool is required for interacting with certain type
of PDU hosts in platform PSU/power related testing. This
change is to have the snmp tool pre-built in the sonic-mgmt
docker image.

Signed-off-by: Xin Wang <xinw@mellanox.com>
@wangxin wangxin deleted the sonic-mgmt-snmp branch February 18, 2022 07:45
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.

2 participants