[bitnami/postgresql-repmgr] Support running scripts on primary node when container starts

[sawntoe]

Revival of #70737

Description of the change

This adds the ability to run scripts on startup always, a feature that is present in other containers such as mariadb with the /docker-entrypoint-startdb.d endpoint

Benefits

It is currently a huge pain in the behind to run scripts at startup everytime the container starts, requiring an undocumented change to the entrypoint for which there is no information anywhere

Applicable issues

• fixes [bitnami/postgresql-repmgr] Running custom script on primary database when container starting #70606

[fmulero]

Great! thanks a lot @sawntoe for your contribution! I appreciate it a lot.

I've just left a comment, Could you give it a glance?

[sawntoe]

Hi @fmulero, so sorry, I am unable to see any comments on the code, may I ask if its still a draft? Thanks so much!

[fmulero]

Sorry, I didn't add the comment to the review. My failure.

[fmulero]

This is almost the same code as in postgresql_custom_init_scripts:

containers/bitnami/postgresql/17/debian-12/rootfs/opt/bitnami/scripts/libpostgresql.sh

Line 784 in da31f8d

postgresql_custom_init_scripts() {

To avoid duplication, could you follow the same approach we've used in mariadb?

containers/bitnami/mariadb/11.4/debian-12/rootfs/opt/bitnami/scripts/libmariadb.sh

Lines 381 to 390 in da31f8d

	########################
	# Run custom scripts
	# Globals:
	# DB_*
	# Arguments:
	# $1 - 'init' or 'start' ('init' runs on first container start, 'start' runs everytime the container starts)
	# Returns:
	# None
	#########################
	mysql_custom_scripts() {

[sawntoe]

Hi, so sorry for the holdup, i've been quite busy. Just a question regarding the handling of scripts:
Should preinit be added to the custom scripts function or should that be a seperate PR?

[fmulero]

Hi, so sorry for the holdup, i've been quite busy. Just a question regarding the handling of scripts: Should preinit be added to the custom scripts function or should that be a seperate PR?

Good question. I think that could be done in a different PR, that function only supports sh files.