Fix for Policy strategy updated

togashidm · uniemimu · commit de0e4f148c7d · 2022-03-29T15:16:15.000+03:00
This fix is related to TAS being unable to update the loaded policy
while running in the cluster.
diff --git a/telemetry-aware-scheduling/pkg/controller/controller.go b/telemetry-aware-scheduling/pkg/controller/controller.go
@@ -129,6 +129,7 @@ func (controller *TelemetryPolicyController) onUpdate(old, new interface{}) {
 			klog.V(2).InfoS(err.Error(), "component", "controller")
 			return
 		}
+		oldStrat.SetPolicyName(polCopy.ObjectMeta.Name)
 		controller.Enforcer.RemoveStrategy(oldStrat, oldStrat.StrategyType())
 		for _, rule := range oldPol.Spec.Strategies[oldStrat.StrategyType()].Rules {
 			err := controller.DeleteMetric(rule.Metricname)

Original file line number	Diff line number	Diff line change
`@@ -129,6 +129,7 @@ func (controller *TelemetryPolicyController) onUpdate(old, new interface{}) {`
`129`	`129`	`klog.V(2).InfoS(err.Error(), "component", "controller")`
`130`	`130`	`return`
`131`	`131`	`}`
	`132`	`+ oldStrat.SetPolicyName(polCopy.ObjectMeta.Name)`
`132`	`133`	`controller.Enforcer.RemoveStrategy(oldStrat, oldStrat.StrategyType())`
`133`	`134`	`for _, rule := range oldPol.Spec.Strategies[oldStrat.StrategyType()].Rules {`
`134`	`135`	`err := controller.DeleteMetric(rule.Metricname)`