Skip to content

Commit 98328c6

Browse files
Merge pull request #79 from perdasilva/perdasilva/bump/operators-api-olm-v1
OPRUN-3590: Update to use api/operator/v1 OLM resource
2 parents 1df7873 + 34c3186 commit 98328c6

File tree

618 files changed

+18783
-12063
lines changed

Some content is hidden

Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.

618 files changed

+18783
-12063
lines changed

Dockerfile

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -7,7 +7,7 @@ FROM registry.ci.openshift.org/ocp/4.18:base-rhel9
77

88
COPY --from=builder /build/bin/cluster-olm-operator /
99
COPY manifests /manifests
10-
COPY vendor/github.com/openshift/api/operator/v1alpha1/zz_generated.crd-manifests/*_olms*.crd.yaml /manifests/
10+
COPY vendor/github.com/openshift/api/operator/v1/zz_generated.crd-manifests/*_olms*.crd.yaml /manifests/
1111
USER 1001
1212

1313
LABEL io.openshift.release.operator=true \

cmd/cluster-olm-operator/main.go

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -10,7 +10,7 @@ import (
1010
configv1 "github.com/openshift/api/config/v1"
1111
operatorv1alpha1 "github.com/openshift/api/operator/v1alpha1"
1212

13-
_ "github.com/openshift/api/operator/v1alpha1/zz_generated.crd-manifests"
13+
_ "github.com/openshift/api/operator/v1/zz_generated.crd-manifests"
1414
"github.com/openshift/library-go/pkg/controller/controllercmd"
1515
"github.com/openshift/library-go/pkg/controller/factory"
1616
"github.com/openshift/library-go/pkg/operator/loglevel"

go.mod

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -9,9 +9,9 @@ replace github.com/openshift/library-go => github.com/gallettilance/library-go v
99
require (
1010
github.com/blang/semver/v4 v4.0.0
1111
github.com/go-logr/logr v1.4.2
12-
github.com/openshift/api v0.0.0-20240909041644-5852b58f4b10
12+
github.com/openshift/api v0.0.0-20241107155230-d37bb9f7e380
1313
github.com/openshift/build-machinery-go v0.0.0-20240613134303-8359781da660
14-
github.com/openshift/client-go v0.0.0-20240906181530-b2f7c4ab0984
14+
github.com/openshift/client-go v0.0.0-20241107164952-923091dd2b1a
1515
github.com/openshift/library-go v0.0.0-20240905123346-5bdbfe35a6f5
1616
github.com/operator-framework/catalogd v0.23.0
1717
github.com/operator-framework/helm-operator-plugins v0.5.0
@@ -29,7 +29,7 @@ require (
2929
k8s.io/client-go v0.31.1
3030
k8s.io/component-base v0.31.1
3131
k8s.io/klog/v2 v2.130.1
32-
k8s.io/utils v0.0.0-20240711033017-18e509b52bc8
32+
k8s.io/utils v0.0.0-20240921022957-49e7df575cb6
3333
sigs.k8s.io/controller-runtime v0.19.0
3434
)
3535

go.sum

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -183,12 +183,12 @@ github.com/onsi/ginkgo/v2 v2.20.2 h1:7NVCeyIWROIAheY21RLS+3j2bb52W0W82tkberYytp4
183183
github.com/onsi/ginkgo/v2 v2.20.2/go.mod h1:K9gyxPIlb+aIvnZ8bd9Ak+YP18w3APlR+5coaZoE2ag=
184184
github.com/onsi/gomega v1.34.2 h1:pNCwDkzrsv7MS9kpaQvVb1aVLahQXyJ/Tv5oAZMI3i8=
185185
github.com/onsi/gomega v1.34.2/go.mod h1:v1xfxRgk0KIsG+QOdm7p8UosrOzPYRo60fd3B/1Dukc=
186-
github.com/openshift/api v0.0.0-20240909041644-5852b58f4b10 h1:/K3I8q4K+LDj0DfdTtnNtj1EhQpiDgZWcth9iMXdgrc=
187-
github.com/openshift/api v0.0.0-20240909041644-5852b58f4b10/go.mod h1:OOh6Qopf21pSzqNVCB5gomomBXb8o5sGKZxG2KNpaXM=
186+
github.com/openshift/api v0.0.0-20241107155230-d37bb9f7e380 h1:EstDpct2ypQv500NPRQk92YBk9ZO0baDd94mna8o6w0=
187+
github.com/openshift/api v0.0.0-20241107155230-d37bb9f7e380/go.mod h1:Shkl4HanLwDiiBzakv+con/aMGnVE2MAGvoKp5oyYUo=
188188
github.com/openshift/build-machinery-go v0.0.0-20240613134303-8359781da660 h1:F0zE2bmdVvaEd18VXuGYQdJJ1FYJu4MIDW9PYZWc9No=
189189
github.com/openshift/build-machinery-go v0.0.0-20240613134303-8359781da660/go.mod h1:8jcm8UPtg2mCAsxfqKil1xrmRMI3a+XU2TZ9fF8A7TE=
190-
github.com/openshift/client-go v0.0.0-20240906181530-b2f7c4ab0984 h1:4OVV/fm6ea+51rZbA/52SFbHdjlzjCKK6OCE7Xtn834=
191-
github.com/openshift/client-go v0.0.0-20240906181530-b2f7c4ab0984/go.mod h1:K+5rEJpGf5LpcwdNtkGsvV3u8wU7m3oHzcVZzuGTRZ4=
190+
github.com/openshift/client-go v0.0.0-20241107164952-923091dd2b1a h1:h3F55x+zOXwSG4okUlXmOlU/5CQ7Ve2HKxYjseJPVoY=
191+
github.com/openshift/client-go v0.0.0-20241107164952-923091dd2b1a/go.mod h1:JBIcn1JfD/JCpGYnRRT0+HLxrleF/Y7T3Y/t0p8o5jk=
192192
github.com/operator-framework/api v0.27.0 h1:OrVaGKZJvbZo58HTv2guz7aURkhVKYhFqZ/6VpifiXI=
193193
github.com/operator-framework/api v0.27.0/go.mod h1:lg2Xx+S8NQWGYlEOvFwQvH46E5EK5IrAIL7HWfAhciM=
194194
github.com/operator-framework/catalogd v0.23.0 h1:LAuiYuFftCQp6Jvol9pWbyvOY04QP7xH0dcf9ZO9RcE=
@@ -398,8 +398,8 @@ k8s.io/kube-aggregator v0.31.0 h1:3DqSpmqHF8rey7fY+qYXLJms0tYPhxrgWvjpnKVnS0Y=
398398
k8s.io/kube-aggregator v0.31.0/go.mod h1:Fa+OVSpMQC7zbTTz7/QG7FXe9jZ8usuJQej5sMdCrkM=
399399
k8s.io/kube-openapi v0.0.0-20240228011516-70dd3763d340 h1:BZqlfIlq5YbRMFko6/PM7FjZpUb45WallggurYhKGag=
400400
k8s.io/kube-openapi v0.0.0-20240228011516-70dd3763d340/go.mod h1:yD4MZYeKMBwQKVht279WycxKyM84kkAx2DPrTXaeb98=
401-
k8s.io/utils v0.0.0-20240711033017-18e509b52bc8 h1:pUdcCO1Lk/tbT5ztQWOBi5HBgbBP1J8+AsQnQCKsi8A=
402-
k8s.io/utils v0.0.0-20240711033017-18e509b52bc8/go.mod h1:OLgZIPagt7ERELqWJFomSt595RzquPNLL48iOWgYOg0=
401+
k8s.io/utils v0.0.0-20240921022957-49e7df575cb6 h1:MDF6h2H/h4tbzmtIKTuctcwZmY0tY9mD9fNT47QO6HI=
402+
k8s.io/utils v0.0.0-20240921022957-49e7df575cb6/go.mod h1:OLgZIPagt7ERELqWJFomSt595RzquPNLL48iOWgYOg0=
403403
sigs.k8s.io/apiserver-network-proxy/konnectivity-client v0.30.3 h1:2770sDpzrjjsAtVhSeUFseziht227YAWYHLGNM8QPwY=
404404
sigs.k8s.io/apiserver-network-proxy/konnectivity-client v0.30.3/go.mod h1:Ve9uj1L+deCXFrPOk1LpFXqTg7LCFzFso6PA48q/XZw=
405405
sigs.k8s.io/controller-runtime v0.19.0 h1:nWVM7aq+Il2ABxwiCizrVDSlmDcshi9llbaFbC0ji/Q=

manifests/0000_51_olm_00-olm-operator.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,4 @@
1-
apiVersion: operator.openshift.io/v1alpha1
1+
apiVersion: operator.openshift.io/v1
22
kind: OLM
33
metadata:
44
name: cluster

pkg/clients/clients.go

Lines changed: 14 additions & 15 deletions
Original file line numberDiff line numberDiff line change
@@ -8,10 +8,9 @@ import (
88
"time"
99

1010
operatorv1 "github.com/openshift/api/operator/v1"
11-
operatorv1alpha1 "github.com/openshift/api/operator/v1alpha1"
1211
configclient "github.com/openshift/client-go/config/clientset/versioned"
1312
configinformer "github.com/openshift/client-go/config/informers/externalversions"
14-
operatorv1alpha1apply "github.com/openshift/client-go/operator/applyconfigurations/operator/v1alpha1"
13+
operatorv1apply "github.com/openshift/client-go/operator/applyconfigurations/operator/v1"
1514
operatorclient "github.com/openshift/client-go/operator/clientset/versioned"
1615
operatorinformers "github.com/openshift/client-go/operator/informers/externalversions"
1716
"github.com/openshift/library-go/pkg/controller/controllercmd"
@@ -191,19 +190,19 @@ type OperatorClient struct {
191190
}
192191

193192
func (o OperatorClient) Informer() cache.SharedIndexInformer {
194-
return o.informers.Operator().V1alpha1().OLMs().Informer()
193+
return o.informers.Operator().V1().OLMs().Informer()
195194
}
196195

197196
func (o OperatorClient) GetObjectMeta() (*metav1.ObjectMeta, error) {
198-
olm, err := o.clientset.OperatorV1alpha1().OLMs().Get(context.TODO(), globalConfigName, metav1.GetOptions{})
197+
olm, err := o.clientset.OperatorV1().OLMs().Get(context.TODO(), globalConfigName, metav1.GetOptions{})
199198
if err != nil {
200199
return nil, err
201200
}
202201
return &olm.ObjectMeta, nil
203202
}
204203

205204
func (o OperatorClient) GetOperatorState() (*operatorv1.OperatorSpec, *operatorv1.OperatorStatus, string, error) {
206-
orig, err := o.informers.Operator().V1alpha1().OLMs().Lister().Get(globalConfigName)
205+
orig, err := o.informers.Operator().V1().OLMs().Lister().Get(globalConfigName)
207206
if err != nil {
208207
return nil, nil, "", err
209208
}
@@ -213,7 +212,7 @@ func (o OperatorClient) GetOperatorState() (*operatorv1.OperatorSpec, *operatorv
213212
}
214213

215214
func (o OperatorClient) GetOperatorStateWithQuorum(ctx context.Context) (*operatorv1.OperatorSpec, *operatorv1.OperatorStatus, string, error) {
216-
orig, err := o.clientset.OperatorV1alpha1().OLMs().Get(ctx, globalConfigName, metav1.GetOptions{})
215+
orig, err := o.clientset.OperatorV1().OLMs().Get(ctx, globalConfigName, metav1.GetOptions{})
217216
if err != nil {
218217
return nil, nil, "", err
219218
}
@@ -228,7 +227,7 @@ func (o OperatorClient) UpdateOperatorSpec(ctx context.Context, oldResourceVersi
228227
return nil, "", fmt.Errorf("error generating patch: %w", err)
229228
}
230229

231-
out, err := o.clientset.OperatorV1alpha1().OLMs().Patch(ctx, globalConfigName, types.ApplyPatchType, patch, metav1.PatchOptions{FieldManager: fieldManager, Force: ptr.To(true)})
230+
out, err := o.clientset.OperatorV1().OLMs().Patch(ctx, globalConfigName, types.ApplyPatchType, patch, metav1.PatchOptions{FieldManager: fieldManager, Force: ptr.To(true)})
232231
if err != nil {
233232
return nil, "", err
234233
}
@@ -241,54 +240,54 @@ func (o OperatorClient) UpdateOperatorStatus(ctx context.Context, oldResourceVer
241240
return nil, fmt.Errorf("error generating patch: %w", err)
242241
}
243242

244-
out, err := o.clientset.OperatorV1alpha1().OLMs().Patch(ctx, globalConfigName, types.ApplyPatchType, patch, metav1.PatchOptions{FieldManager: fieldManager, Force: ptr.To(true)}, "status")
243+
out, err := o.clientset.OperatorV1().OLMs().Patch(ctx, globalConfigName, types.ApplyPatchType, patch, metav1.PatchOptions{FieldManager: fieldManager, Force: ptr.To(true)}, "status")
245244
if err != nil {
246245
return nil, err
247246
}
248247
return &out.Status.OperatorStatus, nil
249248
}
250249

251250
func (o OperatorClient) EnsureFinalizer(ctx context.Context, finalizer string) error {
252-
instance, err := o.informers.Operator().V1alpha1().OLMs().Lister().Get(globalConfigName)
251+
instance, err := o.informers.Operator().V1().OLMs().Lister().Get(globalConfigName)
253252
if err != nil {
254253
return err
255254
}
256255
newFinalizers := sets.List(sets.New(instance.GetFinalizers()...).Insert(finalizer))
257256

258-
olm := operatorv1alpha1apply.OLM(globalConfigName).WithFinalizers(newFinalizers...)
257+
olm := operatorv1apply.OLM(globalConfigName).WithFinalizers(newFinalizers...)
259258
patch, err := json.Marshal(olm)
260259
if err != nil {
261260
return err
262261
}
263262

264-
if _, err := o.clientset.OperatorV1alpha1().OLMs().Patch(ctx, globalConfigName, types.ApplyPatchType, patch, metav1.PatchOptions{FieldManager: fieldManager, Force: ptr.To(true)}); err != nil {
263+
if _, err := o.clientset.OperatorV1().OLMs().Patch(ctx, globalConfigName, types.ApplyPatchType, patch, metav1.PatchOptions{FieldManager: fieldManager, Force: ptr.To(true)}); err != nil {
265264
return err
266265
}
267266
return nil
268267
}
269268

270269
func (o OperatorClient) RemoveFinalizer(ctx context.Context, finalizer string) error {
271-
instance, err := o.informers.Operator().V1alpha1().OLMs().Lister().Get(globalConfigName)
270+
instance, err := o.informers.Operator().V1().OLMs().Lister().Get(globalConfigName)
272271
if err != nil {
273272
return err
274273
}
275274
newFinalizers := sets.List(sets.New(instance.GetFinalizers()...).Delete(finalizer))
276275

277-
olm := operatorv1alpha1apply.OLM(globalConfigName).WithFinalizers(newFinalizers...)
276+
olm := operatorv1apply.OLM(globalConfigName).WithFinalizers(newFinalizers...)
278277
patch, err := json.Marshal(olm)
279278
if err != nil {
280279
return err
281280
}
282281

283-
if _, err := o.clientset.OperatorV1alpha1().OLMs().Patch(ctx, globalConfigName, types.ApplyPatchType, patch, metav1.PatchOptions{FieldManager: fieldManager, Force: ptr.To(true)}); err != nil {
282+
if _, err := o.clientset.OperatorV1().OLMs().Patch(ctx, globalConfigName, types.ApplyPatchType, patch, metav1.PatchOptions{FieldManager: fieldManager, Force: ptr.To(true)}); err != nil {
284283
return err
285284
}
286285
return nil
287286
}
288287

289288
func generateOLMPatch(resourceVersion string, in any, fieldPath ...string) ([]byte, error) {
290289
var u unstructured.Unstructured
291-
u.SetAPIVersion(schema.GroupVersion{Group: operatorv1alpha1.GroupName, Version: "v1alpha1"}.String())
290+
u.SetAPIVersion(schema.GroupVersion{Group: operatorv1.GroupName, Version: "v1"}.String())
292291
u.SetKind("OLM")
293292
u.SetResourceVersion(resourceVersion)
294293

vendor/github.com/openshift/api/apps/v1/generated.proto

Lines changed: 15 additions & 15 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

0 commit comments

Comments
 (0)