linera-protocol icon indicating copy to clipboard operation
linera-protocol copied to clipboard

Issue of `Missing message notification`

Open MathieuDutSik opened this issue 1 year ago • 1 comments

The issue of Missing message notification shows up in the test_end_to_end_repeated_transfers test in the following way:

  • Eliminate the INTEGRATION_TEST_GUARD from the linera_net_tests.rs.
  • Run the tests in the following way. First, set up the validators:
cargo build --features storage-service,remote-net
rm -rf /tmp/WORK && mkdir /tmp/WORK
cargo run --features storage-service --bin linera -- net up --policy-config devnet --path /tmp/WORK --validators 4 --shards 4
  • Then run the Faucet:
export LINERA_WALLET=/tmp/WORK/wallet_0.json
export LINERA_STORAGE=rocksdb:/tmp/WORK/client_0.db
cargo run --features storage-service,remote-net --bin linera -- faucet --amount 1000 --port 8079
  • Then run the tests:
export LINERA_FAUCET_URL=http://localhost:8079
cargo test -p linera-service remote_net_grpc --features remote-net

Then the error sometimes occurs, along with other such as error at publishBytecode. It occurs at the start for i=0, though not systematically.

The questions are the following:

  • Are the notifications ordered in the way that they are obtained from the stream?
  • Is it conceivable that the test itself is not well written and is based on assumptions that are not valid when other operations are occurring?
  • Changing the position of the let notification2 = ... to just after the creation of node_service2 leads to a reliable error in the computation of the block height. Would a switch to ProcessInbox::Skip address the issue?

MathieuDutSik avatar Oct 29 '24 17:10 MathieuDutSik

I can't reproduce this, I think. At least I don't see any Missing message notification logs.

afck avatar Feb 05 '25 17:02 afck