ChainSafe / ChainSafe/forest

Send FIL failed

Open
#2,726 20 comments 0 reactions 0 assignees View on GitHub
Type: Bug
Dominant language
Rust
Stars
697
Forks
200
Avg merge
1d 5h
Merged PRs (30d)
65

Description

**Describe the bug**

The wallet test failed, and the FIL was not sent. The target wallet didn't receive anything and the transaction is not visible in the source wallet. https://calibration.filscan.io/address/general?address=t1ac6ndwj6nghqbmtbovvnwcqo577p6ox2pt52q2y
(it should've happened around 2023-03-23 17:47:17, target t1fye76pfymru4y2fqbgdxswpwqt3ukeems2nzfna)

This means that either we have a bug in the test setup or the send command doesn't always send the funds, which would be a **significant** issue.

**To Reproduce**

It happened in this PR https://github.com/ChainSafe/forest/pull/2709 (which didn't introduce any logic changes)

The job failed: https://github.com/ChainSafe/forest/actions/runs/4503553182/jobs/7926889785

**Log output**

```
Wallet tests
Importing preloaded wallet key
t1ac6ndwj6nghqbmtbovvnwcqo577p6ox2pt52q2y
Exporting key
Fetching metrics
--2023-03-23 17:47:17-- http://localhost:6116/metrics
Resolving localhost (localhost)... ::1, 127.0.0.1
Connecting to localhost (localhost)|::1|:6116... failed: Connection refused.
Connecting to localhost (localhost)|127.0.0.1|:6116... connected.
HTTP request sent, awaiting response... 200 OK
Length: 28751 (28K) [text/plain]
Saving to: ‘metrics.log’

0K .......... .......... ........ 100% 404M=0s

2023-03-23 17:47:17 (404 MB/s) - ‘metrics.log’ saved [28751/28751]

Listing wallet balances
Address Default Balance
t1ac6ndwj6nghqbmtbovvnwcqo577p6ox2pt52q2y ~100 FIL
Creating a new address to send FIL to
t1fye76pfymru4y2fqbgdxswpwqt3ukeems2nzfna
Listing wallet balances
Address Default Balance
t1ac6ndwj6nghqbmtbovvnwcqo577p6ox2pt52q2y X ~100 FIL
t1fye76pfymru4y2fqbgdxswpwqt3ukeems2nzfna 0 atto FIL
Sending FIL to the above address
Checking balance of t1fye76pfymru4y2fqbgdxswpwqt3ukeems2nzfna...
Checking balance 1/10
Checking balance 2/10
Checking balance 3/10
Checking balance 4/10
Checking balance 5/10
Checking balance 6/10
Checking balance 7/10
Checking balance 8/10
Checking balance 9/10
Checking balance 10/10
Address Default Balance
t1ac6ndwj6nghqbmtbovvnwcqo577p6ox2pt52q2y X ~100 FIL
t1fye76pfymru4y2fqbgdxswpwqt3ukeems2nzfna 0 atto FIL
FIL amount should match
Shutting down Forest node
```

**Expected behaviour**

The test passes.

**Screenshots**

**Environment (please complete the following information):**

- OS: buildjet runner, Ubuntu 20.04
- Rust version(e.g. `rustc --version`)
- Branch/commit [2dc3442](https://github.com/ChainSafe/forest/pull/2709/commits/2dc34424b2482a790828afad42620997e67dfa37)

**Other information and links**

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.