Skip to content

PortManager #234

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.

Sign up for GitHub

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

Merged
merged 32 commits into from
Apr 6, 2025
Merged

PortManager #234

Changes from 1 commit
Commits
Show all changes
32 commits
Select commit Hold shift + click to select a range
09dab31
PostgresNode refactoring [PostgresNodePortManager and RO-properties]
dmitry-lipetsk Apr 3, 2025
ef095d3
[FIX] clone_with_new_name_and_base_dir did not respect port_manager
dmitry-lipetsk Apr 4, 2025
0f842fd
PostgresNodePortManager__ThisHost is defined (was: PostgresNodePortMa…
dmitry-lipetsk Apr 4, 2025
e1e609e
PostgresNodePortManager__Generic is added
dmitry-lipetsk Apr 4, 2025
285e5b7
PostgresNodePortManager is added in public API
dmitry-lipetsk Apr 4, 2025
a974866
Test structures were refactored (local, local2, remote)
dmitry-lipetsk Apr 4, 2025
110947d
CI files are updated
dmitry-lipetsk Apr 4, 2025
4f49dde
TestTestgresCommon.test_pgbench is added
dmitry-lipetsk Apr 4, 2025
4fbf51d
PostgresNodePortManager is updated [error messages]
dmitry-lipetsk Apr 4, 2025
17c73cb
PostgresNodePortManager(+company) was moved in own file - port_manage…
dmitry-lipetsk Apr 4, 2025
b1cee19
PortManager was deleted [amen]
dmitry-lipetsk Apr 4, 2025
c5ad907
PostgresNodePortManager was renamed with PortManager
dmitry-lipetsk Apr 4, 2025
0967057
TestTestgresCommon.test_unix_sockets is added
dmitry-lipetsk Apr 4, 2025
1d450b2
TestTestgresCommon.test_the_same_port is added
dmitry-lipetsk Apr 4, 2025
4a38b35
[TestTestgresCommon] New tests are added
dmitry-lipetsk Apr 4, 2025
322fb23
RemoteOperations::is_port_free is updated
dmitry-lipetsk Apr 4, 2025
94da63e
Tests for OsOps::is_port_free are added
dmitry-lipetsk Apr 4, 2025
88f9b73
TestTestgresCommon is corrected [python problems]
dmitry-lipetsk Apr 4, 2025
d9558ce
The call of RaiseError.CommandExecutionError is fixed [message, not m…
dmitry-lipetsk Apr 4, 2025
0da4c21
[CI] ubuntu 24.04 does not have nc
dmitry-lipetsk Apr 4, 2025
0058508
RemoteOperations is update [private method names]
dmitry-lipetsk Apr 5, 2025
8f3a566
test_is_port_free__true is updated
dmitry-lipetsk Apr 5, 2025
d8ebdb7
RemoteOperations::is_port_free is updated (comments)
dmitry-lipetsk Apr 5, 2025
30e472c
setup.py is updated [testgres.helpers was deleted]
dmitry-lipetsk Apr 5, 2025
c94bbb5
Comment in node.py is updated
dmitry-lipetsk Apr 5, 2025
04f88c7
PostgresNode::_node was deleted [use self._os_ops.host]
dmitry-lipetsk Apr 5, 2025
0a3442a
PostgresNode::start is corrected [error message]
dmitry-lipetsk Apr 5, 2025
30124f3
Merge branch 'master' into D20250403_001--port_manager
dmitry-lipetsk Apr 5, 2025
13e71d8
[FIX] PostgresNode.__init__ must not test "os_ops.host" attribute.
dmitry-lipetsk Apr 5, 2025
9e14f4a
PostgresNode.free_port always set a port to None
dmitry-lipetsk Apr 6, 2025
739ef61
[FIX] flake8 (noqa: E721)
dmitry-lipetsk Apr 6, 2025
696cc1e
PortManager__Generic is refactored
dmitry-lipetsk Apr 6, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Comment in node.py is updated
  • Loading branch information
@dmitry-lipetsk
dmitry-lipetsk committed Apr 5, 2025
commit c94bbb58e6a19a2ad6ba94897eacb4b40179bd46
2 changes: 1 addition & 1 deletion testgres/node.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -276,7 +276,7 @@ def _get_port_manager(os_ops: OsOperations) -> PortManager:
if isinstance(os_ops, LocalOperations):
return PortManager__ThisHost()

# TODO: Throw exception "Please define a port manager."
# TODO: Throw the exception "Please define a port manager." ?
return PortManager__Generic(os_ops)

def clone_with_new_name_and_base_dir(self, name: str, base_dir: str):
Expand Down