Skip to content

Conversation

@ayalash
Copy link
Contributor

@ayalash ayalash commented Oct 11, 2024

In case this PR will be merged, the deployment helm-chart will need to be changed as well, for using the heartbeat.
For example:

      containers:
        - name: status-updates
          securityContext:
            {{- toYaml .Values.securityContext | nindent 12 }}
          image: '{{- .Values.images.backend.repository }}:{{ .Values.images.backend.tag }}'
          imagePullPolicy: {{ .Values.images.backend.pullPolicy }}
          command:
            - /root/platz-status-updates
            - --heartbeat-file-path=/tmp/heartbeat
          livenessProbe:
            exec:
              command:
                - /bin/sh
                - '-c'
                - find /tmp/heartbeat -type f -newermt '5 minute ago' | grep -q -e '.'
            initialDelaySeconds: 120
            timeoutSeconds: 1
            periodSeconds: 60
            successThreshold: 1
            failureThreshold: 1

@ayalash ayalash requested a review from popen2 October 11, 2024 07:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

3 participants