flakey: add resource requirements to SuperTuxKart e2e test#4280
Conversation
|
Build Failed 😭 Build Id: 1c170c4e-df97-406d-aa3c-77fa493a13e0 Status: FAILURE To get permission to view the Cloud Build view, join the agones-discuss Google Group. |
|
Same flake: |
|
Build Failed 😭 Build Id: 2b380234-e02d-4379-ab2f-2229ca61010a Status: FAILURE To get permission to view the Cloud Build view, join the agones-discuss Google Group. |
|
Build Succeeded 🥳 Build Id: 18a2ebaa-47f8-4882-ac83-0a8dc07d6ea1 The following development artifacts have been built, and will exist for the next 30 days:
A preview of the website (the last 30 builds are retained): To install this version: |
|
Build Succeeded 🥳 Build Id: 7152b0a6-ef9a-42e5-8da1-0924ffee172c The following development artifacts have been built, and will exist for the next 30 days:
A preview of the website (the last 30 builds are retained): To install this version: |
|
Of course everything passes now 😁 @igooch - any chance there was something changed in Autopilot? This has been less flaky today. |
|
/gcbrun |
|
Build Succeeded 🥳 Build Id: a5bfefe6-22c7-4a82-a3b1-caaa12c2a138 The following development artifacts have been built, and will exist for the next 30 days:
A preview of the website (the last 30 builds are retained): To install this version: |
This test has been flaky on GKE Autopilot. Adding explicit memory (700Mi) and CPU (200m) resource requests to the container to ensure adequate resource allocation and improve test stability on GKE Autopilot clusters.
* Tweaking some resource levels, to see if it makes a difference.
96be4bf to
658225f
Compare
|
Build Failed 😭 Build Id: 37e81e0e-2bb7-44ed-8cd0-f6e04d8d5479 Status: FAILURE To get permission to view the Cloud Build view, join the agones-discuss Google Group. |
|
So this is interesting, this line fails first: Then consistent failure here: Interestingly - this is a standard GKE cluster. |
* Updated log output to be easier to read.
|
Build Succeeded 🥳 Build Id: d6375bd6-0969-454f-9f4c-84f1cf0c2b43 The following development artifacts have been built, and will exist for the next 30 days:
A preview of the website (the last 30 builds are retained): To install this version: |
|
/gcbrun |
|
Build Succeeded 🥳 Build Id: 2eb5509a-8098-4686-afdb-1842299d96c6 The following development artifacts have been built, and will exist for the next 30 days:
A preview of the website (the last 30 builds are retained): To install this version: |
lacroixthomas
left a comment
There was a problem hiding this comment.
LGTM, as you were saying, if it fails again, we should have more info now ! 😄
…v#4280) * flakey: add resource requirements to SuperTuxKart e2e test This test has been flaky on GKE Autopilot. Adding explicit memory (700Mi) and CPU (200m) resource requests to the container to ensure adequate resource allocation and improve test stability on GKE Autopilot clusters. * Add the ability to get Pod logs to the e2e framework. * Also bump the memory a little? * Tweaks to logging functions. * * Review updates * Tweaking some resource levels, to see if it makes a difference. * * Tweaks to test parameters * Updated log output to be easier to read.

What type of PR is this?
/kind bug
What this PR does / Why we need it:
This test has been flaky on GKE Autopilot. Adding explicit memory (700Mi) and CPU (200m) resource requests to the container to ensure adequate resource allocation and improve test stability on GKE Autopilot clusters. 🤞🏻
Which issue(s) this PR fixes:
N/A
Special notes for your reviewer:
This is the same as what we do for Xonotic and the Xonotic one doesn't flake.