Skip to content

submodule update: sonic-gnmi#11961

Closed
renukamanavalan wants to merge 3 commits into
sonic-net:masterfrom
renukamanavalan:remanava_events
Closed

submodule update: sonic-gnmi#11961
renukamanavalan wants to merge 3 commits into
sonic-net:masterfrom
renukamanavalan:remanava_events

Conversation

@renukamanavalan

@renukamanavalan renukamanavalan commented Sep 4, 2022

Copy link
Copy Markdown
Contributor

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

ganglyu commented Sep 4, 2022

Copy link
Copy Markdown
Contributor

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

@ganglyu

ganglyu commented Sep 4, 2022

Copy link
Copy Markdown
Contributor

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
Copy Markdown
Contributor Author

/azp run buildimage

@azure-pipelines

Copy link
Copy Markdown
No pipelines are associated with this pull request.

@renukamanavalan

Copy link
Copy Markdown
Contributor Author

/azp run Azure.sonic-buildimage

@azure-pipelines

Copy link
Copy Markdown
Azure Pipelines successfully started running 1 pipeline(s).

@renukamanavalan

Copy link
Copy Markdown
Contributor Author

/azp run Azure.sonic-buildimage

@azure-pipelines

Copy link
Copy Markdown
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