[YUNIKORN-2293] Flaky E2E Test: Failed asserts in LogTestClusterInfoWrapper() blocked the resources cleanup steps

• Type: Sub-task
• Status: Closed
• Priority: Major
• Resolution: Fixed
• Fix Version/s: 1.5.0
• Component/s: test - e2e
• Labels: pull-request-available
• Target Version: 1.5.0
• Assignee: Yu-Lin Chen
• Reporter: Yu-Lin Chen
• Created: 30/Dec/23 07:53
• Updated: 20/Mar/24 14:32
• Resolved: 02/Jan/24 13:01

Description

If an E2E test fails, we will dump the cluster status through the following functions:

1. test/e2e/wrappers.go#LogTestClusterInfoWrapper()
2. test/e2e/wrappers.go#LogYunikornContainer()

However, these log functions contain several assertions, and a failed assertion will block other cleanup steps in AfterEach. Incomplete cleanup can cause other E2E tests to fail.
 
For example, E2E test (#967) failed due to a flaky assert in gang scheduling. The afterEach status have no application in queue, which caused an assert function failed.  Furthermore, the incompleted resources cleanup caused the following E2E tests to fail as well:

• simple_preemptor
• state_aware_app_scheduling
• user_group_limit

We should remove the assertions in those dump functions and just purely log the error messages.

Issue Links

• Blocked: YUNIKORN-2291 Flaky e2e test
• links to: GitHub Pull Request #755