Skip to content

Add exit clause on Heap full - #74

Merged
pdowler merged 2 commits into
opencadc:mainfrom
at88mph:main
Oct 15, 2026
Merged

pdowler merged 2 commits into
opencadc:mainfrom
at88mph:main

Conversation

@at88mph

@at88mph at88mph commented Oct 13, 2026

Copy link
Copy Markdown
Member

No description provided.

@pdowler pdowler left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Were you able to provoke an OOM and see the jvm and container exit?

Did the heap dump actually work? iirc the attempt to cat the result to stdout so it ends up in logs did not work last time I wanted to see it

@at88mph

at88mph commented Oct 15, 2026

Copy link
Copy Markdown
Member Author

Correct. I ran two Kubernetes deployments; one with the new switch and one without:

One with:
-Xmx32m -XX:+ExitOnOutOfMemoryError

and another with
-Xmx32m

below is a view of the two Pods. The oom-exit one actually died (Error), and the other is still running as it's not registered as an error. To get appropriate restarts, the end application (e.g. cavern) will need to have a proper livenessProbe in place.

Screenshot 2026-10-15 at 08 42 03

@pdowler
pdowler merged commit 8d7b53b into opencadc:main Oct 15, 2026
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants