Skip to content

Conversation

tommyk-gears
Copy link
Contributor

This is needed because when the default Out-Of-Memory handler in Hazelcast detects a OOM error, and terminates Hazelcast, it does so on the node but not via the Hazelcast lifecycle manager - and the results is that it is still possible to successfully execute transactions in Hazelcast after this termination. The health indicator should not report UP after Hazelcast has detected an OOM and terminated itself.

This is needed because when the default Out-Of-Memory handler in Hazelcast detects a OOM error, and terminates Hazelcast, it does so on the _node_ but not via the Hazelcast lifecycle manager - and the results is that it is still possible to successfully execute transactions in Hazelcast after this termination. The health indicator should not report UP after Hazelcast has detected an OOM and terminated itself. Signed-off-by: Tommy Karlsson <tommy.karlsson@leovegas.com>
@spring-projects-issues spring-projects-issues added the status: waiting-for-triage An issue we've not yet triaged label Aug 18, 2025
@snicoll snicoll changed the title Update Hazelcast health indicator to verify that Hazelcast is running Hazelcast health indicator reports the wrong status when Hazelcast has shut down due to an out-of-memory error Aug 18, 2025
@snicoll snicoll added type: bug A general bug and removed status: waiting-for-triage An issue we've not yet triaged labels Aug 18, 2025
@snicoll snicoll added this to the 3.4.x milestone Aug 18, 2025
@snicoll snicoll self-assigned this Aug 19, 2025
@snicoll snicoll modified the milestones: 3.4.x, 3.4.9 Aug 19, 2025
snicoll pushed a commit that referenced this pull request Aug 19, 2025
This commit improves the Hazelcast health indicator to fail if the Lifecycle manager indicates that the instance is no longer running. This is needed because when the default Out-Of-Memory handler in Hazelcast detects a OOM error, and terminates Hazelcast, it does so on the node but not via the Hazelcast lifecycle manager - and the results is that it is still possible to successfully execute transactions in Hazelcast after this termination. The health indicator should not report UP after Hazelcast has detected an OOM and terminated itself. See gh-46877 Signed-off-by: Tommy Karlsson <tommy.karlsson@leovegas.com>
snicoll added a commit that referenced this pull request Aug 19, 2025
@snicoll snicoll closed this in e1db00b Aug 19, 2025
@snicoll
Copy link
Member

snicoll commented Aug 19, 2025

Good stuff, thanks @tommyk-gears for making your first contribution to Spring Boot.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

type: bug A general bug

3 participants