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.

Sign up for GitHub

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

Jump to bottom

[Tier-4]: CEPH-83572752 - RGW multisite sync Error log #686

Merged
merged 1 commit into from
Mar 7, 2025
Merged

[Tier-4]: CEPH-83572752 - RGW multisite sync Error log #686

merged 1 commit into from
Mar 7, 2025

Conversation

anrao19
Copy link
Contributor

@anrao19 anrao19 commented Mar 6, 2025 •
edited
Loading

log :
http://magna002.ceph.redhat.com/cephci-jenkins/Anuchaithra/test_sync_error_list.console.log
http://magna002.ceph.redhat.com/cephci-jenkins/Anuchaithra/test_sync_error_list_pri.console.log

Performing radosgw-admin sync error list in the ceph cluster, if error exist we can trim error using sync error trim. But trimming the error is not recommended as error can be a genuine issue, So in this pr if radosgw-admin sync error list does not have entries as empty that means sync error exist in cluster and failing the testcase.

@anrao19 anrao19 requested a review from a team March 6, 2025 05:17
@anrao19 anrao19 changed the title [Tier-3]: CEPH-83572752 - RGW multisite sync Error log [Tier-4]: CEPH-83572752 - RGW multisite sync Error log Mar 6, 2025
@anrao19 anrao19 mentioned this pull request Mar 6, 2025
Merged
7 tasks
viduship
viduship reviewed Mar 6, 2025
View reviewed changes
Copy link
Contributor

@viduship viduship left a comment

Choose a reason for hiding this comment

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

@anrao19,

Please explain the PR description with the goal of the PR and what it addresses.

@anrao19
Copy link
Contributor Author

anrao19 commented Mar 6, 2025

@anrao19,

Please explain the PR description with the goal of the PR and what it addresses.

@viduship
description updated with testcase discription

@anrao19 anrao19 requested review from viduship and a team March 6, 2025 11:08
@TejasC88
Copy link
Contributor

TejasC88 commented Mar 6, 2025

LGTM

@mergify mergify bot merged commit 3dd5f19 into red-hat-storage:master Mar 7, 2025
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@viduship viduship viduship approved these changes

@ckulal ckulal ckulal approved these changes

@yuva-gayam yuva-gayam yuva-gayam approved these changes

@mkasturi18 mkasturi18 Awaiting requested review from mkasturi18 mkasturi18 is a code owner

@TejasC88 TejasC88 Awaiting requested review from TejasC88 TejasC88 is a code owner

Assignees
No one assigned
Labels
pr-verified RGW RGW Tier-4
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
5 participants
@anrao19 @TejasC88 @viduship @ckulal @yuva-gayam