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

Fix CI error: "AttributeError: channels" #450

Closed
wants to merge 2 commits into from

Conversation

nepet
Copy link
Collaborator

@nepet nepet commented May 19, 2024

There are 2 open PRs that produce the same error AttributeError: channels on the following tests:

  • libs/gl-testing/tests/test_gl_node.py:127 test_configure_close_to_addr
  • libs/gl-testing/tests/test_node.py:143 test_node_invoice_amountless
  • libs/gl-testing/tests/test_node.py:186 test_node_listpays_preimage

It seems that the optional channels array returned by listpeers is not reliable anymore. This PR replaces the listpeers calls by listpeerchannels which will return a channels array.

@nepet nepet force-pushed the check-ci-channel-failures branch 2 times, most recently from 043847c to 4118ac3 Compare May 19, 2024 18:58
nepet added 2 commits May 19, 2024 21:19
The channels array returned by the rpc call `listpeers` is optional and
has been deprecated with v23.02 and will be removed after v24.02

Signed-off-by: Peter Neuroth <[email protected]>
Replace list_peers().channel (which is optional)  with
list_peer_channels().

Signed-off-by: Peter Neuroth <[email protected]>
@nepet nepet force-pushed the check-ci-channel-failures branch from 4118ac3 to 6af1e57 Compare May 19, 2024 19:21
@nepet nepet requested a review from cdecker May 19, 2024 19:41
@nepet nepet marked this pull request as ready for review May 19, 2024 19:41
@nepet nepet closed this May 21, 2024
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.

1 participant