[CNI] Add GET_STATE command to dump CNI state to stdout #891

matmerr · 2021-06-04T22:35:17Z

Reason for Change:

Instead of CNS rehydrating from API server, CNS can use CNI to rehydrate from state stored on disk.

Issue Fixed:

Requirements:

Notes:

cni/api/api.go

cni/network/invoker_cns.go

rbtr

confusing that the lint step didn't catch the error casing, because it does locally for me

cni/client/client.go

cni/client/client_integration_test.go

cni/network/plugin/main.go

cni/api/api.go

cni/network/invoker_azure.go

cni/network/invoker_cns.go

cni/network/plugin/main.go

cns/cnsclient/cnsclient.go

cni/network/plugin/main.go

cni/client/client.go

cni/cni.go

cni/network/invoker_azure.go

cni/client/client.go

cni/network/invoker_cns.go

cni/network/network.go

cni/api/api.go

cni/network/plugin/main.go

cni/api/api.go

neaggarwMS

resolved concerns, stale review

matmerr added 3 commits June 3, 2021 17:03

inital dump state and ipam interface update

5d96c7b

add reconcile command to CNI

984c5a5

add integration test

364dc84

matmerr commented Jun 4, 2021

View reviewed changes

cni/api/api.go Outdated Show resolved Hide resolved

matmerr force-pushed the cnidumpstate branch from e61f703 to bc674b2 Compare June 4, 2021 23:01

pass endpoint id on add

a01039a

matmerr force-pushed the cnidumpstate branch from bc674b2 to a01039a Compare June 4, 2021 23:05

matmerr requested a review from tamilmani1989 June 7, 2021 17:06

matmerr commented Jun 7, 2021

View reviewed changes

cni/network/invoker_cns.go Show resolved Hide resolved

rbtr previously requested changes Jun 7, 2021

View reviewed changes