Skip to content

Conversation

@jbachorik
Copy link
Collaborator

@jbachorik jbachorik commented Jul 22, 2025

What does this PR do?:
It adds an exclude for BoundMethodHandleMetadataSizeTest on IBM JDK 8 where we see this test crashing reliably and holding back the CI pipelines for 3 hours (it seems like the JVM state is in such a bad shape it gets deadlocked on trying to generate the diagnostic files)

Motivation:
Unblock CI pipeline

Additional Notes:
The bug is tracked internally

For Datadog employees:

  • If this PR touches code that signs or publishes builds or packages, or handles
    credentials of any kind, I've requested a review from @DataDog/security-design-and-guidance.
  • This PR doesn't touch any of that.
  • JIRA: PROF-12162

Unsure? Have a question? Request a review!

@jbachorik jbachorik requested a review from zhengyu123 July 22, 2025 09:07
@github-actions
Copy link

github-actions bot commented Jul 22, 2025

🔧 Report generated by pr-comment-cppcheck

CppCheck Report

Errors (2)

Warnings (8)

Style Violations (300)

@github-actions
Copy link

github-actions bot commented Jul 22, 2025

🔧 Report generated by pr-comment-scanbuild

@jbachorik jbachorik force-pushed the jb/exclude_crashing_test branch from c1bf0ed to 3307907 Compare July 23, 2025 15:59
Copy link
Contributor

@zhengyu123 zhengyu123 left a comment

Choose a reason for hiding this comment

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

LGTM

@jbachorik jbachorik merged commit 394a7a0 into main Jul 23, 2025
65 of 95 checks passed
@jbachorik jbachorik deleted the jb/exclude_crashing_test branch July 23, 2025 22:45
@github-actions github-actions bot added this to the 1.29.0 milestone Jul 23, 2025
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.

3 participants