-
Notifications
You must be signed in to change notification settings - Fork 575
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
Give the possibility to pass extensions options to the config template #202
base: master
Are you sure you want to change the base?
Conversation
Looks like test failing due to missing defaults ? |
4235fc5
to
97f587e
Compare
@otakup0pe yes, sorry. Just fixed that. |
@@ -683,3 +683,6 @@ postgresql_pgdg_dists: | |||
postgresql_version_terse: "{{ postgresql_version | replace('.', '') }}" | |||
postgresql_yum_repository_base_url: "http://yum.postgresql.org" | |||
postgresql_yum_repository_url: "{{ postgresql_yum_repository_base_url }}/{{ postgresql_version }}/{{ ansible_os_family | lower }}/rhel-{{ ansible_distribution_major_version }}-{{ ansible_architecture }}/pgdg-{{ postgresql_pgdg_dists[ansible_distribution] }}{{ postgresql_version_terse }}-{{ postgresql_version }}-2.noarch.rpm" | |||
|
|||
# List of options from loaded extensions (optional) | |||
postgresql_extensions_options: [] |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
What is an example of this format? It looks like a free-form way of injecting config entries?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Right. My use-case is the following, I am using the pg_stat_statements
extension and I would like to be able to add pg_stat_statements.track = all
in the config.
Does this sound good to you or would you prefer passing something like { key: 'pg_stat_statements.track', value: 'all' }
?
97f587e
to
deb8066
Compare
I have updated my PR with a dictionary and an example of how to use it in the README. |
deb8066
to
291d5d2
Compare
This pr has been marked 'stale' due to lack of recent activity. If there is no further activity, the issue will be closed in another 30 days. Thank you for your contribution! |
No description provided.