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

[action] [PR:509] [ycabled][active-active] Fix in gRPC channel callback logic by creating swsscommon table within the context (#509) #518

Merged
merged 1 commit into from
Jul 19, 2024

Conversation

mssonicbld
Copy link
Collaborator

  • [ycabled][active-active] Fix in gRPC channel callback logic by creating
    swsscommon table within the context

Signed-off-by: Vaibhav Dahiya vdahiya@microsoft.com

  • fix UT

Signed-off-by: Vaibhav Dahiya vdahiya@microsoft.com

  • add more tests

Signed-off-by: Vaibhav Dahiya vdahiya@microsoft.com

  • typo

Signed-off-by: Vaibhav Dahiya vdahiya@microsoft.com

  • add port

Signed-off-by: Vaibhav Dahiya vdahiya@microsoft.com

  • add logging

Signed-off-by: Vaibhav Dahiya vdahiya@microsoft.com

  • add tests

Signed-off-by: Vaibhav Dahiya vdahiya@microsoft.com


Signed-off-by: Vaibhav Dahiya vdahiya@microsoft.com

…ng swsscommon table within the context (sonic-net#509)

* [ycabled][active-active] Fix in gRPC channel callback logic by creating
swsscommon table within the context

Signed-off-by: Vaibhav Dahiya <vdahiya@microsoft.com>

* fix UT

Signed-off-by: Vaibhav Dahiya <vdahiya@microsoft.com>

* add more tests

Signed-off-by: Vaibhav Dahiya <vdahiya@microsoft.com>

* typo

Signed-off-by: Vaibhav Dahiya <vdahiya@microsoft.com>

* add port

Signed-off-by: Vaibhav Dahiya <vdahiya@microsoft.com>

* add logging

Signed-off-by: Vaibhav Dahiya <vdahiya@microsoft.com>

* add tests

Signed-off-by: Vaibhav Dahiya <vdahiya@microsoft.com>

---------

Signed-off-by: Vaibhav Dahiya <vdahiya@microsoft.com>
@mssonicbld
Copy link
Collaborator Author

Original PR: #509

@mssonicbld mssonicbld merged commit 923958f into sonic-net:202405 Jul 19, 2024
3 checks passed
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.

2 participants