Skip to content

Commit d895383

Browse files
authored
Merge pull request #94 from kubealex/fix-jackett-probe
fix(jackett): readiness probe
2 parents 93df77f + 7ffce6d commit d895383

File tree

9 files changed

+18
-15
lines changed

9 files changed

+18
-15
lines changed

.github/workflows/integration-tests.yml

Lines changed: 3 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -48,7 +48,7 @@ jobs:
4848
cp k8s-mediaserver.yml k8s-mediaserver-operator.yml tests/
4949
sed -i 's/accessMode: ""/accessMode: ReadWriteOnce/g' tests/k8s-mediaserver.yml
5050
sed -i 's/pvcStorageClass: ""/pvcStorageClass: standard/g' tests/k8s-mediaserver.yml
51-
sed 's|quay.io\/kubealex.*|${{ env.image_name }}:${{ github.sha }}|g' tests/k8s-mediaserver-operator.yml
51+
sed -i 's|quay.io\/kubealex.*|${{ env.image_name }}:${{ github.sha }}|g' tests/k8s-mediaserver-operator.yml
5252
5353
- name: Kind Cluster
5454
uses: helm/[email protected]
@@ -76,15 +76,13 @@ jobs:
7676
kubectl apply -f tests/k8s-mediaserver.yml -n mediaserver
7777
sleep 15
7878
for pod in $(kubectl get pods -n mediaserver | awk 'NR>1{ print $1 }'); do while [[ $(kubectl get pods $pod -n mediaserver -o 'jsonpath={..status.conditions[?(@.type=="Ready")].status}') != "True" ]]; do echo "Waiting for mediaserver pods to be ready" && sleep 10; done ; done
79-
sleep 15
79+
sleep 30
8080
- name: Edit hosts file and test apps
8181
run: |
8282
sudo echo "127.0.0.1 ${{ env.plex_ingress_host }} ${{ env.ingress_host }}" | sudo tee -a /etc/hosts
83-
sudo echo "127.0.0.1 ${{ env.jellyfin_ingress_host }} ${{ env.ingress_host }}" | sudo tee -a /etc/hosts
84-
wget ${{ env.ingress_host }}/jackett
8583
wget ${{ env.ingress_host }}/sonarr
8684
wget ${{ env.ingress_host }}/radarr
8785
wget ${{ env.ingress_host }}/sabnzbd
8886
wget ${{ env.ingress_host }}/prowlarr
87+
wget --retry-on-http-error=503,500 ${{ env.ingress_host }}/jackett
8988
curl ${{ env.plex_ingress_host }}
90-
curl ${{ env.jellyfin_ingress_host }}
Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
11
apiVersion: v2
2-
appVersion: 0.9.1
2+
appVersion: 0.9.2
33
description: A Helm chart for Kubernetes mediaserver
44
name: k8s-mediaserver
55
type: application
6-
version: 0.9.1
6+
version: 0.9.2

helm-charts/k8s-mediaserver/templates/jackett-resources.yml

Lines changed: 2 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -72,10 +72,9 @@ spec:
7272
image: "{{ .Values.jackett.container.image }}:{{ .Values.jackett.container.tag | default .Values.general.image_tag }}"
7373
imagePullPolicy: Always
7474
readinessProbe:
75-
httpGet:
75+
tcpSocket:
7676
port: {{ .Values.jackett.container.port }}
77-
path: /{{ .Values.radarr.ingress.path }}/UI/Dashboard
78-
initialDelaySeconds: 5
77+
initialDelaySeconds: 15
7978
periodSeconds: 10
8079
ports:
8180
- name: jackett-port

helm-charts/k8s-mediaserver/templates/plex-resources.yml

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -57,6 +57,7 @@ spec:
5757
path: "/"
5858
initialDelaySeconds: 20
5959
periodSeconds: 15
60+
timeoutSeconds: 20
6061
ports:
6162
- name: plex-port
6263
containerPort: {{ .Values.plex.container.port }}

helm-charts/k8s-mediaserver/templates/prowlarr-resources.yml

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -75,10 +75,11 @@ spec:
7575
imagePullPolicy: Always
7676
readinessProbe:
7777
httpGet:
78-
path: "/{{ .Values.prowlarr.ingress.path }}/ping"
78+
path: "{{ .Values.prowlarr.ingress.path }}/ping"
7979
port: {{ .Values.prowlarr.container.port }}
8080
initialDelaySeconds: 10
8181
periodSeconds: 20
82+
timeoutSeconds: 20
8283
ports:
8384
- name: prowlarr-port
8485
containerPort: {{ .Values.prowlarr.container.port }}

helm-charts/k8s-mediaserver/templates/radarr-resources.yml

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -76,10 +76,11 @@ spec:
7676
imagePullPolicy: Always
7777
readinessProbe:
7878
httpGet:
79-
path: "/{{ .Values.radarr.ingress.path }}/ping"
79+
path: "{{ .Values.radarr.ingress.path }}/ping"
8080
port: {{ .Values.radarr.container.port }}
8181
initialDelaySeconds: 10
8282
periodSeconds: 20
83+
timeoutSeconds: 20
8384
ports:
8485
- name: radarr-port
8586
containerPort: {{ .Values.radarr.container.port }}

helm-charts/k8s-mediaserver/templates/sabnzbd-resources.yml

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -378,10 +378,11 @@ spec:
378378
imagePullPolicy: Always
379379
readinessProbe:
380380
httpGet:
381-
path: /{{ .Values.sabnzbd.ingress.path }}/wizard/
381+
path: "{{ .Values.sabnzbd.ingress.path }}/wizard/"
382382
port: {{ .Values.sabnzbd.container.port.http }}
383383
initialDelaySeconds: 5
384384
periodSeconds: 10
385+
timeoutSeconds: 20
385386
ports:
386387
- name: sabnzbd-http
387388
containerPort: {{ .Values.sabnzbd.container.port.http }}

helm-charts/k8s-mediaserver/templates/sonarr-resources.yml

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -75,10 +75,11 @@ spec:
7575
imagePullPolicy: Always
7676
readinessProbe:
7777
httpGet:
78-
path: "/{{ .Values.sonarr.ingress.path }}/ping"
78+
path: "{{ .Values.sonarr.ingress.path }}/ping"
7979
port: {{ .Values.sonarr.container.port }}
8080
initialDelaySeconds: 10
8181
periodSeconds: 20
82+
timeoutSeconds: 20
8283
ports:
8384
- name: sonarr-port
8485
containerPort: {{ .Values.sonarr.container.port }}

helm-charts/k8s-mediaserver/templates/transmission-resources.yml

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -142,10 +142,11 @@ spec:
142142
imagePullPolicy: Always
143143
readinessProbe:
144144
httpGet:
145-
path: /{{ .Values.transmission.ingress.path }}/
145+
path: "{{ .Values.transmission.ingress.path }}/"
146146
port: {{ .Values.transmission.container.port.utp }}
147147
initialDelaySeconds: 5
148148
periodSeconds: 10
149+
timeoutSeconds: 20
149150
ports:
150151
- name: trans-port
151152
containerPort: {{ .Values.transmission.container.port.utp }}

0 commit comments

Comments
 (0)