Details
-
Test
-
Status: Closed
-
Minor
-
Resolution: Fixed
-
None
-
None
-
None
-
None
Description
https://github.com/apache/yunikorn-k8shim/blob/master/test/e2e/wrappers.go#L112
for _, ns := range namespaces { logErr := k8s.LogNamespaceInfo(ns) Ω(logErr).NotTo(HaveOccurred()) pods, err := k.GetPodsByOptions(metav1.ListOptions{}) Ω(err).NotTo(HaveOccurred()) By("Pod count is " + strconv.Itoa(len(pods.Items))) for _, pod := range pods.Items { By("Pod name is " + pod.Name) By("Pod details: " + pod.String()) } logErr = restClient.LogAppsInfo(ns) Ω(logErr).NotTo(HaveOccurred()) }
`restClient.LogAppsInfo(ns)` tries to get apps from "root.$ns". However, these queues are nonexistent normally (https://github.com/apache/yunikorn-k8shim/blob/master/test/e2e/framework/helpers/yunikorn/rest_api_utils.go#L351)
It logged all apps before https://github.com/apache/yunikorn-k8shim/pull/516 got merged. It seems to me previous behavior makes sense. Logging all apps is better than logging nothing
Attachments
Attachments
Issue Links
- is fixed by
-
YUNIKORN-2305 E2E test: Upload stdout logs to Github Action artifact
- Closed