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

Server: Update windows error codes list #3125

Merged
merged 1 commit into from
Feb 11, 2022
Merged

Conversation

rvl
Copy link
Contributor

@rvl rvl commented Feb 10, 2022

When testing the windows build under wine, the test case for withListeningSocket "patate" ListenOnRandomPort was sometimes failing with "Network.Socket.gai_strerror not supported: 11002". This PR should fix it.

Issue Number

ADP-1432

The test case for `withListeningSocket "patate" ListenOnRandomPort`
was sometimes failing with "Network.Socket.gai_strerror not supported: 11002".
@rvl rvl self-assigned this Feb 10, 2022
Copy link
Contributor

@paweljakubas paweljakubas left a comment

Choose a reason for hiding this comment

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

keeping finger crossed for that 👍

@rvl
Copy link
Contributor Author

rvl commented Feb 10, 2022

bors r+

🤞

@rvl rvl linked an issue Feb 10, 2022 that may be closed by this pull request
1 task
iohk-bors bot added a commit that referenced this pull request Feb 10, 2022
3074: Fix gitRevFromGit with newer process libraries r=rvl a=angerman

This code was initially lifted likely from cardano-node, and now lives in cardano-config, but is replicated here...
It was fixed in cardano config recently by `@newhoggy` in cardano-config#4. Now we need to fix it here as well.

3110: select witnesses only for unique inps r=paweljakubas a=paweljakubas

<!--
Detail in a few bullet points the work accomplished in this PR.

Before you submit, don't forget to:

* Make sure the GitHub PR fields are correct:
   ✓ Set a good Title for your PR.
   ✓ Assign yourself to the PR.
   ✓ Assign one or more reviewer(s).
   ✓ Link to a Jira issue, and/or other GitHub issues or PRs.
   ✓ In the PR description delete any empty sections
     and all text commented in <!--, so that this text does not appear
     in merge commit messages.

* Don't waste reviewers' time:
   ✓ If it's a draft, select the Create Draft PR option.
   ✓ Self-review your changes to make sure nothing unexpected slipped through.

* Try to make your intent clear:
   ✓ Write a good Description that explains what this PR is meant to do.
   ✓ Jira will detect and link to this PR once created, but you can also
     link this PR in the description of the corresponding Jira ticket.
   ✓ Highlight what Testing you have done.
   ✓ Acknowledge any changes required to the Documentation.
-->

adp-1190

- [x] I have laxed condition when checking if tx is partially signed. We take into consideration only unique inputs now
- [x] Integration test

### Comments

<!-- Additional comments, links, or screenshots to attach, if any. -->

### Issue Number

<!-- Reference the Jira/GitHub issue that this PR relates to, and which requirements it tackles.
  Note: Jira issues of the form ADP- will be auto-linked. -->


3125: Server: Update windows error codes list r=rvl a=rvl

When testing the windows build under wine, the test case for `withListeningSocket "patate" ListenOnRandomPort` was sometimes failing with "Network.Socket.gai_strerror not supported: 11002". This PR should fix it.

### Issue Number

ADP-1432

Co-authored-by: Moritz Angermann <[email protected]>
Co-authored-by: Rodney Lorrimar <[email protected]>
Co-authored-by: IOHK <[email protected]>
Co-authored-by: Pawel Jakubas <[email protected]>
Co-authored-by: Piotr Stachyra <[email protected]>
Co-authored-by: Johannes Lund <[email protected]>
@iohk-bors
Copy link
Contributor

iohk-bors bot commented Feb 10, 2022

This PR was included in a batch that timed out, it will be automatically retried

#duplicate

@rvl rvl merged commit c9109cf into master Feb 11, 2022
@rvl rvl deleted the rvl/adp-1432/winsock-error-codes branch February 11, 2022 06:36
WilliamKingNoel-Bot pushed a commit that referenced this pull request Feb 11, 2022
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.

Flaky test: gai_strerror not supported: 11002 from API Server, handles bad host name
2 participants