Add Update, Replace, and Delete #14

DanG100 · 2022-04-28T20:40:20Z

Because of an issue with type inference.
Either:

Update, Watch, Delete accept SingletonQuery and runtime error its not a config path.
Config() needs to specify the types for Lookup Lookup[int64](ctx, c, path.Config)

ygnmi/gnmi.go

ygnmi/types.go

ygnmi/ygnmi.go

ygnmi/ygnmi_test.go

ygnmi/gnmi.go

ygnmi/types.go

ygnmi/gnmi.go

ygnmi/types.go

internal/testutil/schema.go

ygnmi/types.go

first impl

9e3487a

DanG100 requested review from greg-dennis and wenovus April 28, 2022 20:47

wenovus requested changes Apr 28, 2022

View reviewed changes

feedback

46c51f7

DanG100 force-pushed the config-ops branch from 2856995 to 46c51f7 Compare April 28, 2022 23:26

DanG100 requested a review from wenovus April 28, 2022 23:34

wenovus approved these changes Apr 29, 2022

View reviewed changes

greg-dennis suggested changes Apr 29, 2022

View reviewed changes

feedback

20e2e01

DanG100 requested a review from greg-dennis April 29, 2022 17:37

greg-dennis approved these changes Apr 29, 2022

View reviewed changes

internal/testutil/schema.go Outdated Show resolved Hide resolved

ygnmi/types.go Show resolved Hide resolved

feedback

e6caf48

DanG100 merged commit c949a56 into main Apr 29, 2022

DanG100 deleted the config-ops branch April 29, 2022 19:38

Provide feedback