Skip to content

Commit

Permalink
Remove unnecessary application lookup in buildpack metric generation
Browse files Browse the repository at this point in the history
  • Loading branch information
adamspd authored and gmllt committed Jun 18, 2024
1 parent 26be714 commit ce93f84
Showing 1 changed file with 6 additions and 9 deletions.
15 changes: 6 additions & 9 deletions collectors/applications.go
Original file line number Diff line number Diff line change
Expand Up @@ -259,15 +259,12 @@ func (c ApplicationsCollector) reportApp(application models.Application, objs *m
}

// 3. Use the droplet data for the buildpack metric
apps, exists := objs.Apps[application.GUID]
if exists {
for _, bp := range apps.Lifecycle.Data.Buildpacks {
c.applicationBuildpackMetric.WithLabelValues(
application.GUID,
application.Name,
bp,
).Set(float64(1))
}
for _, bp := range application.Lifecycle.Data.Buildpacks {
c.applicationBuildpackMetric.WithLabelValues(
application.GUID,
application.Name,
bp,
).Set(float64(1))
}

c.applicationInfoMetric.WithLabelValues(
Expand Down

0 comments on commit ce93f84

Please sign in to comment.