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

HDDS-12010. Refactor check for running service in ozone repair #7758

Merged
merged 1 commit into from
Jan 30, 2025

Conversation

adoroszlai
Copy link
Contributor

What changes were proposed in this pull request?

Offline repair tools check if Ozone service, e.g. OM is running. This PR centralizes the check, so that tools only need to declare which service they apply to.

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

How was this patch tested?

https://github.com/adoroszlai/ozone/actions/runs/12986816805

@sarvekshayr @Tejaskriya

@adoroszlai adoroszlai added the tools Tools that helps with debugging label Jan 27, 2025
@adoroszlai adoroszlai self-assigned this Jan 27, 2025
Copy link
Contributor

@peterxcli peterxcli left a comment

Choose a reason for hiding this comment

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

👍

Copy link
Contributor

@sarvekshayr sarvekshayr left a comment

Choose a reason for hiding this comment

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

Thanks @adoroszlai for the patch. LGTM.

@nandakumar131 nandakumar131 merged commit 34792ed into apache:master Jan 30, 2025
42 checks passed
@nandakumar131
Copy link
Contributor

Thanks @adoroszlai for the contribution!
Thanks @peterxcli & @sarvekshayr for the reviews.

@adoroszlai adoroszlai deleted the HDDS-12010 branch January 30, 2025 06:53
@adoroszlai
Copy link
Contributor Author

Thanks for the reviews.

nandakumar131 pushed a commit to nandakumar131/ozone that referenced this pull request Feb 10, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
tools Tools that helps with debugging
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants