Skip to content

Conversation

@maflcko
Copy link
Member

@maflcko maflcko commented Apr 25, 2020

Remove incorrect and undocumented connect_nodes(self.nodes[0], 1).

Issue is that transactions are re-relayed (going full circle) between the two nodes, that have two connections between each other.

https://travis-ci.org/github/bitcoin/bitcoin/jobs/679201559#L6992

Also fix some pep8 while touching the file

This bug has been introduced by accident in c1dde3a

@DrahtBot
Copy link
Contributor

The following sections might be updated with supplementary metadata relevant to reviewers and maintainers.

Conflicts

Reviewers, this pull request conflicts with the following ones:

If you consider this pull request important, please also help to review the conflicting pull requests. Ideally, start with the one that should be merged first.

@maflcko
Copy link
Member Author

maflcko commented Apr 28, 2020

cc @achow101 This was accidentally introduced by you, so you seem qualified to review 😬

@achow101
Copy link
Member

The self.sync_all() immediately following can be removed too.

@maflcko maflcko force-pushed the 2004-qaWalletBumpInterm branch from fabfd3c to fa301fe Compare April 28, 2020 15:53
@maflcko
Copy link
Member Author

maflcko commented Apr 28, 2020

Thanks, fixed.

@achow101
Copy link
Member

ACK fa301fe

@maflcko maflcko merged commit 6a60bfc into bitcoin:master Apr 28, 2020
@maflcko maflcko deleted the 2004-qaWalletBumpInterm branch April 28, 2020 17:13
@bitcoin bitcoin locked as resolved and limited conversation to collaborators Feb 15, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants