This is an automated email from the ASF dual-hosted git repository.
ptupitsyn pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/ignite-3.git
The following commit(s) were added to refs/heads/main by this push:
new 7a38a221ab IGNITE-20962 .NET: Fix TestAuthnOnClientNoAuthnOnServer
flakiness (#2880)
7a38a221ab is described below
commit 7a38a221abb39418ed9925d1c6f8185dfbe4300c
Author: Pavel Tupitsyn <[email protected]>
AuthorDate: Mon Nov 27 13:33:24 2023 +0200
IGNITE-20962 .NET: Fix TestAuthnOnClientNoAuthnOnServer flakiness (#2880)
Ensure that all servers have applied the configuration change, not only one.
---
.../dotnet/Apache.Ignite.Tests/BasicAuthenticatorTests.cs | 10 ++++++++--
1 file changed, 8 insertions(+), 2 deletions(-)
diff --git
a/modules/platforms/dotnet/Apache.Ignite.Tests/BasicAuthenticatorTests.cs
b/modules/platforms/dotnet/Apache.Ignite.Tests/BasicAuthenticatorTests.cs
index b6136ebb54..82fe28c9ec 100644
--- a/modules/platforms/dotnet/Apache.Ignite.Tests/BasicAuthenticatorTests.cs
+++ b/modules/platforms/dotnet/Apache.Ignite.Tests/BasicAuthenticatorTests.cs
@@ -29,7 +29,7 @@ public class BasicAuthenticatorTests : IgniteTestsBase
{
private const string EnableAuthnJob =
"org.apache.ignite.internal.runner.app.PlatformTestNodeRunner$EnableAuthenticationJob";
- private bool _authnEnabled;
+ private volatile bool _authnEnabled;
[TearDown]
public async Task DisableAuthenticationAfterTest() => await
EnableAuthn(false);
@@ -118,7 +118,13 @@ public class BasicAuthenticatorTests : IgniteTestsBase
{
try
{
- await Client.Tables.GetTablesAsync();
+ // Ensure that all servers have applied the configuration
change.
+ foreach (var endpoint in GetConfig().Endpoints)
+ {
+ var cfg = new IgniteClientConfiguration(endpoint);
+ using var client2 = await IgniteClient.StartAsync(cfg);
+ }
+
return true;
}
catch (Exception)