Merge branch 'expiry' into 'master'

Clarify artifact expiry

## What does this MR do?

Clarifies documentation about artifact expiry.

## Are there points in the code the reviewer needs to double check?

## Why was this MR needed?

## What are the relevant issue numbers?

## Screenshots (if relevant)

See merge request !4831
Signed-off-by: Rémy Coutable <remy@rymai.me>

CHANGELOG

@@ -9,6 +9,7 @@ v 8.9.1 (unreleased)
   - Ensure we don't send the "access request declined" email to access requesters on project deletion. !4744
   - Display group/project access requesters separately in the admin area. !4798
   - Add documentation and examples for configuring cloud storage for registry images. !4812
+  - Clarifies documentation about artifact expiry. !4831
   - Fix MR-auto-close text added to description. !4836
   - Fix typo in export failure email. !4847
 

doc/ci/yaml/README.md

@@ -726,8 +726,8 @@ time they are uploaded and stored on GitLab.
 You can use the **Keep** button on the build page to override expiration and
 keep artifacts forever.
 
-By default, artifacts are deleted hourly (via a cron job), but they are not
-accessible after expiry.
+After expiry, artifacts are actually deleted hourly by default (via a cron job),
+but they are not accessible after expiry.
 
 The value of `expire_in` is an elapsed time. Examples of parseable values:
 - '3 mins 4 sec'