Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Install GNMI client for mgmt container #16081

Draft
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

ganglyu
Copy link
Contributor

@ganglyu ganglyu commented Aug 9, 2023

Why I did it

Need GNMI client for end2end test.

Work item tracking
  • Microsoft ADO (number only):

How I did it

Download sonic-gnmi package from pipeline, and install gnmi client in container.

How to verify it

Use GNMI client in mgmt container.

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

  • 201811
  • 201911
  • 202006
  • 202012
  • 202106
  • 202111
  • 202205
  • 202211
  • 202305

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)

@ganglyu ganglyu marked this pull request as draft August 9, 2023 05:34
@ganglyu ganglyu marked this pull request as ready for review August 10, 2023 00:49
@ganglyu ganglyu requested a review from Pterosaur August 10, 2023 02:08
# Install GNMI client
RUN mkdir -p /tmp/gnmi \
&& cd /tmp/gnmi \
&& wget "https://sonic-build.azurewebsites.net/api/sonic/artifacts?branchName=master&definitionId=142&artifactName=sonic-buildimage.vs&target=target/debs/bullseye/sonic-gnmi_0.1_amd64.deb" -O sonic-gnmi_0.1_amd64.deb \
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

sonic-gnmi_0.1_amd64.deb

docker-ptf is using https://github.com/lguohan/gnxi.git.
Please check if you can reuse that client.
We intend to reduce using sonic binary in the test controller.

@ganglyu ganglyu marked this pull request as draft August 10, 2023 11:02
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.

3 participants