Skip to content

Commit

Permalink
core: skip counting exporter pods as a ceph pod
Browse files Browse the repository at this point in the history
The exporter pods were being counted as a ceph pod,
so once an exporter pod was added to a node, it would never
be removed since the controller thinks the exporter pod
itself is a ceph pod for which the exporter must be present.

Signed-off-by: Travis Nielsen <[email protected]>
  • Loading branch information
travisn committed Oct 15, 2024
1 parent 1433587 commit a71078b
Showing 1 changed file with 2 additions and 1 deletion.
3 changes: 2 additions & 1 deletion pkg/operator/ceph/cluster/nodedaemon/add.go
Original file line number Diff line number Diff line change
Expand Up @@ -177,7 +177,8 @@ func isCephPod(labels map[string]string, podName string) bool {
// will be empty since the monitors don't exist yet
isCanaryPod := strings.Contains(podName, "-canary-")
isCrashCollectorPod := strings.Contains(podName, "-crashcollector-")
if ok && !isCanaryPod && !isCrashCollectorPod {
isExporterPod := strings.Contains(podName, "-exporter-")
if ok && !isCanaryPod && !isCrashCollectorPod && !isExporterPod {
logger.Debugf("%q is a ceph pod!", podName)
return true
}
Expand Down

0 comments on commit a71078b

Please sign in to comment.