-
Notifications
You must be signed in to change notification settings - Fork 2.8k
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.
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
Upgrade SmallRye Health to 3.2.0 #23206
Conversation
aa8611e
to
b52389c
Compare
This workflow status is outdated as a new workflow run has been triggered. Failing Jobs - Building b52389c
Failures⚙️ Initial JDK 11 Build #- Failing: extensions/smallrye-health/runtime
! Skipped: devtools/bom-descriptor-json docs extensions/agroal/deployment and 218 more 📦 extensions/smallrye-health/runtime✖ |
b52389c
to
04fe3e2
Compare
04fe3e2
to
ece5ceb
Compare
<exclusions> | ||
<exclusion> | ||
<groupId>jakarta.json</groupId> | ||
<artifactId>jakarta.json-api</artifactId> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Question, do you need JSON-P around or is it just optional?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
yes, we also now have a non-CDI plain option of using sr-health so I needed to ship this too, unfortunately.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ah OK, I just checked and the dependency is already there. Fine then.
This workflow status is outdated as a new workflow run has been triggered. Failing Jobs - Building ece5ceb
Full information is available in the Build summary check run. Failures⚙️ Gradle Tests - JDK 11 Windows #- Failing: integration-tests/gradle
📦 integration-tests/gradle✖
|
I requested another CI run as I would have to get the MP TCK tests green (it's CP who made the build stall but we didn't have a clear report of the Health TCK status). |
No description provided.