From 6ee1f3febec8b63e5462adc8636b64342572987d Mon Sep 17 00:00:00 2001
From: EMaksy <emaksymenko@suse.com>
Date: Fri, 24 Jan 2025 13:58:16 +0100
Subject: [PATCH] Add more information

---
 trento/adoc/helm_event_pruning.adoc | 9 ++++++---
 trento/xml/article_sap_trento.xml   | 4 +++-
 trento/xml/helm_event_pruning.xml   | 8 ++++++--
 3 files changed, 15 insertions(+), 6 deletions(-)

diff --git a/trento/adoc/helm_event_pruning.adoc b/trento/adoc/helm_event_pruning.adoc
index af4ce5a1..59b5734b 100644
--- a/trento/adoc/helm_event_pruning.adoc
+++ b/trento/adoc/helm_event_pruning.adoc
@@ -9,7 +9,10 @@ The following configurations are available:
 
 `+pruneEventsOlderThan+`::
   The number of days registered events are stored in the database. The
-  default value is *10*.
+  default value is *10*. _Beware that `+pruneEventsOlderThan+` can be
+  set to *0* but that would mean that all events are deleted every time
+  the cron job runs, making it impossible to analyze/troubleshoot any
+  issue with the application_
 `+pruneEventsCronjobSchedule+`::
   The frequency of the cron job that deletes expired events. The default
   value is *"0 0 * * *"*, which runs daily at midnight.
@@ -35,8 +38,8 @@ Replace the placeholders with the desired values:
 `+ADMIN_PASSWORD+`::
   The password for the admin user.
 
-Example command to retain events for 30 days and schedule pruning daily
-at 3 AM:
+*Example* command to retain events for 30 days and schedule pruning
+daily at 3 AM:
 
 [source,bash]
 ----
diff --git a/trento/xml/article_sap_trento.xml b/trento/xml/article_sap_trento.xml
index 7c868e56..9535cdd1 100644
--- a/trento/xml/article_sap_trento.xml
+++ b/trento/xml/article_sap_trento.xml
@@ -462,7 +462,7 @@ As agreed on https://confluence.suse.com/x/DAEcN on our Trento doc kick off
         </step>
       </procedure>
     </section>
-    <xi:include href="helm_event_pruning.xml"/>
+
     <section xml:id="sec-trento-install-trentoserver-on-k3s">
       <title>Installing &t.server; on K3s</title>
       <para> If you do not have a &k8s; cluster or have one but do not want to use
@@ -562,6 +562,8 @@ As agreed on https://confluence.suse.com/x/DAEcN on our Trento doc kick off
    --set trento-runner.nodeSelector.<replaceable>LABEL</replaceable>=<replaceable>VALUE</replaceable></screen>
     </section>
 
+    <xi:include href="helm_event_pruning.xml"/>
+
     <section xml:id="sec-trento-enabling-email-alerts">
       <title>Enabling email alerts</title>
       <para>
diff --git a/trento/xml/helm_event_pruning.xml b/trento/xml/helm_event_pruning.xml
index 74741fea..83610078 100644
--- a/trento/xml/helm_event_pruning.xml
+++ b/trento/xml/helm_event_pruning.xml
@@ -10,10 +10,14 @@
   <variablelist>
     <varlistentry>
       <term><literal>pruneEventsOlderThan</literal></term>
-      <listitem>
+        <listitem>
         <para>
           The number of days registered events are stored in the database. 
           The default value is <emphasis role="bold">10</emphasis>.
+            <emphasis>Beware that <literal>pruneEventsOlderThan</literal> can be set to <emphasis role="bold">0</emphasis>
+             but that would mean that all events are deleted every time
+             the cron job runs, making it impossible to analyze/troubleshoot
+             any issue with the application</emphasis>
         </para>
       </listitem>
     </varlistentry>
@@ -65,7 +69,7 @@
     </varlistentry>
   </variablelist>
   <para>
-    Example command to retain events for 30 days and schedule pruning daily at 3 AM:
+    <emphasis role="bold">Example</emphasis> command to retain events for 30 days and schedule pruning daily at 3 AM:
   </para>
   <programlisting language="bash">helm install trento-server . \
   --set trento-web.pruneEventsOlderThan=30 \