Skip to content

Commit 2977d3e

Browse files
committed
Add support for locale and encoding, fix #406
1 parent 83c71ac commit 2977d3e

File tree

3 files changed

+37
-2
lines changed

3 files changed

+37
-2
lines changed

src/root/usr/share/container-scripts/postgresql/README.md

+7
Original file line numberDiff line numberDiff line change
@@ -74,6 +74,13 @@ initialization by passing `-e VAR=VALUE` to the Docker run command.
7474
**`POSTGRESQL_ADMIN_PASSWORD`**
7575
Password for the `postgres` admin account (optional)
7676

77+
The following environment variables are optional, and only used when the database is initialzed
78+
79+
**`POSTGRESQL_ENCODING`**
80+
Database encoding. Default to UTF8
81+
82+
**`POSTGRESQL_LOCALE`**
83+
Database locale. Default to en_US
7784

7885
Alternatively, the following options are related to migration scenario:
7986

src/root/usr/share/container-scripts/postgresql/common.sh

+13-2
Original file line numberDiff line numberDiff line change
@@ -190,7 +190,9 @@ initdb_wrapper ()
190190
# Initialize the database cluster with utf8 support enabled by default.
191191
# This might affect performance, see:
192192
# http://www.postgresql.org/docs/{{ spec.version }}/static/locale.html
193-
LANG=${LANG:-en_US.utf8} "$@"
193+
ENCODING=${POSTGRESQL_ENCODING:-UTF8}
194+
LOCALE=${POSTGRESQL_LOCALE:-en_US}
195+
LANG=${LANG:-$LOCALE.$ENCODING} "$@" -E $ENCODING
194196
}
195197

196198
function initialize_database() {
@@ -223,7 +225,16 @@ EOF
223225
function create_users() {
224226
if [[ ",$postinitdb_actions," = *,simple_db,* ]]; then
225227
createuser "$POSTGRESQL_USER"
226-
createdb --owner="$POSTGRESQL_USER" "$POSTGRESQL_DATABASE"
228+
229+
EXTRA_ARGS=""
230+
if [ -v POSTGRESQL_ENCODING ]; then
231+
EXTRA_ARGS="$EXTRA_ARGS -E $POSTGRESQL_ENCODING"
232+
fi
233+
if [ -v POSTGRESQL_LOCALE ]; then
234+
EXTRA_ARGS="$EXTRA_ARGS -l $POSTGRESQL_LOCALE"
235+
fi
236+
237+
createdb $EXTRA_ARGS --owner="$POSTGRESQL_USER" "$POSTGRESQL_DATABASE"
227238
fi
228239

229240
if [ -v POSTGRESQL_MASTER_USER ]; then

test/run_test

+17
Original file line numberDiff line numberDiff line change
@@ -29,6 +29,7 @@ run_s2i_enable_ssl_test
2929
run_upgrade_test
3030
run_migration_test
3131
run_pgaudit_test
32+
run_locales_test
3233
"
3334

3435
test $# -eq 1 -a "${1-}" == --list && echo "$TEST_LIST" && exit 0
@@ -898,6 +899,22 @@ EOSQL"
898899
grep -E 'AUDIT: SESSION,.*,.*,READ,SELECT,,,SELECT' "${data_dir}"/userdata/log/postgresql-*.log
899900
}
900901

902+
function run_locales_test() {
903+
local name=pg-test-locales
904+
DOCKER_ARGS="-e POSTGRESQL_ENCODING=ISO_8859_6 -e POSTGRESQL_LOCALE=C" create_container $name
905+
wait_ready "$name"
906+
907+
docker exec -i $(get_cid "$name") bash -c "psql <<EOSQL
908+
SHOW SERVER_ENCODING;
909+
EOSQL" | grep ISO_8859_6
910+
911+
docker exec -i $(get_cid "$name") bash -c "psql <<EOSQL
912+
SHOW LC_COLLATE;
913+
EOSQL" | grep -vi LC_COLLA | grep C
914+
915+
docker stop "$(get_cid "$name")"
916+
}
917+
901918
function run_all_tests() {
902919
for test_case in $TEST_LIST; do
903920
: "Running test $test_case"

0 commit comments

Comments
 (0)