[7.x] Remove warm phase from default ILM policy (backport #6322) (#6325)

* Remove warm phase from default ILM policy (#6322) (cherry picked from commit c3d27d4) # Conflicts: # changelogs/head.asciidoc * Delete head.asciidoc Co-authored-by: Andrew Wilkins <[email protected]>
elastic · Oct 12, 2021 · ada91c1 · ada91c1
1 parent 2ff4d3b
commit ada91c1
Show file tree

Hide file tree

Showing 7 changed files with 5 additions and 40 deletions.
diff --git a/_meta/beat.yml b/_meta/beat.yml
@@ -468,12 +468,6 @@ apm-server:
                     #max_age: "30d"
                   #set_priority:
                     #priority: 100
-              #warm:
-                #min_age: "30d"
-                #actions:
-                  #set_priority:
-                    #priority: 50
-                  #readonly: {}
 
 
 

diff --git a/apm-server.docker.yml b/apm-server.docker.yml
@@ -468,12 +468,6 @@ apm-server:
                     #max_age: "30d"
                   #set_priority:
                     #priority: 100
-              #warm:
-                #min_age: "30d"
-                #actions:
-                  #set_priority:
-                    #priority: 50
-                  #readonly: {}
 
 
 

diff --git a/apm-server.yml b/apm-server.yml
@@ -468,12 +468,6 @@ apm-server:
                     #max_age: "30d"
                   #set_priority:
                     #priority: 100
-              #warm:
-                #min_age: "30d"
-                #actions:
-                  #set_priority:
-                    #priority: 50
-                  #readonly: {}
 
 
 

diff --git a/docs/apm-package/apm-integration.asciidoc b/docs/apm-package/apm-integration.asciidoc
@@ -46,8 +46,7 @@ Stack monitoring::
 
 Index lifecycle management (ILM)::
 A default ILM policy, named `traces-apm.traces-default_policy` is created for all event types.
-This policy moves indices to a warm data tier after 30 days.
-No default cold or delete data tier is defined.
+No default warm, cold, or delete data tiers are defined.
 It is not possible to configure this policy in APM Server or {agent}–
 it must be configured with {es} or {kib}.
 See {ref}/example-using-index-lifecycle-policy.html[Customize built-in ILM policies] for more information.

diff --git a/docs/ilm-reference.asciidoc b/docs/ilm-reference.asciidoc
@@ -28,15 +28,14 @@ will be applied to all APM indices as long as all of the following conditions ar
 * `output.elasticsearch` is enabled.
 * Custom `index` or `indices` settings are not configured.
 
-The default ILM policy applies *hot* and *warm* phases to all APM events:
+The default ILM policy applies only a *hot* phase to all APM events:
 `span`, `transaction`, `error`, and `metric`.
-*Cold* and *delete* phases are not defined.
+
+*Warm*, *cold*, and *delete* phases are not defined.
 
 * *Hot* -- Rollover data when the index reaches a maximum size of 50gb or a maximum age of 30 days:
 `max_size: 50gb`, `max_age: 30d`
 
-* *Warm* -- Move to warm phase after 30 days: `min_age: 30d`
-
 [float]
 [[ilm-default-config]]
 === Default configuration
@@ -74,12 +73,6 @@ apm-server:
                     max_age: "30d"
                   set_priority:
                     priority: 100
-              warm:
-                min_age: "30d"
-                actions:
-                  set_priority:
-                    priority: 50
-                  readonly: {}
 ----
 
 [float]

diff --git a/idxmgmt/ilm/config.go b/idxmgmt/ilm/config.go
@@ -246,15 +246,6 @@ func defaultPolicies() map[string]Policy {
 								},
 							},
 						},
-						"warm": map[string]interface{}{
-							"min_age": "30d",
-							"actions": map[string]interface{}{
-								"set_priority": map[string]interface{}{
-									"priority": 50,
-								},
-								"readonly": map[string]interface{}{},
-							},
-						},
 					},
 				},
 			},

diff --git a/tests/system/test_export.py b/tests/system/test_export.py
@@ -79,7 +79,7 @@ def test_export_ilm_policy_to_files(self):
                 with open(os.path.join(dir, file)) as f:
                     policy = json.load(f)
                 assert "hot" in policy["policy"]["phases"]
-                assert "warm" in policy["policy"]["phases"]
+                assert "warm" not in policy["policy"]["phases"]
                 assert "delete" not in policy["policy"]["phases"]