Skip to content

De-register smallrye context manager on shutdown #18877

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
gsmet merged 1 commit into from
Jul 21, 2021
Merged

De-register smallrye context manager on shutdown #18877

gsmet merged 1 commit into from
Jul 21, 2021

Conversation

@stuartwdouglas
Copy link
Collaborator

@stuartwdouglas stuartwdouglas commented Jul 21, 2021

Fixes #18864

@quarkus-bot
Copy link

quarkus-bot bot commented Jul 21, 2021 •
edited
Loading

This workflow status is outdated as a new workflow run has been triggered.

Failing Jobs - Building 2470b1b

Status Name Step Test failures Logs Raw logs
✖ Initial JDK 11 Build Build ⚠️ Check → Logs Raw logs

@quarkus-bot
Copy link

quarkus-bot bot commented Jul 21, 2021

Failing Jobs - Building 78e5635

Status Name Step Test failures Logs Raw logs
✔️ JVM Tests - JDK 11
✖ JVM Tests - JDK 11 Windows Build ⚠️ Check → Logs Raw logs
✔️ JVM Tests - JDK 16

@gsmet gsmet merged commit d34e20b into quarkusio:main Jul 21, 2021
@quarkus-bot quarkus-bot bot added this to the 2.2 - main milestone Jul 21, 2021
@gsmet gsmet modified the milestones: 2.2 - main, 2.0.3.Final Jul 21, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@ebullient ebullient ebullient approved these changes

@geoand geoand geoand approved these changes

Assignees

No one assigned

Labels

area/smallrye

Projects

None yet

Milestone

2.0.3.Final

Development

Successfully merging this pull request may close these issues.

Classloader leak in MP ContextManagerProvider / Smallrye SmallRyeContextManagerProvider

4 participants

@stuartwdouglas @ebullient @geoand @gsmet