Add compliance testing CLI tool. by robshakir · Pull Request #31 · openconfig/gribigo

robshakir · 2021-06-18T06:24:36Z

commit c118419e96a62a955d61844d10b641bbab9b7814
Author: Rob Shakir <robjs@google.com>
Date:   Thu Jun 17 23:18:42 2021 -0700

    Add changes to run tests to remote gRIBI server.
    
      * (A) cmd/ccli/ccli.go
      * (A) cmd/ccli/ccli_test.go
        - This package is a test package that can be run using
          go test -addr <addr> of gRIBI server against a real
          server.
      * (M) cmd/rtr/main.go
        - Add pprof debug HTTP server.
      * (M) compliance/compliance.go
      * (M) compliance/compliance_test.go
        - Generalise tests.
        - Ensure that election ID doesn't go backwards during tests
          by implementing a poor man's chubby :-)
      * (M) server/server.go
        - Add logging.

* (M) aft/oc.go * (M) aft/update.sh - Update generated code. * (M) go.mod * (M) go.sum - Go mod tidying. * (M) rib/rib.go * (M) rib/rib_test.go - Add IPv4 entry add and delete.

* (M) go.mod * (M) go.sum - Tidy up modules. * (M) rib/rib.go * (M) rib/rib_test.go - Add support for network instances in the RIB rather than a single RIB.

* (M) client/client.go - TODO comment about handling coalescing. * (M) rib/rib.go * (M) rib/rib_test.go - Add support for adding NHG and NHs to the RIB.

* (M) rib/rib.go * (M) rib/rib_test.go - Validate any candidate RIB against the OpenConfig schema before adding it to the RIB.

* (M) aft/oc.go * (M) aft/update.sh - Add delete methods. * (M) go.mod * (M) go.sum - Tidy up modules. * (M) rib/rib.go - Add support for a simple callback function that is run after every update to the RIB. - Fix a defect with implicit replace whereby we did not delete the RIB entry before doing MergeStructInto meaning that for some leaf-lists we ended up appending to the contents of the RIB. * (M) rib/rib_test.go - Add test cases for the above functionality - including a test case demonstrating how gNMI Notifications can be generated.

* Add an initial implementation of a resolved-RIB. Background -- we want the gRIBI server to be able build both a RIB that is efficient for it to do gRIBI operations with, as well as a software emulated 'resolved RIB' / 'FIB' whereby we can say something like "where will this packet go based on what gRIBI is programmed?". This package starts this implementation. * (M) aft/oc.go - Update generated code. * (M) go.mod * (M) go.sum - New dependencies on kentik/patricia * (A) ocrt/oc.go * (A) ocrt/update.sh - Generated routing specific OpenConfig Go structs. * (A) ocrt/yang/deviations.yang - Deviations to reduce the size of the generated oc.go. * (M) rib/rib.go - Fix handling of implicit replace. * (A) sysrib/sysrib.go * (A) sysrib/sysrib_test.go - Initial system RIB implementation. This package currently parses OpenConfig to build a list of all of the connected routes that are on the system. * Address review comments.

* (A) gnmit/gnmit.go * (A) gnmit/gnmit_test.go - Wrap the openconfig/gnmi libraries to create a single-device cache that can be used to send gNMI updates to from the overall fake device.

* (M) go.mod * (M) go.sum - move off ygot development branch. * (M) rib/rib.go - Adopt change to protomap API in ygot. * (M) server/server.go - Add goroutines to server Modify handler to allow concurrent processing.

* (M) gnmit/gnmit_test.go - deflake test by: - checking for presence of correct leaves before syncresponse. - ignoring update order.

* (M) fluent/fluent.go * (M) fluent/fluent_test.go - Add support for generating basic IPv4Entry types in the fluent API. - Ensure coverage of the builder functions in the fluent library.

* (M) client/client.go - Fix duplicate queueing for AFTOperations during send queueing. - Handle received operations from the server by removing them from the pending queue. * (M) fluent/fluent.go - Simplfy mapping to canonical reason codes. * (M) fluent/fluent_test.go - Add an integration test for a simple IPv4 entry. * (M) rib/rib.go * (M) rib/rib_test.go - Define a type for the RIB function hook. - add locking across network instances. - change ribHolder to be publicly accessible. * (M) server/server.go - Define a const for the default network instance name. - Add handling for specifying a RIBHook when initialising the server. - Add handling for doing an IPv4 add operation.

* (M) fluent/fluent.go * (M) fluent/fluent_test.go - Make gRIBIClient public so that more reusable tests can be built. - Add builder for results messages.

* (M) client/client.go - add close method, and operation details to returned messages.

* (M) constants/const.go - Mapping of constants to protobuf constants. - fix MODIFY->REPLACE>

robshakir added 30 commits June 8, 2021 14:49

Add initial RIB management.

c9606fd

* (M) aft/oc.go * (M) aft/update.sh - Update generated code. * (M) go.mod * (M) go.sum - Go mod tidying. * (M) rib/rib.go * (M) rib/rib_test.go - Add IPv4 entry add and delete.

Add support for different network instances in the RIB.

74a88d6

* (M) go.mod * (M) go.sum - Tidy up modules. * (M) rib/rib.go * (M) rib/rib_test.go - Add support for network instances in the RIB rather than a single RIB.

Add support for adding NHG and NH values.

fb91477

* (M) client/client.go - TODO comment about handling coalescing. * (M) rib/rib.go * (M) rib/rib_test.go - Add support for adding NHG and NHs to the RIB.

validate entries against schema before allowing them.#

b1b0104

* (M) rib/rib.go * (M) rib/rib_test.go - Validate any candidate RIB against the OpenConfig schema before adding it to the RIB.

Update comments.

2bd0bc1

Address review comments.

e73db09

Merge branch 'rib1' into rib2

7514349

Merge branch 'main' into rib2

7a7caaa

Merge branch 'main' into rib2

c4d17e1

Add gNMI cache for target to use.

8208a66

* (A) gnmit/gnmit.go * (A) gnmit/gnmit_test.go - Wrap the openconfig/gnmi libraries to create a single-device cache that can be used to send gNMI updates to from the overall fake device.

Improve commenting and avoid hostname repetition.

d6a2a06

Merge branch 'main' into rib4

3d957e1

Fix bad merge.

2eee535

Fix commenting.

6996899

start to integrate services for device.

b5bc01b

Add test file.

fd1a4b8

Move to latest version of ygot, server goroutines.

12cc0a0

* (M) go.mod * (M) go.sum - move off ygot development branch. * (M) rib/rib.go - Adopt change to protomap API in ygot. * (M) server/server.go - Add goroutines to server Modify handler to allow concurrent processing.

Deflake test by validating correct semantics.

85ce82d

* (M) gnmit/gnmit_test.go - deflake test by: - checking for presence of correct leaves before syncresponse. - ignoring update order.

Merge branch 'rib4' into rib4.5

f49693f

Add support for entries in fluent, improve coverage.

4b20ff9

* (M) fluent/fluent.go * (M) fluent/fluent_test.go - Add support for generating basic IPv4Entry types in the fluent API. - Ensure coverage of the builder functions in the fluent library.

Merge branch 'rib4.6' into rib5

0540a61

Refactor RIB implementation.

ae26c50

Add initial 'device' function which acts as a combined server.

a9afd5f

Add main package.

bb0b789

Split into separate tests rather than using subtests.

245e4c0

Merge branch 'rib4' into rib4.5

aa061b0

Merge branch 'main' into rib4.5

85795df

robshakir and others added 14 commits June 17, 2021 23:30

Make sure that ccli doesn't fail in CI.

455b093

Fix doc comment.

220e2a7

Merge branch 'rib6.5' into rib7

bf30106

Merge branch 'main' into rib7

9828bb0

Fix review comments.

9f38fbd

Merge branch 'rib7' into rib8

bc97001

Merge branch 'main' into rib8

15492c1

Add handling for errors in fluent code.

15b7c0f

* (M) fluent/fluent.go * (M) fluent/fluent_test.go - Make gRIBIClient public so that more reusable tests can be built. - Add builder for results messages.

Add details of results to client code.

14655db

* (M) client/client.go - add close method, and operation details to returned messages.

Update gRIBI constants.

27faf4e

* (M) constants/const.go - Mapping of constants to protobuf constants. - fix MODIFY->REPLACE>

Adopt new constants in rib.go.

1ac9172

Merge branch 'main' into rib7.1

3528915

Merge branch 'rib7.1' into rib8

095499d

Merge branch 'rib8' into rib8.1-pprof

5892b4a

robshakir added this to the public-release milestone Jun 18, 2021

robshakir added 3 commits June 18, 2021 08:37

Address review comment.

fad7f36

Merge branch 'rib7.1' into rib8

b0ae2d6

Merge branch 'main' into rib8

18d09d0

robshakir requested a review from sthesayi June 18, 2021 15:40

sthesayi approved these changes Jun 18, 2021

View reviewed changes

robshakir added 5 commits June 19, 2021 06:37

Update package comment.

c46f191

Merge branch 'rib8' of github.com:openconfig/gribigo into rib8

09dd0cb

Add debugging to debug gnmit flkae.

ae829d8

Ensure that we call the right cancel function.

30c80a3

Ensure our map is completely written before testing.

a39daa8

Base automatically changed from rib8 to main June 19, 2021 13:58

robshakir added 2 commits June 19, 2021 06:58

Merge branch 'rib8' into rib8.1-pprof

ca0f26b

Merge branch 'main' into rib8.1-pprof

a9bc335

robshakir merged commit 2ce63e6 into main Jun 19, 2021

robshakir deleted the rib8.1-pprof branch June 19, 2021 14:04

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Add compliance testing CLI tool.#31

Add compliance testing CLI tool.#31
robshakir merged 112 commits intomainfrom
rib8.1-pprof

robshakir commented Jun 18, 2021

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

Conversation

robshakir commented Jun 18, 2021

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants