Skip to content

[9.2] Fix log formatting in SnapshotLifecycleTask (#136709)#136756

Merged
elasticsearchmachine merged 1 commit intoelastic:9.2from
nielsbauman:backport/9.2/pr-136709
Oct 17, 2025
Merged

[9.2] Fix log formatting in SnapshotLifecycleTask (#136709)#136756
elasticsearchmachine merged 1 commit intoelastic:9.2from
nielsbauman:backport/9.2/pr-136709

Conversation

@nielsbauman
Copy link
Contributor

@nielsbauman nielsbauman commented Oct 17, 2025

Backports the following commits to 9.2:
@nielsbauman
Fix log formatting in SnapshotLifecycleTask (elastic#136709)
8bacd2c 
While converting an error log from `logger.error` to
`logger.error(format())` in elastic#131646, we forgot to update the syntax from
`{}` to `%s`. This resulted in the logs `failed to create snapshot for
snapshot lifecycle policy [{}]`. This commit fixes the syntax and
ensures we use the right logger.
@nielsbauman nielsbauman added :Data Management/ILM+SLM DO NOT USE. Use ":StorageEngine/ILM" or ":Distributed Coordination/SLM" instead. >bug auto-merge-without-approval Automatically merge pull request when CI checks pass (NB doesn't wait for reviews!) backport Team:Data Management (obsolete) DO NOT USE. This team no longer exists. labels Oct 17, 2025
@elasticsearchmachine elasticsearchmachine mentioned this pull request Oct 17, 2025
Merged
@elasticsearchmachine elasticsearchmachine merged commit af0a2a0 into elastic:9.2 Oct 17, 2025
34 checks passed
@nielsbauman nielsbauman deleted the backport/9.2/pr-136709 branch October 17, 2025 16:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

auto-merge-without-approval Automatically merge pull request when CI checks pass (NB doesn't wait for reviews!) backport >bug :Data Management/ILM+SLM DO NOT USE. Use ":StorageEngine/ILM" or ":Distributed Coordination/SLM" instead. Team:Data Management (obsolete) DO NOT USE. This team no longer exists. v9.2.1

2 participants

@nielsbauman @elasticsearchmachine