-
Notifications
You must be signed in to change notification settings - Fork 38.7k
ci: Enable feature_asmap.py test on native Windows #23103
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
Conversation
Fixed when, with what change? Probably easier to just mention this in #23102. |
In #23084.
Agree. @MarcoFalke ? |
b9ea277 to
b14415d
Compare
.cirrus.yml
Outdated
| # See: https://docs.microsoft.com/en-us/biztalk/technical-guides/settings-that-can-be-modified-to-improve-network-performance | ||
| - 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'. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👀
There was a problem hiding this comment.
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
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Also, wondering if
--extendedpass by now
b14415d to
4befc8f
Compare
|
Updated comment. |
|
cr ACK 4befc8f |
This PR is a follow up of #23084. It enables the
feature_asmap.pyfunctional test which is fixed in #23084.