Skip to content
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

fix: trace only first recovery log failure #30507

Merged

Conversation

joe-chacko
Copy link
Member

@joe-chacko joe-chacko self-assigned this Jan 9, 2025
@joe-chacko joe-chacko requested review from ngmr and jonhawkes January 9, 2025 10:19
@joe-chacko
Copy link
Member Author

joe-chacko commented Jan 9, 2025

!build (view Open Liberty Personal Build - ❌ completed with errors/failures.)

Note: Target locations of links might be accessible only to IBM employees.

@LibbyBot
Copy link

LibbyBot commented Jan 9, 2025

Code analysis and actions

DO NOT DELETE THIS COMMENT.
  • 1 product code files were changed.
  • Please describe in a separate comment how you tested your changes.

jonhawkes
jonhawkes previously approved these changes Jan 9, 2025
@joe-chacko joe-chacko force-pushed the trace-first-recovery-log-fail branch 2 times, most recently from f2ec6c0 to 3a7f59c Compare January 9, 2025 10:43
@joe-chacko joe-chacko marked this pull request as draft January 9, 2025 10:51
@joe-chacko joe-chacko marked this pull request as ready for review January 13, 2025 17:32
@joe-chacko joe-chacko force-pushed the trace-first-recovery-log-fail branch from 3a7f59c to ca509ec Compare January 13, 2025 17:57
@joe-chacko
Copy link
Member Author

joe-chacko commented Jan 13, 2025

!build (view Open Liberty Personal Build - ❌ completed with errors/failures.)

Note: Target locations of links might be accessible only to IBM employees.

@LibbyBot
Copy link

Code analysis and actions

DO NOT DELETE THIS COMMENT.
  • 1 product code files were changed.
  • Please describe in a separate comment how you tested your changes.

@joe-chacko joe-chacko merged commit 2cceab1 into OpenLiberty:integration Jan 14, 2025
2 of 3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants