linter fixes
This commit is contained in:
@@ -126,11 +126,11 @@ func (r *NodeTainterConfigReconciler) Reconcile(ctx context.Context, req ctrl.Re
|
|||||||
// Converting map[string]string into map[string]corev1.Taint for convenience
|
// Converting map[string]string into map[string]corev1.Taint for convenience
|
||||||
parsedRules, parseErrs := parseLabelRulesFromSpec(specLabelRules)
|
parsedRules, parseErrs := parseLabelRulesFromSpec(specLabelRules)
|
||||||
if len(parseErrs) > 0 {
|
if len(parseErrs) > 0 {
|
||||||
errMsg := fmt.Sprintf("Invalid rules found in NodeTainterConfig %s: %v", config.Name, parseErrs)
|
parsingError := fmt.Errorf("invalid rules found in NodeTainterConfig %s: %v", config.Name, parseErrs)
|
||||||
log.Error(fmt.Errorf(errMsg), "Rule parsing failed")
|
log.Error(parsingError, "Rule parsing failed", "configName", config.Name, "parsingErrors", parseErrs)
|
||||||
r.Recorder.Eventf(&config, corev1.EventTypeWarning, "InvalidConfig", errMsg)
|
r.Recorder.Eventf(&config, corev1.EventTypeWarning, "InvalidConfig", parsingError.Error())
|
||||||
_ = r.updateCRDStatus(ctx, &config, metav1.ConditionFalse, ConditionReasonConfigParsingError, errMsg)
|
_ = r.updateCRDStatus(ctx, &config, metav1.ConditionFalse, ConditionReasonConfigParsingError, parsingError.Error())
|
||||||
_ = r.updateNodeTaintStatus(ctx, &node, nil, errMsg)
|
_ = r.updateNodeTaintStatus(ctx, &node, nil, parsingError.Error())
|
||||||
return ctrl.Result{}, nil
|
return ctrl.Result{}, nil
|
||||||
}
|
}
|
||||||
|
|
||||||
|
Reference in New Issue
Block a user