Skip to content

Commit

Permalink
remove type aliases for leasemanager/contentstore
Browse files Browse the repository at this point in the history
Linter does not understand these and shows bogus warnings.

Signed-off-by: Tonis Tiigi <tonistiigi@gmail.com>
  • Loading branch information
tonistiigi committed May 3, 2023
1 parent 7a3c4e1 commit bc3443a
Show file tree
Hide file tree
Showing 2 changed files with 27 additions and 31 deletions.
36 changes: 17 additions & 19 deletions snapshot/containerd/content.go
Original file line number Diff line number Diff line change
Expand Up @@ -11,76 +11,74 @@ import (
"github.com/pkg/errors"
)

type Store = nsContent

func NewContentStore(store content.Store, ns string) *Store {
return &nsContent{ns, store}
return &Store{ns, store}
}

type nsContent struct {
type Store struct {
ns string
content.Store
}

func (c *nsContent) Namespace() string {
func (c *Store) Namespace() string {
return c.ns
}

func (c *nsContent) WithNamespace(ns string) *Store {
func (c *Store) WithNamespace(ns string) *Store {
return NewContentStore(c.Store, ns)
}

func (c *nsContent) Info(ctx context.Context, dgst digest.Digest) (content.Info, error) {
func (c *Store) Info(ctx context.Context, dgst digest.Digest) (content.Info, error) {
ctx = namespaces.WithNamespace(ctx, c.ns)
return c.Store.Info(ctx, dgst)
}

func (c *nsContent) Update(ctx context.Context, info content.Info, fieldpaths ...string) (content.Info, error) {
func (c *Store) Update(ctx context.Context, info content.Info, fieldpaths ...string) (content.Info, error) {
ctx = namespaces.WithNamespace(ctx, c.ns)
return c.Store.Update(ctx, info, fieldpaths...)
}

func (c *nsContent) Walk(ctx context.Context, fn content.WalkFunc, filters ...string) error {
func (c *Store) Walk(ctx context.Context, fn content.WalkFunc, filters ...string) error {
ctx = namespaces.WithNamespace(ctx, c.ns)
return c.Store.Walk(ctx, fn, filters...)
}

func (c *nsContent) Delete(ctx context.Context, dgst digest.Digest) error {
func (c *Store) Delete(ctx context.Context, dgst digest.Digest) error {
return errors.Errorf("contentstore.Delete usage is forbidden")
}

func (c *nsContent) Status(ctx context.Context, ref string) (content.Status, error) {
func (c *Store) Status(ctx context.Context, ref string) (content.Status, error) {
ctx = namespaces.WithNamespace(ctx, c.ns)
return c.Store.Status(ctx, ref)
}

func (c *nsContent) ListStatuses(ctx context.Context, filters ...string) ([]content.Status, error) {
func (c *Store) ListStatuses(ctx context.Context, filters ...string) ([]content.Status, error) {
ctx = namespaces.WithNamespace(ctx, c.ns)
return c.Store.ListStatuses(ctx, filters...)
}

func (c *nsContent) Abort(ctx context.Context, ref string) error {
func (c *Store) Abort(ctx context.Context, ref string) error {
ctx = namespaces.WithNamespace(ctx, c.ns)
return c.Store.Abort(ctx, ref)
}

func (c *nsContent) ReaderAt(ctx context.Context, desc ocispecs.Descriptor) (content.ReaderAt, error) {
func (c *Store) ReaderAt(ctx context.Context, desc ocispecs.Descriptor) (content.ReaderAt, error) {
ctx = namespaces.WithNamespace(ctx, c.ns)
return c.Store.ReaderAt(ctx, desc)
}

func (c *nsContent) Writer(ctx context.Context, opts ...content.WriterOpt) (content.Writer, error) {
func (c *Store) Writer(ctx context.Context, opts ...content.WriterOpt) (content.Writer, error) {
return c.writer(ctx, 3, opts...)
}

func (c *nsContent) WithFallbackNS(ns string) content.Store {
func (c *Store) WithFallbackNS(ns string) content.Store {
return &nsFallbackStore{
main: c,
fb: c.WithNamespace(ns),
}
}

func (c *nsContent) writer(ctx context.Context, retries int, opts ...content.WriterOpt) (content.Writer, error) {
func (c *Store) writer(ctx context.Context, retries int, opts ...content.WriterOpt) (content.Writer, error) {
ctx = namespaces.WithNamespace(ctx, c.ns)
w, err := c.Store.Writer(ctx, opts...)
if err != nil {
Expand All @@ -100,8 +98,8 @@ func (w *nsWriter) Commit(ctx context.Context, size int64, expected digest.Diges
}

type nsFallbackStore struct {
main *nsContent
fb *nsContent
main *Store
fb *Store
}

var _ content.Store = &nsFallbackStore{}
Expand Down
22 changes: 10 additions & 12 deletions util/leaseutil/manager.go
Original file line number Diff line number Diff line change
Expand Up @@ -35,51 +35,49 @@ func MakeTemporary(l *leases.Lease) error {
return nil
}

type Manager = nsLM

func WithNamespace(lm leases.Manager, ns string) *Manager {
return &nsLM{manager: lm, ns: ns}
return &Manager{manager: lm, ns: ns}
}

type nsLM struct {
type Manager struct {
manager leases.Manager
ns string
}

func (l *nsLM) Namespace() string {
func (l *Manager) Namespace() string {
return l.ns
}

func (l *nsLM) WithNamespace(ns string) *Manager {
func (l *Manager) WithNamespace(ns string) *Manager {
return WithNamespace(l.manager, ns)
}

func (l *nsLM) Create(ctx context.Context, opts ...leases.Opt) (leases.Lease, error) {
func (l *Manager) Create(ctx context.Context, opts ...leases.Opt) (leases.Lease, error) {
ctx = namespaces.WithNamespace(ctx, l.ns)
return l.manager.Create(ctx, opts...)
}

func (l *nsLM) Delete(ctx context.Context, lease leases.Lease, opts ...leases.DeleteOpt) error {
func (l *Manager) Delete(ctx context.Context, lease leases.Lease, opts ...leases.DeleteOpt) error {
ctx = namespaces.WithNamespace(ctx, l.ns)
return l.manager.Delete(ctx, lease, opts...)
}

func (l *nsLM) List(ctx context.Context, filters ...string) ([]leases.Lease, error) {
func (l *Manager) List(ctx context.Context, filters ...string) ([]leases.Lease, error) {
ctx = namespaces.WithNamespace(ctx, l.ns)
return l.manager.List(ctx, filters...)
}

func (l *nsLM) AddResource(ctx context.Context, lease leases.Lease, resource leases.Resource) error {
func (l *Manager) AddResource(ctx context.Context, lease leases.Lease, resource leases.Resource) error {
ctx = namespaces.WithNamespace(ctx, l.ns)
return l.manager.AddResource(ctx, lease, resource)
}

func (l *nsLM) DeleteResource(ctx context.Context, lease leases.Lease, resource leases.Resource) error {
func (l *Manager) DeleteResource(ctx context.Context, lease leases.Lease, resource leases.Resource) error {
ctx = namespaces.WithNamespace(ctx, l.ns)
return l.manager.DeleteResource(ctx, lease, resource)
}

func (l *nsLM) ListResources(ctx context.Context, lease leases.Lease) ([]leases.Resource, error) {
func (l *Manager) ListResources(ctx context.Context, lease leases.Lease) ([]leases.Resource, error) {
ctx = namespaces.WithNamespace(ctx, l.ns)
return l.manager.ListResources(ctx, lease)
}

0 comments on commit bc3443a

Please sign in to comment.