Skip to content

Commit 202375e

Browse files
avanthakkarsp98
authored andcommitted
monitoring: enable exporter for downstream 4.14
All regression for ceph-exporter are fixed in downstream Ceph v6.1z2, so enabling the exporter again. (cherry picked from commit c5e35cd) Signed-off-by: sp98 <[email protected]>
1 parent a246ada commit 202375e

File tree

3 files changed

+2
-6
lines changed

3 files changed

+2
-6
lines changed

pkg/operator/ceph/cluster/nodedaemon/exporter.go

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -51,13 +51,11 @@ const (
5151
)
5252

5353
var (
54-
MinVersionForCephExporter = cephver.CephVersion{Major: 18, Minor: 0, Extra: 0}
54+
MinVersionForCephExporter = cephver.CephVersion{Major: 17, Minor: 2, Extra: 6}
5555
)
5656

5757
// createOrUpdateCephExporter is a wrapper around controllerutil.CreateOrUpdate
5858
func (r *ReconcileNode) createOrUpdateCephExporter(node corev1.Node, tolerations []corev1.Toleration, cephCluster cephv1.CephCluster, cephVersion *cephver.CephVersion) (controllerutil.OperationResult, error) {
59-
// CephVersion change is done temporarily, as some regression was detected in Ceph version 17.2.6 which is summarised here https://github.com/ceph/ceph/pull/50718#issuecomment-1505608312.
60-
// Thus, disabling ceph-exporter for now until all the regression are fixed.
6159
if !cephVersion.IsAtLeast(MinVersionForCephExporter) {
6260
logger.Infof("Skipping exporter reconcile on ceph version %q", cephVersion.String())
6361
return controllerutil.OperationResultNone, nil

pkg/operator/ceph/cluster/nodedaemon/exporter_test.go

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -68,7 +68,7 @@ func TestCreateOrUpdateCephExporter(t *testing.T) {
6868
}
6969
cephCluster.Spec.Labels = cephv1.LabelsSpec{}
7070
cephCluster.Spec.PriorityClassNames = cephv1.PriorityClassNamesSpec{}
71-
cephVersion := &cephver.CephVersion{Major: 18, Minor: 0, Extra: 0}
71+
cephVersion := &cephver.CephVersion{Major: 17, Minor: 2, Extra: 6}
7272
ctx := context.TODO()
7373
context := &clusterd.Context{
7474
Clientset: test.New(t, 1),

pkg/operator/ceph/cluster/nodedaemon/reconcile.go

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -257,8 +257,6 @@ func (r *ReconcileNode) createOrUpdateNodeDaemons(node corev1.Node, tolerations
257257
return errors.Wrapf(err, "ceph exporter reconcile failed on op %q", op)
258258
}
259259
} else {
260-
// CephVersion change is done temporarily, as some regression was detected in Ceph version 17.2.6 which is summarised here https://github.com/ceph/ceph/pull/50718#issuecomment-1505608312.
261-
// Thus, disabling ceph-exporter for now until all the regression are fixed.
262260
if cephVersion.IsAtLeast(MinVersionForCephExporter) {
263261
logger.Debugf("ceph exporter successfully reconciled for node %q. operation: %q", node.Name, op)
264262
// create the metrics service

0 commit comments

Comments
 (0)