Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

ci: attempts to fix #31 #48

Closed

Conversation

vincenzopalazzo
Copy link
Collaborator

For now, this enables only the parallel execution on the CI, required more debugging

Fixes #31

Signed-off-by: Vincenzo Palazzo vincenzopalazzodev@gmail.com

@vincenzopalazzo vincenzopalazzo marked this pull request as draft March 31, 2022 08:29
@vincenzopalazzo vincenzopalazzo force-pushed the macros/double_tx branch 2 times, most recently from 43ef2c4 to 1902f7e Compare March 31, 2022 08:57
@vincenzopalazzo
Copy link
Collaborator Author

vincenzopalazzo commented Mar 31, 2022

@niftynei I'm a little bit too young in the dual-funding spec, do you know why we have the same transaction in the blockchain in the test_bolt2-20-open_channel_accepter.py in particular in the tests test_open_accepter_with_inputs?

It is strange to me, but I'm sure that I'm missing some things in the code

more details are described in this issue #31

Stacktrace here https://github.com/rustyrussell/lnprototest/runs/5773967845?check_suite_focus=true#step:3:4672

@vincenzopalazzo vincenzopalazzo added the 🐛 bug 🐛 Something isn't working label Mar 31, 2022
Signed-off-by: Vincenzo Palazzo <vincenzopalazzodev@gmail.com>
Signed-off-by: Vincenzo Palazzo <vincenzopalazzodev@gmail.com>
Signed-off-by: Vincenzo Palazzo <vincenzopalazzodev@gmail.com>
@vincenzopalazzo vincenzopalazzo deleted the macros/double_tx branch May 31, 2022 19:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🐛 bug 🐛 Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

flaky test ran on lightningd
2 participants