Skip to content

Commit

Permalink
remove reduandent function
Browse files Browse the repository at this point in the history
Signed-off-by: Ezra Silvera <ezra@il.ibm.com>
  • Loading branch information
ezrasilvera committed Apr 7, 2021
1 parent 5882dae commit 1011abe
Showing 1 changed file with 3 additions and 10 deletions.
13 changes: 3 additions & 10 deletions pkg/monitoring/vms/prometheus/prometheus.go
Expand Up @@ -469,22 +469,15 @@ func updateVMIEvictionBlocker(nodeName string, vmis []*k6tv1.VirtualMachineInsta
}
}

func (metrics *vmiMetrics) updateVMIEvictionBlockerXX() {
metrics.pushCommonMetric(
"kubevirt_vmi_non_evictable",
"Indication for a VirtualMachine that its eviction strategy is set to Live Migration but is not migratable.",
prometheus.GaugeValue,
checkNonEvictableVMAndSetMetric(metrics.vmi),
)
}

func checkNonEvictableVMAndSetMetric(vmi *k6tv1.VirtualMachineInstance) float64 {
setVal := 0.0
if vmi.IsEvictable() {
vmiIsMigratableCond := controller.NewVirtualMachineInstanceConditionManager().
GetCondition(vmi, k6tv1.VirtualMachineInstanceIsMigratable)

if vmiIsMigratableCond != nil && vmiIsMigratableCond.Status == k8sv1.ConditionFalse {
//As this metric is used for user alert we refer to be conservative - so if the VirtualMachineInstanceIsMigratable
//condition is still not set we treat the VM as if it's "not migratable"
if vmiIsMigratableCond == nil || vmiIsMigratableCond.Status == k8sv1.ConditionFalse {
setVal = 1.0
}

Expand Down

0 comments on commit 1011abe

Please sign in to comment.