馃尡 assert E2E error responses when waiting for MD nodes #8501
Conversation
k8s-ci-robot
added
cncf-cla: yes
jackfrancis
force-pushed
the
e2e-machinedeployment-wait-nodes-err
branch
from
779d38f
to
386ec58
Compare
jackfrancis
force-pushed
the
e2e-machinedeployment-wait-nodes-err
branch
from
386ec58
to
2d5222d
Compare
k8s-ci-robot
added
the
lgtm
|
LGTM label has been added. Git tree hash: 53ebf5db53c0f4d1db46e4b5abe64279d5b1ef1b
|
|
/cherry-pick release-1.4 |
|
@jackfrancis: once the present PR merges, I will cherry-pick it on top of release-1.4 in a new PR and assign it to you. 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/test-infra repository. |
|
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: fabriziopandini 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
|
@jackfrancis: new pull request created: #8516 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/test-infra repository. |
|
/area e2e-testing |
What this PR does / why we need it:
This pull request improves the clarity of error messages during end-to-end validation of the expected node count in MachineDeployments. It does this by wrapping error responses, ensuring that the root cause of the error is displayed in the console output (stdout/stderr).
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 #