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

PostgreSQL OpenSSL build #10

Open
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

Chanakya-TS
Copy link

@Chanakya-TS Chanakya-TS commented Jun 10, 2024

Building PostgreSQL with OpenSSL to allow connections to secure databases.

Fixes: google#187

Description

Updates source files and defines in postgresql.BUILD.
Updates docker images to have necessary libraries installed.

How Has This Been Tested?

Used model registry operator to initiate a PostgreSQL secure database server and ran MLMD postgres test suite (All 7 tests pass)

Merge criteria:

  • The commits are squashed in a cohesive manner and have meaningful messages.
  • Testing instructions have been added in the PR body (for PRs involving changes that are not immediately obvious).
  • The developer has manually tested the changes and verified that the changes work

changed defines

more cleanup

more updates

updated dockerfiles to allow postgre build

removed testing artifcats

removed more testing artificats
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.

Using ML metadata postgres SSL connection options don't work for sslMode=require
1 participant