Skip to content

Conversation

hebasto
Copy link
Member

@hebasto hebasto commented Sep 27, 2021

This PR is a follow up of #23084. It enables the feature_asmap.py functional test which is fixed in #23084.

@fanquake
Copy link
Member

It enables the feature_asmap.py functional test which is fixed now.

Fixed when, with what change? Probably easier to just mention this in #23102.

@hebasto
Copy link
Member Author

hebasto commented Sep 27, 2021

Fixed when, with what change?

In #23084.

Probably easier to just mention this in #23102.

Agree. @MarcoFalke ?

@hebasto
Copy link
Member Author

hebasto commented Sep 27, 2021

Rebased on top of the merged #23102.

Probably easier to just mention this in #23102.

Is no longer relevant.

.cirrus.yml Outdated
@@ -152,7 +151,7 @@ task:
- netsh int ipv4 set dynamicport tcp start=1025 num=64511
- netsh int ipv6 set dynamicport tcp start=1025 num=64511
# TODO enable '--extended' and drop '--exclude'.
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👀

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Also, wondering if --extended pass by now

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Also, wondering if --extended pass by now

Timed out.

@hebasto
Copy link
Member Author

hebasto commented Sep 27, 2021

Updated comment.

@maflcko
Copy link
Member

maflcko commented Sep 27, 2021

cr ACK 4befc8f

@maflcko maflcko merged commit f036c35 into bitcoin:master Sep 27, 2021
@hebasto hebasto deleted the 210927-asmap branch September 27, 2021 17:23
@bitcoin bitcoin locked and limited conversation to collaborators Oct 30, 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.

4 participants