This is an automated email from the ASF dual-hosted git repository.

ccondit pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/yunikorn-core.git


The following commit(s) were added to refs/heads/master by this push:
     new c56c2883 [YUNIKORN-2592] Remove spurious warning when setting queue 
resources (#1016)
c56c2883 is described below

commit c56c28831df040bda477b9e6fc55034158b81ae9
Author: Craig Condit <ccon...@apache.org>
AuthorDate: Thu Mar 6 13:14:01 2025 -0600

    [YUNIKORN-2592] Remove spurious warning when setting queue resources (#1016)
    
    Closes: #1016
---
 pkg/scheduler/partition.go | 5 +----
 1 file changed, 1 insertion(+), 4 deletions(-)

diff --git a/pkg/scheduler/partition.go b/pkg/scheduler/partition.go
index e912f5dc..49a2fa2a 100644
--- a/pkg/scheduler/partition.go
+++ b/pkg/scheduler/partition.go
@@ -356,10 +356,7 @@ func (pc *PartitionContext) AddApplication(app 
*objects.Application) error {
        maxApps := app.GetMaxApps()
        if !isRecoveryQueue && (guaranteedRes != nil || maxRes != nil || 
maxApps != 0) {
                // set resources based on tags, but only if the queue is 
dynamic (unmanaged)
-               if queue.IsManaged() {
-                       log.Log(log.SchedQueue).Warn("Trying to set resources 
on a queue that is not an unmanaged leaf",
-                               zap.String("queueName", queue.QueuePath))
-               } else {
+               if !queue.IsManaged() {
                        if maxApps != 0 {
                                queue.SetMaxRunningApps(maxApps)
                        }


---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscr...@yunikorn.apache.org
For additional commands, e-mail: issues-h...@yunikorn.apache.org

Reply via email to