Merge pull request #165 from Nordix/nsp

NSP API Update function removed
Nordix · Feb 16, 2022 · d31a8a9 · d31a8a9
2 parents 77cca74 + d698b49
commit d31a8a9
Show file tree

Hide file tree

Showing 3 changed files with 15 additions and 62 deletions.
diff --git a/api/nsp/v1/targetregistry.pb.go b/api/nsp/v1/targetregistry.pb.go
diff --git a/api/nsp/v1/targetregistry.proto b/api/nsp/v1/targetregistry.proto
@@ -24,7 +24,6 @@ import "api/nsp/v1/model.proto";
 service TargetRegistry {
     rpc Register(Target) returns (google.protobuf.Empty) {}
     rpc Unregister(Target) returns (google.protobuf.Empty) {}
-    rpc Update(Target) returns (google.protobuf.Empty) {}
     rpc Watch(Target) returns (stream TargetResponse) {}
 }
 

diff --git a/pkg/nsp/server.go b/pkg/nsp/server.go
@@ -48,11 +48,6 @@ func (s *Server) Unregister(ctx context.Context, target *nspAPI.Target) (*empty.
 	return &empty.Empty{}, s.TargetRegistry.Remove(ctx, target)
 }
 
-func (s *Server) Update(ctx context.Context, target *nspAPI.Target) (*empty.Empty, error) {
-	logrus.Infof("Update: %v", target)
-	return &empty.Empty{}, s.TargetRegistry.Set(ctx, target)
-}
-
 func (s *Server) Watch(t *nspAPI.Target, watcher nspAPI.TargetRegistry_WatchServer) error {
 	targetWatcher, err := s.TargetRegistry.Watch(context.TODO(), t)
 	if err != nil {