Răsfoiți Sursa

Merge pull request #1343 from opencost/bug-fix/asset-drilldown-sharing

filter shared assets in aggby
Sean Holcomb 3 ani în urmă
părinte
comite
fee5942783
1 a modificat fișierele cu 13 adăugiri și 3 ștergeri
  1. 13 3
      pkg/kubecost/asset.go

+ 13 - 3
pkg/kubecost/asset.go

@@ -2484,9 +2484,19 @@ func (as *AssetSet) AggregateBy(aggregateBy []string, opts *AssetAggregationOpti
 		sa := NewSharedAsset(name, as.Window.Clone())
 		sa := NewSharedAsset(name, as.Window.Clone())
 		sa.Cost = hourlyCost * hours
 		sa.Cost = hourlyCost * hours
 
 
-		err := aggSet.Insert(sa)
-		if err != nil {
-			return err
+		// Insert shared asset if it passes all filters
+		insert := true
+		for _, ff := range opts.FilterFuncs {
+			if !ff(sa) {
+				insert = false
+				break
+			}
+		}
+		if insert {
+			err := aggSet.Insert(sa)
+			if err != nil {
+				return err
+			}
 		}
 		}
 	}
 	}