robjuz / helm-charts

https://robjuz.github.io/helm-charts/index.yaml
34 stars 30 forks source link

Added option to specify threads in update job #65

Closed dbt-lucka closed 7 months ago

dbt-lucka commented 8 months ago

The --threads parameter on nominatiom replication helps us to drastically speed up the replication process on multi-cpu machines. Hence, I also added an option to customizie the update-job hardware resources on the cluster.

robjuz commented 8 months ago

This looks solid. Thx for the PR

dbt-lucka commented 8 months ago

🛑 please don't merge this. Just figured that nominatim replication --check-for-updates is returning an Exit Code 2 when there are no further updates. This causes the POD to go into a crashloop.

dbt-lucka commented 8 months ago

Removed nominatim replication --check-for-updates and it works again.

Using project directory: /nominatim
Sleeping for 18835 sec before next update.
dbt-lucka commented 7 months ago

Hi @robjuz

after reverting the check-for-updates option everything worked. I think this could be merged now.

robjuz commented 7 months ago

Thx for the contribution!