Skip to content

Commit

Permalink
build: Force deployments on config change (#26)
Browse files Browse the repository at this point in the history
* fix: removed duplicated storages

* fix: merge local and azure volume decorations

* fix: merge local and aws and azure volume decorations

* fix: bad jinja template typo

* fix: bad jinja template typo AGAIN

* fix: use old azure volume sizes as can't shrink exisiting ones

* fix: restore RWO for aws

* fix: don't change the provisioner

* fix: correct ansible ternary syntax

* chore: tidy ordering, restore non-azure sizings

* feat: add checksum for CKAN ini file

* feat: add checksum for CKAN ini file

* fix: correct jinja syntax
  • Loading branch information
cooper667 authored Aug 19, 2024
1 parent 1273e7c commit 090e057
Showing 1 changed file with 4 additions and 9 deletions.
13 changes: 4 additions & 9 deletions roles/ckan/templates/kubernetes/ckan.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -48,6 +48,8 @@ spec:
maxSurge: 2
template:
metadata:
annotations:
checksum/config: "{{ lookup('template', 'templates/ckan/ckan_production.ini') | hash('md5') }}" ## This forces a rolling deployment if the config changes
creationTimestamp: null
labels:
app: ckan
Expand Down Expand Up @@ -152,13 +154,7 @@ spec:
value: "/usr/lib/ckan"
{% endif %}
image: {{ ckan_image }}:{{ ckan_image_tag }}


{% if fjelltopp_env_type == 'local' %}
imagePullPolicy: IfNotPresent
{% else %}
imagePullPolicy: Always
{% endif %}
imagePullPolicy: {{ (fjelltopp_env_type == 'local') | ternary('IfNotPresent', 'Always') }}
name: ckan

# assume local user uid
Expand All @@ -175,7 +171,7 @@ spec:
httpGet:
path: /
port: 5000
initialDelaySeconds: 30
initialDelaySeconds: 60
periodSeconds: 5
timeoutSeconds: 10
failureThreshold: 15
Expand All @@ -201,7 +197,6 @@ spec:
- mountPath: /etc/ckan
name: ckan-configs
readOnly: true

{% if fjelltopp_env_type == 'local' %}
- mountPath: /usr/lib/ckan
name: localckan
Expand Down

0 comments on commit 090e057

Please sign in to comment.