xtf-cz / xtf

MIT License
12 stars 55 forks source link

Fix getAnyPod throwing IllegalArgumentException when no pod found #561

Closed vsalbaba closed 5 months ago

vsalbaba commented 6 months ago

Fix getAnyPod throwing IllegalArgumentException when no pod with specified labels is found.

Now it throws an explained RuntimeException.

Please make sure your PR meets the following requirements:

mnovak1 commented 6 months ago

@vsalbaba do you have any CI run with EAP OCP TS to make sure it does not break anyhting?

Edit: Could you create an issue as well, please?

vsalbaba commented 5 months ago

I've added an issue for this https://github.com/xtf-cz/xtf/issues/563

vsalbaba commented 5 months ago

CI run - https://jenkins.eapqe.psi.redhat.com/view/EAP8/view/OpenShift/job/eap-8.x-openshift-4-80-openjdk17/536/

There are 4 tests failing, but they are consistent fails with current XTF - CI run with those jobs and current XTF - https://jenkins.eapqe.psi.redhat.com/view/EAP8/view/OpenShift/job/eap-8.x-openshift-4-80-openjdk17/537/

mnovak1 commented 5 months ago

@vsalbaba Thanks, LGTM. Merging.