search

cetic / helm-fadi

Helm Chart for FADI
https://github.com/cetic/fadi
Apache License 2.0
13 stars 4 forks source link

Update pg-ldap-sync cron job frequency #32

Closed fzalila closed 3 years ago

fzalila commented 3 years ago

According to FADI newcomers feedback, the actual pg-ldap-sync cron job frequency (every 1 minute) is considered as a barrier to install FADI for the first time. They are usually facing a large list of failing and/or pending pods. Therefore, I propose this PR to reduce the cron job frequency.

banzo commented 3 years ago

lgtm, please bump the version in Chart.yml

alexnuttinck commented 3 years ago

Is it not better to change the pullPolicy of the cron?

alexnuttinck commented 3 years ago

Ok my bad, it is another subject .. lgtm 🙂

alexnuttinck commented 3 years ago

An improvement maybe: use a Helm Hook to be sure that LDAP and postgresql are started before launching the cron.