oben01 / charts

https://oben01.github.io/charts/
Apache License 2.0
4 stars 3 forks source link

"Start update process" screen shows every time a pod is deleted #61

Open BarryWilliams opened 1 month ago

BarryWilliams commented 1 month ago

Chart Name

Homarr

Chart Version

homarr-1.2.5

Bug Description

When a pod gets deleted (and a new pod is created) - Homarr keeps initially showing the "Start update process", which includes asking to set the admin user and password. This is rather annoying.

My dashboards still persist, which is great.

Recreation steps:

  1. Deploy the chart
  2. Do initial setup and add some dashboard creation
  3. Delete the pod (kubectl delete pod homarr-xx-xx -n homarr)
  4. Go to the URL again - it will go through "Start update process"
  5. Delete the pod again - it will go through "Start update process" again Note: dashboards persist

RKE 1.4.10 Longhorn for PV

values.yaml

ingress:
  enabled: true
  hosts:
    - host: launch.kube.local
      paths:
        - path: /
          pathType: Prefix
env:
  DEFAULT_COLOR_SCHEME: dark
persistence:
  - name: homarr-config
    # -- Enable this persistent storage
    enabled: true
    # -- Storage class name
    storageClassName: longhorn
    # -- Access mode
    accessMode: "ReadWriteOnce"
    # -- Storage size
    size: "50Mi"
    # -- Mount path inside the pod
    mountPath: "/app/data/configs"
  - name: homarr-database
    enabled: true
    storageClassName: longhorn
    accessMode: "ReadWriteOnce"
    size: "50Mi"
    mountPath: "/app/database"
  - name: homarr-icons
    enabled: true
    storageClassName: longhorn
    accessMode: "ReadWriteOnce"
    size: "50Mi"
    mountPath: "/app/public/icons"

Relevant log output

Several back-to-back entries of the following:

ℹ Instance is using a database on the file system. Checking if file '/data/db.sqlite' is writable...
ℹ Database is writeable
BarryWilliams commented 1 month ago

It looks like the sqlite location is set as an environment variable in the 1.15.3 docker image

looking at the environment variables in my pod, I see it is this:

DATABASE_URL=file:/data/db.sqlite

It looks like this is fixed on the dev branch. Hopefully that comes in the next release of the image.

BarryWilliams commented 1 month ago

Updating the environment variables in the values file fixed it.

Mine looks like this now:

env:
  DEFAULT_COLOR_SCHEME: dark
  DATABASE_URL: file:/app/database/db.sqlite

Perhaps this can be added to the templates

oben01 commented 1 month ago

@BarryWilliams thank you for the pull request. I'll take a look next week

oben01 commented 1 month ago

@BarryWilliams could u please follow instructions on this PR : https://github.com/oben01/charts/pull/41 to update documentation and chart metadata. Thank u

oben01 commented 3 weeks ago

@BarryWilliams i just added a contribution guide https://github.com/oben01/charts/blob/main/.github/CONTRIBUTING.md could u please follow the instructions.