fix: make sure autopermissions sets the group owner correctly (#994)

* fix: make sure autopermissions sets the group owner correctly

* add test fix
This commit is contained in:
Kjeld Schouten-Lebbing
2021-09-14 23:59:19 +02:00
committed by GitHub
parent 2ecefa844d
commit 0fb8f8bb90
4 changed files with 5 additions and 8 deletions

View File

@@ -22,4 +22,4 @@ name: common-test
sources:
- https://github.com/truecharts/apps/tree/master/charts/library/common-test
type: application
version: 3.1.1
version: 3.1.2

View File

@@ -12,10 +12,7 @@ maintainers:
- email: info@truecharts.org
name: TrueCharts
url: truecharts.org
- email: kjeld@schouten-lebbing.nl
name: Ornias1993
url: truecharts.org
name: common
sources: null
type: library
version: 7.0.13
version: 7.0.14

View File

@@ -27,7 +27,7 @@ before chart installation.
command:
- "/bin/sh"
- "-c"
- "echo 'Automatically correcting permissions...';{{ range $_, $hpm := $hostPathMounts }}chown -R {{ $group }} {{ $hpm.mountPath | squote }}; chmod -R g+w {{ $hpm.mountPath | squote }};{{ end }}"
- "echo 'Automatically correcting permissions...';{{ range $_, $hpm := $hostPathMounts }}chown -R :{{ $group }} {{ $hpm.mountPath | squote }}; chmod -R g+w {{ $hpm.mountPath | squote }};{{ end }}"
volumeMounts:
{{- range $name, $hpm := $hostPathMounts }}
- name: {{ $name }}

View File

@@ -80,8 +80,8 @@ func (suite *PersistenceVolumeClaimTestSuite) TestMetaData() {
"app.kubernetes.io/instance": "common-test",
"app.kubernetes.io/managed-by": "Helm",
"app.kubernetes.io/name": "common-test",
"app.kubernetes.io/version":"none",
"helm.sh/chart": "common-test-3.1.0",
"app.kubernetes.io/version":"latest",
"helm.sh/chart": "common-test-3.1.2",
}
tests := map[string]struct {