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

Add docs: ssl mode must be used when connecting to postgre as currently configured. #49

Open
JoeHCQ1 opened this issue Jul 9, 2024 · 3 comments
Labels
good first issue Good for newcomers tech-debt 💳 Debt that the team has charged and needs to repay

Comments

@JoeHCQ1
Copy link

JoeHCQ1 commented Jul 9, 2024

Describe what should be investigated or refactored

Documentation should be improved to include instruction to use SSL when connecting to postgres. If you don't the pg_conf will drop your connection.

Note, you can view pg_conf by shelling into the postgres container and then catting the pg related yaml.

Links to any relevant code

You can see here how I had to instruct Harbor to use ssl when connecting to postgre. Until it did that, I received back a fatal connection error.

You can see this Harbor setting here: https://github.com/goharbor/harbor-helm/blob/main/values.yaml#L943

@JoeHCQ1 JoeHCQ1 added the tech-debt 💳 Debt that the team has charged and needs to repay label Jul 9, 2024
@Racer159
Copy link
Contributor

Racer159 commented Jul 9, 2024

Looks like there may have been two links? do you have the one to your harbor package?

@Racer159
Copy link
Contributor

Racer159 commented Jul 9, 2024

I think this is a decent good first issue - we have been following a pattern of docs/configuration.md and one could be started for this repo with a note about that config - example for GitLab: https://github.com/defenseunicorns/uds-package-gitlab/blob/main/docs/configuration.md

@Racer159 Racer159 added the good first issue Good for newcomers label Jul 9, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
good first issue Good for newcomers tech-debt 💳 Debt that the team has charged and needs to repay
Projects
None yet
Development

No branches or pull requests

2 participants