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.

Sign up for GitHub

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

Jump to bottom

Once CI supports IPV6, TestTCP/UDP should make the 'EADDRNOTAVAIL' fatal #2698

Closed
andy31415 opened this issue Sep 17, 2020 · 4 comments · Fixed by #13080
Closed

Once CI supports IPV6, TestTCP/UDP should make the 'EADDRNOTAVAIL' fatal #2698

andy31415 opened this issue Sep 17, 2020 · 4 comments · Fixed by #13080
Assignees
@kpschoedel
Labels
feature work p1 priority 1 work V1.0
Milestone
V1.0

Comments

@andy31415
Copy link
Contributor

Problem

We specifically allow transport tests to pass on IPV6 when we get ADDRNOTAVAIL.

Proposed Solution

IPV6 failures should be fatal as CHIP requires IPV6.
@issue-label-bot
Copy link

Issue-Label Bot is automatically applying the label feature_request to this issue, with a confidence of 0.55. Please mark this comment with 👍 or 👎 to give our bot feedback!

Links: app homepage, dashboard and code for this bot.

@andy31415 andy31415 mentioned this issue Sep 17, 2020
Merged
@woody-apple woody-apple added this to the V1.0 milestone Sep 21, 2020
@woody-apple woody-apple added the p1 priority 1 work label Oct 21, 2020
@woody-apple woody-apple added V1.0 and removed V0.7 labels Nov 5, 2020
@todo todo bot mentioned this issue Mar 12, 2021
Merged
@stale
Copy link

stale bot commented Nov 6, 2021

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs.

@stale stale bot added the stale Stale issue or PR label Nov 6, 2021
@stale
Copy link

stale bot commented Nov 13, 2021

This stale issue has been automatically closed. Thank you for your contributions.

@stale stale bot closed this as completed Nov 13, 2021
@kpschoedel
Copy link
Contributor

Reopening because this still exists (and I'm about to fix it).

@kpschoedel kpschoedel reopened this Dec 16, 2021
@stale stale bot removed the stale Stale issue or PR label Dec 16, 2021
@kpschoedel kpschoedel self-assigned this Dec 16, 2021
@kpschoedel kpschoedel mentioned this issue Dec 16, 2021
Closed
kpschoedel added a commit to kpschoedel/connectedhomeip that referenced this issue Dec 16, 2021
@kpschoedel
[Inet] Make tests fail on EADDRNOTAVAIL
3d0e165 
#### Problem

Fixes project-chip#2698 Once CI supports IPV6, TestTCP/UDP should make the 'EADDRNOTAVAIL' fatal

#### Change overview

Remove the special handling of `EADDRNOTAVAIL`.

#### Testing

CI
@kpschoedel kpschoedel mentioned this issue Dec 16, 2021
Merged
kpschoedel added a commit that referenced this issue Dec 16, 2021
@kpschoedel
[Inet] Make tests fail on EADDRNOTAVAIL (#13080)
9adf764 
#### Problem

Fixes #2698 Once CI supports IPV6, TestTCP/UDP should make the 'EADDRNOTAVAIL' fatal

#### Change overview

Remove the special handling of `EADDRNOTAVAIL`.

#### Testing

CI
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@kpschoedel kpschoedel

Labels
feature work p1 priority 1 work V1.0
Projects
None yet
Milestone
V1.0
Development

Successfully merging a pull request may close this issue.

[Inet] Make tests fail on EADDRNOTAVAIL kpschoedel/connectedhomeip
3 participants
@andy31415 @woody-apple @kpschoedel