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

cpp/Ice/binding failure (Windows, release) #3510

Open
bernardnormier opened this issue Feb 6, 2025 · 1 comment
Open

cpp/Ice/binding failure (Windows, release) #3510

bernardnormier opened this issue Feb 6, 2025 · 1 comment
Milestone

Comments

@bernardnormier
Copy link
Member

From https://github.com/zeroc-ice/ice/actions/runs/13181700314/job/36793870237

*** [21/272] Running cpp/Ice/binding tests ***
[ running client/server test - 02/06/25 15:48:09 ]
- Config: ssl,mx,x64
(D:\a\ice\ice\cpp\test\Ice\binding\msbuild\server\x64\Release\server.exe --Ice.Default.Host=127.0.0.1 --Test.BasePort=14200 --Ice.Warn.Connections=1 --Ice.Default.Protocol=ssl --Ice.IPv6=0 --Ice.Admin.Endpoints="tcp -h 127.0.0.1" --Ice.Admin.InstanceName=Server --IceMX.Metrics.Debug.GroupBy=id --IceMX.Metrics.Parent.GroupBy=parent --IceMX.Metrics.All.GroupBy=none --IceSSL.*** --IceSSL.DefaultDir=D:\a\ice\ice\certs --IceSSL.CAs=cacert.pem --IceSSL.CertFile=server.p12 --Ice.PrintStackTraces=1 --Ice.ThreadPool.Server.Size=1 --Ice.ThreadPool.Server.SizeMax=3 --Ice.ThreadPool.Server.SizeWarn=0 --Ice.PrintAdapterReady=1 env={'PATH': 'D:\\a\\ice\\ice\\cpp\\test\\Common\\msbuild\\x64\\Release;D:\\a\\ice\\ice\\cpp\\bin\\x64\\Release'})
(D:\a\ice\ice\cpp\test\Ice\binding\msbuild\client\x64\Release\client.exe --Ice.Default.Host=127.0.0.1 --Test.BasePort=14200 --Ice.Warn.Connections=1 --Ice.Default.Protocol=ssl --Ice.IPv6=0 --Ice.Admin.Endpoints="tcp -h 127.0.0.1" --Ice.Admin.InstanceName=Client --IceMX.Metrics.Debug.GroupBy=id --IceMX.Metrics.Parent.GroupBy=parent --IceMX.Metrics.All.GroupBy=none --IceSSL.*** --IceSSL.DefaultDir=D:\a\ice\ice\certs --IceSSL.CAs=cacert.pem --IceSSL.CertFile=client.p12 --Ice.PrintStackTraces=1 env={'PATH': 'D:\\a\\ice\\ice\\cpp\\test\\Common\\msbuild\\x64\\Release;D:\\a\\ice\\ice\\cpp\\bin\\x64\\Release'})
testing binding with single endpoint... ok
testing binding with multiple endpoints... ok
testing binding with multiple random endpoints... ok
testing binding with multiple endpoints and AMI... ok
testing random endpoint selection... ok
testing ordered endpoint selection... ok
testing per request binding with single endpoint... ok
testing per request binding with multiple endpoints... ok
testing per request binding with multiple endpoints and AMI... ok
testing per request binding and ordered endpoint selection... failed!
D:\a\ice\ice\cpp\test\Ice\binding\AllTests.cpp:684: assertion `i == nRetry' failed
!! 02/06/25 15:49:54.469 error: communicator not destroyed during global destruction.
@bernardnormier bernardnormier added this to the 3.8.0 milestone Feb 6, 2025
@pepone
Copy link
Member

pepone commented Feb 6, 2025

2025-02-06T15:56:30.0313602Z testing per request binding and ordered endpoint selection... failed!
2025-02-06T15:56:30.0315671Z D:\a\ice\ice\cpp\test\Ice\binding\AllTests.cpp:684: assertion `i == nRetry' failed
2025-02-06T15:56:30.0317722Z !! 02/06/25 15:49:54.469 error: communicator not destroyed during global destruction.
2025-02-06T15:56:30.0318535Z 
2025-02-06T15:56:30.0319246Z test in Ice/binding failed:

adapters.push_back(com->createObjectAdapter("Adapter64", endpoints[0]->toString()));
for (i = 0; i < nRetry && test->getAdapterName() == "Adapter64"; i++)
;
#if TARGET_OS_IPHONE > 0
test(i >= nRetry - 1); // WORKAROUND: for connection establishment hang.
#else
test(i == nRetry);
#endif

It is not obvious what are we testing here. We reactivated "Adapter64", and we expect that nRetry count request to use another endpoint? and that is why i == nRetry?

@bernardnormier bernardnormier changed the title cpp/Ice/binging failure (Windows, release) cpp/Ice/binding failure (Windows, release) Feb 6, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants