Skip to content

fix(snapshot): allow runtime storage directory override #13170

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.

Sign up for GitHub

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

Closed
dfeigin-nv wants to merge 1 commit into from
Closed

fix(snapshot): allow runtime storage directory override #13170

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
4 changes: 3 additions & 1 deletion deploy/helm/charts/snapshot/templates/_helpers.tpl
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -102,7 +102,9 @@ Host directory holding per-container storage (overlay upperdirs the agent
reads for rootfs-diff capture, and CRI-O config.json fallback).
*/}}
{{- define "snapshot.runtimeStorageDir" -}}
{{- if eq .Values.runtime.type "crio" -}}/var/lib/containers{{- else -}}/var/lib/containerd{{- end -}}
{{- if .Values.runtime.storageDir -}}
{{- .Values.runtime.storageDir -}}
{{- else if eq .Values.runtime.type "crio" -}}/var/lib/containers{{- else -}}/var/lib/containerd{{- end -}}
{{- end }}

{{- define "snapshot.pageBrokerControlPath" -}}/pagebroker/control{{- end -}}
Expand Down
3 changes: 3 additions & 0 deletions deploy/helm/charts/snapshot/values.yaml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -22,6 +22,9 @@
runtime:
type: containerd # containerd | crio
socketPath: ""
# Host directory containing container runtime snapshots and overlay upperdirs.
# Leave empty to use /var/lib/containerd (or /var/lib/containers for CRI-O).
storageDir: ""

# Gates the OCP RBAC template and the DS required-scc annotation. Keep false
# on vanilla Kubernetes (it references an OCP-only built-in SCC).
Expand Down
Loading