mirror of
https://github.com/k3s-io/kubernetes.git
synced 2025-07-21 02:41:25 +00:00
The recently introduced failure handling in ExpectNoError depends on error wrapping: if an error prefix gets added with `fmt.Errorf("foo: %v", err)`, then ExpectNoError cannot detect that the root cause is an assertion failure and then will add another useless "unexpected error" prefix and will not dump the additional failure information (currently the backtrace inside the E2E framework). Instead of manually deciding on a case-by-case basis where %w is needed, all error wrapping was updated automatically with sed -i "s/fmt.Errorf\(.*\): '*\(%s\|%v\)'*\",\(.* err)\)/fmt.Errorf\1: %w\",\3/" $(git grep -l 'fmt.Errorf' test/e2e*) This may be unnecessary in some cases, but it's not wrong.
181 lines
7.4 KiB
Go
181 lines
7.4 KiB
Go
/*
|
|
Copyright 2019 The Kubernetes 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 node
|
|
|
|
import (
|
|
"context"
|
|
"encoding/json"
|
|
"fmt"
|
|
"time"
|
|
|
|
coordinationv1 "k8s.io/api/coordination/v1"
|
|
apiequality "k8s.io/apimachinery/pkg/api/equality"
|
|
apierrors "k8s.io/apimachinery/pkg/api/errors"
|
|
metav1 "k8s.io/apimachinery/pkg/apis/meta/v1"
|
|
"k8s.io/apimachinery/pkg/labels"
|
|
"k8s.io/apimachinery/pkg/types"
|
|
"k8s.io/apimachinery/pkg/util/strategicpatch"
|
|
"k8s.io/kubernetes/test/e2e/framework"
|
|
admissionapi "k8s.io/pod-security-admission/api"
|
|
"k8s.io/utils/pointer"
|
|
|
|
"github.com/google/go-cmp/cmp"
|
|
)
|
|
|
|
func getPatchBytes(oldLease, newLease *coordinationv1.Lease) ([]byte, error) {
|
|
oldData, err := json.Marshal(oldLease)
|
|
if err != nil {
|
|
return nil, fmt.Errorf("failed to Marshal oldData: %w", err)
|
|
}
|
|
newData, err := json.Marshal(newLease)
|
|
if err != nil {
|
|
return nil, fmt.Errorf("failed to Marshal newData: %w", err)
|
|
}
|
|
patchBytes, err := strategicpatch.CreateTwoWayMergePatch(oldData, newData, coordinationv1.Lease{})
|
|
if err != nil {
|
|
return nil, fmt.Errorf("failed to CreateTwoWayMergePatch: %w", err)
|
|
}
|
|
return patchBytes, nil
|
|
}
|
|
|
|
var _ = SIGDescribe("Lease", func() {
|
|
f := framework.NewDefaultFramework("lease-test")
|
|
f.NamespacePodSecurityEnforceLevel = admissionapi.LevelPrivileged
|
|
|
|
/*
|
|
Release: v1.17
|
|
Testname: lease API should be available
|
|
Description: Create Lease object, and get it; create and get MUST be successful and Spec of the
|
|
read Lease MUST match Spec of original Lease. Update the Lease and get it; update and get MUST
|
|
be successful and Spec of the read Lease MUST match Spec of updated Lease. Patch the Lease and
|
|
get it; patch and get MUST be successful and Spec of the read Lease MUST match Spec of patched
|
|
Lease. Create a second Lease with labels and list Leases; create and list MUST be successful and
|
|
list MUST return both leases. Delete the labels lease via delete collection; the delete MUST be
|
|
successful and MUST delete only the labels lease. List leases; list MUST be successful and MUST
|
|
return just the remaining lease. Delete the lease; delete MUST be successful. Get the lease; get
|
|
MUST return not found error.
|
|
*/
|
|
framework.ConformanceIt("lease API should be available", func(ctx context.Context) {
|
|
leaseClient := f.ClientSet.CoordinationV1().Leases(f.Namespace.Name)
|
|
|
|
name := "lease"
|
|
lease := &coordinationv1.Lease{
|
|
ObjectMeta: metav1.ObjectMeta{
|
|
Name: name,
|
|
},
|
|
Spec: coordinationv1.LeaseSpec{
|
|
HolderIdentity: pointer.String("holder"),
|
|
LeaseDurationSeconds: pointer.Int32(30),
|
|
AcquireTime: &metav1.MicroTime{Time: time.Time{}.Add(2 * time.Second)},
|
|
RenewTime: &metav1.MicroTime{Time: time.Time{}.Add(5 * time.Second)},
|
|
LeaseTransitions: pointer.Int32(0),
|
|
},
|
|
}
|
|
|
|
createdLease, err := leaseClient.Create(ctx, lease, metav1.CreateOptions{})
|
|
framework.ExpectNoError(err, "creating Lease failed")
|
|
|
|
readLease, err := leaseClient.Get(ctx, name, metav1.GetOptions{})
|
|
framework.ExpectNoError(err, "couldn't read Lease")
|
|
if !apiequality.Semantic.DeepEqual(lease.Spec, readLease.Spec) {
|
|
framework.Failf("Leases don't match. Diff (- for expected, + for actual):\n%s", cmp.Diff(lease.Spec, readLease.Spec))
|
|
}
|
|
|
|
createdLease.Spec = coordinationv1.LeaseSpec{
|
|
HolderIdentity: pointer.String("holder2"),
|
|
LeaseDurationSeconds: pointer.Int32(30),
|
|
AcquireTime: &metav1.MicroTime{Time: time.Time{}.Add(20 * time.Second)},
|
|
RenewTime: &metav1.MicroTime{Time: time.Time{}.Add(50 * time.Second)},
|
|
LeaseTransitions: pointer.Int32(1),
|
|
}
|
|
|
|
_, err = leaseClient.Update(ctx, createdLease, metav1.UpdateOptions{})
|
|
framework.ExpectNoError(err, "updating Lease failed")
|
|
|
|
readLease, err = leaseClient.Get(ctx, name, metav1.GetOptions{})
|
|
framework.ExpectNoError(err, "couldn't read Lease")
|
|
if !apiequality.Semantic.DeepEqual(createdLease.Spec, readLease.Spec) {
|
|
framework.Failf("Leases don't match. Diff (- for expected, + for actual):\n%s", cmp.Diff(createdLease.Spec, readLease.Spec))
|
|
}
|
|
|
|
patchedLease := readLease.DeepCopy()
|
|
patchedLease.Spec = coordinationv1.LeaseSpec{
|
|
HolderIdentity: pointer.String("holder3"),
|
|
LeaseDurationSeconds: pointer.Int32(60),
|
|
AcquireTime: &metav1.MicroTime{Time: time.Time{}.Add(50 * time.Second)},
|
|
RenewTime: &metav1.MicroTime{Time: time.Time{}.Add(70 * time.Second)},
|
|
LeaseTransitions: pointer.Int32(2),
|
|
}
|
|
patchBytes, err := getPatchBytes(readLease, patchedLease)
|
|
framework.ExpectNoError(err, "creating patch failed")
|
|
|
|
_, err = leaseClient.Patch(ctx, name, types.StrategicMergePatchType, patchBytes, metav1.PatchOptions{})
|
|
framework.ExpectNoError(err, "patching Lease failed")
|
|
|
|
readLease, err = leaseClient.Get(ctx, name, metav1.GetOptions{})
|
|
framework.ExpectNoError(err, "couldn't read Lease")
|
|
if !apiequality.Semantic.DeepEqual(patchedLease.Spec, readLease.Spec) {
|
|
framework.Failf("Leases don't match. Diff (- for expected, + for actual):\n%s", cmp.Diff(patchedLease.Spec, readLease.Spec))
|
|
}
|
|
|
|
name2 := "lease2"
|
|
lease2 := &coordinationv1.Lease{
|
|
ObjectMeta: metav1.ObjectMeta{
|
|
Name: name2,
|
|
Labels: map[string]string{"deletecollection": "true"},
|
|
},
|
|
Spec: coordinationv1.LeaseSpec{
|
|
HolderIdentity: pointer.String("holder"),
|
|
LeaseDurationSeconds: pointer.Int32(30),
|
|
AcquireTime: &metav1.MicroTime{Time: time.Time{}.Add(2 * time.Second)},
|
|
RenewTime: &metav1.MicroTime{Time: time.Time{}.Add(5 * time.Second)},
|
|
LeaseTransitions: pointer.Int32(0),
|
|
},
|
|
}
|
|
_, err = leaseClient.Create(ctx, lease2, metav1.CreateOptions{})
|
|
framework.ExpectNoError(err, "creating Lease failed")
|
|
|
|
leases, err := leaseClient.List(ctx, metav1.ListOptions{})
|
|
framework.ExpectNoError(err, "couldn't list Leases")
|
|
framework.ExpectEqual(len(leases.Items), 2)
|
|
|
|
selector := labels.Set(map[string]string{"deletecollection": "true"}).AsSelector()
|
|
err = leaseClient.DeleteCollection(ctx, metav1.DeleteOptions{}, metav1.ListOptions{LabelSelector: selector.String()})
|
|
framework.ExpectNoError(err, "couldn't delete collection")
|
|
|
|
leases, err = leaseClient.List(ctx, metav1.ListOptions{})
|
|
framework.ExpectNoError(err, "couldn't list Leases")
|
|
framework.ExpectEqual(len(leases.Items), 1)
|
|
|
|
err = leaseClient.Delete(ctx, name, metav1.DeleteOptions{})
|
|
framework.ExpectNoError(err, "deleting Lease failed")
|
|
|
|
_, err = leaseClient.Get(ctx, name, metav1.GetOptions{})
|
|
if !apierrors.IsNotFound(err) {
|
|
framework.Failf("expected IsNotFound error, got %#v", err)
|
|
}
|
|
|
|
leaseClient = f.ClientSet.CoordinationV1().Leases(metav1.NamespaceAll)
|
|
// Number of leases may be high in large clusters, as Lease object is
|
|
// created for every node by the corresponding Kubelet.
|
|
// That said, the objects themselves are small (~300B), so even with 5000
|
|
// of them, that gives ~1.5MB, which is acceptable.
|
|
_, err = leaseClient.List(ctx, metav1.ListOptions{})
|
|
framework.ExpectNoError(err, "couldn't list Leases from all namespace")
|
|
})
|
|
})
|