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

feat(helm): configure MIW Helm chart to use pgadmin4 subchart #189

Closed
wants to merge 2 commits into from
Closed

feat(helm): configure MIW Helm chart to use pgadmin4 subchart #189

wants to merge 2 commits into from

Conversation

pmoscode
Copy link
Contributor

@pmoscode pmoscode commented Dec 6, 2023

Description

This PR configures the pgadmin Helm chart for usage inside the MIW Helm chart as part of the series of PRs to implement issue #147

Pgadmin is disabled by default, so it has to be enabled / configured in the values.yaml.
A default user / password is set. (The pgadmin chart itself sets also a default password, which is overwritten here)

Beside that, the MIW Postgresql DB is preconfigured and can be instantly used. You just have to provide the password for the DB.
This can be addressed in a new issue / PR.

Pre-review checks

Please ensure to do as many of the following checks as possible, before asking for committer review:

...until everything is complete and/ or it is enabled on demand later on then.
Copy link

sonarqubecloud bot commented Dec 6, 2023

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
No Duplication information No Duplication information

@pmoscode
Copy link
Contributor Author

pmoscode commented Dec 6, 2023

These errors are due to the need to split up the original PR to avoid IP check.
Hence, the PRs are additive and these errors will be fixed automatically with the previous PR completing the missing things.

I.e. please merge the PR anyway and let the previous one be merged timely, too.

@pmoscode pmoscode marked this pull request as ready for review December 6, 2023 09:44
@pmoscode pmoscode closed this Dec 14, 2023
@pmoscode pmoscode deleted the configure-pgadmin-chart branch December 14, 2023 11:31
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