Skip to content

Commit d164287

Browse files
committed
address review 3
1 parent 00e5d02 commit d164287

File tree

1 file changed

+2
-1
lines changed

1 file changed

+2
-1
lines changed

pkg/quotaplugins/quota-forest/quota-manager/quota/quotamanager.go

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -376,7 +376,7 @@ func (m *Manager) AllocateForest(forestName string, consumerID string) (response
376376
if consumerInfo == nil {
377377
return nil, fmt.Errorf("consumer %s does not exist, create and add first", consumerID)
378378
}
379-
//allocation can pass if consumers inside the forest is not released
379+
//allocation can pass if consumers inside the forest are not released
380380
//make sure that appwrapper (consumer) does not exist in target namespace
381381
namespace, awName := util.ParseId(consumerID)
382382
config, err := rest.InClusterConfig()
@@ -401,6 +401,7 @@ func (m *Manager) AllocateForest(forestName string, consumerID string) (response
401401
}
402402
} else {
403403
if forestController.IsConsumerAllocated(consumerID) {
404+
klog.V(4).Infof("[AllocateForest] Found consumer %v in quota tree but didn't find its matching AppWrapper. Deleting consumer from the tree", consumerID)
404405
m.DeAllocateForest(forestName, consumerID)
405406
}
406407
}

0 commit comments

Comments
 (0)