capistrant commented on PR #18580: URL: https://github.com/apache/druid/pull/18580#issuecomment-3344384111
> The `KubernetesClusterWithOperatorDockerTest` succeeded with the original code. > > @capistrant , it seems that we can't get rid of the invocation of `startWatchingLogs()` in `KubernetesWorkItem.shutdown()`. I haven't dug very deep but it is most likely because `KubernetesWorkItem.shutdown()` calls `KubernetesPeonLifecycle.shutdown()` which calls `kubernetesClient.deletePeonJob(taskId)`. > > After this, `join()` calls `startWatchingLogs()` but it fails to initialize `logWatch` since the peon job is already deleted. Damn ok. https://github.com/apache/druid/pull/18556 did upgrade fabric8 to 7.4.0 which attempts to address the bug that is seemingly behind the LogWatch issues (https://github.com/fabric8io/kubernetes-client/issues/7163) -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
