Skip to content

Commit

Permalink
trying with just empty, fixing chart, need to update it.
Browse files Browse the repository at this point in the history
  • Loading branch information
bdw617 committed Oct 30, 2023
1 parent c1ed163 commit b12026d
Show file tree
Hide file tree
Showing 2 changed files with 11 additions and 13 deletions.
8 changes: 2 additions & 6 deletions packages/zarf-registry/chart/templates/pvc.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -17,11 +17,7 @@ spec:
requests:
storage: {{ .Values.persistence.size | quote }}
{{- if .Values.persistence.storageClass }}
{{- if (eq "-" .Values.persistence.storageClass) }}
storageClassName: ""
{{- else }}
storageClassName: "{{ .Values.persistence.storageClass }}"
{{- end }}
{{- end }}
storageClassName: {{ .Values.persistence.storageClass }}
{{- end }}
{{- end -}}
{{- end }}
16 changes: 9 additions & 7 deletions src/pkg/packager/deploy.go
Original file line number Diff line number Diff line change
Expand Up @@ -216,13 +216,15 @@ func (p *Packager) deployInitComponent(component types.ZarfComponent) (charts []
if isSeedRegistry {
p.cluster.StartInjectionMadness(p.layout.Base, p.layout.Images.Base, component.Images)

// Retrieve the default storage class from the cluster, it will be the default
if p.cfg.PkgOpts.SetVariables == nil {
p.cfg.PkgOpts.SetVariables = map[string]string{}
}
if c := p.cfg.PkgOpts.SetVariables["REGISTRY_STORAGE_CLASS"]; c == "" {
p.cfg.PkgOpts.SetVariables["REGISTRY_STORAGE_CLASS"], err = p.cluster.GetDefaultStorageClass()
}
/*
// Retrieve the default storage class from the cluster, it will be the default
if p.cfg.PkgOpts.SetVariables == nil {
p.cfg.PkgOpts.SetVariables = map[string]string{}
}
if c := p.cfg.PkgOpts.SetVariables["REGISTRY_STORAGE_CLASS"]; c == "" {
p.cfg.PkgOpts.SetVariables["REGISTRY_STORAGE_CLASS"], err = p.cluster.GetDefaultStorageClass()
}
*/
if err != nil {
message.WarnErr(err, "Unable to get the default storage class from the cluster")
}
Expand Down

0 comments on commit b12026d

Please sign in to comment.