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

[bitnami/postegresql-repmgr] Added missing postgresql environment variables #52254

Merged
merged 1 commit into from
Dec 4, 2023

Conversation

dcamarenaAZO
Copy link
Contributor

Description of the change

Added missing postgresql environment variables regarding connections to README.md

Benefits

Developers can see them without looking into the code.

Possible drawbacks

None

Applicable issues

Additional information

POSTGRESQL_EXTRA_FLAGS is ignored and it is not possible to pass start parameters to PostgreSQL server.

connections to README.md. Defa

Signed-off-by: Daniel Camarena <daniel.camarena@azo.com>
@github-actions github-actions bot added the triage Triage is needed label Oct 31, 2023
@carrodher carrodher added the verify Execute verification workflow for these changes label Nov 2, 2023
@github-actions github-actions bot added in-progress and removed triage Triage is needed labels Nov 2, 2023
@bitnami-bot bitnami-bot removed the request for review from javsalgar November 2, 2023 08:33
@Mauraza
Copy link
Contributor

Mauraza commented Nov 7, 2023

Hi @dcamarenaAZO,

Thanks for the contribution! To add environment variables, you need to add them in all postgresql-env.sh.
Could you update this PR with those changes?

@dcamarenaAZO
Copy link
Contributor Author

dcamarenaAZO commented Nov 8, 2023

Hi @dcamarenaAZO,

Thanks for the contribution! To add environment variables, you need to add them in all postgresql-env.sh. Could you update this PR with those changes?

Hi @Mauraza,

thank you for your answer. Did you have a look at version 11 postgresql-env.sh?
In version-12, 13, 14, 15 and 16 they are also present.

As I stated in the PR, the variables were just missing in the documentation of bitnami/postegresql-repmgr. They are already in the postgresql-env.sh. Technically everything worked. It was just a lack of documentation.

Thanks for your effort. Let me know if you need anything else.

Another question: I'm now using version 15.4. I just saw that version 16 of bitnami/postgresql-repmgr was released on docker hub. Can you tell me where I can find the release notes, changelog or something to know about changes or compatibility from 15.4 to 16?

Thanks, Cheers, Daniel

@Mauraza
Copy link
Contributor

Mauraza commented Nov 9, 2023

Hi @dcamarenaAZO,

You are right. The env variables are already included, and the configuration is here:

postgresql_configure_connections() {
[[ -n "$POSTGRESQL_MAX_CONNECTIONS" ]] && postgresql_set_property "max_connections" "$POSTGRESQL_MAX_CONNECTIONS"
[[ -n "$POSTGRESQL_TCP_KEEPALIVES_IDLE" ]] && postgresql_set_property "tcp_keepalives_idle" "$POSTGRESQL_TCP_KEEPALIVES_IDLE"
[[ -n "$POSTGRESQL_TCP_KEEPALIVES_INTERVAL" ]] && postgresql_set_property "tcp_keepalives_interval" "$POSTGRESQL_TCP_KEEPALIVES_INTERVAL"
[[ -n "$POSTGRESQL_TCP_KEEPALIVES_COUNT" ]] && postgresql_set_property "tcp_keepalives_count" "$POSTGRESQL_TCP_KEEPALIVES_COUNT"
([[ -n "$POSTGRESQL_STATEMENT_TIMEOUT" ]] && postgresql_set_property "statement_timeout" "$POSTGRESQL_STATEMENT_TIMEOUT") || true
}

Another question: I'm now using version 15.4. I just saw that version 16 of bitnami/postgresql-repmgr was released on docker hub. Can you tell me where I can find the release notes, changelog or something to know about changes or compatibility from 15.4 to 16?

You can check the official documentation https://www.postgresql.org/docs/16/upgrading.html

Copy link

This Pull Request has been automatically marked as "stale" because it has not had recent activity (for 15 days). It will be closed if no further activity occurs. Thank you for your contribution.

@github-actions github-actions bot added the stale 15 days without activity label Nov 25, 2023
@dcamarenaAZO
Copy link
Contributor Author

The pull request is just adding some missing environment variables to the documentation.

@github-actions github-actions bot removed the stale 15 days without activity label Nov 29, 2023
Copy link
Contributor

@Mauraza Mauraza left a comment

Choose a reason for hiding this comment

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

Hi @dcamarenaAZO,

Thanks for the contribution! LGTM!!

@Mauraza Mauraza merged commit 831df63 into bitnami:main Dec 4, 2023
13 checks passed
christian-scheible pushed a commit to christian-scheible/containers that referenced this pull request Dec 7, 2023
…iables (bitnami#52254)

Added postgresql environment variables regarding
connections to README.md. Defa

Signed-off-by: Daniel Camarena <daniel.camarena@azo.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
postgresql-repmgr solved verify Execute verification workflow for these changes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants