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

Added error handle to orphaned objects monitoring #247

Merged
merged 4 commits into from
Nov 5, 2024

Conversation

NikitaUnisikhin
Copy link
Contributor

No description provided.

@NikitaUnisikhin NikitaUnisikhin force-pushed the orp-objects-error-field branch from eb5cca9 to b65b633 Compare October 31, 2024 07:34
@NikitaUnisikhin NikitaUnisikhin force-pushed the orp-objects-error-field branch from b65b633 to 351fcdf Compare October 31, 2024 08:06
@NikitaUnisikhin NikitaUnisikhin changed the title Added ORPHANED_OBJECTS_ERROR_MSG_FIELD [no publish] Added ORPHANED_OBJECTS_ERROR_MSG_FIELD [WIP] Oct 31, 2024
@NikitaUnisikhin NikitaUnisikhin force-pushed the orp-objects-error-field branch 14 times, most recently from d14b1f7 to fb29816 Compare November 1, 2024 08:24
@NikitaUnisikhin NikitaUnisikhin force-pushed the orp-objects-error-field branch from 9a9b434 to e93101e Compare November 1, 2024 08:59
@NikitaUnisikhin NikitaUnisikhin changed the title Added ORPHANED_OBJECTS_ERROR_MSG_FIELD [WIP] Added error handle to orphaned objects clean Nov 1, 2024
@NikitaUnisikhin NikitaUnisikhin changed the title Added error handle to orphaned objects clean Added error handle to orphaned objects monitoring Nov 1, 2024
@@ -195,3 +211,10 @@ def _table_formatter(stats):
print_response(
ctx, clean_stats, default_format="table", table_formatter=_table_formatter
)


def create_orphaned_objects_state(total_size: int, error_msg: str) -> dict:
Copy link
Contributor

Choose a reason for hiding this comment

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

How about creating a dataclass for this purpose with methods to_json() and from_json() ?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I agree, done

@NikitaUnisikhin NikitaUnisikhin force-pushed the orp-objects-error-field branch 2 times, most recently from 4794cbe to 9aee5e8 Compare November 3, 2024 11:40
@NikitaUnisikhin NikitaUnisikhin force-pushed the orp-objects-error-field branch from 9aee5e8 to a12d555 Compare November 3, 2024 11:53
@aalexfvk aalexfvk merged commit d49b7c9 into main Nov 5, 2024
24 of 25 checks passed
@aalexfvk aalexfvk deleted the orp-objects-error-field branch November 5, 2024 08:50
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.

2 participants