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

fix the wrong key in DPU address get from DHCP_SERVER_IPV4_PORT table Issue #288 #289

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

aronovic
Copy link

Fixes #288

Why I did it

gnmi_set.go:203] Set failed: rpc error: code = NotFound desc = Get ZMQ client failed: Get ZMQ address failed: Get DPU address failed: Can't read bridge-midplane|dpu1:ips from DHCP_SERVER_IPV4_PORT table

How I did it

Modified the key from ips to ips@ as expected in the table

How to verify it

Executed my gnmi client and was able to retrieve the entry successfully

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

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

Description for the changelog

Modified the key used for DPU address retrieval by gnmi_set from DHCP_SERVER_IPV4_PORT table

Link to config_db schema for YANG module changes

N/A

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

Copy link

linux-foundation-easycla bot commented Sep 19, 2024

CLA Signed

The committers listed above are authorized under a signed CLA.

@ganglyu
Copy link
Contributor

ganglyu commented Sep 23, 2024

Please fix EasyCLA

@sallylsy
Copy link
Contributor

sallylsy commented Oct 7, 2024

Component test failed

=== RUN   TestGetDpuAddress
    client_test.go:741: get DPU address failed: Can't read bridge-midplane|dpu0:ips@ from DHCP_SERVER_IPV4_PORT table
    client_test.go:745: get DPU address failed: 
    client_test.go:761: get invalid DPU address failed
--- FAIL: TestGetDpuAddress (0.01s)

Please fix

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.

gnmi_set fails to get DPU address
3 participants