diff --git a/server/config.go b/server/config.go index 764f6bdf2..d6c2ca644 100644 --- a/server/config.go +++ b/server/config.go @@ -33,12 +33,14 @@ type Config struct { md toml.MetaData } +// GRPCConfig provides GRPC configuration for the socket type GRPCConfig struct { Address string `toml:"address"` Uid int `toml:"uid"` Gid int `toml:"gid"` } +// Debug provides debug configuration type Debug struct { Address string `toml:"address"` Uid int `toml:"uid"` @@ -46,10 +48,12 @@ type Debug struct { Level string `toml:"level"` } +// MetricsConfig provides metrics configuration type MetricsConfig struct { Address string `toml:"address"` } +// CgroupConfig provides cgroup configuration type CgroupConfig struct { Path string `toml:"path"` } diff --git a/services/containers/service.go b/services/containers/service.go index c5322f874..81af3f169 100644 --- a/services/containers/service.go +++ b/services/containers/service.go @@ -34,21 +34,22 @@ func init() { }) } -type Service struct { +type service struct { db *metadata.DB publisher events.Publisher } +// NewService returns the container GRPC server func NewService(db *metadata.DB, publisher events.Publisher) api.ContainersServer { - return &Service{db: db, publisher: publisher} + return &service{db: db, publisher: publisher} } -func (s *Service) Register(server *grpc.Server) error { +func (s *service) Register(server *grpc.Server) error { api.RegisterContainersServer(server, s) return nil } -func (s *Service) Get(ctx context.Context, req *api.GetContainerRequest) (*api.GetContainerResponse, error) { +func (s *service) Get(ctx context.Context, req *api.GetContainerRequest) (*api.GetContainerResponse, error) { var resp api.GetContainerResponse return &resp, errdefs.ToGRPC(s.withStoreView(ctx, func(ctx context.Context, store containers.Store) error { @@ -63,7 +64,7 @@ func (s *Service) Get(ctx context.Context, req *api.GetContainerRequest) (*api.G })) } -func (s *Service) List(ctx context.Context, req *api.ListContainersRequest) (*api.ListContainersResponse, error) { +func (s *service) List(ctx context.Context, req *api.ListContainersRequest) (*api.ListContainersResponse, error) { var resp api.ListContainersResponse return &resp, errdefs.ToGRPC(s.withStoreView(ctx, func(ctx context.Context, store containers.Store) error { @@ -77,7 +78,7 @@ func (s *Service) List(ctx context.Context, req *api.ListContainersRequest) (*ap })) } -func (s *Service) Create(ctx context.Context, req *api.CreateContainerRequest) (*api.CreateContainerResponse, error) { +func (s *service) Create(ctx context.Context, req *api.CreateContainerRequest) (*api.CreateContainerResponse, error) { var resp api.CreateContainerResponse if err := s.withStoreUpdate(ctx, func(ctx context.Context, store containers.Store) error { @@ -108,7 +109,7 @@ func (s *Service) Create(ctx context.Context, req *api.CreateContainerRequest) ( return &resp, nil } -func (s *Service) Update(ctx context.Context, req *api.UpdateContainerRequest) (*api.UpdateContainerResponse, error) { +func (s *service) Update(ctx context.Context, req *api.UpdateContainerRequest) (*api.UpdateContainerResponse, error) { if req.Container.ID == "" { return nil, status.Errorf(codes.InvalidArgument, "Container.ID required") } @@ -148,7 +149,7 @@ func (s *Service) Update(ctx context.Context, req *api.UpdateContainerRequest) ( return &resp, nil } -func (s *Service) Delete(ctx context.Context, req *api.DeleteContainerRequest) (*empty.Empty, error) { +func (s *service) Delete(ctx context.Context, req *api.DeleteContainerRequest) (*empty.Empty, error) { if err := s.withStoreUpdate(ctx, func(ctx context.Context, store containers.Store) error { return store.Delete(ctx, req.ID) }); err != nil { @@ -168,14 +169,14 @@ func (s *Service) Delete(ctx context.Context, req *api.DeleteContainerRequest) ( return &empty.Empty{}, nil } -func (s *Service) withStore(ctx context.Context, fn func(ctx context.Context, store containers.Store) error) func(tx *bolt.Tx) error { +func (s *service) withStore(ctx context.Context, fn func(ctx context.Context, store containers.Store) error) func(tx *bolt.Tx) error { return func(tx *bolt.Tx) error { return fn(ctx, metadata.NewContainerStore(tx)) } } -func (s *Service) withStoreView(ctx context.Context, fn func(ctx context.Context, store containers.Store) error) error { +func (s *service) withStoreView(ctx context.Context, fn func(ctx context.Context, store containers.Store) error) error { return s.db.View(s.withStore(ctx, fn)) } -func (s *Service) withStoreUpdate(ctx context.Context, fn func(ctx context.Context, store containers.Store) error) error { +func (s *service) withStoreUpdate(ctx context.Context, fn func(ctx context.Context, store containers.Store) error) error { return s.db.Update(s.withStore(ctx, fn)) } diff --git a/services/content/service.go b/services/content/service.go index 8289e1e64..3784579d5 100644 --- a/services/content/service.go +++ b/services/content/service.go @@ -21,7 +21,7 @@ import ( "google.golang.org/grpc/codes" ) -type Service struct { +type service struct { store content.Store publisher events.Publisher } @@ -32,7 +32,7 @@ var bufPool = sync.Pool{ }, } -var _ api.ContentServer = &Service{} +var _ api.ContentServer = &service{} func init() { plugin.Register(&plugin.Registration{ @@ -53,19 +53,20 @@ func init() { }) } -func NewService(cs content.Store, publisher events.Publisher) (*Service, error) { - return &Service{ +// NewService returns the content GRPC server +func NewService(cs content.Store, publisher events.Publisher) (api.ContentServer, error) { + return &service{ store: cs, publisher: publisher, }, nil } -func (s *Service) Register(server *grpc.Server) error { +func (s *service) Register(server *grpc.Server) error { api.RegisterContentServer(server, s) return nil } -func (s *Service) Info(ctx context.Context, req *api.InfoRequest) (*api.InfoResponse, error) { +func (s *service) Info(ctx context.Context, req *api.InfoRequest) (*api.InfoResponse, error) { if err := req.Digest.Validate(); err != nil { return nil, grpc.Errorf(codes.InvalidArgument, "%q failed validation", req.Digest) } @@ -80,7 +81,7 @@ func (s *Service) Info(ctx context.Context, req *api.InfoRequest) (*api.InfoResp }, nil } -func (s *Service) Update(ctx context.Context, req *api.UpdateRequest) (*api.UpdateResponse, error) { +func (s *service) Update(ctx context.Context, req *api.UpdateRequest) (*api.UpdateResponse, error) { if err := req.Info.Digest.Validate(); err != nil { return nil, grpc.Errorf(codes.InvalidArgument, "%q failed validation", req.Info.Digest) } @@ -95,7 +96,7 @@ func (s *Service) Update(ctx context.Context, req *api.UpdateRequest) (*api.Upda }, nil } -func (s *Service) List(req *api.ListContentRequest, session api.Content_ListServer) error { +func (s *service) List(req *api.ListContentRequest, session api.Content_ListServer) error { var ( buffer []api.Info sendBlock = func(block []api.Info) error { @@ -137,7 +138,7 @@ func (s *Service) List(req *api.ListContentRequest, session api.Content_ListServ return nil } -func (s *Service) Delete(ctx context.Context, req *api.DeleteContentRequest) (*empty.Empty, error) { +func (s *service) Delete(ctx context.Context, req *api.DeleteContentRequest) (*empty.Empty, error) { if err := req.Digest.Validate(); err != nil { return nil, grpc.Errorf(codes.InvalidArgument, err.Error()) } @@ -155,7 +156,7 @@ func (s *Service) Delete(ctx context.Context, req *api.DeleteContentRequest) (*e return &empty.Empty{}, nil } -func (s *Service) Read(req *api.ReadContentRequest, session api.Content_ReadServer) error { +func (s *service) Read(req *api.ReadContentRequest, session api.Content_ReadServer) error { if err := req.Digest.Validate(); err != nil { return grpc.Errorf(codes.InvalidArgument, "%v: %v", req.Digest, err) } @@ -223,7 +224,7 @@ func (rw *readResponseWriter) Write(p []byte) (n int, err error) { return len(p), nil } -func (s *Service) Status(ctx context.Context, req *api.StatusRequest) (*api.StatusResponse, error) { +func (s *service) Status(ctx context.Context, req *api.StatusRequest) (*api.StatusResponse, error) { status, err := s.store.Status(ctx, req.Ref) if err != nil { return nil, errdefs.ToGRPCf(err, "could not get status for ref %q", req.Ref) @@ -242,7 +243,7 @@ func (s *Service) Status(ctx context.Context, req *api.StatusRequest) (*api.Stat return &resp, nil } -func (s *Service) ListStatuses(ctx context.Context, req *api.ListStatusesRequest) (*api.ListStatusesResponse, error) { +func (s *service) ListStatuses(ctx context.Context, req *api.ListStatusesRequest) (*api.ListStatusesResponse, error) { statuses, err := s.store.ListStatuses(ctx, req.Filters...) if err != nil { return nil, errdefs.ToGRPC(err) @@ -263,7 +264,7 @@ func (s *Service) ListStatuses(ctx context.Context, req *api.ListStatusesRequest return &resp, nil } -func (s *Service) Write(session api.Content_WriteServer) (err error) { +func (s *service) Write(session api.Content_WriteServer) (err error) { var ( ctx = session.Context() msg api.WriteContentResponse @@ -283,7 +284,7 @@ func (s *Service) Write(session api.Content_WriteServer) (err error) { // identically across all GRPC methods. // // This is pretty noisy, so we can remove it but leave it for now. - log.G(ctx).WithError(err).Error("(*Service).Write failed") + log.G(ctx).WithError(err).Error("(*service).Write failed") } return @@ -319,7 +320,7 @@ func (s *Service) Write(session api.Content_WriteServer) (err error) { ctx = log.WithLogger(ctx, log.G(ctx).WithFields(fields)) - log.G(ctx).Debug("(*Service).Write started") + log.G(ctx).Debug("(*service).Write started") // this action locks the writer for the session. wr, err := s.store.Writer(ctx, ref, total, expected) if err != nil { @@ -444,7 +445,7 @@ func (s *Service) Write(session api.Content_WriteServer) (err error) { } } -func (s *Service) Abort(ctx context.Context, req *api.AbortRequest) (*empty.Empty, error) { +func (s *service) Abort(ctx context.Context, req *api.AbortRequest) (*empty.Empty, error) { if err := s.store.Abort(ctx, req.Ref); err != nil { return nil, errdefs.ToGRPC(err) } diff --git a/services/events/service.go b/services/events/service.go index ae2d3913f..c6f080b8c 100644 --- a/services/events/service.go +++ b/services/events/service.go @@ -21,20 +21,21 @@ func init() { }) } -type Service struct { +type service struct { events *events.Exchange } +// NewService returns the GRPC events server func NewService(events *events.Exchange) api.EventsServer { - return &Service{events: events} + return &service{events: events} } -func (s *Service) Register(server *grpc.Server) error { +func (s *service) Register(server *grpc.Server) error { api.RegisterEventsServer(server, s) return nil } -func (s *Service) Publish(ctx context.Context, r *api.PublishRequest) (*empty.Empty, error) { +func (s *service) Publish(ctx context.Context, r *api.PublishRequest) (*empty.Empty, error) { if err := s.events.Publish(ctx, r.Topic, r.Event); err != nil { return nil, errdefs.ToGRPC(err) } @@ -42,7 +43,7 @@ func (s *Service) Publish(ctx context.Context, r *api.PublishRequest) (*empty.Em return &empty.Empty{}, nil } -func (s *Service) Forward(ctx context.Context, r *api.ForwardRequest) (*empty.Empty, error) { +func (s *service) Forward(ctx context.Context, r *api.ForwardRequest) (*empty.Empty, error) { if err := s.events.Forward(ctx, r.Envelope); err != nil { return nil, errdefs.ToGRPC(err) } @@ -50,7 +51,7 @@ func (s *Service) Forward(ctx context.Context, r *api.ForwardRequest) (*empty.Em return &empty.Empty{}, nil } -func (s *Service) Subscribe(req *api.SubscribeRequest, srv api.Events_SubscribeServer) error { +func (s *service) Subscribe(req *api.SubscribeRequest, srv api.Events_SubscribeServer) error { ctx, cancel := context.WithCancel(srv.Context()) defer cancel() diff --git a/services/healthcheck/service.go b/services/healthcheck/service.go index bb7d33e5f..85ffdab67 100644 --- a/services/healthcheck/service.go +++ b/services/healthcheck/service.go @@ -8,7 +8,7 @@ import ( "google.golang.org/grpc/health/grpc_health_v1" ) -type Service struct { +type service struct { serve *health.Server } @@ -22,13 +22,14 @@ func init() { }) } -func NewService() (*Service, error) { - return &Service{ +// NewService returns the GRPC health server +func NewService() (*service, error) { + return &service{ health.NewServer(), }, nil } -func (s *Service) Register(server *grpc.Server) error { +func (s *service) Register(server *grpc.Server) error { grpc_health_v1.RegisterHealthServer(server, s.serve) return nil } diff --git a/services/images/service.go b/services/images/service.go index fa8a00aae..3843df554 100644 --- a/services/images/service.go +++ b/services/images/service.go @@ -34,24 +34,25 @@ func init() { }) } -type Service struct { +type service struct { db *metadata.DB publisher events.Publisher } +// NewService returns the GRPC image server func NewService(db *metadata.DB, publisher events.Publisher) imagesapi.ImagesServer { - return &Service{ + return &service{ db: db, publisher: publisher, } } -func (s *Service) Register(server *grpc.Server) error { +func (s *service) Register(server *grpc.Server) error { imagesapi.RegisterImagesServer(server, s) return nil } -func (s *Service) Get(ctx context.Context, req *imagesapi.GetImageRequest) (*imagesapi.GetImageResponse, error) { +func (s *service) Get(ctx context.Context, req *imagesapi.GetImageRequest) (*imagesapi.GetImageResponse, error) { var resp imagesapi.GetImageResponse return &resp, errdefs.ToGRPC(s.withStoreView(ctx, func(ctx context.Context, store images.Store) error { @@ -65,7 +66,7 @@ func (s *Service) Get(ctx context.Context, req *imagesapi.GetImageRequest) (*ima })) } -func (s *Service) List(ctx context.Context, req *imagesapi.ListImagesRequest) (*imagesapi.ListImagesResponse, error) { +func (s *service) List(ctx context.Context, req *imagesapi.ListImagesRequest) (*imagesapi.ListImagesResponse, error) { var resp imagesapi.ListImagesResponse return &resp, errdefs.ToGRPC(s.withStoreView(ctx, func(ctx context.Context, store images.Store) error { @@ -79,7 +80,7 @@ func (s *Service) List(ctx context.Context, req *imagesapi.ListImagesRequest) (* })) } -func (s *Service) Create(ctx context.Context, req *imagesapi.CreateImageRequest) (*imagesapi.CreateImageResponse, error) { +func (s *service) Create(ctx context.Context, req *imagesapi.CreateImageRequest) (*imagesapi.CreateImageResponse, error) { if req.Image.Name == "" { return nil, status.Errorf(codes.InvalidArgument, "Image.Name required") } @@ -111,7 +112,7 @@ func (s *Service) Create(ctx context.Context, req *imagesapi.CreateImageRequest) } -func (s *Service) Update(ctx context.Context, req *imagesapi.UpdateImageRequest) (*imagesapi.UpdateImageResponse, error) { +func (s *service) Update(ctx context.Context, req *imagesapi.UpdateImageRequest) (*imagesapi.UpdateImageResponse, error) { if req.Image.Name == "" { return nil, status.Errorf(codes.InvalidArgument, "Image.Name required") } @@ -149,7 +150,7 @@ func (s *Service) Update(ctx context.Context, req *imagesapi.UpdateImageRequest) return &resp, nil } -func (s *Service) Delete(ctx context.Context, req *imagesapi.DeleteImageRequest) (*empty.Empty, error) { +func (s *service) Delete(ctx context.Context, req *imagesapi.DeleteImageRequest) (*empty.Empty, error) { if err := s.withStoreUpdate(ctx, func(ctx context.Context, store images.Store) error { return errdefs.ToGRPC(store.Delete(ctx, req.Name)) }); err != nil { @@ -169,14 +170,14 @@ func (s *Service) Delete(ctx context.Context, req *imagesapi.DeleteImageRequest) return &empty.Empty{}, nil } -func (s *Service) withStore(ctx context.Context, fn func(ctx context.Context, store images.Store) error) func(tx *bolt.Tx) error { +func (s *service) withStore(ctx context.Context, fn func(ctx context.Context, store images.Store) error) func(tx *bolt.Tx) error { return func(tx *bolt.Tx) error { return fn(ctx, metadata.NewImageStore(tx)) } } -func (s *Service) withStoreView(ctx context.Context, fn func(ctx context.Context, store images.Store) error) error { +func (s *service) withStoreView(ctx context.Context, fn func(ctx context.Context, store images.Store) error) error { return s.db.View(s.withStore(ctx, fn)) } -func (s *Service) withStoreUpdate(ctx context.Context, fn func(ctx context.Context, store images.Store) error) error { +func (s *service) withStoreUpdate(ctx context.Context, fn func(ctx context.Context, store images.Store) error) error { return s.db.Update(s.withStore(ctx, fn)) } diff --git a/services/introspection/service.go b/services/introspection/service.go index 34c2307aa..001db8e53 100644 --- a/services/introspection/service.go +++ b/services/introspection/service.go @@ -28,22 +28,23 @@ func init() { }) } -type Service struct { +type service struct { plugins []api.Plugin } +// NewService returns the GRPC introspection server func NewService(plugins []api.Plugin) api.IntrospectionServer { - return &Service{ + return &service{ plugins: plugins, } } -func (s *Service) Register(server *grpc.Server) error { +func (s *service) Register(server *grpc.Server) error { api.RegisterIntrospectionServer(server, s) return nil } -func (s *Service) Plugins(ctx context.Context, req *api.PluginsRequest) (*api.PluginsResponse, error) { +func (s *service) Plugins(ctx context.Context, req *api.PluginsRequest) (*api.PluginsResponse, error) { filter, err := filters.ParseAll(req.Filters...) if err != nil { return nil, errdefs.ToGRPCf(errdefs.ErrInvalidArgument, err.Error()) diff --git a/services/namespaces/service.go b/services/namespaces/service.go index c0f6eb733..b795ab52f 100644 --- a/services/namespaces/service.go +++ b/services/namespaces/service.go @@ -34,26 +34,27 @@ func init() { }) } -type Service struct { +type service struct { db *metadata.DB publisher events.Publisher } -var _ api.NamespacesServer = &Service{} +var _ api.NamespacesServer = &service{} +// NewService returns the GRPC namespaces server func NewService(db *metadata.DB, publisher events.Publisher) api.NamespacesServer { - return &Service{ + return &service{ db: db, publisher: publisher, } } -func (s *Service) Register(server *grpc.Server) error { +func (s *service) Register(server *grpc.Server) error { api.RegisterNamespacesServer(server, s) return nil } -func (s *Service) Get(ctx context.Context, req *api.GetNamespaceRequest) (*api.GetNamespaceResponse, error) { +func (s *service) Get(ctx context.Context, req *api.GetNamespaceRequest) (*api.GetNamespaceResponse, error) { var resp api.GetNamespaceResponse return &resp, s.withStoreView(ctx, func(ctx context.Context, store namespaces.Store) error { @@ -71,7 +72,7 @@ func (s *Service) Get(ctx context.Context, req *api.GetNamespaceRequest) (*api.G }) } -func (s *Service) List(ctx context.Context, req *api.ListNamespacesRequest) (*api.ListNamespacesResponse, error) { +func (s *service) List(ctx context.Context, req *api.ListNamespacesRequest) (*api.ListNamespacesResponse, error) { var resp api.ListNamespacesResponse return &resp, s.withStoreView(ctx, func(ctx context.Context, store namespaces.Store) error { @@ -98,7 +99,7 @@ func (s *Service) List(ctx context.Context, req *api.ListNamespacesRequest) (*ap }) } -func (s *Service) Create(ctx context.Context, req *api.CreateNamespaceRequest) (*api.CreateNamespaceResponse, error) { +func (s *service) Create(ctx context.Context, req *api.CreateNamespaceRequest) (*api.CreateNamespaceResponse, error) { var resp api.CreateNamespaceResponse if err := s.withStoreUpdate(ctx, func(ctx context.Context, store namespaces.Store) error { @@ -129,7 +130,7 @@ func (s *Service) Create(ctx context.Context, req *api.CreateNamespaceRequest) ( } -func (s *Service) Update(ctx context.Context, req *api.UpdateNamespaceRequest) (*api.UpdateNamespaceResponse, error) { +func (s *service) Update(ctx context.Context, req *api.UpdateNamespaceRequest) (*api.UpdateNamespaceResponse, error) { var resp api.UpdateNamespaceResponse if err := s.withStoreUpdate(ctx, func(ctx context.Context, store namespaces.Store) error { if req.UpdateMask != nil && len(req.UpdateMask.Paths) > 0 { @@ -181,7 +182,7 @@ func (s *Service) Update(ctx context.Context, req *api.UpdateNamespaceRequest) ( return &resp, nil } -func (s *Service) Delete(ctx context.Context, req *api.DeleteNamespaceRequest) (*empty.Empty, error) { +func (s *service) Delete(ctx context.Context, req *api.DeleteNamespaceRequest) (*empty.Empty, error) { if err := s.withStoreUpdate(ctx, func(ctx context.Context, store namespaces.Store) error { return errdefs.ToGRPC(store.Delete(ctx, req.Name)) }); err != nil { @@ -198,14 +199,14 @@ func (s *Service) Delete(ctx context.Context, req *api.DeleteNamespaceRequest) ( return &empty.Empty{}, nil } -func (s *Service) withStore(ctx context.Context, fn func(ctx context.Context, store namespaces.Store) error) func(tx *bolt.Tx) error { +func (s *service) withStore(ctx context.Context, fn func(ctx context.Context, store namespaces.Store) error) func(tx *bolt.Tx) error { return func(tx *bolt.Tx) error { return fn(ctx, metadata.NewNamespaceStore(tx)) } } -func (s *Service) withStoreView(ctx context.Context, fn func(ctx context.Context, store namespaces.Store) error) error { +func (s *service) withStoreView(ctx context.Context, fn func(ctx context.Context, store namespaces.Store) error) error { return s.db.View(s.withStore(ctx, fn)) } -func (s *Service) withStoreUpdate(ctx context.Context, fn func(ctx context.Context, store namespaces.Store) error) error { +func (s *service) withStoreUpdate(ctx context.Context, fn func(ctx context.Context, store namespaces.Store) error) error { return s.db.Update(s.withStore(ctx, fn)) } diff --git a/services/tasks/service.go b/services/tasks/service.go index 75904a98c..2c58c9d72 100644 --- a/services/tasks/service.go +++ b/services/tasks/service.go @@ -34,7 +34,7 @@ import ( ) var ( - _ = (api.TasksServer)(&Service{}) + _ = (api.TasksServer)(&service{}) empty = &google_protobuf.Empty{} ) @@ -46,11 +46,11 @@ func init() { plugin.RuntimePlugin, plugin.MetadataPlugin, }, - InitFn: New, + InitFn: initFunc, }) } -func New(ic *plugin.InitContext) (interface{}, error) { +func initFunc(ic *plugin.InitContext) (interface{}, error) { rt, err := ic.GetByType(plugin.RuntimePlugin) if err != nil { return nil, err @@ -75,7 +75,7 @@ func New(ic *plugin.InitContext) (interface{}, error) { if len(runtimes) == 0 { return nil, errors.New("no runtimes available to create task service") } - return &Service{ + return &service{ runtimes: runtimes, db: m.(*metadata.DB), store: cs, @@ -83,19 +83,19 @@ func New(ic *plugin.InitContext) (interface{}, error) { }, nil } -type Service struct { +type service struct { runtimes map[string]runtime.Runtime db *metadata.DB store content.Store publisher events.Publisher } -func (s *Service) Register(server *grpc.Server) error { +func (s *service) Register(server *grpc.Server) error { api.RegisterTasksServer(server, s) return nil } -func (s *Service) Create(ctx context.Context, r *api.CreateTaskRequest) (*api.CreateTaskResponse, error) { +func (s *service) Create(ctx context.Context, r *api.CreateTaskRequest) (*api.CreateTaskResponse, error) { var ( checkpointPath string err error @@ -160,7 +160,7 @@ func (s *Service) Create(ctx context.Context, r *api.CreateTaskRequest) (*api.Cr }, nil } -func (s *Service) Start(ctx context.Context, r *api.StartRequest) (*api.StartResponse, error) { +func (s *service) Start(ctx context.Context, r *api.StartRequest) (*api.StartResponse, error) { t, err := s.getTask(ctx, r.ContainerID) if err != nil { return nil, err @@ -183,7 +183,7 @@ func (s *Service) Start(ctx context.Context, r *api.StartRequest) (*api.StartRes }, nil } -func (s *Service) Delete(ctx context.Context, r *api.DeleteTaskRequest) (*api.DeleteResponse, error) { +func (s *service) Delete(ctx context.Context, r *api.DeleteTaskRequest) (*api.DeleteResponse, error) { t, err := s.getTask(ctx, r.ContainerID) if err != nil { return nil, err @@ -203,7 +203,7 @@ func (s *Service) Delete(ctx context.Context, r *api.DeleteTaskRequest) (*api.De }, nil } -func (s *Service) DeleteProcess(ctx context.Context, r *api.DeleteProcessRequest) (*api.DeleteResponse, error) { +func (s *service) DeleteProcess(ctx context.Context, r *api.DeleteProcessRequest) (*api.DeleteResponse, error) { t, err := s.getTask(ctx, r.ContainerID) if err != nil { return nil, err @@ -253,7 +253,7 @@ func processFromContainerd(ctx context.Context, p runtime.Process) (*task.Proces }, nil } -func (s *Service) Get(ctx context.Context, r *api.GetRequest) (*api.GetResponse, error) { +func (s *service) Get(ctx context.Context, r *api.GetRequest) (*api.GetResponse, error) { task, err := s.getTask(ctx, r.ContainerID) if err != nil { return nil, err @@ -273,7 +273,7 @@ func (s *Service) Get(ctx context.Context, r *api.GetRequest) (*api.GetResponse, }, nil } -func (s *Service) List(ctx context.Context, r *api.ListTasksRequest) (*api.ListTasksResponse, error) { +func (s *service) List(ctx context.Context, r *api.ListTasksRequest) (*api.ListTasksResponse, error) { resp := &api.ListTasksResponse{} for _, r := range s.runtimes { tasks, err := r.Tasks(ctx) @@ -296,7 +296,7 @@ func addTasks(ctx context.Context, r *api.ListTasksResponse, tasks []runtime.Tas } } -func (s *Service) Pause(ctx context.Context, r *api.PauseTaskRequest) (*google_protobuf.Empty, error) { +func (s *service) Pause(ctx context.Context, r *api.PauseTaskRequest) (*google_protobuf.Empty, error) { t, err := s.getTask(ctx, r.ContainerID) if err != nil { return nil, err @@ -308,7 +308,7 @@ func (s *Service) Pause(ctx context.Context, r *api.PauseTaskRequest) (*google_p return empty, nil } -func (s *Service) Resume(ctx context.Context, r *api.ResumeTaskRequest) (*google_protobuf.Empty, error) { +func (s *service) Resume(ctx context.Context, r *api.ResumeTaskRequest) (*google_protobuf.Empty, error) { t, err := s.getTask(ctx, r.ContainerID) if err != nil { return nil, err @@ -320,7 +320,7 @@ func (s *Service) Resume(ctx context.Context, r *api.ResumeTaskRequest) (*google return empty, nil } -func (s *Service) Kill(ctx context.Context, r *api.KillRequest) (*google_protobuf.Empty, error) { +func (s *service) Kill(ctx context.Context, r *api.KillRequest) (*google_protobuf.Empty, error) { t, err := s.getTask(ctx, r.ContainerID) if err != nil { return nil, err @@ -337,7 +337,7 @@ func (s *Service) Kill(ctx context.Context, r *api.KillRequest) (*google_protobu return empty, nil } -func (s *Service) ListPids(ctx context.Context, r *api.ListPidsRequest) (*api.ListPidsResponse, error) { +func (s *service) ListPids(ctx context.Context, r *api.ListPidsRequest) (*api.ListPidsResponse, error) { t, err := s.getTask(ctx, r.ContainerID) if err != nil { return nil, err @@ -365,7 +365,7 @@ func (s *Service) ListPids(ctx context.Context, r *api.ListPidsRequest) (*api.Li }, nil } -func (s *Service) Exec(ctx context.Context, r *api.ExecProcessRequest) (*google_protobuf.Empty, error) { +func (s *service) Exec(ctx context.Context, r *api.ExecProcessRequest) (*google_protobuf.Empty, error) { if r.ExecID == "" { return nil, grpc.Errorf(codes.InvalidArgument, "exec id cannot be empty") } @@ -387,7 +387,7 @@ func (s *Service) Exec(ctx context.Context, r *api.ExecProcessRequest) (*google_ return empty, nil } -func (s *Service) ResizePty(ctx context.Context, r *api.ResizePtyRequest) (*google_protobuf.Empty, error) { +func (s *service) ResizePty(ctx context.Context, r *api.ResizePtyRequest) (*google_protobuf.Empty, error) { t, err := s.getTask(ctx, r.ContainerID) if err != nil { return nil, err @@ -407,7 +407,7 @@ func (s *Service) ResizePty(ctx context.Context, r *api.ResizePtyRequest) (*goog return empty, nil } -func (s *Service) CloseIO(ctx context.Context, r *api.CloseIORequest) (*google_protobuf.Empty, error) { +func (s *service) CloseIO(ctx context.Context, r *api.CloseIORequest) (*google_protobuf.Empty, error) { t, err := s.getTask(ctx, r.ContainerID) if err != nil { return nil, err @@ -426,7 +426,7 @@ func (s *Service) CloseIO(ctx context.Context, r *api.CloseIORequest) (*google_p return empty, nil } -func (s *Service) Checkpoint(ctx context.Context, r *api.CheckpointTaskRequest) (*api.CheckpointTaskResponse, error) { +func (s *service) Checkpoint(ctx context.Context, r *api.CheckpointTaskRequest) (*api.CheckpointTaskResponse, error) { container, err := s.getContainer(ctx, r.ContainerID) if err != nil { return nil, err @@ -471,7 +471,7 @@ func (s *Service) Checkpoint(ctx context.Context, r *api.CheckpointTaskRequest) }, nil } -func (s *Service) Update(ctx context.Context, r *api.UpdateTaskRequest) (*google_protobuf.Empty, error) { +func (s *service) Update(ctx context.Context, r *api.UpdateTaskRequest) (*google_protobuf.Empty, error) { t, err := s.getTask(ctx, r.ContainerID) if err != nil { return nil, err @@ -482,7 +482,7 @@ func (s *Service) Update(ctx context.Context, r *api.UpdateTaskRequest) (*google return empty, nil } -func (s *Service) Metrics(ctx context.Context, r *api.MetricsRequest) (*api.MetricsResponse, error) { +func (s *service) Metrics(ctx context.Context, r *api.MetricsRequest) (*api.MetricsResponse, error) { filter, err := filters.ParseAll(r.Filters...) if err != nil { return nil, err @@ -498,7 +498,7 @@ func (s *Service) Metrics(ctx context.Context, r *api.MetricsRequest) (*api.Metr return &resp, nil } -func (s *Service) Wait(ctx context.Context, r *api.WaitRequest) (*api.WaitResponse, error) { +func (s *service) Wait(ctx context.Context, r *api.WaitRequest) (*api.WaitResponse, error) { t, err := s.getTask(ctx, r.ContainerID) if err != nil { return nil, err @@ -557,7 +557,7 @@ func getTasksMetrics(ctx context.Context, filter filters.Filter, tasks []runtime } } -func (s *Service) writeContent(ctx context.Context, mediaType, ref string, r io.Reader) (*types.Descriptor, error) { +func (s *service) writeContent(ctx context.Context, mediaType, ref string, r io.Reader) (*types.Descriptor, error) { writer, err := s.store.Writer(ctx, ref, 0, "") if err != nil { return nil, err @@ -580,7 +580,7 @@ func (s *Service) writeContent(ctx context.Context, mediaType, ref string, r io. }, nil } -func (s *Service) getContainer(ctx context.Context, id string) (*containers.Container, error) { +func (s *service) getContainer(ctx context.Context, id string) (*containers.Container, error) { var container containers.Container if err := s.db.View(func(tx *bolt.Tx) error { store := metadata.NewContainerStore(tx) @@ -593,7 +593,7 @@ func (s *Service) getContainer(ctx context.Context, id string) (*containers.Cont return &container, nil } -func (s *Service) getTask(ctx context.Context, id string) (runtime.Task, error) { +func (s *service) getTask(ctx context.Context, id string) (runtime.Task, error) { container, err := s.getContainer(ctx, id) if err != nil { return nil, err @@ -601,7 +601,7 @@ func (s *Service) getTask(ctx context.Context, id string) (runtime.Task, error) return s.getTaskFromContainer(ctx, container) } -func (s *Service) getTaskFromContainer(ctx context.Context, container *containers.Container) (runtime.Task, error) { +func (s *service) getTaskFromContainer(ctx context.Context, container *containers.Container) (runtime.Task, error) { runtime, err := s.getRuntime(container.Runtime.Name) if err != nil { return nil, errdefs.ToGRPCf(err, "runtime for task %s", container.Runtime.Name) @@ -613,7 +613,7 @@ func (s *Service) getTaskFromContainer(ctx context.Context, container *container return t, nil } -func (s *Service) getRuntime(name string) (runtime.Runtime, error) { +func (s *service) getRuntime(name string) (runtime.Runtime, error) { runtime, ok := s.runtimes[name] if !ok { return nil, grpc.Errorf(codes.NotFound, "unknown runtime %q", name) diff --git a/services/version/service.go b/services/version/service.go index b0e1ee6da..5a7454f24 100644 --- a/services/version/service.go +++ b/services/version/service.go @@ -9,29 +9,29 @@ import ( "google.golang.org/grpc" ) -var _ api.VersionServer = &Service{} +var _ api.VersionServer = &service{} func init() { plugin.Register(&plugin.Registration{ Type: plugin.GRPCPlugin, ID: "version", - InitFn: New, + InitFn: initFunc, }) } -func New(ic *plugin.InitContext) (interface{}, error) { - return &Service{}, nil +func initFunc(ic *plugin.InitContext) (interface{}, error) { + return &service{}, nil } -type Service struct { +type service struct { } -func (s *Service) Register(server *grpc.Server) error { +func (s *service) Register(server *grpc.Server) error { api.RegisterVersionServer(server, s) return nil } -func (s *Service) Version(ctx context.Context, _ *empty.Empty) (*api.VersionResponse, error) { +func (s *service) Version(ctx context.Context, _ *empty.Empty) (*api.VersionResponse, error) { return &api.VersionResponse{ Version: ctrdversion.Version, Revision: ctrdversion.Revision, diff --git a/windows/hcsshimtypes/doc.go b/windows/hcsshimtypes/doc.go index 1712b1aed..4b1b4b341 100644 --- a/windows/hcsshimtypes/doc.go +++ b/windows/hcsshimtypes/doc.go @@ -1,2 +1,2 @@ -// hcsshimtypes holds the windows runtime specific types +// Package hcsshimtypes holds the windows runtime specific types package hcsshimtypes