sdk

sdk copied to clipboard

We should not log into the span after calling `span.Finish()`. Instead, we should start a new span with [`FollowsFrom`] relation. Current `trace.Log` design: the span lifetime matches the request lifetime,...

xzfc

Add coverage check on CI

2

## Motivation The coverage check might help to review PRs with tests.

denis-tingaikin

Add benchmark testing on CI

1

We need to add benchmark testing on CI. We already have benchmark tests but they not run on CI: pkg/tools/serialize/serialize_test.go

denis-tingaikin

https://github.com/networkservicemesh/cmd-nse-vl3-vpp/issues/261 cc @denis-tingaikin @glazychev-art

VitalyGushin

## Failure Information (for bugs) https://github.com/networkservicemesh/sdk/actions/runs/4553291328/jobs/8029678084?pr=1443 ## Failure Logs [logs_21005.zip](https://github.com/networkservicemesh/sdk/files/11099978/logs_21005.zip)

NikitaSkrynnik

Get rid of deprecated grpc options

3

## Expected Behavior All applications use the same grpc options ## Current Behavior Some cmds use a different set of options ## Failure Information (for bugs) Need to figure out...

glazychev-art

[R&D] Improve the recovery of NSM clients

3

We have done a lot of work to improve the recovery of NSM in the past. Recently we've found a few use-cases that conceptually do not work with existing code....

denis-tingaikin

opa: token_expired policy should check only the right side of Path

1

## Description ## Issue link Closes: https://github.com/networkservicemesh/sdk/issues/1512 ## How Has This Been Tested? - [x] Added unit testing to cover - [ ] Tested manually - [ ] Tested by...

glazychev-art

Add client chain element to monitor dead NSEs

1

## Description Added a new client chain element that monitors NSE it has a connection to. If it get `deleted:true` event from the registry for this NSE it calls Close...

NikitaSkrynnik

Add an option to change reselect function

2

## Description Add an ability to change connection modifications during a reselect, because vl3 healing should not delete NSE name ## Issue link https://github.com/networkservicemesh/cmd-nse-vl3-vpp/issues/289 ## How Has This Been Tested?...

NikitaSkrynnik