search

woodpecker-ci / helm

Woodpecker CI Helm Charts
Apache License 2.0
21 stars 17 forks source link

Remove PV from server #162

Closed pat-s closed 7 months ago

pat-s commented 7 months ago

see #160

The server doesn't need a PV as all data is stored in the DB and all config is done via env vars/secrets.

I just checked our own install and the PV is empty.

Not sure if this is an enhancement of bugfix but it shouldn't be breaking as I can't see how anyone has anything stored in that PV. Hence deleting shouldn't harm existing installations.

anbraten commented 7 months ago

Isn't it used for sqlite?

pat-s commented 7 months ago

Ahh yes 😵‍💫 Always forget that it exists and is file-based. Closing here. But we should highlight in the docs that persistence can be disabled if a non-sqlite DB is used.