🐛 fix: self hosted e2e test #5288
Conversation
k8s-ci-robot
added
release-note
|
/test ? |
|
@richardcase: The following commands are available to trigger required jobs: The following commands are available to trigger optional jobs: Use In response to this:
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository. |
|
/test pull-cluster-api-provider-aws-e2e |
1 similar comment
|
/test pull-cluster-api-provider-aws-e2e |
k8s-ci-robot
added
the
lgtm
Yeah still a bit flakey. I know the fix works locally so i will monitor this. |
|
Seems there was still a node not having the E2E image: failing capa pod artifact |
The self hosted e2e was failing as the control plane node was encountering disk pressure. This caused CAPA to be evicted and the CAPA images on the original node where deleted to reclaim disk space. When CAPA moved back to the node the image couldn't be found. Signed-off-by: Richard Case <richard.case@outlook.com>
richardcase
force-pushed
the
e2e_fix_selfhosted
branch
from
16b1129 to
c259906
Compare
k8s-ci-robot
removed
the
lgtm
|
/test pull-cluster-api-provider-aws-e2e |
|
https://storage.googleapis.com/kubernetes-ci-logs/pr-logs/pull/kubernetes-sigs_cluster-api-provider-aws/5288/pull-cluster-api-provider-aws-e2e/1881316234645999616/artifacts/clusters/self-hosted-2ka0nh/resources/Node/ip-10-0-160-246.us-west-2.compute.internal.yaml is still missing the e2e image, and the node shows no disk pressure 🤔 |
k8s-ci-robot
added
size/M
|
/test pull-cluster-api-provider-aws-e2e |
I misunderstood the "generated" cluster definitions process, so the root volume size wasn't being added. Just testing a change to that. |
|
/test pull-cluster-api-provider-aws-e2e |
1 similar comment
|
/test pull-cluster-api-provider-aws-e2e |
|
The e2e test passed: https://prow.k8s.io/view/gs/kubernetes-ci-logs/pr-logs/pull/kubernetes-sigs_cluster-api-provider-aws/5288/pull-cluster-api-provider-aws-e2e/1881396398931316736 The commits have been squashed. |
|
/lgtm |
k8s-ci-robot
added
the
lgtm
|
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: nrb The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
k8s-ci-robot
added
the
approved
|
/cherrypick release-2.7 |
|
@richardcase: new pull request created: #5293 In response to this:
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository. |
|
/cherrypick release-2.6 |
|
@richardcase: new pull request created: #5294 In response to this:
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository. |
|
@richardcase: new pull request created: #5295 In response to this:
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository. |
|
@richardcase: new pull request created: #5296 In response to this:
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository. |
What type of PR is this?
/kind bug
What this PR does / why we need it:
The self hosted e2e was failing as the control plane node was encountering disk pressure. This caused CAPA to be evicted and the CAPA images on the original node where deleted to reclaim disk space. When CAPA moved back to the node the image couldn't be found.
Which issue(s) this PR fixes (optional, in
fixes #<issue number>(, fixes #<issue_number>, ...)format, will close the issue(s) when PR gets merged):Fixes #5252
Special notes for your reviewer:
Tested this locally:
Checklist:
Release note: