allows for specifying a path for NewAuditPolicyPathGetter function

openshift · Jul 31, 2020 · 0f5f426 · 0f5f426
1 parent 918750c
commit 0f5f426
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 5 deletions.
diff --git a/pkg/operator/apiserver/audit/audit_policies.go b/pkg/operator/apiserver/audit/audit_policies.go
@@ -57,13 +57,12 @@ func getRawAuditPolicies(targetName, targetNamespace string) ([]byte, error) {
 	return auditPoliciesForTargetNs, nil
 }
 
-// NewAuditPolicyPathGetter returns a path getter for audit policy file mounted into
-// the '/var/run/configmaps/audit' directory of a Pod.
+// NewAuditPolicyPathGetter returns a path getter for audit policy file mounted into the given path of a Pod as a directory.
 //
 // openshift-apiserver and oauth-apiserver mounts the audit policy ConfigMap into
 // the above path inside the Pod.
-func NewAuditPolicyPathGetter() (libgoapiserver.AuditPolicyPathGetterFunc, error) {
-	return newAuditPolicyPathGetter("/var/run/configmaps/audit")
+func NewAuditPolicyPathGetter(path string) (libgoapiserver.AuditPolicyPathGetterFunc, error) {
+	return newAuditPolicyPathGetter(path)
 }
 
 func newAuditPolicyPathGetter(path string) (libgoapiserver.AuditPolicyPathGetterFunc, error) {

diff --git a/pkg/operator/apiserver/audit/audit_policies_test.go b/pkg/operator/apiserver/audit/audit_policies_test.go
@@ -128,7 +128,7 @@ func TestNewAuditPolicyPathGetter(t *testing.T) {
 		},
 	}
 
-	pathGetter, err := NewAuditPolicyPathGetter()
+	pathGetter, err := NewAuditPolicyPathGetter("/var/run/configmaps/audit")
 	if err != nil {
 		t.Fatal(err)
 	}