openconfig / lemming Goto Github PK

View Code? Open in Web Editor NEW

24.0 24.0 6.0 74.15 MB

An OpenConfig reference device implementation

License: Apache License 2.0

Go 97.21% Shell 0.03% Makefile 0.02% Starlark 0.63% C++ 1.83% C 0.28% Dockerfile 0.01%

lemming's Issues

Provide howto on having a reference device container

Provide registration function for linking g* services into the main event loop

Subscribe() requires a target while the spec says it shouldn't

Hey!

Subscribe() requires a target, while the spec says it should not. In 2.2.2.1:

This field is optional for clients.

Commercial implementations do not require a target. Currently, Subscribe() implementation is delegated to openconfig/gnmi which requires a target. See openconfig/gnmi#151.

Test for configure replace Root

Improve linter config

Its very slow because the large amount of generated code and flaky.

Links to Linux-only libraries causing test failures on OS X.

@wenovus migrated a number of other projects to use lemming as the fake that they test against, great. Unfortunately, today since the dataplane links against Linux-only libraries, this means that these tests are now broken on OS X targets.

e.g., openconfig/gribigo tests fail like this:

[4:16] github.com/openconfig/gribigo   reconnreset ✔                                                                                                                                                                                  10m ⚑  
▶ go test -race $(go list ./... | egrep -v "(compliance|cmd|device)")
package github.com/openconfig/gribigo/cmd/rtr
	imports github.com/openconfig/lemming
	imports github.com/openconfig/lemming/dataplane
	imports github.com/openconfig/lemming/dataplane/forwarding
	imports github.com/openconfig/lemming/dataplane/forwarding/fwdport/ports
	imports github.com/google/gopacket/afpacket: build constraints exclude all Go files in /Users/rjs/go/pkg/mod/github.com/google/[email protected]/afpacket
go: downloading github.com/openconfig/testt v0.0.0-20220311054427-efbb1a32ec07
go: downloading github.com/pmezard/go-difflib v1.0.0
# github.com/openconfig/gribigo/fluent
package github.com/openconfig/gribigo/fluent (test)
	imports github.com/openconfig/lemming
	imports github.com/openconfig/lemming/dataplane
	imports github.com/openconfig/lemming/dataplane/forwarding
	imports github.com/openconfig/lemming/dataplane/forwarding/fwdport/ports
	imports github.com/google/gopacket/afpacket: build constraints exclude all Go files in /Users/rjs/go/pkg/mod/github.com/google/[email protected]/afpacket

Given that these tests do not use the dataplane, this seems undesirable - and makes lemming unsuitable for being a fake for development for these projects.

Please investigate using build tags or some other solution to fix this :-)

gnmiclient Replace/Update helpers do not handle non-leafs correctly

Since they're wrappers to ConfigQuery, the GoStruct is marshalled using the wrong PreferShadowPath setting. So setting state using a non-leaf doesn't work.

@DanG100

Create fork of GoBGP's ZAPI library

ZAPI is unlikely to change, so we're unlikely to miss many enhancements from upstream. This enables lemming to use more zebra message types that's currently unexported in GoBGP as well as making other improvements.

openconfig / lemming Goto Github PK

lemming's People

Contributors

Stargazers

Watchers

Forkers

lemming's Issues

Provide howto on having a reference device container

Provide registration function for linking g* services into the main event loop

Change to klog

Subscribe() requires a target while the spec says it shouldn't

Test for configure replace Root

Improve linter config

Links to Linux-only libraries causing test failures on OS X.

gnmiclient Replace/Update helpers do not handle non-leafs correctly

Create fork of GoBGP's ZAPI library

Recommend Projects

React

Vue.js

Typescript

TensorFlow

Django

Laravel

D3

Recommend Topics

javascript

web

server

Machine learning

Visualization

Game

Recommend Org

Facebook

Microsoft

Google

Alibaba

D3

Tencent

Jobs