Skip to content

Conversation

@pinheadmz
Copy link
Contributor

addrs must revealed by clicking "> description" but we can document that.

Screenshot 2024-10-09 at 2 42 05 PM
Screenshot 2024-10-09 at 2 41 59 PM

@willcl-ark
Copy link
Contributor

willcl-ark commented Oct 9, 2024

Test works for me locally using k3d cluster:

Network not connected
Tank george peers expected: 1, actual: 1
Tank john peers expected: 1, actual: 1
Tank paul peers expected: 0, actual: 0
Tank ringo peers expected: 0, actual: 0
Network connected
2024-10-09 22:04:24 | INFO    | test     | Creating chain split
2024-10-09 22:04:24 | DEBUG   | test     | Executing warnet command: bitcoin rpc john createwallet miner
2024-10-09 22:04:25 | DEBUG   | test     | Executing warnet command: bitcoin rpc john -generate 1
2024-10-09 22:04:26 | DEBUG   | test     | Waiting for predicate with timeout 300s and interval 5s
2024-10-09 22:04:31 | INFO    | test     | Fork observed!
2024-10-09 22:04:31 | INFO    | test     | Checking node description...
2024-10-09 22:04:31 | INFO    | test     | Checking reachable address is provided...
2024-10-09 22:04:31 | DEBUG   | test     | Executing warnet command: bitcoin rpc george addnode john.default.svc:18444 onetry
2024-10-09 22:04:31 | DEBUG   | test     | Waiting for predicate with timeout 300s and interval 5s
2024-10-09 22:04:31 | DEBUG   | test     | Executing warnet command: bitcoin rpc george getpeerinfo
2024-10-09 22:04:32 | INFO    | test     | Stopping network
2024-10-09 22:04:32 | DEBUG   | test     | Executing warnet command: down --force
2024-10-09 22:04:32 | DEBUG   | test     | Waiting for predicate with timeout 60s and interval 1s

@pinheadmz
Copy link
Contributor Author

works for me with docker desktop locally too. I dont think this PR affected ingress so im going to try just adding a bit of sleep before we try the API

@pinheadmz pinheadmz force-pushed the fo-addr branch 3 times, most recently from 6f95964 to 4f19381 Compare October 9, 2024 23:45
@pinheadmz
Copy link
Contributor Author

there we go, finally got the test to work

@pinheadmz pinheadmz merged commit 26b1910 into bitcoin-dev-project:main Oct 10, 2024
11 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants