Skip to content

Conversation

@boozallendanny
Copy link
Contributor

Description:

Technical Details:

Requirements for PR Merge:

  1. Unit & integration tests updated
  2. API documentation updated (examples listed below)
    1. API Contracts
    2. API UI
    3. Comments
  3. Data validation completed (examples listed below)
    1. Does this work well with the current frontend? Or is the frontend aware of a needed change?
    2. Is performance impacted in the changes (e.g., API, pipeline, downloads, etc.)?
    3. Is the expected data returned with the expected format?
  4. Appropriate Operations ticket(s) created
  5. Jira Ticket(s)
    1. DEV-0

Explain N/A in above checklist:

sethstoudenmier and others added 7 commits November 7, 2025 15:33
…emporary-update-for-spark-downloads

[DEV-13706] Use delta location to support EMR cutover
…ilter-by-prefix-on-s3-retrieval

[DEV-13988] Limit the S3 objects we check during download zip process
…ilter-by-prefix-on-s3-retrieval

Promote hotfix to master
Comment on lines +14 to +19
RUN apt-get update && apt-get install -y wget gnupg lsb-release \
&& wget -qO - https://www.postgresql.org/media/keys/ACCC4CF8.asc | apt-key add - \
&& echo "deb http://apt.postgresql.org/pub/repos/apt/ $(lsb_release -cs)-pgdg main" \
> /etc/apt/sources.list.d/pgdg.list \
&& apt-get update \
&& apt-get install -y postgresql-16
Copy link
Contributor

Choose a reason for hiding this comment

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

What is the benefit of installing this way vs updating the existing pattern to apt install postgresql-16?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

That was the first thing I tested. It seems the base image does not come with postgresql-16 package

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Copy link
Contributor

Choose a reason for hiding this comment

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

Looks like the postgresql-16 package is only available in Debian's unstable branch, so installing it from Postgres' repo is the easiest option.

@zachflanders-frb
Copy link
Contributor

zachflanders-frb commented Dec 12, 2025

Are we going straight to the master branch with this change? (vs. qat->staging->master.)

@zachflanders-frb zachflanders-frb changed the base branch from master to qat December 17, 2025 16:00
@zachflanders-frb zachflanders-frb dismissed their stale review December 17, 2025 16:00

The base branch was changed.

@zachflanders-frb zachflanders-frb merged commit 581da18 into qat Dec 17, 2025
19 checks passed
@zachflanders-frb zachflanders-frb deleted the mod/postgresql16 branch December 17, 2025 17:07
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.

6 participants