diff --git a/CHANGELOG.md b/CHANGELOG.md index 934a14c9..5b5c85ae 100644 --- a/CHANGELOG.md +++ b/CHANGELOG.md @@ -17,6 +17,7 @@ Changelog for Cass Operator, new PRs should update the `main / unreleased` secti * [ENHANCEMENT] [#628](https://github.com/k8ssandra/cass-operator/issues/628) Replace pod task can replace any node, including those that have crashed * [ENHANCEMENT] [#532](https://github.com/k8ssandra/cass-operator/issues/532) Instead of rejecting updates/creates with deprecated fields, return kubectl warnings. * [ENHANCEMENT] [#637](https://github.com/k8ssandra/cass-operator/issues/637) Expand nodeStatuses to include IPs and racks. +* [BUGFIX] [#639](https://github.com/k8ssandra/cass-operator/issues/639) Remove InvalidState check, there's no need to block reconcile here. Keep the InvalidState as information for the user only. ## v1.19.1 diff --git a/pkg/reconciliation/reconcile_racks.go b/pkg/reconciliation/reconcile_racks.go index 79cd72a5..d048cf6a 100644 --- a/pkg/reconciliation/reconcile_racks.go +++ b/pkg/reconciliation/reconcile_racks.go @@ -2279,16 +2279,6 @@ func (rc *ReconciliationContext) CheckClearActionConditions() result.ReconcileRe return result.Continue() } -func (rc *ReconciliationContext) CheckForInvalidState() result.ReconcileResult { - cond, isSet := rc.Datacenter.GetCondition(api.DatacenterValid) - if isSet && cond.Status == corev1.ConditionFalse { - err := fmt.Errorf("datacenter %s is not in a valid state: %s", rc.Datacenter.Name, cond.Message) - return result.Error(err) - } - - return result.Continue() -} - func (rc *ReconciliationContext) CheckStatefulSetControllerCaughtUp() result.ReconcileResult { if hasStatefulSetControllerCaughtUp(rc.statefulSets, rc.dcPods) { // We do this here instead of in CheckPodsReady where we fix stuck pods @@ -2341,10 +2331,6 @@ func (rc *ReconciliationContext) fixMissingPVC() (bool, error) { func (rc *ReconciliationContext) ReconcileAllRacks() (reconcile.Result, error) { rc.ReqLogger.Info("reconciliationContext::reconcileAllRacks") - if recResult := rc.CheckForInvalidState(); recResult.Completed() { - return recResult.Output() - } - logger := rc.ReqLogger podList, err := rc.listPods(rc.Datacenter.GetClusterLabels())