Skip to content

Commit 6dd0da4

Browse files
dkokkinoeshulman2
authored andcommitted
Create static secret to replace sycretsync
- Previously the csi-manila-secrets secret in the openshift-manila-csi-driver namespace required secretsync.go to generate the secret dynamically. Since we are changing how the authentication occurs for the manila driver that process can be replaced with a static secret asset.
1 parent 729e602 commit 6dd0da4

7 files changed

Lines changed: 43 additions & 164 deletions

File tree

Lines changed: 9 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,9 @@
1+
apiVersion: v1
2+
kind: Secret
3+
metadata:
4+
name: csi-manila-secrets
5+
namespace: openshift-manila-csi-driver
6+
stringData:
7+
os-cloud: openstack
8+
os-cloudsFile: /etc/openstack/clouds.yaml
9+
os-useClouds: "true"
Lines changed: 15 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,15 @@
1+
# Generated file. Do not edit. Update using "make update".
2+
#
3+
# Loaded from overlays/openstack-manila/base/config_secret.yaml
4+
#
5+
#
6+
7+
apiVersion: v1
8+
kind: Secret
9+
metadata:
10+
name: csi-manila-secrets
11+
namespace: openshift-manila-csi-driver
12+
stringData:
13+
os-cloud: openstack
14+
os-cloudsFile: /etc/openstack/clouds.yaml
15+
os-useClouds: "true"

assets/overlays/openstack-manila/generated/hypershift/manifests.yaml

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -5,6 +5,7 @@ controllerStaticAssetNames:
55
- controller_sa.yaml
66
- service.yaml
77
guestStaticAssetNames:
8+
- config_secret.yaml
89
- csidriver.yaml
910
- lease_leader_election_binding.yaml
1011
- lease_leader_election_role.yaml
Lines changed: 15 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,15 @@
1+
# Generated file. Do not edit. Update using "make update".
2+
#
3+
# Loaded from overlays/openstack-manila/base/config_secret.yaml
4+
#
5+
#
6+
7+
apiVersion: v1
8+
kind: Secret
9+
metadata:
10+
name: csi-manila-secrets
11+
namespace: openshift-manila-csi-driver
12+
stringData:
13+
os-cloud: openstack
14+
os-cloudsFile: /etc/openstack/clouds.yaml
15+
os-useClouds: "true"

assets/overlays/openstack-manila/generated/standalone/manifests.yaml

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -10,6 +10,7 @@ controllerStaticAssetNames:
1010
- service.yaml
1111
- servicemonitor.yaml
1212
guestStaticAssetNames:
13+
- config_secret.yaml
1314
- csidriver.yaml
1415
- lease_leader_election_binding.yaml
1516
- lease_leader_election_role.yaml

pkg/driver/openstack-manila/openstack_manila.go

Lines changed: 2 additions & 19 deletions
Original file line numberDiff line numberDiff line change
@@ -16,7 +16,6 @@ import (
1616
"github.com/openshift/csi-operator/pkg/driver/common/operator"
1717
"github.com/openshift/csi-operator/pkg/generator"
1818
"github.com/openshift/csi-operator/pkg/openstack-manila/client"
19-
"github.com/openshift/csi-operator/pkg/openstack-manila/secret"
2019
"github.com/openshift/csi-operator/pkg/openstack-manila/util"
2120
"github.com/openshift/csi-operator/pkg/operator/config"
2221
"github.com/openshift/library-go/pkg/controller/factory"
@@ -98,6 +97,7 @@ func GetOpenStackManilaGeneratorConfig() *generator.CSIDriverGeneratorConfig {
9897
"overlays/openstack-manila/base/csidriver.yaml",
9998
"overlays/openstack-manila/base/volumesnapshotclass.yaml",
10099
"overlays/openstack-manila/base/node_nfs.yaml",
100+
"overlays/openstack-manila/base/config_secret.yaml",
101101
),
102102
AssetPatches: commongenerator.DefaultGuestAssetPatches,
103103
},
@@ -187,11 +187,7 @@ func GetOpenStackManilaOperatorControllerConfig(ctx context.Context, flavour gen
187187
}
188188

189189
cfg.PreconditionInformers = []factory.Informer{c.GetCSIDriverInformer().Informer(), c.GetStorageClassInformer().Informer()}
190-
secretSyncer, err := createSecretSyncer(c)
191-
if err != nil {
192-
return nil, err
193-
}
194-
cfg.ExtraControlPlaneControllers = append(cfg.ExtraControlPlaneControllers, configMapSyncer, secretSyncer, nfsCSIDriverController)
190+
cfg.ExtraControlPlaneControllers = append(cfg.ExtraControlPlaneControllers, configMapSyncer, nfsCSIDriverController)
195191

196192
cfg.ExtraReplacementsFunc = func() []string {
197193
pairs := []string{}
@@ -324,19 +320,6 @@ func withCABundleDaemonSetHook(c *clients.Clients) (csidrivernodeservicecontroll
324320
return hook, informers
325321
}
326322

327-
func createSecretSyncer(c *clients.Clients) (factory.Controller, error) {
328-
secretSyncController := secret.NewSecretSyncController(
329-
c.OperatorClient,
330-
c.KubeClient,
331-
c.ControlPlaneKubeInformers,
332-
c.ControlPlaneNamespace,
333-
c.GuestNamespace,
334-
resyncInterval,
335-
c.EventRecorder)
336-
337-
return secretSyncController, nil
338-
}
339-
340323
func createConfigMapSyncer(c *clients.Clients) (factory.Controller, error) {
341324
// sync config map with OpenStack CA certificate to the operand namespace,
342325
// so the driver can get it as a ConfigMap volume.

pkg/openstack-manila/secret/secretsync.go

Lines changed: 0 additions & 145 deletions
This file was deleted.

0 commit comments

Comments
 (0)