Skip to content

Commit 6d31f70

Browse files
committed
Fix IT Test
1 parent f43bba2 commit 6d31f70

File tree

2 files changed

+6
-6
lines changed

2 files changed

+6
-6
lines changed

integration-tests/kyuubi-kubernetes-it/src/test/scala/org/apache/kyuubi/kubernetes/test/spark/SparkOnKubernetesTestsSuite.scala

Lines changed: 4 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -208,18 +208,19 @@ class KyuubiOperationKubernetesClusterClusterModeSuite
208208
batchRequest.getConf.asScala.toMap,
209209
batchRequest)
210210

211-
val session = sessionManager.getSession(sessionHandle).asInstanceOf[KyuubiBatchSessionImpl]
212-
// val batchJobSubmissionOp = session.batchJobSubmissionOp
211+
sessionManager.getSession(sessionHandle).asInstanceOf[KyuubiBatchSessionImpl]
213212

214213
eventually(timeout(3.minutes), interval(50.milliseconds)) {
214+
// trigger k8sOperation init here
215215
val appInfo = k8sOperation.getApplicationInfoByTag(sessionHandle.identifier.toString)
216216
assert(appInfo.state == RUNNING)
217217
assert(appInfo.name.startsWith(driverPodNamePrefix))
218218
}
219219

220220
val killResponse = k8sOperation.killApplicationByTag(sessionHandle.identifier.toString)
221221
assert(killResponse._1)
222-
assert(killResponse._2 startsWith "Operation of deleted appId:")
222+
assert(killResponse._2 endsWith "is completed")
223+
assert(killResponse._2 contains sessionHandle.identifier.toString)
223224

224225
eventually(timeout(3.minutes), interval(50.milliseconds)) {
225226
val appInfo = k8sOperation.getApplicationInfoByTag(sessionHandle.identifier.toString)

kyuubi-server/src/main/scala/org/apache/kyuubi/engine/KubernetesApplicationOperation.scala

Lines changed: 2 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -87,8 +87,7 @@ class KubernetesApplicationOperation extends ApplicationOperation with Logging {
8787
debug(s"Deleting application info from Kubernetes cluster by $tag tag")
8888
try {
8989
val info = appInfoStore.getOrDefault(tag, ApplicationInfo.notFound)
90-
logger.info(appInfoStore.toString)
91-
logger.info(info.toMap.toString())
90+
debug(s"Application info[tag: ${tag}] is in ${info.state}")
9291
info.state match {
9392
case NOT_FOUND | FAILED | UNKNOWN =>
9493
(
@@ -128,7 +127,7 @@ class KubernetesApplicationOperation extends ApplicationOperation with Logging {
128127
case NOT_FOUND =>
129128
ApplicationInfo(id = null, name = null, ApplicationState.NOT_FOUND)
130129
case _ =>
131-
logger.info(s"Successfully got application info by $tag: $info")
130+
debug(s"Successfully got application info by $tag: $info")
132131
info
133132
}
134133
} catch {

0 commit comments

Comments
 (0)