Skip to content

Commit 4ed30be

Browse files
committed
Don't divide by zero.
1 parent 5ee4f43 commit 4ed30be

File tree

1 file changed

+8
-2
lines changed

1 file changed

+8
-2
lines changed

pkg/sync/syncer.go

Lines changed: 8 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -75,11 +75,14 @@ func (p *ProgressCounts) LogResourcesProgress(ctx context.Context, resourceType
7575

7676
func (p *ProgressCounts) LogEntitlementsProgress(ctx context.Context, resourceType string) {
7777
// TODO: only log progress if we complete or we haven't logged in the past 10 seconds
78-
l := ctxzap.Extract(ctx)
7978
entitlementsProgress := p.EntitlementsProgress[resourceType]
8079
resources := p.Resources[resourceType]
80+
if resources == 0 {
81+
return
82+
}
8183
percentComplete := (entitlementsProgress * 100) / resources
8284

85+
l := ctxzap.Extract(ctx)
8386
switch {
8487
case entitlementsProgress > resources:
8588
l.Error("more entitlement resources than resources",
@@ -106,11 +109,14 @@ func (p *ProgressCounts) LogEntitlementsProgress(ctx context.Context, resourceTy
106109
}
107110

108111
func (p *ProgressCounts) LogGrantsProgress(ctx context.Context, resourceType string) {
109-
l := ctxzap.Extract(ctx)
110112
grantsProgress := p.GrantsProgress[resourceType]
111113
resources := p.Resources[resourceType]
114+
if resources == 0 {
115+
return
116+
}
112117
percentComplete := (grantsProgress * 100) / resources
113118

119+
l := ctxzap.Extract(ctx)
114120
switch {
115121
case grantsProgress > resources:
116122
l.Error("more grant resources than resources",

0 commit comments

Comments
 (0)