Skip to content
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.

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

Unify and differentiate default values for ports #341

Closed
ckunki opened this issue Jun 7, 2023 · 2 comments · Fixed by #354
Closed

Unify and differentiate default values for ports #341

ckunki opened this issue Jun 7, 2023 · 2 comments · Fixed by #354
Assignees
Labels
refactoring Code improvement without behavior change

Comments

@ckunki
Copy link
Contributor

ckunki commented Jun 7, 2023

ITDE implementation specifies port numbers in multiple places.
Default Ports depend on version of Exasol database and deployment variant.
The current ticket requests to harmonize the default values, documentation and variants in a uniform way.
Maybe class PortForwarding can be used to grab all variants and provide a uniform access to them.

Default Port Earlier versions Later versions
For database in Docker Container 8888 8563
For database w/o Docker Container 8563 8563
For Bucketfs 6583 2580

See also file cli/options/test_environment_options.py

This should be respected by ITDE implementation but should be released with incremented major version.

@ckunki ckunki added the refactoring Code improvement without behavior change label Jun 7, 2023
@ckunki ckunki changed the title Unify default values for ports Unify and differentiate default values for ports Jun 7, 2023
@tkilias tkilias mentioned this issue Jun 7, 2023
7 tasks
@ckunki
Copy link
Contributor Author

ckunki commented Jun 9, 2023

This ticket goes beyond #340 and is not in conflict with the changes describes.

@ckunki
Copy link
Contributor Author

ckunki commented Jul 4, 2023

Efforts are recorded in ticket #308

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
refactoring Code improvement without behavior change
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant