From d38f3d00db2e9e124f7136f4a2f86337c8ce2a54 Mon Sep 17 00:00:00 2001
From: Corey Daley <cdaley@redhat.com>
Date: Thu, 14 Nov 2024 13:09:45 -0500
Subject: [PATCH] feature: Remove Audit Log PVC

https://issues.redhat.com/browse/RHIDP-4922
---
 charts/backstage/Chart.yaml          |  2 +-
 charts/backstage/README.md           |  2 +-
 charts/backstage/templates/pvcs.yaml | 13 +------------
 charts/backstage/values.schema.json  | 15 ---------------
 charts/backstage/values.yaml         |  9 ---------
 5 files changed, 3 insertions(+), 38 deletions(-)

diff --git a/charts/backstage/Chart.yaml b/charts/backstage/Chart.yaml
index 3c12586..d1c3bb8 100644
--- a/charts/backstage/Chart.yaml
+++ b/charts/backstage/Chart.yaml
@@ -45,4 +45,4 @@ sources:
 # This is the chart version. This version number should be incremented each time you make changes
 # to the chart and its templates, including the app version.
 # Versions are expected to follow Semantic Versioning (https://semver.org/)
-version: 2.21.0
+version: 2.22.0
diff --git a/charts/backstage/README.md b/charts/backstage/README.md
index 8e4bd62..826dcc3 100644
--- a/charts/backstage/README.md
+++ b/charts/backstage/README.md
@@ -2,7 +2,7 @@
 # RHDH Backstage Helm Chart for OpenShift (Community Version)
 
 [![Artifact Hub](https://img.shields.io/endpoint?url=https://artifacthub.io/badge/repository/rhdh-chart&style=flat-square)](https://artifacthub.io/packages/search?repo=rhdh-chart)
-![Version: 2.21.0](https://img.shields.io/badge/Version-2.21.0-informational?style=flat-square)
+![Version: 2.22.0](https://img.shields.io/badge/Version-2.22.0-informational?style=flat-square)
 ![Type: application](https://img.shields.io/badge/Type-application-informational?style=flat-square)
 
 A Helm chart for deploying Red Hat Developer Hub.
diff --git a/charts/backstage/templates/pvcs.yaml b/charts/backstage/templates/pvcs.yaml
index 4f85c39..f441e2d 100644
--- a/charts/backstage/templates/pvcs.yaml
+++ b/charts/backstage/templates/pvcs.yaml
@@ -1,16 +1,5 @@
 kind: PersistentVolumeClaim
 apiVersion: v1
-metadata:
-  name: {{ printf "%s-audit-log" .Release.Name }}
-spec:
-  accessModes:
-    - ReadWriteOnce
-  resources:
-    requests:
-      storage: 1Gi
----
-kind: PersistentVolumeClaim
-apiVersion: v1
 metadata:
   name: {{ printf "%s-dynamic-plugins-root" .Release.Name }}
 spec:
@@ -18,4 +7,4 @@ spec:
     - ReadWriteOnce
   resources:
     requests:
-      storage: 5Gi
\ No newline at end of file
+      storage: 5Gi
diff --git a/charts/backstage/values.schema.json b/charts/backstage/values.schema.json
index 22f97a0..b9256d1 100644
--- a/charts/backstage/values.schema.json
+++ b/charts/backstage/values.schema.json
@@ -954,11 +954,6 @@
                                 "app": {
                                     "baseUrl": "https://{{- include \"janus-idp.hostname\" . }}"
                                 },
-                                "auditLog": {
-                                    "rotateFile": {
-                                        "enabled": true
-                                    }
-                                },
                                 "auth": {
                                     "providers": {}
                                 },
@@ -2596,10 +2591,6 @@
                                 {
                                     "mountPath": "/opt/app-root/src/dynamic-plugins-root",
                                     "name": "dynamic-plugins-root"
-                                },
-                                {
-                                    "mountPath": "/var/log/redhat-developer-hub/audit",
-                                    "name": "audit-log-data"
                                 }
                             ],
                             "items": {
@@ -2651,12 +2642,6 @@
                                         "claimName": "{{ printf \"%s-dynamic-plugins-root\" .Release.Name }}"
                                     }
                                 },
-                                {
-                                    "name": "audit-log-data",
-                                    "persistentVolumeClaim": {
-                                        "claimName": "{{ printf \"%s-audit-log\" .Release.Name }}"
-                                    }
-                                },
                                 {
                                     "configMap": {
                                         "defaultMode": 420,
diff --git a/charts/backstage/values.yaml b/charts/backstage/values.yaml
index 62ef72e..54c0fd3 100644
--- a/charts/backstage/values.yaml
+++ b/charts/backstage/values.yaml
@@ -47,9 +47,6 @@ upstream:
     # create multiple DBs. Since Backstage requires by default 5 different DBs, we
     # can't accommodate that properly.
     appConfig:
-      auditLog:
-        rotateFile:
-          enabled: true
       auth:
         providers: {}
       app:
@@ -123,16 +120,10 @@ upstream:
       # The initContainer below will install dynamic plugins in this volume mount.
       - name: dynamic-plugins-root
         mountPath: /opt/app-root/src/dynamic-plugins-root
-      # Audit Log data will be stored in this volume mount.
-      - name: audit-log-data
-        mountPath: /var/log/redhat-developer-hub/audit
     extraVolumes:
       - name: dynamic-plugins-root
         persistentVolumeClaim:
           claimName: '{{ printf "%s-dynamic-plugins-root" .Release.Name }}'
-      - name: audit-log-data
-        persistentVolumeClaim:
-          claimName: '{{ printf "%s-audit-log" .Release.Name }}'
       # Volume that will expose the `dynamic-plugins.yaml` file from the `dynamic-plugins` config map.
       # The `dynamic-plugins` config map is created by the helm chart from the content of the `global.dynamic` field.
       - name: dynamic-plugins