You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@yunikorn.apache.org by ma...@apache.org on 2024/07/05 11:09:34 UTC

(yunikorn-core) branch master updated: [YUNIKORN-2699] Preemption e2e tests fail in latest master (#909)

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

mani 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 108ed0d2 [YUNIKORN-2699] Preemption e2e tests fail in latest master (#909)
108ed0d2 is described below

commit 108ed0d25768782232aa27e3ee26b35962f41416
Author: Manikandan R <ma...@gmail.com>
AuthorDate: Fri Jul 5 16:39:23 2024 +0530

    [YUNIKORN-2699] Preemption e2e tests fail in latest master (#909)
    
    Closes: #909
    
    Signed-off-by: Manikandan R <ma...@gmail.com>
---
 pkg/scheduler/objects/preemption.go | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/pkg/scheduler/objects/preemption.go b/pkg/scheduler/objects/preemption.go
index d91dbe32..2bcd7867 100644
--- a/pkg/scheduler/objects/preemption.go
+++ b/pkg/scheduler/objects/preemption.go
@@ -641,7 +641,7 @@ func (p *Preemptor) TryPreemption() (*AllocationResult, bool) {
 	p.ask.MarkTriggeredPreemption()
 
 	// notify RM that victims should be released
-	p.application.notifyRMAllocationReleased(victims, si.TerminationType_PREEMPTED_BY_SCHEDULER,
+	p.application.notifyRMAllocationReleased(finalVictims, si.TerminationType_PREEMPTED_BY_SCHEDULER,
 		"preempting allocations to free up resources to run ask: "+p.ask.GetAllocationKey())
 
 	// reserve the selected node for the new allocation if it will fit


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