ARMmbed / htrun

MOVED: https://github.com/ARMmbed/mbed-os-tools (Flash, reset and run host supervised tests on mbed platforms)
8 stars 37 forks source link

Wait forever in case global resource manager #166

Closed mazimkhan closed 7 years ago

mazimkhan commented 7 years ago

Currently connection sub-process is started and htrun waits for process_start_timeout secs. This is not correct when global resource manager is used. As resource allocation can take really long time.

@bridadan @studavekar Please review. Required for use with RAAS.

adbridge commented 7 years ago

@theotherjimmy Azim says this one is urgent can you ensure it moves through swiftly

theotherjimmy commented 7 years ago

@mazimkhan What's up with that merge commit?

theotherjimmy commented 7 years ago

I re-triggered Travis to check for Python 3 compatibility.

mazimkhan commented 7 years ago

Removed merge commit.

theotherjimmy commented 7 years ago

@mazimkhan Thanks! I'll merge this when testing passes.

theotherjimmy commented 7 years ago

Should we also make specifying the timeout and the use of a global resource manager mutually exclusive options? This can come as another PR.

theotherjimmy commented 7 years ago

Testing went green. I'm going to merge this, and I'll investigate my own question above.

mazimkhan commented 7 years ago

Can we release htrun please. So that we can get it in Oulu CI.

bridadan commented 7 years ago

Y'all are too quick for me 😄