305652a432
This test wishes to observe a watch event. In order to do this in the past, the test chose a well-known `Service` object, fetched it, and did arithmetic on the returned `resourceVersion` in order to start a watch that was guaranteed to see an event. It is not valid to parse the `resourceVersion` as an integer or to do arithmetic on it, so in order to make the test conformant to an appropriate use of the API it now: - creates a namespace - fetches the current `resourceVersion` - creates an object - watches from the previous `resourceVersion` that was read This ensures that an event is seen by the watch, but uses the publically supported API. `ConfigMap`s are used instead of `Service`s as they do not require a valid `spec` for creation and make the test terser. Signed-off-by: Steve Kuznetsov <skuznets@redhat.com>
97 lines
3.3 KiB
Go
97 lines
3.3 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 apimachinery
|
|
|
|
import (
|
|
"context"
|
|
"fmt"
|
|
|
|
g "github.com/onsi/ginkgo"
|
|
o "github.com/onsi/gomega"
|
|
|
|
v1 "k8s.io/api/core/v1"
|
|
apierrors "k8s.io/apimachinery/pkg/api/errors"
|
|
metav1 "k8s.io/apimachinery/pkg/apis/meta/v1"
|
|
"k8s.io/apimachinery/pkg/fields"
|
|
"k8s.io/apimachinery/pkg/watch"
|
|
"k8s.io/client-go/kubernetes"
|
|
|
|
"k8s.io/kubernetes/test/e2e/framework"
|
|
)
|
|
|
|
var _ = SIGDescribe("client-go should negotiate", func() {
|
|
f := framework.NewDefaultFramework("protocol")
|
|
|
|
for _, s := range []string{
|
|
"application/json",
|
|
"application/vnd.kubernetes.protobuf",
|
|
"application/vnd.kubernetes.protobuf,application/json",
|
|
"application/json,application/vnd.kubernetes.protobuf",
|
|
} {
|
|
accept := s
|
|
g.It(fmt.Sprintf("watch and report errors with accept %q", accept), func() {
|
|
g.By("creating an object for which we will watch")
|
|
ns := f.Namespace.Name
|
|
client := f.ClientSet.CoreV1().ConfigMaps(ns)
|
|
configMapName := "e2e-client-go-test-negotiation"
|
|
testConfigMap := &v1.ConfigMap{ObjectMeta: metav1.ObjectMeta{Name: configMapName}}
|
|
before, err := client.List(context.TODO(), metav1.ListOptions{})
|
|
framework.ExpectNoError(err)
|
|
_, err = client.Create(context.TODO(), testConfigMap, metav1.CreateOptions{})
|
|
framework.ExpectNoError(err)
|
|
opts := metav1.ListOptions{
|
|
ResourceVersion: before.ResourceVersion,
|
|
FieldSelector: fields.SelectorFromSet(fields.Set{"metadata.name": configMapName}).String(),
|
|
}
|
|
|
|
g.By("watching for changes on the object")
|
|
cfg, err := framework.LoadConfig()
|
|
framework.ExpectNoError(err)
|
|
|
|
cfg.AcceptContentTypes = accept
|
|
|
|
c := kubernetes.NewForConfigOrDie(cfg)
|
|
w, err := c.CoreV1().ConfigMaps(ns).Watch(context.TODO(), opts)
|
|
framework.ExpectNoError(err)
|
|
defer w.Stop()
|
|
|
|
evt, ok := <-w.ResultChan()
|
|
o.Expect(ok).To(o.BeTrue())
|
|
switch evt.Type {
|
|
case watch.Added, watch.Modified:
|
|
// this is allowed
|
|
case watch.Error:
|
|
err := apierrors.FromObject(evt.Object)
|
|
// In Kubernetes 1.17 and earlier, the api server returns both apierrors.StatusReasonExpired and
|
|
// apierrors.StatusReasonGone for HTTP 410 (Gone) status code responses. In 1.18 the kube server is more consistent
|
|
// and always returns apierrors.StatusReasonExpired. For backward compatibility we can only remove the apierrs.IsGone
|
|
// check when we fully drop support for Kubernetes 1.17 servers from reflectors.
|
|
if apierrors.IsGone(err) || apierrors.IsResourceExpired(err) {
|
|
// this is allowed, since the kubernetes object could be very old
|
|
break
|
|
}
|
|
if apierrors.IsUnexpectedObjectError(err) {
|
|
g.Fail(fmt.Sprintf("unexpected object, wanted v1.Status: %#v", evt.Object))
|
|
}
|
|
g.Fail(fmt.Sprintf("unexpected error: %#v", evt.Object))
|
|
default:
|
|
g.Fail(fmt.Sprintf("unexpected type %s: %#v", evt.Type, evt.Object))
|
|
}
|
|
})
|
|
}
|
|
})
|