Skip to content

Fix: correct PG port configuration in pgversion.yml#663

Merged
estebanzimanyi merged 1 commit intoMobilityDB:masterfrom
ImreSamu:master
Mar 18, 2025
Merged

Fix: correct PG port configuration in pgversion.yml#663
estebanzimanyi merged 1 commit intoMobilityDB:masterfrom
ImreSamu:master

Conversation

@ImreSamu
Copy link
Copy Markdown
Contributor

  • Removes any existing PostgreSQL installations
  • Sets a consistent port (PGPORT=5432)
  • Installs PostgreSQL from the official repository
  • Uses the standard port for all PostgreSQL operations

Problem:

  • the PostgreSQL service was started before configuring its port. Once started, changing the PGPORT environment variable doesn't affect which port the server listens on.

My proposal:

  • Removing existing PostgreSQL installations: So no conflicts between Ubuntu's default PostgreSQL and the version from PostgreSQL's official repository.
  • Simplifying port configuration: Set a fixed port (PGPORT=5432) which is PostgreSQL's default port, eliminating the conditional logic that was causing issues.

Additional Notes:

This update specifically addresses the port configuration issue and does not resolve all existing problems.
Expected workflow outcomes:

  • PG 16 and 17: Expected to work correctly.
  • PG 13, 14, and 15: 070_tpoint_spatialrels_tbl (Failed)
  • PG 17 (coverage=1): May show the error: geninfo: ERROR: 'include' pattern '*/MobilityDB/meos/src/*' is unused.

-  Removes any existing PostgreSQL installations
- Sets a consistent port (PGPORT=5432)
- Installs PostgreSQL from the official repository
- Uses the standard port for all PostgreSQL operations
@estebanzimanyi estebanzimanyi self-requested a review March 18, 2025 07:21
@estebanzimanyi estebanzimanyi merged commit 74a4268 into MobilityDB:master Mar 18, 2025
5 of 9 checks passed
mschoema pushed a commit to mschoema/MobilityDB that referenced this pull request Sep 30, 2025
-  Removes any existing PostgreSQL installations
- Sets a consistent port (PGPORT=5432)
- Installs PostgreSQL from the official repository
- Uses the standard port for all PostgreSQL operations
mschoema added a commit that referenced this pull request Oct 2, 2025
-  Removes any existing PostgreSQL installations
- Sets a consistent port (PGPORT=5432)
- Installs PostgreSQL from the official repository
- Uses the standard port for all PostgreSQL operations

Cherry-pick of 74a4268

Co-authored-by: ImreSamu <ImreSamu@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants