PostgresNode::release_resources(self) is added

postgrespro · dmitry-lipetsk · Jun 30, 2025 · Jun 26, 2025 · Jun 26, 2025 · Jun 26, 2025
commit 0547b0b92db8efb7e5dba97479a21a99d187ed19
diff --git a/testgres/node.py b/testgres/node.py
@@ -690,9 +690,6 @@ def _try_shutdown(self, max_attempts, with_force=False):
             ps_output,
             ps_command)
 
-    def _release_resources(self):
-        self.free_port()
-
     @staticmethod
     def _throw_bugcheck__unexpected_result_of_ps(result, cmd):
         assert type(result) == str  # noqa: E721
@@ -1324,25 +1321,18 @@ def pg_ctl(self, params):
 
         return execute_utility2(self.os_ops, _params, self.utils_log_file)
 
+    def release_resources(self):
+        """
+        Release resorces owned by this node.
+        """
+        return self._release_resources()
+
     def free_port(self):
         """
         Reclaim port owned by this node.
         NOTE: this method does not release manually defined port but reset it.
         """
-        assert type(self._should_free_port) == bool  # noqa: E721
-
-        if not self._should_free_port:
-            self._port = None
-        else:
-            assert type(self._port) == int  # noqa: E721
-
-            assert self._port_manager is not None
-            assert isinstance(self._port_manager, PortManager)
-
-            port = self._port
-            self._should_free_port = False
-            self._port = None
-            self._port_manager.release_port(port)
+        return self._free_port()
 
     def cleanup(self, max_attempts=3, full=False, release_resources=False):
         """
@@ -2156,6 +2146,25 @@ def upgrade_from(self, old_node, options=None, expect_error=False):
 
         return self.os_ops.exec_command(upgrade_command, expect_error=expect_error)
 
+    def _release_resources(self):
+        self._free_port()
+
+    def _free_port(self):
+        assert type(self._should_free_port) == bool  # noqa: E721
+
+        if not self._should_free_port:
+            self._port = None
+        else:
+            assert type(self._port) == int  # noqa: E721
+
+            assert self._port_manager is not None
+            assert isinstance(self._port_manager, PortManager)
+
+            port = self._port
+            self._should_free_port = False
+            self._port = None
+            self._port_manager.release_port(port)
+
     def _get_bin_path(self, filename):
         assert self._os_ops is not None
         assert isinstance(self._os_ops, OsOperations)