This is an automated email from the ASF dual-hosted git repository.

apolovtsev 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 2b2c8d75fb0 IGNITE-26227 Fix server shutdown in case of an error 
(#6441)
2b2c8d75fb0 is described below

commit 2b2c8d75fb0414559718bd0f6a715e8eac23f209
Author: Ivan Zlenko <[email protected]>
AuthorDate: Mon Aug 18 16:24:42 2025 +0400

    IGNITE-26227 Fix server shutdown in case of an error (#6441)
---
 .../ItNodeBootstrapConfigurationTest.java          | 41 ++++++++++++----------
 1 file changed, 22 insertions(+), 19 deletions(-)

diff --git 
a/modules/runner/src/integrationTest/java/org/apache/ignite/internal/configuration/ItNodeBootstrapConfigurationTest.java
 
b/modules/runner/src/integrationTest/java/org/apache/ignite/internal/configuration/ItNodeBootstrapConfigurationTest.java
index 6ca5aa8fb5e..158cfa9931d 100644
--- 
a/modules/runner/src/integrationTest/java/org/apache/ignite/internal/configuration/ItNodeBootstrapConfigurationTest.java
+++ 
b/modules/runner/src/integrationTest/java/org/apache/ignite/internal/configuration/ItNodeBootstrapConfigurationTest.java
@@ -128,31 +128,34 @@ public class ItNodeBootstrapConfigurationTest {
                 + "}";
 
         IgniteServer igniteServer = 
TestIgnitionManager.startWithProductionDefaults(testNodeName(testInfo, 0), 
config, workDir);
-        
igniteServer.initCluster(InitParameters.builder().clusterName("name").build());
-        Ignite ignite = igniteServer.api();
 
-        Path configFile = 
workDir.resolve(TestIgnitionManager.DEFAULT_CONFIG_NAME);
-        String storedConfig = Files.readString(configFile);
-        Config hoconConfig = ConfigFactory.parseString(config);
-        assertThat(ConfigFactory.parseString(storedConfig), is(hoconConfig));
+        try {
+            
igniteServer.initCluster(InitParameters.builder().clusterName("name").build());
+            Ignite ignite = igniteServer.api();
 
-        IgniteImpl igniteImpl = TestWrappers.unwrapIgniteImpl(ignite);
-        ConfigurationRegistry nodeCfgRegistry = igniteImpl.nodeConfiguration();
-        HoconPresentation hoconPresentation = new 
HoconPresentation(nodeCfgRegistry);
+            Path configFile = 
workDir.resolve(TestIgnitionManager.DEFAULT_CONFIG_NAME);
+            String storedConfig = Files.readString(configFile);
+            Config hoconConfig = ConfigFactory.parseString(config);
+            assertThat(ConfigFactory.parseString(storedConfig), 
is(hoconConfig));
 
-        String configUpdate = "ignite {\n"
-                + "  rest: {\n"
-                + "    httpToHttpsRedirection: true\n"
-                + "  }\n"
-                + "}\n";
+            IgniteImpl igniteImpl = TestWrappers.unwrapIgniteImpl(ignite);
+            ConfigurationRegistry nodeCfgRegistry = 
igniteImpl.nodeConfiguration();
+            HoconPresentation hoconPresentation = new 
HoconPresentation(nodeCfgRegistry);
 
-        CompletableFuture<Void> update = 
hoconPresentation.update(configUpdate);
+            String configUpdate = "ignite {\n"
+                    + "  rest: {\n"
+                    + "    httpToHttpsRedirection: true\n"
+                    + "  }\n"
+                    + "}\n";
 
-        assertThat(update, willCompleteSuccessfully());
-        storedConfig = Files.readString(configFile);
+            CompletableFuture<Void> update = 
hoconPresentation.update(configUpdate);
 
-        assertThat(ConfigFactory.parseString(storedConfig), 
is(hoconConfig.withFallback(ConfigFactory.parseString(configUpdate))));
+            assertThat(update, willCompleteSuccessfully());
+            storedConfig = Files.readString(configFile);
 
-        igniteServer.shutdown();
+            assertThat(ConfigFactory.parseString(storedConfig), 
is(hoconConfig.withFallback(ConfigFactory.parseString(configUpdate))));
+        } finally {
+            igniteServer.shutdown();
+        }
     }
 }

Reply via email to