Skip to content

Commit b8e53a5

Browse files
author
Per Goncalves da Silva
committed
move scheme creation to main
Signed-off-by: Per Goncalves da Silva <[email protected]>
1 parent cc4930e commit b8e53a5

File tree

1 file changed

+12
-1
lines changed

1 file changed

+12
-1
lines changed

cmd/catalog/main.go

Lines changed: 12 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -7,6 +7,9 @@ import (
77
"os"
88
"time"
99

10+
apiextensionsv1 "k8s.io/apiextensions-apiserver/pkg/apis/apiextensions/v1"
11+
apiregistrationv1 "k8s.io/kube-aggregator/pkg/apis/apiregistration/v1"
12+
1013
configv1client "github.com/openshift/client-go/config/clientset/versioned/typed/config/v1"
1114
"github.com/sirupsen/logrus"
1215
k8sscheme "k8s.io/client-go/kubernetes/scheme"
@@ -88,6 +91,14 @@ func (o *options) run(ctx context.Context, logger *logrus.Logger) error {
8891
if o.setWorkloadUserID {
8992
workloadUserID = defaultWorkLoadUserID
9093
}
94+
95+
// the scheme is used by the catalog operator to create
96+
// a validatingroundtripper that ensures that all created
97+
// resources are appropriately labeled
98+
scheme := k8sscheme.Scheme
99+
_ = apiextensionsv1.AddToScheme(scheme) // required by opClient
100+
_ = apiregistrationv1.AddToScheme(scheme) // required by opClient
101+
91102
// TODO(tflannag): Use options pattern for catalog operator
92103
// Create a new instance of the operator.
93104
op, err := catalog.NewOperator(
@@ -100,7 +111,7 @@ func (o *options) run(ctx context.Context, logger *logrus.Logger) error {
100111
o.opmImage,
101112
o.utilImage,
102113
o.catalogNamespace,
103-
k8sscheme.Scheme,
114+
scheme,
104115
o.installPlanTimeout,
105116
o.bundleUnpackTimeout,
106117
workloadUserID,

0 commit comments

Comments
 (0)