search

Kuadrant / testsuite

3 stars 12 forks source link

Fix TLSPolicy timeouts #460

Closed azgabur closed 2 months ago

azgabur commented 2 months ago

Some previous change lowered the TLSPolicy wait timeout resulting in TLSPolicy not getting ready erros, this PR will raise it to 180s which is sufficient even for the Letsecrypt test. From my testing when the TLSPolicy is Enforced: true additional waits are not needed, thats why I removed the sleep in the client fixture. I also added parameter for the wait_for_ready function to set the timeout/timelimit duration more easily.

azgabur commented 2 months ago

I agree with you @pehala, the timeout handling needs bigger refactor then this. I will leave it for some future PR.

Closing in favor of #461