Merge pull request #51128 from thaJeztah/28.x_backport_gcpolicy-invalid-calculations

austinvazquez · web-flow · commit 17db0cd834df · 2025-10-07T13:56:48.000-07:00
[28.x backport] builder: use proper percentage calculations for default gc policy
diff --git a/builder/builder-next/worker/gc.go b/builder/builder-next/worker/gc.go
@@ -72,6 +72,6 @@ func DefaultGCPolicy(p string, reservedSpace, maxUsedSpace, minFreeSpace int64)
 }
 
 func diskPercentage(dstat disk.DiskStat, percentage int64) int64 {
-	avail := dstat.Total / percentage
+	avail := dstat.Total * percentage / 100
 	return (avail/(1<<30) + 1) * 1e9 // round up
 }

Original file line number	Diff line number	Diff line change
`@@ -72,6 +72,6 @@ func DefaultGCPolicy(p string, reservedSpace, maxUsedSpace, minFreeSpace int64)`
`72`	`72`	`}`
`73`	`73`
`74`	`74`	`func diskPercentage(dstat disk.DiskStat, percentage int64) int64 {`
`75`		`- avail := dstat.Total / percentage`
	`75`	`+ avail := dstat.Total * percentage / 100`
`76`	`76`	`return (avail/(1<<30) + 1) * 1e9 // round up`
`77`	`77`	`}`