[tor-bugs] #22132 [Core Tor/Chutney]: Chutney should avoid waiting for set times: wait for conditions instead

[Tor Bug Tracker & Wiki]

#22132: Chutney should avoid waiting for set times: wait for conditions instead
--------------------------------------------+------------------------------
 Reporter:  teor                            |          Owner:  nickm
     Type:  defect                          |         Status:
                                            |  needs_revision
 Priority:  Medium                          |      Milestone:
Component:  Core Tor/Chutney                |        Version:
 Severity:  Normal                          |     Resolution:
 Keywords:  network-team-roadmap-2019-Q1Q2  |  Actual Points:  1.2
Parent ID:  #20647                          |         Points:  2
 Reviewer:                                  |        Sponsor:  Sponsor19
--------------------------------------------+------------------------------
Changes (by teor):

 * status:  needs_review => needs_revision


Comment:

 I did a rollup branch for #22132 and #28203:
 https://travis-ci.org/teor2345/chutney/builds/506043823

 There are two bugs in this branch:

 1. CHUTNEY_START_TIME is too low:
 https://travis-ci.org/teor2345/chutney/jobs/506043826
 https://travis-ci.org/teor2345/chutney/jobs/506043830
 https://travis-ci.org/teor2345/chutney/jobs/506043835

 Let's try setting it to 60 seconds by default?

 2. The logs say that all the nodes are done, but the script still fails:
 https://travis-ci.org/teor2345/chutney/jobs/506043837

 Maybe there's a race condition here, because we load the logs for
 c.isBootstrapped(), then we load them again for
 c.getLastBootstrapStatus().

--
Ticket URL: <https://trac.torproject.org/projects/tor/ticket/22132#comment:26>
Tor Bug Tracker & Wiki <https://trac.torproject.org/>
The Tor Project: anonymity online