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

[Customer-BZ][Automation]: radosgw-admin lc process --bucket <bucket>… #657

Merged

Conversation

ckulal
Copy link
Contributor

@ckulal ckulal commented Jan 8, 2025

… doesn't remove objects if versioning is suspended

[Customer-BZ][Automation]: radosgw-admin lc process --bucket doesn't remove objects if versioning is suspended
BZ: https://bugzilla.redhat.com/show_bug.cgi?id=2319199

Log: http://magna002.ceph.redhat.com/ceph-qe-logs/Chaithra/lc_process_with_ver_suspended/test_lc_process_with_versioning_suspended.console.log_final

Failure expected since BZ not fixed yet.

pass logs for some existing configs:
http://magna002.ceph.redhat.com/ceph-qe-logs/Chaithra/cephci-run-Q0APPH/

@ckulal ckulal added DNM close-loop-automation close-loop-automation customer BZ automated as part of close-loop RGW RGW labels Jan 8, 2025
@ckulal ckulal force-pushed the lc_process_with_ver_suspend branch 2 times, most recently from b047b33 to e91db90 Compare January 8, 2025 10:00
@ckulal ckulal force-pushed the lc_process_with_ver_suspend branch 2 times, most recently from 4b6381e to f0740e8 Compare January 9, 2025 16:20
Copy link
Contributor

mergify bot commented Jan 9, 2025

"This pull request now has conflicts with the target branch. Could you please resolve conflicts and force push the corrected changes?"

@ckulal ckulal force-pushed the lc_process_with_ver_suspend branch 3 times, most recently from 13ed8a2 to c321a75 Compare January 9, 2025 16:39
@ckulal ckulal added the DNM label Jan 9, 2025
@ckulal ckulal force-pushed the lc_process_with_ver_suspend branch from b5085cb to 0ebc333 Compare January 10, 2025 04:45
@ckulal ckulal force-pushed the lc_process_with_ver_suspend branch from 0ebc333 to 65d7307 Compare January 10, 2025 04:51
… doesn't remove objects if versioning is suspended

Signed-off-by: Chaithra <ckulal@redhat.com>
@openshift-ci openshift-ci bot added the lgtm Looks good to me label Jan 13, 2025
@mkasturi18 mkasturi18 added the approved Approving PR. label Jan 15, 2025
Copy link
Contributor

openshift-ci bot commented Jan 15, 2025

[APPROVALNOTIFIER] This PR is APPROVED

Approval requirements bypassed by manually added approval.

This pull-request has been approved by: ckulal, mkasturi18, TejasC88, viduship

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@openshift-merge-bot openshift-merge-bot bot merged commit e28cd2a into red-hat-storage:master Jan 15, 2025
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
approved Approving PR. close-loop-automation close-loop-automation customer BZ automated as part of close-loop lgtm Looks good to me pr-verified RGW RGW Tier-2
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants