feat(models): log model source url

Signed-off-by: Sunil Thaha <sthaha@redhat.com>

pkg/model/node_component_energy.go

@@ -55,11 +55,14 @@ func CreateNodeComponentPowerEstimatorModel(nodeFeatureNames, systemMetaDataFeat
 	var err error
 	nodeComponentPowerModel, err = createPowerModelEstimator(modelConfig)
 	if err != nil {
-		klog.Errorf("Failed to create %s/%s Model to estimate Node Component Power: %v", modelConfig.ModelType, modelConfig.ModelOutputType, err)
+		klog.Errorf("Failed to create %s/%s Model from %s to estimate Node Component Power: %v",
+			modelConfig.ModelType, modelConfig.ModelOutputType,
+			modelConfig.SourceURL(), err)
 		return
 	}
 
-	klog.V(1).Infof("Using the %s/%s Model to estimate Node Component Power", modelConfig.ModelType, modelConfig.ModelOutputType)
+	klog.V(1).Infof("Using the %s/%s Model from %s to estimate Node Component Power",
+		modelConfig.ModelType, modelConfig.ModelOutputType, modelConfig.SourceURL())
 }
 
 // IsNodeComponentPowerModelEnabled returns if the estimator has been enabled or not

pkg/model/node_platform_energy.go

@@ -51,10 +51,13 @@ func CreateNodePlatformPowerEstimatorModel(nodeFeatureNames, systemMetaDataFeatu
 	var err error
 	nodePlatformPowerModel, err = createPowerModelEstimator(modelConfig)
 	if err != nil {
-		klog.Errorf("Failed to create %s/%s Model to estimate Node Platform Power: %v", modelConfig.ModelType, modelConfig.ModelOutputType, err)
+		klog.Errorf("Failed to create %s/%s Model from %s to estimate Node Platform Power: %v",
+			modelConfig.ModelType, modelConfig.ModelOutputType,
+			modelConfig.SourceURL(), err)
 		return
 	}
-	klog.V(1).Infof("Using the %s/%s Model to estimate Node Platform Power", modelConfig.ModelType, modelConfig.ModelOutputType)
+	klog.V(1).Infof("Using the %s/%s Model from %s to estimate Node Platform Power",
+		modelConfig.ModelType, modelConfig.ModelOutputType, modelConfig.SourceURL())
 }
 
 // IsNodePlatformPowerModelEnabled returns if the estimator has been enabled or not

pkg/model/types/types.go

@@ -16,15 +16,18 @@ limitations under the License.
 
 package types
 
-type ModelType int
-type ModelOutputType int
+type (
+	ModelType       int
+	ModelOutputType int
+)
 
 const (
 	// Power Model types
 	Ratio            ModelType = iota + 1 // estimation happens within kepler without using Model Server
 	Regressor                             // estimation happens within kepler, but pre-trained model parameters are downloaded externally
 	EstimatorSidecar                      // estimation happens in the sidecar with a loaded pre-trained power model
 )
+
 const (
 	// Power Model Output types
 	// Absolute Power Model (AbsPower): is the power model trained by measured power (including the idle power)
@@ -33,6 +36,7 @@ const (
 	DynPower
 	Unsupported
 )
+
 const (
 	// Define energy source
 	PlatformEnergySource    = "acpi"
@@ -99,3 +103,11 @@ type ModelConfig struct {
 	SystemMetaDataFeatureNames  []string
 	SystemMetaDataFeatureValues []string
 }
+
+func (c *ModelConfig) SourceURL() string {
+	if c.InitModelURL != "" {
+		return c.InitModelURL
+	}
+
+	return c.InitModelFilepath
+}