Skip to content

Commit e6b4d3a

Browse files
Moving code from catalogd/internal to internal/catalogd
This partially fixes #1707. The intent is to consolidate internal code from operator-controller and catalogd within internal e.g.: catalogd/ - code specific to catalogd internal/ - holds internal code of catalogd and operator-controller. internal/catalogd - holds unexported code specific to catalogd internal/operator-controller - holds unexported code specific to operator-comntroller internal/shared - shared code between catalogd and operator-controller which can not be exported outside. Signed-off-by: Lalatendu Mohanty <lmohanty@redhat.com>
1 parent 6dace7c commit e6b4d3a

23 files changed

+33
-20
lines changed

catalogd/cmd/catalogd/main.go

Lines changed: 8 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -56,14 +56,14 @@ import (
5656
crwebhook "sigs.k8s.io/controller-runtime/pkg/webhook"
5757

5858
catalogdv1 "github.com/operator-framework/operator-controller/catalogd/api/v1"
59-
corecontrollers "github.com/operator-framework/operator-controller/catalogd/internal/controllers/core"
60-
"github.com/operator-framework/operator-controller/catalogd/internal/features"
61-
"github.com/operator-framework/operator-controller/catalogd/internal/garbagecollection"
62-
catalogdmetrics "github.com/operator-framework/operator-controller/catalogd/internal/metrics"
63-
"github.com/operator-framework/operator-controller/catalogd/internal/serverutil"
64-
"github.com/operator-framework/operator-controller/catalogd/internal/source"
65-
"github.com/operator-framework/operator-controller/catalogd/internal/storage"
66-
"github.com/operator-framework/operator-controller/catalogd/internal/webhook"
59+
corecontrollers "github.com/operator-framework/operator-controller/internal/catalogd/controllers/core"
60+
"github.com/operator-framework/operator-controller/internal/catalogd/features"
61+
"github.com/operator-framework/operator-controller/internal/catalogd/garbagecollection"
62+
catalogdmetrics "github.com/operator-framework/operator-controller/internal/catalogd/metrics"
63+
"github.com/operator-framework/operator-controller/internal/catalogd/serverutil"
64+
"github.com/operator-framework/operator-controller/internal/catalogd/source"
65+
"github.com/operator-framework/operator-controller/internal/catalogd/storage"
66+
"github.com/operator-framework/operator-controller/internal/catalogd/webhook"
6767
fsutil "github.com/operator-framework/operator-controller/internal/shared/util/fs"
6868
"github.com/operator-framework/operator-controller/internal/shared/version"
6969
)

config/base/rbac/role.yaml

Lines changed: 17 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -21,25 +21,38 @@ rules:
2121
resources:
2222
- clustercatalogs
2323
verbs:
24+
- create
25+
- delete
2426
- get
2527
- list
28+
- patch
29+
- update
2630
- watch
2731
- apiGroups:
2832
- olm.operatorframework.io
2933
resources:
30-
- clusterextensions
34+
- clustercatalogs/finalizers
35+
- clusterextensions/finalizers
36+
verbs:
37+
- update
38+
- apiGroups:
39+
- olm.operatorframework.io
40+
resources:
41+
- clustercatalogs/status
3142
verbs:
3243
- get
33-
- list
3444
- patch
3545
- update
36-
- watch
3746
- apiGroups:
3847
- olm.operatorframework.io
3948
resources:
40-
- clusterextensions/finalizers
49+
- clusterextensions
4150
verbs:
51+
- get
52+
- list
53+
- patch
4254
- update
55+
- watch
4356
- apiGroups:
4457
- olm.operatorframework.io
4558
resources:

catalogd/internal/controllers/core/clustercatalog_controller.go renamed to internal/catalogd/controllers/core/clustercatalog_controller.go

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -38,8 +38,8 @@ import (
3838
"sigs.k8s.io/controller-runtime/pkg/reconcile"
3939

4040
catalogdv1 "github.com/operator-framework/operator-controller/catalogd/api/v1"
41-
"github.com/operator-framework/operator-controller/catalogd/internal/source"
42-
"github.com/operator-framework/operator-controller/catalogd/internal/storage"
41+
"github.com/operator-framework/operator-controller/internal/catalogd/source"
42+
"github.com/operator-framework/operator-controller/internal/catalogd/storage"
4343
)
4444

4545
const (

catalogd/internal/controllers/core/clustercatalog_controller_test.go renamed to internal/catalogd/controllers/core/clustercatalog_controller_test.go

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -21,8 +21,8 @@ import (
2121
"sigs.k8s.io/controller-runtime/pkg/reconcile"
2222

2323
catalogdv1 "github.com/operator-framework/operator-controller/catalogd/api/v1"
24-
"github.com/operator-framework/operator-controller/catalogd/internal/source"
25-
"github.com/operator-framework/operator-controller/catalogd/internal/storage"
24+
"github.com/operator-framework/operator-controller/internal/catalogd/source"
25+
"github.com/operator-framework/operator-controller/internal/catalogd/storage"
2626
)
2727

2828
var _ source.Unpacker = &MockSource{}
File renamed without changes.
File renamed without changes.
File renamed without changes.
File renamed without changes.
File renamed without changes.
File renamed without changes.

0 commit comments

Comments
 (0)