cnti-testcatalog / testsuite

📞📱☎️📡🌐 Cloud Native Telecom Initiative (CNTI) Test Catalog is a tool to check for and provide feedback on the use of K8s + cloud native best practices in networking applications and platforms
https://wiki.lfnetworking.org/display/LN/Test+Catalog
Apache License 2.0
173 stars 71 forks source link

[Feature] Replace sleep <n> commands with proper waiters #2042

Open kosstennbl opened 4 months ago

kosstennbl commented 4 months ago

Is your feature request related to a problem? Please describe. Having sleep commands with fixed variables of significant value doesn't feel quite right. (Next is LLM-generated reasoning): Using proper waiters instead of fixed sleep commands improves code precision and maintainability by allowing specification of maximum wait times and conditions to be met, reducing the risk of unnecessary delays or premature execution.

Describe the solution you'd like Design a proper waiter for each case of "sleep" command. Examples of where it's needed:

Once this issue is addressed how will the fix be verified? Absence of fixed long "sleeps" and successful specs.