quarkus-qe / quarkus-test-suite

16 stars 32 forks source link

Test OpenShift serving certificates and TLS registry with REST client #1987

Closed michalvavrik closed 1 week ago

michalvavrik commented 1 week ago

Summary

This is quite similar to what I have done in QE Test Framework https://github.com/quarkus-qe/quarkus-test-framework/pull/1270 because I think it tests all we need from TLS Registry integration with REST client and OpenShift certificate serving. Few differences:

Please select the relevant options.

Checklist:

michalvavrik commented 1 week ago

run tests

michalvavrik commented 1 week ago

run tests

michalvavrik commented 1 week ago

Native failed over Caused by: java.security.cert.CertificateException: No subject alternative DNS name matching server.ts-jblpfaeiqm.svc.cluster.local found which is the original reason why I added untilAsserted. I think it is just flaky and we should add couple of seconds to waiting as it depends on OpenShift and not us. Let me add few seconds and retry.

michalvavrik commented 1 week ago

run tests

michalvavrik commented 1 week ago

yeah, OCP is green now so I'll merge it.

fedinskiy commented 1 week ago

FYI: our weekly runs now fail due to : java.security.cert.CertificateException

michalvavrik commented 1 week ago

FYI: our weekly runs now fail due to : java.security.cert.CertificateException

thanks, I'll look. this seems like OCP setup or longer waiting, I don't know what could change in code to avoid this, but let see

michalvavrik commented 1 week ago

in OpenShift they say that pod won't start until secret is mounted, so I think I need to inspect what is actually mounted in cases when it fails and whether it is retry or just randomness that is makes it sometimes fail and sometimes pass

michalvavrik commented 1 week ago

it's not quite easy to reproduce, didn't happen to me once so far :-( maybe it is more likely to happen when OpenShift cluster is busy, don't know.

michalvavrik commented 1 week ago

I figured it out: https://github.com/quarkus-qe/quarkus-test-framework/pull/1296