Skip to content

chore: update database service port configuration in docker-compose-[next].yml#1768

Merged
gustavobtflores merged 1 commit intokernelci:mainfrom
gustavobtflores:chore/dynamic-port-db-docker
Mar 2, 2026
Merged

chore: update database service port configuration in docker-compose-[next].yml#1768
gustavobtflores merged 1 commit intokernelci:mainfrom
gustavobtflores:chore/dynamic-port-db-docker

Conversation

@gustavobtflores
Copy link
Copy Markdown
Contributor

@gustavobtflores gustavobtflores commented Mar 2, 2026

Description

Makes the host-side database port configurable via an environment variable instead of being hardcoded, while preserving 5434 as the default.

Changes

  • Replace hardcoded 5434 host port with ${DB_PORT:-5434} in docker-compose.yml
  • Apply the same dynamic port configuration to docker-compose-next.yml

Closes #1769

@gustavobtflores gustavobtflores self-assigned this Mar 2, 2026
@gustavobtflores gustavobtflores added the CI/CD Most or all of the changes are about automated tests / Github's CI label Mar 2, 2026
Copy link
Copy Markdown
Collaborator

@MarceloRobert MarceloRobert left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@nuclearcat this might help with the port conflict, just remember to change the value then

@gustavobtflores gustavobtflores added this pull request to the merge queue Mar 2, 2026
Merged via the queue into kernelci:main with commit 868835d Mar 2, 2026
7 checks passed
@gustavobtflores gustavobtflores deleted the chore/dynamic-port-db-docker branch March 30, 2026 18:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CI/CD Most or all of the changes are about automated tests / Github's CI

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Make database host port configurable via environment variable

2 participants