forked from rook/rook
retrieveMultisiteZone is meant to gate the object-store reconcile on the backing Ceph zone existing: it runs "radosgw-admin zone get" and, when that fails, returns a non-nil error so the caller requeues. That gate is dead code. The ENOENT check declares an inner err from exec.ExtractExitCode that shadows the outer command error, and ExtractExitCode returns a nil error for the ordinary exit failures radosgw-admin produces. Both the ENOENT branch and the else branch then wrap that shadowed nil, and errors.Wrapf(nil, ...) is nil, so the function returns (waitForRequeueIfObjectStoreNotReady, nil). The caller only propagates the requeue when the error is non-nil, so the requeue is dropped and reconcile runs on. The result is that a failed "zone get" no longer backs off. Reconcile proceeds to stand up the object store anyway -- the RGW service, the admin-ops endpoint, the deployment, and the pools radosgw scaffolds as it comes up -- for a multisite store whose backing zone does not exist. This is not the "normal multisite bootstrap" transient the original wording suggested. getMultisiteResourceNames runs immediately before this and already requeues until the CephObjectZone CR reports Ready, and the zone controller marks it Ready only after it has created the Ceph zone, so a healthy bootstrap never reaches this gate with a missing zone. That CR-Ready gate, not this one, is what actually blocks bootstrap. Where the dead gate does bite is the cases the CR status cannot cover: - the Ceph zone deleted or renamed out of band while the CR still reads Ready - a zone controller that reports Ready without leaving a usable zone behind - any non-ENOENT "zone get" failure, e.g. a permission or connectivity error, which the else branch swallows the same way The check was correct whenfc579f4520introduced it in 2020 with exec.ExitStatus, which returns (code, ok) and leaves err unshadowed.bd58790c31("ceph: proxy ceph commands when multus is configured", 2021) swapped it to exec.ExtractExitCode as an unrelated drive-by, inverting the contract and killing the gate. The sibling realm, zonegroup, and zone controllers were left untouched and still use exec.ExitStatus today. Restore exec.ExitStatus so the outer error is no longer shadowed, both branches wrap the real command error, and the caller requeues until the zone exists -- matching the sibling controllers. Signed-off-by: Anas Khan <83116240+anxkhn@users.noreply.github.com> Signed-off-by: Joshua Hoblitt <josh@hoblitt.com>
735 lines
32 KiB
Go
735 lines
32 KiB
Go
/*
|
|
Copyright 2016 The Rook Authors. All rights reserved.
|
|
|
|
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 object
|
|
|
|
import (
|
|
"context"
|
|
"fmt"
|
|
"reflect"
|
|
"strings"
|
|
"syscall"
|
|
"time"
|
|
|
|
"github.com/coreos/pkg/capnslog"
|
|
bktclient "github.com/kube-object-storage/lib-bucket-provisioner/pkg/client/clientset/versioned"
|
|
"github.com/pkg/errors"
|
|
cephv1 "github.com/rook/rook/pkg/apis/ceph.rook.io/v1"
|
|
"github.com/rook/rook/pkg/clusterd"
|
|
cephclient "github.com/rook/rook/pkg/daemon/ceph/client"
|
|
"github.com/rook/rook/pkg/operator/ceph/config"
|
|
"github.com/rook/rook/pkg/operator/ceph/config/keyring"
|
|
opcontroller "github.com/rook/rook/pkg/operator/ceph/controller"
|
|
"github.com/rook/rook/pkg/operator/ceph/reporting"
|
|
"github.com/rook/rook/pkg/operator/k8sutil"
|
|
"github.com/rook/rook/pkg/util/exec"
|
|
"github.com/rook/rook/pkg/util/log"
|
|
appsv1 "k8s.io/api/apps/v1"
|
|
corev1 "k8s.io/api/core/v1"
|
|
kerrors "k8s.io/apimachinery/pkg/api/errors"
|
|
metav1 "k8s.io/apimachinery/pkg/apis/meta/v1"
|
|
"k8s.io/apimachinery/pkg/runtime"
|
|
"k8s.io/apimachinery/pkg/types"
|
|
"k8s.io/client-go/tools/events"
|
|
"sigs.k8s.io/controller-runtime/pkg/client"
|
|
"sigs.k8s.io/controller-runtime/pkg/controller"
|
|
"sigs.k8s.io/controller-runtime/pkg/event"
|
|
"sigs.k8s.io/controller-runtime/pkg/handler"
|
|
"sigs.k8s.io/controller-runtime/pkg/manager"
|
|
"sigs.k8s.io/controller-runtime/pkg/predicate"
|
|
"sigs.k8s.io/controller-runtime/pkg/reconcile"
|
|
"sigs.k8s.io/controller-runtime/pkg/source"
|
|
)
|
|
|
|
const (
|
|
controllerName = "ceph-object-controller"
|
|
// DisableOBCEnvVar environment variable, if set to "true", will skip watching Object Bucket and Notification resources.
|
|
// This variable can be added to container spec of the `rook-ceph-operator` deployment.
|
|
DisableOBCEnvVar = "ROOK_DISABLE_OBJECT_BUCKET_CLAIM"
|
|
)
|
|
|
|
var waitForRequeueIfObjectStoreNotReady = reconcile.Result{Requeue: true, RequeueAfter: 10 * time.Second}
|
|
|
|
var logger = capnslog.NewPackageLogger("github.com/rook/rook", "object-controller")
|
|
|
|
// List of object resources to watch by the controller
|
|
var objectsToWatch = []client.Object{
|
|
&corev1.Secret{TypeMeta: metav1.TypeMeta{Kind: "Secret", APIVersion: corev1.SchemeGroupVersion.String()}},
|
|
&corev1.Service{TypeMeta: metav1.TypeMeta{Kind: "Service", APIVersion: corev1.SchemeGroupVersion.String()}},
|
|
&appsv1.Deployment{TypeMeta: metav1.TypeMeta{Kind: "Deployment", APIVersion: appsv1.SchemeGroupVersion.String()}},
|
|
}
|
|
|
|
// Sets the type meta for the controller main object
|
|
var controllerTypeMeta = metav1.TypeMeta{
|
|
Kind: reflect.TypeFor[cephv1.CephObjectStore]().Name(),
|
|
APIVersion: fmt.Sprintf("%s/%s", cephv1.CustomResourceGroup, cephv1.Version),
|
|
}
|
|
|
|
var currentAndDesiredCephVersion = opcontroller.CurrentAndDesiredCephVersion
|
|
|
|
// allow this to be overridden for unit tests
|
|
var cephObjectStoreDependents = CephObjectStoreDependents
|
|
|
|
// ReconcileCephObjectStore reconciles a cephObjectStore object
|
|
type ReconcileCephObjectStore struct {
|
|
client client.Client
|
|
bktclient bktclient.Interface
|
|
scheme *runtime.Scheme
|
|
context *clusterd.Context
|
|
clusterSpec *cephv1.ClusterSpec
|
|
clusterInfo *cephclient.ClusterInfo
|
|
recorder events.EventRecorder
|
|
opManagerContext context.Context
|
|
opConfig opcontroller.OperatorConfig
|
|
}
|
|
|
|
// Add creates a new cephObjectStore Controller and adds it to the Manager. The Manager will set fields on the Controller
|
|
// and Start it when the Manager is Started.
|
|
func Add(mgr manager.Manager, context *clusterd.Context, opManagerContext context.Context, opConfig opcontroller.OperatorConfig) error {
|
|
return add(mgr, newReconciler(mgr, context, opManagerContext, opConfig))
|
|
}
|
|
|
|
// newReconciler returns a new reconcile.Reconciler
|
|
func newReconciler(mgr manager.Manager, context *clusterd.Context, opManagerContext context.Context, opConfig opcontroller.OperatorConfig) reconcile.Reconciler {
|
|
context.Client = mgr.GetClient()
|
|
return &ReconcileCephObjectStore{
|
|
client: mgr.GetClient(),
|
|
scheme: mgr.GetScheme(),
|
|
context: context,
|
|
bktclient: bktclient.NewForConfigOrDie(context.KubeConfig),
|
|
recorder: mgr.GetEventRecorder("rook-" + controllerName),
|
|
opManagerContext: opManagerContext,
|
|
opConfig: opConfig,
|
|
}
|
|
}
|
|
|
|
func watchOwnedCoreObject[T client.Object](c controller.Controller, mgr manager.Manager, obj T) error {
|
|
return c.Watch(
|
|
source.Kind(
|
|
mgr.GetCache(),
|
|
obj,
|
|
handler.TypedEnqueueRequestForOwner[T](
|
|
mgr.GetScheme(),
|
|
mgr.GetRESTMapper(),
|
|
&cephv1.CephObjectStore{},
|
|
),
|
|
opcontroller.WatchPredicateForNonCRDObject[T](&cephv1.CephObjectStore{TypeMeta: controllerTypeMeta}, mgr.GetScheme()),
|
|
),
|
|
)
|
|
}
|
|
|
|
func add(mgr manager.Manager, r reconcile.Reconciler) error {
|
|
// Create a new controller
|
|
c, err := controller.New(controllerName, mgr, controller.Options{Reconciler: r})
|
|
if err != nil {
|
|
return err
|
|
}
|
|
logger.Info("successfully started")
|
|
|
|
// Watch for changes on the cephObjectStore CRD object
|
|
err = c.Watch(
|
|
source.Kind(
|
|
mgr.GetCache(),
|
|
&cephv1.CephObjectStore{TypeMeta: controllerTypeMeta},
|
|
&handler.TypedEnqueueRequestForObject[*cephv1.CephObjectStore]{},
|
|
opcontroller.WatchControllerPredicate[*cephv1.CephObjectStore](mgr.GetScheme()),
|
|
),
|
|
)
|
|
if err != nil {
|
|
return err
|
|
}
|
|
|
|
// Watch all other resources
|
|
for _, t := range objectsToWatch {
|
|
err = watchOwnedCoreObject(c, mgr, t)
|
|
if err != nil {
|
|
return err
|
|
}
|
|
}
|
|
|
|
// Watch secrets annotated for the object store
|
|
err = c.Watch(
|
|
source.Kind(
|
|
mgr.GetCache(),
|
|
&corev1.Secret{TypeMeta: metav1.TypeMeta{Kind: "Secret", APIVersion: corev1.SchemeGroupVersion.String()}},
|
|
handler.TypedEnqueueRequestsFromMapFunc(mapSecretToCR(mgr.GetClient())),
|
|
secretPredicate(),
|
|
),
|
|
)
|
|
if err != nil {
|
|
return err
|
|
}
|
|
|
|
return nil
|
|
}
|
|
|
|
// Watch all secrets not owned by Rook
|
|
func secretPredicate[T *corev1.Secret]() predicate.TypedPredicate[T] {
|
|
rookGV := cephv1.SchemeGroupVersion.String()
|
|
return predicate.TypedFuncs[T]{
|
|
UpdateFunc: func(e event.TypedUpdateEvent[T]) bool {
|
|
secret := (*corev1.Secret)(e.ObjectNew)
|
|
|
|
// check if secret already owned by Rook:
|
|
for _, owner := range secret.OwnerReferences {
|
|
if owner.APIVersion == rookGV {
|
|
// already owned by Rook CR
|
|
return false
|
|
}
|
|
}
|
|
return true
|
|
},
|
|
CreateFunc: func(e event.TypedCreateEvent[T]) bool {
|
|
secret := (*corev1.Secret)(e.Object)
|
|
|
|
// check if secret already owned by Rook:
|
|
for _, owner := range secret.OwnerReferences {
|
|
if owner.APIVersion == rookGV {
|
|
// already owned by Rook CR
|
|
return false
|
|
}
|
|
}
|
|
return true
|
|
},
|
|
DeleteFunc: func(e event.TypedDeleteEvent[T]) bool {
|
|
secret := (*corev1.Secret)(e.Object)
|
|
// check if secret already owned by Rook:
|
|
for _, owner := range secret.OwnerReferences {
|
|
if owner.APIVersion == rookGV {
|
|
// already owned by Rook CR
|
|
return false
|
|
}
|
|
}
|
|
return true
|
|
},
|
|
}
|
|
}
|
|
|
|
// Maps secret referenced by object store to the object store CR
|
|
func mapSecretToCR(k8sClient client.Client) handler.TypedMapFunc[*corev1.Secret, reconcile.Request] {
|
|
return func(ctx context.Context, secret *corev1.Secret) []reconcile.Request {
|
|
// lookup object store CRs by name
|
|
objStores := cephv1.CephObjectStoreList{}
|
|
err := k8sClient.List(ctx, &objStores, client.InNamespace(secret.Namespace))
|
|
if err != nil {
|
|
if kerrors.IsNotFound(err) {
|
|
logger.Debugf("cephObjectStore resource for referenced secret %q not found. Ignoring since object must be deleted.", secret.Name)
|
|
return nil
|
|
}
|
|
logger.Errorf("failed to list cephObjectStore resources for referenced secret %q", secret.Name)
|
|
return nil
|
|
}
|
|
|
|
var requests []reconcile.Request
|
|
for _, objStore := range objStores.Items {
|
|
// reconcile ObjectStore if it refers to the secret
|
|
if isObjStoreSpecContainsSecret(&objStore.Spec, secret) {
|
|
requests = append(requests, reconcile.Request{
|
|
NamespacedName: types.NamespacedName{
|
|
Name: objStore.Name,
|
|
Namespace: objStore.Namespace,
|
|
},
|
|
})
|
|
}
|
|
}
|
|
return requests
|
|
}
|
|
}
|
|
|
|
func isObjStoreSpecContainsSecret(spec *cephv1.ObjectStoreSpec, secret *corev1.Secret) bool {
|
|
// check if secret is referred in object store rgwConfigFromSecret:
|
|
for _, sec := range spec.Gateway.RgwConfigFromSecret {
|
|
if sec.Name == secret.Name {
|
|
return true
|
|
}
|
|
}
|
|
// check if secret is referred in object store keystone service user secret:
|
|
if spec.Auth.Keystone != nil && spec.Auth.Keystone.ServiceUserSecretName == secret.Name {
|
|
return true
|
|
}
|
|
return false
|
|
}
|
|
|
|
// Reconcile reads that state of the cluster for a cephObjectStore object and makes changes based on the state read
|
|
// and what is in the cephObjectStore.Spec
|
|
// The Controller will requeue the Request to be processed again if the returned error is non-nil or
|
|
// Result.Requeue is true, otherwise upon completion it will remove the work from the queue.
|
|
func (r *ReconcileCephObjectStore) Reconcile(context context.Context, request reconcile.Request) (reconcile.Result, error) {
|
|
defer opcontroller.RecoverAndLogException()
|
|
// workaround because the rook logging mechanism is not compatible with the controller-runtime logging interface
|
|
reconcileResponse, objectStore, err := r.reconcile(request)
|
|
|
|
return reporting.ReportReconcileResult(logger, r.recorder, request,
|
|
&objectStore, reconcileResponse, err)
|
|
}
|
|
|
|
func (r *ReconcileCephObjectStore) reconcile(request reconcile.Request) (reconcile.Result, cephv1.CephObjectStore, error) {
|
|
// Fetch the cephObjectStore instance
|
|
cephObjectStore := &cephv1.CephObjectStore{}
|
|
err := r.client.Get(r.opManagerContext, request.NamespacedName, cephObjectStore)
|
|
if err != nil {
|
|
if kerrors.IsNotFound(err) {
|
|
log.NamedDebug(request.NamespacedName, logger, "cephObjectStore resource not found. Ignoring since object must be deleted.")
|
|
return reconcile.Result{}, *cephObjectStore, nil
|
|
}
|
|
// Error reading the object - requeue the request.
|
|
return reconcile.Result{}, *cephObjectStore, errors.Wrap(err, "failed to get cephObjectStore")
|
|
}
|
|
// update observedGeneration local variable with current generation value,
|
|
// because generation can be changed before reconcile got completed
|
|
// CR status will be updated at end of reconcile, so to reflect the reconcile has finished
|
|
observedGeneration := cephObjectStore.ObjectMeta.Generation
|
|
|
|
// Set a finalizer so we can do cleanup before the object goes away
|
|
generationUpdated, err := opcontroller.AddFinalizerIfNotPresent(r.opManagerContext, r.client, cephObjectStore)
|
|
if err != nil {
|
|
return reconcile.Result{}, *cephObjectStore, errors.Wrap(err, "failed to add finalizer")
|
|
}
|
|
if generationUpdated {
|
|
log.NamedInfo(request.NamespacedName, logger, "reconciling the object store after adding finalizer")
|
|
return reconcile.Result{}, *cephObjectStore, nil
|
|
}
|
|
|
|
// The CR was just created, initializing status fields
|
|
if cephObjectStore.Status == nil {
|
|
// The store is not available so let's not build the status Info yet
|
|
cephxUninitialized := keyring.UninitializedCephxStatus()
|
|
err := updateStatus(r.opManagerContext, k8sutil.ObservedGenerationNotAvailable, replicaCountNotAvailable, r.client, request.NamespacedName, cephv1.ConditionProgressing, map[string]string{}, &cephxUninitialized)
|
|
if err != nil {
|
|
return reconcile.Result{}, *cephObjectStore, errors.Wrapf(err, "failed to initialize status and cephx status for cephObjectStore %q", request.NamespacedName)
|
|
}
|
|
cephObjectStore.Status = &cephv1.ObjectStoreStatus{
|
|
Cephx: cephv1.LocalCephxStatus{
|
|
Daemon: cephxUninitialized,
|
|
},
|
|
}
|
|
} else {
|
|
var nilCephxStatus *cephv1.CephxStatus = nil // leave cephx status as-is
|
|
err := updateStatus(r.opManagerContext, k8sutil.ObservedGenerationNotAvailable, replicaCountNotAvailable, r.client, request.NamespacedName, cephv1.ConditionProgressing, buildStatusInfo(cephObjectStore), nilCephxStatus)
|
|
if err != nil {
|
|
return reconcile.Result{}, *cephObjectStore, errors.Wrapf(err, "failed to initialize cephx status for cephObjectStore %q", request.NamespacedName)
|
|
}
|
|
}
|
|
|
|
// Make sure a CephCluster is present otherwise do nothing
|
|
cephCluster, isReadyToReconcile, cephClusterExists, reconcileResponse := opcontroller.IsReadyToReconcile(r.opManagerContext, r.client, request.NamespacedName, controllerName)
|
|
if !isReadyToReconcile {
|
|
// This handles the case where the Ceph Cluster is gone and we want to delete that CR
|
|
// We skip the deleteStore() function since everything is gone already
|
|
//
|
|
// Also, only remove the finalizer if the CephCluster is gone
|
|
// If not, we should wait for it to be ready
|
|
// This handles the case where the operator is not ready to accept Ceph command but the cluster exists
|
|
if !cephObjectStore.GetDeletionTimestamp().IsZero() && !cephClusterExists {
|
|
// Remove finalizer
|
|
err := opcontroller.RemoveFinalizer(r.opManagerContext, r.client, cephObjectStore)
|
|
if err != nil {
|
|
return reconcile.Result{}, *cephObjectStore, errors.Wrap(err, "failed to remove finalizer")
|
|
}
|
|
|
|
// Return and do not requeue. Successful deletion.
|
|
return reconcile.Result{}, *cephObjectStore, nil
|
|
}
|
|
|
|
return reconcileResponse, *cephObjectStore, nil
|
|
}
|
|
r.clusterSpec = &cephCluster.Spec
|
|
|
|
// Populate clusterInfo during each reconcile
|
|
r.clusterInfo, _, _, err = opcontroller.LoadClusterInfo(r.context, r.opManagerContext, request.NamespacedName.Namespace, r.clusterSpec)
|
|
if err != nil {
|
|
return reconcile.Result{}, *cephObjectStore, errors.Wrap(err, "failed to populate cluster info")
|
|
}
|
|
|
|
// DELETE: the CR was deleted
|
|
if !cephObjectStore.GetDeletionTimestamp().IsZero() {
|
|
err := updateStatus(r.opManagerContext, k8sutil.ObservedGenerationNotAvailable, replicaCountNotAvailable, r.client, request.NamespacedName, cephv1.ConditionDeleting, buildStatusInfo(cephObjectStore), nil)
|
|
if err != nil {
|
|
return reconcile.Result{}, *cephObjectStore, errors.Wrapf(err, "failed to set deleting status for cephObjectStore %q", request.NamespacedName)
|
|
}
|
|
|
|
// Detect running Ceph version
|
|
runningCephVersion, err := cephclient.LeastUptodateDaemonVersion(r.context, r.clusterInfo, config.MonType)
|
|
if err != nil {
|
|
return reconcile.Result{}, *cephObjectStore, errors.Wrapf(err, "failed to retrieve current ceph %q version", config.MonType)
|
|
}
|
|
r.clusterInfo.CephVersion = runningCephVersion
|
|
r.clusterInfo.Context = r.opManagerContext
|
|
|
|
// get the latest version of the object to check dependencies
|
|
err = r.client.Get(r.opManagerContext, request.NamespacedName, cephObjectStore)
|
|
if err != nil {
|
|
return reconcile.Result{}, *cephObjectStore, errors.Wrapf(err, "failed to get latest CephObjectStore %q", request.NamespacedName.String())
|
|
}
|
|
objCtx, err := NewMultisiteContext(r.context, r.clusterInfo, cephObjectStore)
|
|
if err != nil && !kerrors.IsNotFound(err) {
|
|
return reconcile.Result{}, *cephObjectStore, errors.Wrapf(err, "failed to get object context")
|
|
}
|
|
|
|
// Check dependents only if we have a working multisite context.
|
|
// Context may be nil when zone/zonegroup/realm CRDs are already deleted.
|
|
if objCtx != nil {
|
|
if !poolsExistForNonRawOps(objCtx) {
|
|
log.NamedInfo(request.NamespacedName, logger, "skipping bucket dependency check during deletion")
|
|
} else {
|
|
opsCtx, err := NewMultisiteAdminOpsContext(objCtx, &cephObjectStore.Spec)
|
|
if err != nil {
|
|
log.NamedWarning(request.NamespacedName, logger, "failed to get admin ops context during deletion, skipping dependency check: %v", err)
|
|
} else {
|
|
deps, err := cephObjectStoreDependents(r.context, r.clusterInfo, cephObjectStore, objCtx, opsCtx)
|
|
if err != nil {
|
|
return reconcile.Result{}, *cephObjectStore, err
|
|
}
|
|
if !deps.Empty() {
|
|
err := reporting.ReportDeletionBlockedDueToDependents(r.opManagerContext, logger, r.client, cephObjectStore, deps)
|
|
return opcontroller.WaitForRequeueIfFinalizerBlocked, *cephObjectStore, err
|
|
}
|
|
reporting.ReportDeletionNotBlockedDueToDependents(r.opManagerContext, logger, r.client, r.recorder, cephObjectStore)
|
|
}
|
|
}
|
|
}
|
|
|
|
cfg := clusterConfig{
|
|
context: r.context,
|
|
store: cephObjectStore,
|
|
clusterSpec: r.clusterSpec,
|
|
clusterInfo: r.clusterInfo,
|
|
}
|
|
cfg.deleteStore()
|
|
|
|
// Remove finalizer
|
|
err = opcontroller.RemoveFinalizer(r.opManagerContext, r.client, cephObjectStore)
|
|
if err != nil {
|
|
return reconcile.Result{}, *cephObjectStore, errors.Wrap(err, "failed to remove finalizer")
|
|
}
|
|
|
|
// Return and do not requeue. Successful deletion.
|
|
return reconcile.Result{}, *cephObjectStore, nil
|
|
}
|
|
|
|
shouldRotateCephxKeys := false
|
|
if cephObjectStore.Spec.IsExternal() {
|
|
// Check the ceph version of the running monitors
|
|
desiredCephVersion, err := cephclient.LeastUptodateDaemonVersion(r.context, r.clusterInfo, config.MonType)
|
|
if err != nil {
|
|
return reconcile.Result{}, *cephObjectStore, errors.Wrapf(err, "failed to retrieve current ceph %q version", config.MonType)
|
|
}
|
|
r.clusterInfo.CephVersion = desiredCephVersion
|
|
} else {
|
|
// Detect desired CephCluster version
|
|
runningCephVersion, desiredCephVersion, err := currentAndDesiredCephVersion(
|
|
r.opManagerContext,
|
|
r.opConfig.Image,
|
|
cephObjectStore.Namespace,
|
|
controllerName,
|
|
k8sutil.NewOwnerInfo(cephObjectStore, r.scheme),
|
|
r.context,
|
|
r.clusterSpec,
|
|
r.clusterInfo,
|
|
)
|
|
if err != nil {
|
|
if strings.Contains(err.Error(), opcontroller.UninitializedCephConfigError) {
|
|
log.NamedInfo(request.NamespacedName, logger, opcontroller.OperatorNotInitializedMessage)
|
|
return opcontroller.WaitForRequeueIfOperatorNotInitialized, *cephObjectStore, nil
|
|
}
|
|
return reconcile.Result{}, *cephObjectStore, errors.Wrap(err, "failed to detect running and desired ceph version")
|
|
}
|
|
|
|
// If the version of the Ceph monitor differs from the CephCluster CR image version we assume
|
|
// the cluster is being upgraded. So the controller will just wait for the upgrade to finish and
|
|
// then versions should match. Obviously using the cmd reporter job adds up to the deployment time
|
|
// Skip waiting for upgrades to finish in case of external cluster.
|
|
if !cephCluster.Spec.External.Enable && !reflect.DeepEqual(*runningCephVersion, *desiredCephVersion) {
|
|
// Upgrade is in progress, let's wait for the mons to be done
|
|
return opcontroller.WaitForRequeueIfCephClusterIsUpgrading,
|
|
*cephObjectStore,
|
|
opcontroller.ErrorCephUpgradingRequeue(desiredCephVersion, runningCephVersion)
|
|
}
|
|
r.clusterInfo.CephVersion = *runningCephVersion
|
|
|
|
shouldRotateCephxKeys, err = keyring.ShouldRotateCephxKeys(
|
|
cephCluster.Spec.Security.CephX.Daemon, *runningCephVersion, *desiredCephVersion, cephObjectStore.Status.Cephx.Daemon)
|
|
if err != nil {
|
|
return reconcile.Result{}, *cephObjectStore, errors.Wrap(err, "failed to determine if cephx keys should be rotated")
|
|
}
|
|
if shouldRotateCephxKeys {
|
|
log.NamedInfo(request.NamespacedName, logger, "cephx keys for CephObjectStore %q will be rotated", request.NamespacedName)
|
|
}
|
|
}
|
|
|
|
// validate the store settings
|
|
if err := r.validateStore(cephObjectStore); err != nil {
|
|
return reconcile.Result{}, *cephObjectStore, errors.Wrapf(err, "invalid object store %q arguments", cephObjectStore.Name)
|
|
}
|
|
|
|
ownerInfo := k8sutil.NewOwnerInfo(cephObjectStore, r.scheme)
|
|
cfg := clusterConfig{
|
|
context: r.context,
|
|
clusterInfo: r.clusterInfo,
|
|
store: cephObjectStore,
|
|
rookVersion: r.clusterSpec.CephVersion.Image,
|
|
clusterSpec: r.clusterSpec,
|
|
DataPathMap: config.NewStatelessDaemonDataPathMap(config.RgwType, cephObjectStore.Name, cephObjectStore.Namespace, r.clusterSpec.DataDirHostPath),
|
|
client: r.client,
|
|
ownerInfo: ownerInfo,
|
|
shouldRotateCephxKeys: shouldRotateCephxKeys,
|
|
}
|
|
|
|
// CREATE/UPDATE
|
|
reconcileResult, err := r.reconcileCreateObjectStore(cephObjectStore, request.NamespacedName, cfg)
|
|
if err != nil && kerrors.IsNotFound(err) {
|
|
// A not found error may mean ceph is still initializing, but there might be some other error
|
|
// so we log the error and requeue
|
|
log.NamedWarning(request.NamespacedName, logger, "object store %q reconcile failed, ceph may still be initializing. %v", request.NamespacedName.String(), err)
|
|
return opcontroller.WaitForRequeueIfOperatorNotInitialized, *cephObjectStore, nil
|
|
} else if err != nil {
|
|
// If reconcileCreateObjectStore returned a requeue result (e.g. waiting for zone Ready),
|
|
// propagate it instead of setting Failure status. This avoids flapping between
|
|
// Progressing and Failure while waiting for dependencies.
|
|
if reconcileResult.RequeueAfter > 0 {
|
|
log.NamedWarning(request.NamespacedName, logger, "object store %q reconcile: %v. requeueing", request.NamespacedName.String(), err)
|
|
return reconcileResult, *cephObjectStore, nil
|
|
}
|
|
result, err := r.setFailedStatus(k8sutil.ObservedGenerationNotAvailable, request.NamespacedName, "failed to create object store deployments", err)
|
|
return result, *cephObjectStore, err
|
|
}
|
|
|
|
// update ObservedGeneration in status at the end of reconcile
|
|
// Set Progressing status, we are done reconciling, the health check go routine will update the status
|
|
cephxStatus := keyring.UpdatedCephxStatus(shouldRotateCephxKeys, cephCluster.Spec.Security.CephX.Daemon, r.clusterInfo.CephVersion, cephObjectStore.Status.Cephx.Daemon)
|
|
err = updateStatus(r.opManagerContext, observedGeneration, cephObjectStore.Spec.Gateway.Instances, r.client, request.NamespacedName, cephv1.ConditionReady, buildStatusInfo(cephObjectStore), &cephxStatus)
|
|
if err != nil {
|
|
return reconcile.Result{}, *cephObjectStore, errors.Wrapf(err, "failed to set final status for cephObjectStore %q", request.NamespacedName)
|
|
}
|
|
|
|
// Return and do not requeue
|
|
log.NamedDebug(request.NamespacedName, logger, "done reconciling")
|
|
return reconcile.Result{}, *cephObjectStore, nil
|
|
}
|
|
|
|
func (r *ReconcileCephObjectStore) reconcileCreateObjectStore(cephObjectStore *cephv1.CephObjectStore, namespacedName types.NamespacedName, cfg clusterConfig) (reconcile.Result, error) {
|
|
objContext, err := NewMultisiteContext(r.context, r.clusterInfo, cephObjectStore)
|
|
if err != nil {
|
|
return r.setFailedStatus(k8sutil.ObservedGenerationNotAvailable, namespacedName, "failed to setup object store context", err)
|
|
}
|
|
|
|
if cephObjectStore.Spec.IsExternal() {
|
|
log.NamedInfo(namespacedName, logger, "reconciling external object store")
|
|
|
|
// Before v1.11, Rook created a Service and custom Endpoints that routed to external RGW
|
|
// endpoints. This causes problems if the external endpoint has TLS certificates that block
|
|
// connections to other endpoints. This also makes it impossible to create an external mode
|
|
// CephObjectStore that references another CephObjectStore's Service endpoint in the same
|
|
// Kubernetes cluster.
|
|
//
|
|
// TODO: this code block can be removed once OBCs are no longer supported. The legacy
|
|
// service can also be removed at that point.
|
|
service := cfg.generateService(cephObjectStore)
|
|
clientset := cfg.context.Clientset
|
|
clusterCtx := cfg.clusterInfo.Context
|
|
_, err = clientset.CoreV1().Services(service.Namespace).Get(clusterCtx, service.Name, metav1.GetOptions{})
|
|
if err != nil {
|
|
if kerrors.IsNotFound(err) {
|
|
// We do not need to create Services/Endpoints for new CephObjectStores.
|
|
} else {
|
|
return r.setFailedStatus(k8sutil.ObservedGenerationNotAvailable, namespacedName,
|
|
"failed to determine if legacy external service exists", err)
|
|
}
|
|
} else {
|
|
// For any legacy users that have an external mode CephObjectStore successfully using
|
|
// the Service/Endpoints and who have already created OBCs,we leave the legacy
|
|
// Service/Endpoints in place. We need to update legacy services if the user edits the
|
|
// externalRgwEndpoint -- perhaps their RGW node changed IPs.
|
|
|
|
// RECONCILE SERVICE
|
|
log.NamedInfo(namespacedName, logger, "reconciling legacy external object store service")
|
|
err = cfg.reconcileService(cephObjectStore)
|
|
if err != nil {
|
|
return r.setFailedStatus(k8sutil.ObservedGenerationNotAvailable, namespacedName, "failed to reconcile service", err)
|
|
}
|
|
|
|
// RECONCILE ENDPOINTS
|
|
// Always add the endpoint AFTER the service otherwise it will get overridden
|
|
log.NamedInfo(namespacedName, logger, "reconciling legacy external object store endpoint")
|
|
err = cfg.reconcileExternalEndpoint(cephObjectStore)
|
|
if err != nil {
|
|
return r.setFailedStatus(k8sutil.ObservedGenerationNotAvailable, namespacedName, "failed to reconcile external endpoint", err)
|
|
}
|
|
}
|
|
|
|
if err := UpdateEndpointForAdminOps(objContext, cephObjectStore); err != nil {
|
|
return r.setFailedStatus(k8sutil.ObservedGenerationNotAvailable, namespacedName, "failed to set endpoint", err)
|
|
}
|
|
|
|
// Validate that the admin ops user credentials exist
|
|
if _, _, err := GetAdminOPSUserCredentials(objContext, &cephObjectStore.Spec); err != nil {
|
|
return r.setFailedStatus(k8sutil.ObservedGenerationNotAvailable, namespacedName, "failed to get admin ops user credentials", err)
|
|
}
|
|
} else {
|
|
log.NamedInfo(namespacedName, logger, "reconciling object store deployments")
|
|
|
|
// Reconcile realm/zonegroup/zone CRs & update their names
|
|
realmName, zoneGroupName, zoneName, zone, reconcileResponse, err := r.getMultisiteResourceNames(cephObjectStore)
|
|
if err != nil {
|
|
return reconcileResponse, err
|
|
}
|
|
|
|
// Reconcile Ceph Zone if Multisite to ensure it exists, or else requeue the request
|
|
if cephObjectStore.Spec.IsMultisite() {
|
|
reconcileResponse, err := r.retrieveMultisiteZone(cephObjectStore, zoneGroupName, realmName)
|
|
if err != nil {
|
|
return reconcileResponse, err
|
|
}
|
|
}
|
|
|
|
objContext.Realm = realmName
|
|
objContext.ZoneGroup = zoneGroupName
|
|
objContext.Zone = zoneName
|
|
log.NamedDebug(namespacedName, logger, "realm is %q, zone group is %q, zone is %q", objContext.Realm, objContext.ZoneGroup, objContext.Zone)
|
|
|
|
// RECONCILE SERVICE
|
|
log.NamedDebug(namespacedName, logger, "reconciling object store service")
|
|
err = cfg.reconcileService(cephObjectStore)
|
|
if err != nil {
|
|
return r.setFailedStatus(k8sutil.ObservedGenerationNotAvailable, namespacedName, "failed to reconcile service", err)
|
|
}
|
|
|
|
if err := UpdateEndpointForAdminOps(objContext, cephObjectStore); err != nil {
|
|
return r.setFailedStatus(k8sutil.ObservedGenerationNotAvailable, namespacedName, "failed to set endpoint", err)
|
|
}
|
|
|
|
err = ValidateObjectStorePoolsConfig(cephObjectStore.Spec.MetadataPool, cephObjectStore.Spec.DataPool, cephObjectStore.Spec.SharedPools)
|
|
if err != nil {
|
|
return r.setFailedStatus(k8sutil.ObservedGenerationNotAvailable, namespacedName, "invalid pool configuration", err)
|
|
}
|
|
// Reconcile Pool Creation
|
|
if !cephObjectStore.Spec.IsMultisite() {
|
|
log.NamedInfo(namespacedName, logger, "reconciling object store pools")
|
|
|
|
if IsNeedToCreateObjectStorePools(cephObjectStore.Spec.SharedPools) {
|
|
err = CreateObjectStorePools(objContext, r.clusterSpec, cephObjectStore.Spec.MetadataPool, cephObjectStore.Spec.DataPool)
|
|
if err != nil {
|
|
return r.setFailedStatus(k8sutil.ObservedGenerationNotAvailable, namespacedName, "failed to create object pools", err)
|
|
}
|
|
}
|
|
}
|
|
|
|
// Reconcile the object store
|
|
log.NamedInfo(namespacedName, logger, "configuring object store")
|
|
err = configureObjectStore(objContext, cephObjectStore, zone)
|
|
if err != nil && kerrors.IsNotFound(err) {
|
|
return reconcile.Result{}, err
|
|
} else if err != nil {
|
|
return r.setFailedStatus(k8sutil.ObservedGenerationNotAvailable, namespacedName, "failed to configure multisite for object store", err)
|
|
}
|
|
|
|
// Retrieve the keystone secret if specified
|
|
var keystoneSecret *corev1.Secret
|
|
if ks := cephObjectStore.Spec.Auth.Keystone; ks != nil {
|
|
keystoneSecret, err = objContext.Context.Clientset.CoreV1().Secrets(objContext.clusterInfo.Namespace).Get(objContext.clusterInfo.Context, ks.ServiceUserSecretName, metav1.GetOptions{})
|
|
if err != nil {
|
|
return reconcile.Result{}, errors.Wrapf(err, "failed to get the keystone credential secret")
|
|
}
|
|
}
|
|
|
|
// Create or Update store
|
|
err = cfg.createOrUpdateStore(realmName, zoneGroupName, zoneName, keystoneSecret)
|
|
if err != nil {
|
|
return reconcile.Result{}, errors.Wrapf(err, "failed to create object store %q", cephObjectStore.Name)
|
|
}
|
|
if cephObjectStore.Spec.DefaultRealm {
|
|
log.NamedDebug(namespacedName, logger, "marking realm %q as default for object store", realmName)
|
|
if err := SetDefaultRealm(objContext, realmName); err != nil {
|
|
return reconcile.Result{}, errors.Wrapf(err, "failed to set realm %q as default", realmName)
|
|
}
|
|
}
|
|
}
|
|
|
|
return reconcile.Result{}, nil
|
|
}
|
|
|
|
func (r *ReconcileCephObjectStore) retrieveMultisiteZone(store *cephv1.CephObjectStore, zoneGroupName string, realmName string) (reconcile.Result, error) {
|
|
nsName := opcontroller.NsName(store.Namespace, store.Name)
|
|
realmArg := fmt.Sprintf("--rgw-realm=%s", realmName)
|
|
zoneGroupArg := fmt.Sprintf("--rgw-zonegroup=%s", zoneGroupName)
|
|
zoneArg := fmt.Sprintf("--rgw-zone=%s", store.Spec.Zone.Name)
|
|
objContext := NewContext(r.context, r.clusterInfo, store.Name)
|
|
|
|
_, err := RunAdminCommandNoMultisite(objContext, true, "zone", "get", realmArg, zoneGroupArg, zoneArg)
|
|
if err != nil {
|
|
// ENOENT mean "No such file or directory"
|
|
if code, ok := exec.ExitStatus(err); ok && code == int(syscall.ENOENT) {
|
|
return waitForRequeueIfObjectStoreNotReady, errors.Wrapf(err, "ceph zone %q not found", store.Spec.Zone.Name)
|
|
} else {
|
|
return waitForRequeueIfObjectStoreNotReady, errors.Wrapf(err, "radosgw-admin zone get failed with code %d", code)
|
|
}
|
|
}
|
|
|
|
log.NamedInfo(nsName, logger, "Zone %q found in Ceph cluster will include object store %q", store.Spec.Zone.Name, store.Name)
|
|
return reconcile.Result{}, nil
|
|
}
|
|
|
|
func (r *ReconcileCephObjectStore) getMultisiteResourceNames(cephObjectStore *cephv1.CephObjectStore) (string, string, string, *cephv1.CephObjectZone, reconcile.Result, error) {
|
|
nsName := opcontroller.NsName(cephObjectStore.Namespace, cephObjectStore.Name)
|
|
if !cephObjectStore.Spec.IsMultisite() {
|
|
return cephObjectStore.Name, cephObjectStore.Name, cephObjectStore.Name, nil, reconcile.Result{}, nil
|
|
}
|
|
|
|
zoneName := cephObjectStore.Spec.Zone.Name
|
|
zone := &cephv1.CephObjectZone{}
|
|
err := r.client.Get(r.opManagerContext, types.NamespacedName{Name: zoneName, Namespace: cephObjectStore.Namespace}, zone)
|
|
if err != nil {
|
|
if kerrors.IsNotFound(err) {
|
|
return "", "", "", nil, waitForRequeueIfObjectStoreNotReady, err
|
|
}
|
|
return "", "", "", nil, waitForRequeueIfObjectStoreNotReady, errors.Wrapf(err, "error getting CephObjectZone %q", cephObjectStore.Spec.Zone.Name)
|
|
}
|
|
log.NamedDebug(nsName, logger, "CephObjectZone resource %s found", zone.Name)
|
|
|
|
// Wait for the zone to be fully configured (pools, placements, period commit)
|
|
// to avoid race condition in period commit in objectStore and Zone controllers
|
|
if zone.Status == nil || zone.Status.Phase != k8sutil.ReadyStatus {
|
|
gotPhase := ""
|
|
if zone.Status != nil {
|
|
gotPhase = zone.Status.Phase
|
|
}
|
|
logger.Infof("CephObjectZone %q is not ready yet (phase %q). waiting for it to be configured", zoneName, gotPhase)
|
|
return "", "", "", nil, waitForRequeueIfObjectStoreNotReady, fmt.Errorf("CephObjectZone %q is not ready", zoneName)
|
|
}
|
|
|
|
zonegroup := &cephv1.CephObjectZoneGroup{}
|
|
err = r.client.Get(r.opManagerContext, types.NamespacedName{Name: zone.Spec.ZoneGroup, Namespace: cephObjectStore.Namespace}, zonegroup)
|
|
if err != nil {
|
|
if kerrors.IsNotFound(err) {
|
|
return "", "", "", nil, waitForRequeueIfObjectStoreNotReady, err
|
|
}
|
|
return "", "", "", nil, waitForRequeueIfObjectStoreNotReady, errors.Wrapf(err, "error getting CephObjectZoneGroup %q", zone.Spec.ZoneGroup)
|
|
}
|
|
log.NamedDebug(nsName, logger, "CephObjectZoneGroup resource %s found", zonegroup.Name)
|
|
|
|
realm := &cephv1.CephObjectRealm{}
|
|
err = r.client.Get(r.opManagerContext, types.NamespacedName{Name: zonegroup.Spec.Realm, Namespace: cephObjectStore.Namespace}, realm)
|
|
if err != nil {
|
|
if kerrors.IsNotFound(err) {
|
|
return "", "", "", nil, waitForRequeueIfObjectStoreNotReady, err
|
|
}
|
|
return "", "", "", nil, waitForRequeueIfObjectStoreNotReady, errors.Wrapf(err, "error getting CephObjectRealm %q", zonegroup.Spec.Realm)
|
|
}
|
|
log.NamedDebug(nsName, logger, "CephObjectRealm resource %s found", realm.Name)
|
|
|
|
return realm.Name, zonegroup.Name, zone.Name, zone, reconcile.Result{}, nil
|
|
}
|