Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

qa_crowbarsetup.sh has too many sleeps #293

Open
aspiers opened this issue Mar 27, 2015 · 4 comments
Open

qa_crowbarsetup.sh has too many sleeps #293

aspiers opened this issue Mar 27, 2015 · 4 comments

Comments

@aspiers
Copy link
Contributor

aspiers commented Mar 27, 2015

qa_crowbarsetup.sh has loads of sleep calls. This is unreliable and also typically results in everything taking longer than it should. They should all be replaced with busy waits.

@jdsn
Copy link
Member

jdsn commented Apr 10, 2015

We totally agree :)
Most sleeps are actually meaning "FIXME, I have to be replaced by a proper query if the needed ressource is available". But we never had the time to replace them.

@aspiers
Copy link
Contributor Author

aspiers commented Feb 23, 2016

I think some progress has been made on this, but there are still 9 sleep calls left.

@aspiers
Copy link
Contributor Author

aspiers commented Aug 30, 2016

The remaining sleep calls are now:

  • sleep 1 in resize_partition
  • Three sleeps in reboot_nodes_via_ipmi
  • sleep 10 in onadmin_crowbar_register
  • sleep 10 in update_one_proposal
  • sleep 5 in a loop oncontroller_testsetup waiting for the newly uploaded glance image to appear (should use wait_for instead) -- see PR qa_crowbarsetup: use wait_for instead of sleep-loop #1192
  • sleep 60 in onadmin_cloudupgrade_reboot_and_redeploy_clients

@seife
Copy link
Contributor

seife commented Jun 15, 2018

one sleep (not in qa_crowbarsetub, but in freshadminvm) is removed by #2564 :-)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants