Archives/xmr-btc-swap
2
0
Fork 0
mirror of https://github.com/comit-network/xmr-btc-swap.git synced 2024-11-19 09:25:33 +00:00
Code Issues Projects Releases Wiki Activity

Merge #453

Browse Source 
453: Fix messages for awaiting containers ready r=da-kami a=da-kami

The message `JOINING all threads` is unfortunately not deterministic, it can happen that it just is not printed in the logs.
For Monerod container the message is set to `RPC server started ok`. This message appears in both a test run that was hanging with `JOINING all threads` and a successful run. Initially the message was set to `core RPC server started ok` with `core` being a variable value. We assume that `core` does not change, but did not to further code analysis what values it can be.

For Monero Wallet RPC container the message is set to `Run server thread name: RPC` which is what it was set to initially. After several container runs this message seems to be reasonable - there are no recorded issues of the Wallet RPC container hanging, but we had problems with Monerod in the past.

Co-authored-by: Daniel Karzel <daniel@comit.network>
This commit is contained in:
bors[bot] 2021-04-28 05:37:34 +00:00 committed by GitHub
commit 7174303496
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
1 changed files with 2 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
monero-harness/src/image.rs
View File

@ -34,7 +34,7 @@ impl Image for Monerod {
container container
.logs() .logs()
.stdout .stdout
.wait_for_message("JOINING all threads") .wait_for_message("RPC server started ok")
.unwrap(); .unwrap();
} }
@ -87,7 +87,7 @@ impl Image for MoneroWalletRpc {
container container
.logs() .logs()
.stdout .stdout
.wait_for_message("JOINING all threads") .wait_for_message("Run server thread name: RPC")
.unwrap(); .unwrap();
} }