Merge pull request #76227 from smunje1/OBSDOCS-1070

/lgtm OBSDOCS-1070

Author: JoeAldinger

modules/logging-loki-retention.adoc

@@ -1,19 +1,21 @@
 // Module included in the following assemblies:
-//
-// logging/cluster-logging-loki.adoc
+// 
+// * observability/logging/log_storage/cluster-logging-loki.adoc
 
 :_mod-docs-content-type: PROCEDURE
 [id="logging-loki-retention_{context}"]
 = Enabling stream-based retention with Loki
 
 With Logging version 5.6 and higher, you can configure retention policies based on log streams. Rules for these may be set globally, per tenant, or both. If you configure both, tenant rules apply before global rules.
 
+include::snippets/logging-retention-period-snip.adoc[]
+
 [NOTE]
 ====
 Although logging version 5.9 and higher supports schema v12, v13 is recommended.
 ====
 
-. To enable stream-based retention, create a `LokiStack` custom resource (CR):
+. To enable stream-based retention, create a `LokiStack` CR:
 +
 .Example global stream-based retention for AWS
 [source,yaml]

snippets/logging-retention-period-snip.adoc

@@ -0,0 +1,14 @@
+// Text snippet included in the following assemblies:
+//
+// * observability/logging/log_storage/cluster-logging-loki.adoc
+//
+// Text snippet included in the following modules:
+//
+// * logging-loki-retention.adoc
+
+:_mod-docs-content-type: SNIPPET
+
+[IMPORTANT]
+====
+If there is no retention period defined on the s3 bucket or in the LokiStack custom resource (CR), then the logs are not pruned and they stay in the s3 bucket forever, which might fill up the s3 storage.
+====