Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

remove probes #56

Merged
merged 2 commits into from
Feb 26, 2024
Merged

remove probes #56

merged 2 commits into from
Feb 26, 2024

Conversation

jgilfoil
Copy link
Owner

flux keeps uninstall plex for some reason, i think because the probes are failing.

flux keeps uninstall plex for some reason, i think because the probes are failing.
Copy link

github-actions bot commented Feb 26, 2024

--- kubernetes/apps/media/plex/app Kustomization: flux-system/plex HelmRelease: media/plex

+++ kubernetes/apps/media/plex/app Kustomization: flux-system/plex HelmRelease: media/plex

@@ -46,34 +46,12 @@

             env:
               TZ: America/Denver
             image:
               repository: ghcr.io/onedr0p/plex-beta
               tag: 1.40.0.7775-456fbaf97@sha256:cd6bafd8cf9f75c2d5b6517de312a82a986049f89f089da74b6bdb23928b8b37
             probes:
-              liveness:
-                custom: true
-                enabled: true
-                spec:
-                  failureThreshold: 3
-                  httpGet:
-                    path: /identity
-                    port: 32400
-                  initialDelaySeconds: 0
-                  periodSeconds: 10
-                  timeoutSeconds: 1
-              readiness:
-                custom: true
-                enabled: true
-                spec:
-                  failureThreshold: 3
-                  httpGet:
-                    path: /identity
-                    port: 32400
-                  initialDelaySeconds: 0
-                  periodSeconds: 10
-                  timeoutSeconds: 1
               startup:
                 enabled: false
             resources:
               limits:
                 gpu.intel.com/i915: 1
                 memory: 16Gi

Copy link

github-actions bot commented Feb 26, 2024

--- HelmRelease: media/plex Deployment: media/plex

+++ HelmRelease: media/plex Deployment: media/plex

@@ -50,26 +50,24 @@

       - env:
         - name: TZ
           value: America/Denver
         image: ghcr.io/onedr0p/plex-beta:1.40.0.7775-456fbaf97@sha256:cd6bafd8cf9f75c2d5b6517de312a82a986049f89f089da74b6bdb23928b8b37
         livenessProbe:
           failureThreshold: 3
-          httpGet:
-            path: /identity
-            port: 32400
           initialDelaySeconds: 0
           periodSeconds: 10
+          tcpSocket:
+            port: 32400
           timeoutSeconds: 1
         name: main
         readinessProbe:
           failureThreshold: 3
-          httpGet:
-            path: /identity
-            port: 32400
           initialDelaySeconds: 0
           periodSeconds: 10
+          tcpSocket:
+            port: 32400
           timeoutSeconds: 1
         resources:
           limits:
             gpu.intel.com/i915: 1
             memory: 16Gi
           requests:

it was trying to use the default i think.
@jgilfoil jgilfoil merged commit 2866e26 into main Feb 26, 2024
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant