zalando / spilo

Highly available elephant herd: HA PostgreSQL cluster using Docker
Apache License 2.0
1.52k stars 374 forks source link

Add errorhandling for ulimit #943

Closed glemsom closed 5 months ago

glemsom commented 8 months ago

In case the container is not spawned with SYS_RESOURCE, setting unlimited size for coredumps will never work. Since the defaults in the script is to abort on any uncaught errors - I propose we catch the ulimit error, and notify the user about the possible missing capability.

This fixes issues like https://github.com/zalando/postgres-operator/issues/2387

hughcapet commented 5 months ago

I would abstain from adding any specific configuration suggestions to the logging here but let's add the error handling anyway. Thanks.

idanovinda commented 5 months ago

👍

hughcapet commented 5 months ago

👍