-
Notifications
You must be signed in to change notification settings - Fork 93
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
job-assets-precompile
fails for Mastodon v4.3.0
#158
Comments
I am also dealing with it failing (using
I use a fork of this chart, but it's mostly just to keep it actually released until this chart moves to the new repo. Here's the job yaml that's loaded in right now: click em for full assets-precompile yaml# kubectl get job mastodon-web-app-assets-precompile -o yaml
apiVersion: batch/v1
kind: Job
metadata:
annotations:
helm.sh/hook: post-install
helm.sh/hook-delete-policy: before-hook-creation,hook-succeeded
helm.sh/hook-weight: "-2"
creationTimestamp: "2024-10-22T10:33:03Z"
generation: 1
labels:
app.kubernetes.io/instance: mastodon-web-app
app.kubernetes.io/managed-by: Helm
app.kubernetes.io/name: mastodon
app.kubernetes.io/version: v4.3.1
argocd.argoproj.io/instance: mastodon-web-app
helm.sh/chart: mastodon-9.5.0
name: mastodon-web-app-assets-precompile
namespace: mastodon
resourceVersion: "70931996"
uid: 9382b09e-d9d1-4a00-970b-48732ff99315
spec:
backoffLimit: 6
completionMode: NonIndexed
completions: 1
manualSelector: false
parallelism: 1
podReplacementPolicy: TerminatingOrFailed
selector:
matchLabels:
batch.kubernetes.io/controller-uid: 9382b09e-d9d1-4a00-970b-48732ff99315
suspend: false
template:
metadata:
creationTimestamp: null
labels:
batch.kubernetes.io/controller-uid: 9382b09e-d9d1-4a00-970b-48732ff99315
batch.kubernetes.io/job-name: mastodon-web-app-assets-precompile
controller-uid: 9382b09e-d9d1-4a00-970b-48732ff99315
job-name: mastodon-web-app-assets-precompile
name: mastodon-web-app-assets-precompile
spec:
containers:
- command:
- bash
- -c
- |
bundle exec rake assets:precompile && yarn cache clean
env:
- name: DB_PASS
valueFrom:
secretKeyRef:
key: password
name: mastodon-pgsql-credentials
- name: REDIS_PASSWORD
valueFrom:
secretKeyRef:
key: valkey-password
name: mastodon-valkey-credentials
- name: PORT
value: "3000"
envFrom:
- configMapRef:
name: mastodon-web-app-env
- secretRef:
name: mastodon-server-secrets
image: ghcr.io/mastodon/mastodon:v4.3.1
imagePullPolicy: IfNotPresent
name: mastodon-web-app-assets-precompile
resources: {}
terminationMessagePath: /dev/termination-log
terminationMessagePolicy: File
volumeMounts:
- mountPath: /etc/secrets/ca
name: postgres-ca
- mountPath: /etc/secrets/mastodon
name: postgres-client-certs
dnsPolicy: ClusterFirst
restartPolicy: Never
schedulerName: default-scheduler
securityContext: {}
terminationGracePeriodSeconds: 30
volumes:
- name: postgres-ca
secret:
defaultMode: 288
secretName: mastodon-postgres-server-ca-key-pair
- name: postgres-client-certs
secret:
defaultMode: 288
secretName: mastodon-postgres-mastodon-cert
status:
failed: 4
ready: 0
startTime: "2024-10-22T10:33:03Z"
terminating: 0
uncountedTerminatedPods: {} I don't actually see precompile assets in the default setup here though, so maybe we don't need it?: |
You should never precompile assets on deploy. Either you use our official container image, which already contains the assets, or you should build your own. |
Awesome, thanks for the fast response! we should disable it by default on this chart then, to reduce confusion. We could also delete it entirely from the chart in #157, if it doesn't serve a purpose anymore (or never did). |
Apparently, Node.js is missing in the new images:
Is this hook/job still needed or can it safely be disabled via https://github.com/mastodon/chart/blob/main/values.yaml#L30?
The text was updated successfully, but these errors were encountered: