Skip to content

Commit

Permalink
fix(osd): ceph/osd/remove, reduce retry timeout to 15s
Browse files Browse the repository at this point in the history
This reduces the safe-to-remove retry interval from 1m to 15s.

Closes #14247

Signed-off-by: Brian Dowling <bjd-dev@simplicity.net>
  • Loading branch information
bdowling committed May 23, 2024
1 parent 8dc88c3 commit 1619704
Showing 1 changed file with 3 additions and 3 deletions.
6 changes: 3 additions & 3 deletions pkg/daemon/ceph/osd/remove.go
Original file line number Diff line number Diff line change
Expand Up @@ -84,7 +84,7 @@ func removeOSD(clusterdContext *clusterd.Context, clusterInfo *client.ClusterInf
logger.Errorf("failed to exclude osd.%d out of the crush map. %v", osdID, err)
}

// Check we can remove the OSD
// Check if we can safely remove the OSD
// Loop forever until the osd is safe-to-destroy
for {
isSafeToDestroy, err := client.OsdSafeToDestroy(clusterdContext, clusterInfo, osdID)
Expand Down Expand Up @@ -112,8 +112,8 @@ func removeOSD(clusterdContext *clusterd.Context, clusterInfo *client.ClusterInf
break
}
// Else we wait until the OSD can be removed
logger.Warningf("osd.%d is NOT ok to destroy, retrying in 1m until success", osdID)
time.Sleep(1 * time.Minute)
logger.Warningf("osd.%d is NOT ok to destroy, retrying in 15s until success", osdID)
time.Sleep(15 * time.Second)
}
}

Expand Down

0 comments on commit 1619704

Please sign in to comment.