Skip to content

add db_extra_options parameter - #1195

Open
dmaes wants to merge 1 commit into
theforeman:masterfrom
dmaes:db_extra_options
Open

add db_extra_options parameter#1195
dmaes wants to merge 1 commit into
theforeman:masterfrom
dmaes:db_extra_options

Conversation

@dmaes

@dmaes dmaes commented Nov 13, 2024

Copy link
Copy Markdown
Contributor

This PR adds a db_extra_options parameter, to allow adding custom parameters in the database.yml config.

Fixes #1194

@dmaes

dmaes commented Jan 7, 2025

Copy link
Copy Markdown
Contributor Author

@ekohl @evgeni anything still missing to get this reviewed and merged?

@dmaes
dmaes force-pushed the db_extra_options branch from cbd76b2 to 98ed32c Compare January 9, 2025 14:19

@ekohl ekohl left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

I was a bit slow getting back from the holiday break.

Comment thread templates/database.yml.epp Outdated
Comment thread templates/database.yml.epp Outdated
Comment thread manifests/init.pp Outdated
@dmaes

dmaes commented Jan 20, 2025

Copy link
Copy Markdown
Contributor Author

@ekohl thanks for the feedback, I implemented the suggestions

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Allow more database.yml settings

3 participants