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

submodule update: sonic-gnmi #11961

Closed

Conversation

renukamanavalan
Copy link
Contributor

@renukamanavalan renukamanavalan commented Sep 4, 2022

Included commits:

  • e89b594 (HEAD, origin/master, origin/HEAD) Workaround gomonkey for armhf
  • 3beaadc (HEAD, origin/master, origin/HEAD, master) Improve unit test for gnmi (# 38)
  • a12fcd8 Merge pull request # 13 from renukamanavalan/statistics
  • | 28b2f27 Enable unit test
  • | 65e3c8f Add log for authentication and add unit test. # 12
  • | 1b3b838 Merge pull request # 8 from ganglyu/fix_pipeline

Why I did it

How I did it

How to verify it

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

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

Description for the changelog

Link to config_db schema for YANG module changes

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

* 3beaadc (HEAD, origin/master, origin/HEAD, master) Improve unit test for gnmi (#38)
*   a12fcd8 Merge pull request #13 from renukamanavalan/statistics
* |   28b2f27 Enable unit test
* |   65e3c8f Add log for authentication and add unit test. #12
* |   1b3b838 Merge pull request #8 from ganglyu/fix_pipeline
@ganglyu
Copy link
Contributor

ganglyu commented Sep 4, 2022

I use gomonkey to provide mock for unit test, but this version does not support ARM.
I need to investigate this issue.

@ganglyu
Copy link
Contributor

ganglyu commented Sep 4, 2022

sonic-net/sonic-gnmi#39
Use gomonkey v2 to support arm

73ce325 Disable test.bin for arm.
3beaadc (HEAD, origin/master, origin/HEAD, master) Improve unit test for gnmi (# 38)
a12fcd8 Merge pull request # 13 from renukamanavalan/statistics
| 28b2f27 Enable unit test
| 65e3c8f Add log for authentication and add unit test. # 12
| 1b3b838 Merge pull request # 8 from ganglyu/fix_pipeline
@renukamanavalan
Copy link
Contributor Author

/azp run buildimage

@azure-pipelines
Copy link

No pipelines are associated with this pull request.

@renukamanavalan
Copy link
Contributor Author

/azp run Azure.sonic-buildimage

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@renukamanavalan
Copy link
Contributor Author

/azp run Azure.sonic-buildimage

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

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