Skip to content

Conversation

@wenbenz
Copy link

@wenbenz wenbenz commented Jan 5, 2026

Description of the change

Add parameter for creating a PV for an existing data directory

Benefits

Enable users to mount existing data directory.

Possible drawbacks

None.

Applicable issues

Additional information

Checklist

Signed-off-by: Ben <ben.zhao@benzhao.dev>
@wrenix
Copy link
Collaborator

wrenix commented Jan 5, 2026

If we want to support hostPath im this helm-chart, i would prefer a different solution:

@wenbenz
Copy link
Author

wenbenz commented Jan 5, 2026

What if there's a cluster with >1 node? Wouldn't we want the PVC with a hostPath PV so they read the same data?

@wrenix
Copy link
Collaborator

wrenix commented Jan 6, 2026

nodeSelector:

OR

affinity:

{{- with .Values.affinity }}
affinity:
{{- toYaml . | nindent 8 }}

@wenbenz wenbenz force-pushed the add-host-path branch 2 times, most recently from 0412a79 to ccbfe7d Compare January 8, 2026 04:06
Signed-off-by: Ben <ben.zhao@benzhao.dev>
Signed-off-by: Ben <ben.zhao@benzhao.dev>
Signed-off-by: Ben <ben.zhao@benzhao.dev>
@wenbenz
Copy link
Author

wenbenz commented Jan 10, 2026

Does this look good?

Signed-off-by: WrenIX <133280015+wrenix@users.noreply.github.com>
@wrenix wrenix changed the title added hostpath volumes feat(nextcloud): added hostPath volumes option Jan 14, 2026
@wrenix
Copy link
Collaborator

wrenix commented Jan 14, 2026

Yet, now. i will test it next.

@wrenix wrenix added the 3. to review Waiting for reviews label Jan 14, 2026
Signed-off-by: WrenIX <133280015+wrenix@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

3. to review Waiting for reviews

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Feature: Allow to mount custom volume for data

2 participants