mirror of
https://github.com/ceph/ceph-csi.git
synced 2024-11-10 00:10:20 +00:00
caf4090657
As we added support to set the metadata on the rbd images created for the PVC and volume snapshot, by default metadata is set on all the images. As we have seen we are hitting issues#2327 a lot of times with this, we start to leave a lot of stale images. Currently, we rely on `--extra-create-metadata=true` to decide to set the metadata or not, we cannot set this option to false to disable setting metadata because we use this for encryption too. This changes is to provide an option to disable setting the image metadata when starting cephcsi. Fixes: #3009 Signed-off-by: Prasanna Kumar Kalever <prasanna.kalever@redhat.com>
89 lines
2.4 KiB
Go
89 lines
2.4 KiB
Go
/*
|
|
Copyright 2020 The Ceph-CSI Authors.
|
|
|
|
Licensed under the Apache License, Version 2.0 (the "License");
|
|
you may not use this file except in compliance with the License.
|
|
You may obtain a copy of the License at
|
|
|
|
http://www.apache.org/licenses/LICENSE-2.0
|
|
|
|
Unless required by applicable law or agreed to in writing, software
|
|
distributed under the License is distributed on an "AS IS" BASIS,
|
|
WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
|
|
See the License for the specific language governing permissions and
|
|
limitations under the License.
|
|
*/
|
|
package controller
|
|
|
|
import (
|
|
"fmt"
|
|
|
|
"github.com/ceph/ceph-csi/internal/util/log"
|
|
|
|
"k8s.io/client-go/tools/leaderelection/resourcelock"
|
|
clientConfig "sigs.k8s.io/controller-runtime/pkg/client/config"
|
|
"sigs.k8s.io/controller-runtime/pkg/manager"
|
|
"sigs.k8s.io/controller-runtime/pkg/manager/signals"
|
|
)
|
|
|
|
// Manager is the interface that will wrap Add function.
|
|
// The New controllers which gets added, as to implement Add function to get
|
|
// started by the manager.
|
|
type Manager interface {
|
|
Add(manager.Manager, Config) error
|
|
}
|
|
|
|
// Config holds the drivername and namespace name.
|
|
type Config struct {
|
|
DriverName string
|
|
Namespace string
|
|
ClusterName string
|
|
SetMetadata bool
|
|
}
|
|
|
|
// ControllerList holds the list of managers need to be started.
|
|
var ControllerList []Manager
|
|
|
|
// addToManager calls the registered managers Add method.
|
|
func addToManager(mgr manager.Manager, config Config) error {
|
|
for _, c := range ControllerList {
|
|
err := c.Add(mgr, config)
|
|
if err != nil {
|
|
return fmt.Errorf("failed to add: %w", err)
|
|
}
|
|
}
|
|
|
|
return nil
|
|
}
|
|
|
|
// Start will start all the registered managers.
|
|
func Start(config Config) error {
|
|
electionID := config.DriverName + "-" + config.Namespace
|
|
opts := manager.Options{
|
|
LeaderElection: true,
|
|
// disable metrics
|
|
MetricsBindAddress: "0",
|
|
LeaderElectionNamespace: config.Namespace,
|
|
LeaderElectionResourceLock: resourcelock.LeasesResourceLock,
|
|
LeaderElectionID: electionID,
|
|
}
|
|
mgr, err := manager.New(clientConfig.GetConfigOrDie(), opts)
|
|
if err != nil {
|
|
log.ErrorLogMsg("failed to create manager %s", err)
|
|
|
|
return err
|
|
}
|
|
err = addToManager(mgr, config)
|
|
if err != nil {
|
|
log.ErrorLogMsg("failed to add manager %s", err)
|
|
|
|
return err
|
|
}
|
|
err = mgr.Start(signals.SetupSignalHandler())
|
|
if err != nil {
|
|
log.ErrorLogMsg("failed to start manager %s", err)
|
|
}
|
|
|
|
return err
|
|
}
|