Skip to content

HDDS-14862. Log volume failures and database errors as errors#9950

Open
ptlrs wants to merge 2 commits intoapache:masterfrom
ptlrs:HDDS-14862-Log-volume-failures-as-errors
Open

HDDS-14862. Log volume failures and database errors as errors#9950
ptlrs wants to merge 2 commits intoapache:masterfrom
ptlrs:HDDS-14862-Log-volume-failures-as-errors

Conversation

@ptlrs
Copy link
Contributor

@ptlrs ptlrs commented Mar 19, 2026

What changes were proposed in this pull request?

Volume failure is currently logged at INFO level. It should be marked as an ERROR as a volume failing is an actual problem in the system and also searching for ERRORs in the log files should flag this.

What is the link to the Apache JIRA

https://issues.apache.org/jira/browse/HDDS-14862

How was this patch tested?

CI: https://github.com/ptlrs/ozone/actions/runs/23277693312

Copy link
Contributor

@adoroszlai adoroszlai left a comment

Choose a reason for hiding this comment

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

Thanks @ptlrs for working on this.

@errose28 errose28 self-requested a review March 23, 2026 16:23
Copy link
Contributor

@errose28 errose28 left a comment

Choose a reason for hiding this comment

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

Thanks for looking into this @ptlrs.

@ptlrs
Copy link
Contributor Author

ptlrs commented Mar 24, 2026

Thanks for the reviews @adoroszlai @errose28. I have made some changes. Could you please take another look.

@ptlrs ptlrs requested review from adoroszlai and errose28 March 24, 2026 20:14
Copy link
Contributor

@errose28 errose28 left a comment

Choose a reason for hiding this comment

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

LGTM and CI on the fork is green. @ptlrs do you want to take this out of draft?

@ptlrs ptlrs marked this pull request as ready for review March 25, 2026 19:40
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