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)

Reply via email to