8da4346686
A race occurs today where the cleanup runs after the lease has been deleted, making all the content and statuses eligible for collection during the cleanup. There is a case where a status could be listed but removed before the abort is called. In this case, abort will return a not found and the test cleanup should ignore it. Signed-off-by: Derek McGowan <derek@mcgstyle.net>
89 lines
2.2 KiB
Go
89 lines
2.2 KiB
Go
/*
|
|
Copyright The containerd Authors.
|
|
|
|
Licensed under the Apache License, Version 2.0 (the "License");
|
|
you may not use this file except in compliance with the License.
|
|
You may obtain a copy of the License at
|
|
|
|
http://www.apache.org/licenses/LICENSE-2.0
|
|
|
|
Unless required by applicable law or agreed to in writing, software
|
|
distributed under the License is distributed on an "AS IS" BASIS,
|
|
WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
|
|
See the License for the specific language governing permissions and
|
|
limitations under the License.
|
|
*/
|
|
|
|
package containerd
|
|
|
|
import (
|
|
"context"
|
|
"fmt"
|
|
"sync/atomic"
|
|
"testing"
|
|
|
|
"github.com/containerd/containerd/content"
|
|
"github.com/containerd/containerd/content/testsuite"
|
|
"github.com/containerd/containerd/errdefs"
|
|
"github.com/containerd/containerd/namespaces"
|
|
"github.com/pkg/errors"
|
|
)
|
|
|
|
func newContentStore(ctx context.Context, root string) (context.Context, content.Store, func() error, error) {
|
|
client, err := New(address)
|
|
if err != nil {
|
|
return nil, nil, nil, err
|
|
}
|
|
|
|
var (
|
|
count uint64
|
|
cs = client.ContentStore()
|
|
name = testsuite.Name(ctx)
|
|
)
|
|
|
|
wrap := func(ctx context.Context) (context.Context, func(context.Context) error, error) {
|
|
n := atomic.AddUint64(&count, 1)
|
|
ctx = namespaces.WithNamespace(ctx, fmt.Sprintf("%s-n%d", name, n))
|
|
return client.WithLease(ctx)
|
|
}
|
|
|
|
ctx = testsuite.SetContextWrapper(ctx, wrap)
|
|
|
|
return ctx, cs, func() error {
|
|
for i := uint64(1); i <= count; i++ {
|
|
ctx = namespaces.WithNamespace(ctx, fmt.Sprintf("%s-n%d", name, i))
|
|
statuses, err := cs.ListStatuses(ctx)
|
|
if err != nil {
|
|
return err
|
|
}
|
|
for _, st := range statuses {
|
|
if err := cs.Abort(ctx, st.Ref); err != nil && !errdefs.IsNotFound(err) {
|
|
return errors.Wrapf(err, "failed to abort %s", st.Ref)
|
|
}
|
|
}
|
|
err = cs.Walk(ctx, func(info content.Info) error {
|
|
if err := cs.Delete(ctx, info.Digest); err != nil {
|
|
if errdefs.IsNotFound(err) {
|
|
return nil
|
|
}
|
|
|
|
return err
|
|
}
|
|
return nil
|
|
})
|
|
if err != nil {
|
|
return err
|
|
}
|
|
}
|
|
return nil
|
|
|
|
}, nil
|
|
}
|
|
|
|
func TestContentClient(t *testing.T) {
|
|
if testing.Short() {
|
|
t.Skip()
|
|
}
|
|
testsuite.ContentSuite(t, "ContentClient", newContentStore)
|
|
}
|