Skip to content

Commit 481a9d3

Browse files
committed
[sig-imageregistry]
1 parent 182505c commit 481a9d3

File tree

2 files changed

+10
-9
lines changed

2 files changed

+10
-9
lines changed

test/extended/images/resolve.go

+9-8
Original file line numberDiff line numberDiff line change
@@ -8,7 +8,6 @@ import (
88
g "github.com/onsi/ginkgo/v2"
99
o "github.com/onsi/gomega"
1010

11-
appsv1 "github.com/openshift/api/apps/v1"
1211
configv1 "github.com/openshift/api/config/v1"
1312
kappsv1 "k8s.io/api/apps/v1"
1413
kbatchv1 "k8s.io/api/batch/v1"
@@ -177,7 +176,7 @@ var _ = g.Describe("[sig-imageregistry][Feature:ImageLookup] Image policy", func
177176
o.Expect(sts.Spec.Template.Spec.Containers[0].Image).To(o.Equal(internalImageReference))
178177
})
179178

180-
g.It("should update OpenShift object image fields when local names are on [apigroup:image.openshift.io][apigroup:apps.openshift.io]", func() {
179+
g.It("should update OpenShift object image fields when local names are on [apigroup:image.openshift.io]", func() {
181180
err := oc.Run("tag").Args(k8simage.GetE2EImage(k8simage.BusyBox), "busybox:latest").Execute()
182181
o.Expect(err).NotTo(o.HaveOccurred())
183182
err = oc.Run("set", "image-lookup").Args("busybox").Execute()
@@ -201,11 +200,13 @@ var _ = g.Describe("[sig-imageregistry][Feature:ImageLookup] Image policy", func
201200
o.Expect(err).NotTo(o.HaveOccurred(), fmt.Sprintf("unable to wait for image to be imported: %v", lastErr))
202201

203202
// deployment configs should auto replace local references
204-
dc, err := oc.AppsClient().AppsV1().DeploymentConfigs(oc.Namespace()).Create(ctx, &appsv1.DeploymentConfig{
203+
dc, err := oc.AdminKubeClient().AppsV1().Deployments(oc.Namespace()).Create(ctx, &kappsv1.Deployment{
205204
ObjectMeta: metav1.ObjectMeta{Name: "resolve"},
206-
Spec: appsv1.DeploymentConfigSpec{
207-
Selector: map[string]string{"resolve": "true"},
208-
Template: &kapiv1.PodTemplateSpec{
205+
Spec: kappsv1.DeploymentSpec{
206+
Selector: &metav1.LabelSelector{
207+
MatchLabels: map[string]string{"resolve": "true"},
208+
},
209+
Template: kapiv1.PodTemplateSpec{
209210
ObjectMeta: metav1.ObjectMeta{
210211
Labels: map[string]string{"resolve": "true"},
211212
},
@@ -221,10 +222,10 @@ var _ = g.Describe("[sig-imageregistry][Feature:ImageLookup] Image policy", func
221222
o.Expect(err).NotTo(o.HaveOccurred())
222223
o.Expect(dc.Spec.Template.Spec.Containers[0].Image).To(o.Equal(internalImageReference))
223224
defer func() {
224-
oc.AppsClient().AppsV1().DeploymentConfigs(oc.Namespace()).Delete(ctx, dc.Name, metav1.DeleteOptions{})
225+
oc.AdminKubeClient().AppsV1().Deployments(oc.Namespace()).Delete(ctx, dc.Name, metav1.DeleteOptions{})
225226
}()
226227

227-
dc, err = oc.AppsClient().AppsV1().DeploymentConfigs(oc.Namespace()).Patch(context.Background(), dc.Name, types.StrategicMergePatchType, []byte(`{"spec": {"template": {"spec": {"containers": [{"name": "resolve", "image": "busybox:latest"}]}}}}`), metav1.PatchOptions{})
228+
dc, err = oc.AdminKubeClient().AppsV1().Deployments(oc.Namespace()).Patch(context.Background(), dc.Name, types.StrategicMergePatchType, []byte(`{"spec": {"template": {"spec": {"containers": [{"name": "resolve", "image": "busybox:latest"}]}}}}`), metav1.PatchOptions{})
228229
o.Expect(err).NotTo(o.HaveOccurred())
229230
o.Expect(dc.Spec.Template.Spec.Containers[0].Image).To(o.Equal(internalImageReference))
230231
})

test/extended/util/annotate/generated/zz_generated.annotations.go

+1-1
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

0 commit comments

Comments
 (0)