You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@yunikorn.apache.org by cc...@apache.org on 2023/08/24 18:13:14 UTC
[yunikorn-core] branch master updated: [YUNIKORN-1932] Ignore not-started app when reporting app summary (#627)
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 e9d703bf [YUNIKORN-1932] Ignore not-started app when reporting app summary (#627)
e9d703bf is described below
commit e9d703bf87e95740629267af379b9d30e25cb84f
Author: Yongjun Zhang <yo...@pinterest.com>
AuthorDate: Thu Aug 24 13:12:44 2023 -0500
[YUNIKORN-1932] Ignore not-started app when reporting app summary (#627)
Closes: #627
Signed-off-by: Craig Condit <cc...@apache.org>
---
pkg/scheduler/objects/application.go | 3 +++
1 file changed, 3 insertions(+)
diff --git a/pkg/scheduler/objects/application.go b/pkg/scheduler/objects/application.go
index 1f47c7e1..3f58c1a7 100644
--- a/pkg/scheduler/objects/application.go
+++ b/pkg/scheduler/objects/application.go
@@ -2013,6 +2013,9 @@ func (sa *Application) CleanupUsedResource() {
}
func (sa *Application) LogAppSummary(rmID string) {
+ if sa.startTime.IsZero() {
+ return
+ }
appSummary := sa.GetApplicationSummary(rmID)
appSummary.DoLogging()
appSummary.ResourceUsage = nil
---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@yunikorn.apache.org
For additional commands, e-mail: issues-help@yunikorn.apache.org