Merge pull request #59 from intel/fix_remove_sys_metric_dependency_for_htmlblob

mlim19 · web-flow · commit f48eba856fe8 · 2025-06-20T09:51:21.000-07:00
Remove the dependency of cpu/mem metrics for htmlblob
diff --git a/src/gprofiler_indexer/callstacks.go b/src/gprofiler_indexer/callstacks.go
@@ -290,7 +290,7 @@ func (pw *ProfilesWriter) ParseStackFrameFile(sess *session.Session, task SQSMes
 		}
 	}
 
-	if fileInfo.Metrics.CPUAvg != 0 && fileInfo.Metrics.MemoryAvg != 0 {
+	if htmlBlobPath != "" || (fileInfo.Metrics.CPUAvg != 0 && fileInfo.Metrics.MemoryAvg != 0) {
 		pw.writeMetrics(uint32(serviceId), fileInfo.Metadata.CloudInfo.InstanceType,
 			fileInfo.Metadata.Hostname, timestamp, fileInfo.Metrics.CPUAvg,
 			fileInfo.Metrics.MemoryAvg, htmlBlobPath)

Original file line number	Diff line number	Diff line change
`@@ -290,7 +290,7 @@ func (pw ProfilesWriter) ParseStackFrameFile(sess session.Session, task SQSMes`
`290`	`290`	`}`
`291`	`291`	`}`
`292`	`292`
`293`		`- if fileInfo.Metrics.CPUAvg != 0 && fileInfo.Metrics.MemoryAvg != 0 {`
	`293`	`+ if htmlBlobPath != "" \|\| (fileInfo.Metrics.CPUAvg != 0 && fileInfo.Metrics.MemoryAvg != 0) {`
`294`	`294`	`pw.writeMetrics(uint32(serviceId), fileInfo.Metadata.CloudInfo.InstanceType,`
`295`	`295`	`fileInfo.Metadata.Hostname, timestamp, fileInfo.Metrics.CPUAvg,`
`296`	`296`	`fileInfo.Metrics.MemoryAvg, htmlBlobPath)`