Connect_DualMode_DnsConnect_RetrievedEndPoints_Success - Assert ::1 #54677

Activity

1. karelz

   added

   area-System.Net.Sockets

   test-run-coreTest failures in .NET Core test runsTest failures in .NET Core test runs

   on Jun 24, 2021
2. karelz

   added this to the 6.0.0 milestone on Jun 24, 2021
4. ghost added

   untriagedNew issue has not been triaged by the area ownerNew issue has not been triaged by the area owner

   on Jun 24, 2021
5. karelz

   removed

   untriagedNew issue has not been triaged by the area ownerNew issue has not been triaged by the area owner

   on Jun 24, 2021
6. karelz commented on Jun 24, 2021

   

   karelz

   on Jun 24, 2021

   MemberAuthor

   More actions

   @antonfirsov this looks like recent regression -- any idea why?
   We should disable the test ASAP -- 50 failures in 18 days.
7. antonfirsov commented on Jun 24, 2021

   

   antonfirsov

   on Jun 24, 2021

   Contributor

   More actions

   This is a new test introduced in #53581. Looks like client.LocalEndpoint doesn't get mapped to IPv6:
   

   runtime/src/libraries/System.Net.Sockets/tests/FunctionalTests/Connect.cs

   Line 132 in 0d31ddb

   Assert.Equal(IPAddress.Loopback.MapToIPv6(), localEndPoint.Address);

   Likely a product bug, only affects Linux.

   @tmds any tips what is going wrong here?
8. antonfirsov

   added

   os-linuxLinux OS (any supported distro)Linux OS (any supported distro)

   on Jun 24, 2021
9. antonfirsov commented on Jun 24, 2021

   

   antonfirsov

   on Jun 24, 2021

   Last edited by antonfirsov

   Contributor

   More actions

   If I get #53447 (comment) right, the bug pointed out in that issue is probably still present.
10. tmds commented on Jun 24, 2021

    

    tmds

    on Jun 24, 2021

    Member

    More actions

    @antonfirsov I think we need to update the test expectation and allow both

     ::ffff:127.0.0.1
     ::1
    

    as values for LocalEndPoint.
11. ghost added

    in-prThere is an active PR which will close this issue when it is mergedThere is an active PR which will close this issue when it is merged

    on Jun 24, 2021
12. danmoseley commented on Jun 25, 2021

    

    danmoseley

    on Jun 25, 2021

    Contributor

    More actions

    Hit this just now on SLES15 Arm64 in the preview 6 branch.
13. antonfirsov

    closed this as completedin #54681on Jun 29, 2021
14. ghost removed

    in-prThere is an active PR which will close this issue when it is mergedThere is an active PR which will close this issue when it is merged

    on Jun 29, 2021
15. karelz commented on Jul 20, 2021

    

    karelz

    on Jul 20, 2021

    Last edited by karelz

    MemberAuthor

    More actions

    This is still happening -- see updated table above, reopening

    Also note that after attempted fix, some failures changed to different manifestation - see #55709
16. karelz

    reopened this on Jul 20, 2021
17. karelz

    changed the title [-]Connect_DualMode_DnsConnect_RetrievedEndPoints_Success fails[/-] [+]Connect_DualMode_DnsConnect_RetrievedEndPoints_Success - Assert ::1[/+] on Jul 20, 2021
18. karelz

    added 2 commits that reference this issue on Jul 20, 2021

    Disable test Connect_DualMode_DnsConnect_RetrievedEndPoints_Success o…

    ...

    5f02794

    Disable test Connect_DualMode_DnsConnect_RetrievedEndPoints_Success o…

    ...

    5026c25
19. karelz

    added

    disabled-testThe test is disabled in source code against the issueThe test is disabled in source code against the issue

    on Jul 20, 2021
20. karelz commented on Jul 20, 2021

    

    karelz

    on Jul 20, 2021

    Last edited by karelz

    MemberAuthor

    More actions

    Test disabled on Linux in PR #56003 on 7/20
21. karelz

    assigned

    antonfirsov

    on Jul 22, 2021
22. antonfirsov

    closed this as completedin a85d36fon Jul 26, 2021
23. ghost locked as resolved and limited conversation to collaborators on Aug 25, 2021