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

rgw: udpate radosgw-admin usage with bi purge #18245

Merged
merged 1 commit into from Oct 13, 2017

Conversation

Projects
None yet
3 participants
@yaozongyou
Copy link
Contributor

commented Oct 11, 2017

Signed-off-by: Yao Zongyou yaozongyou@vip.qq.com

@yaozongyou yaozongyou force-pushed the yaozongyou:wip-radosgw-admin-bi-fix branch from 5c0c8db to fc02ca3 Oct 11, 2017

@yaozongyou yaozongyou changed the title rgw: udpate radosgw-admin usage with bi purge. rgw: udpate radosgw-admin usage with bi purge Oct 11, 2017

@yaozongyou

This comment has been minimized.

Copy link
Contributor Author

commented Oct 11, 2017

retest this please

@joscollin joscollin added the rgw label Oct 11, 2017

@joscollin
Copy link
Member

left a comment

Please fix:

/home/jenkins-build/build/workspace/ceph-pull-requests/src/test/cli/radosgw-admin/help.t: failed
--- /home/jenkins-build/build/workspace/ceph-pull-requests/src/test/cli/radosgw-admin/help.t
+++ /home/jenkins-build/build/workspace/ceph-pull-requests/src/test/cli/radosgw-admin/help.t.err
@@ -30,6 +30,7 @@
     bi get                     retrieve bucket index object entries
     bi put                     store bucket index object entries
     bi list                    list raw bucket index entries
+    bi purge                   purge bucket index entries
     object rm                  remove object
     object stat                stat an object for its metadata
     object unlink              unlink object from bucket index
rgw: udpate radosgw-admin usage with bi purge.
Signed-off-by: Yao Zongyou <yaozongyou@vip.qq.com>

@yaozongyou yaozongyou force-pushed the yaozongyou:wip-radosgw-admin-bi-fix branch from fc02ca3 to 8b27df9 Oct 11, 2017

@yaozongyou

This comment has been minimized.

Copy link
Contributor Author

commented Oct 12, 2017

retest this please

1 similar comment
@yaozongyou

This comment has been minimized.

Copy link
Contributor Author

commented Oct 12, 2017

retest this please

@yaozongyou

This comment has been minimized.

Copy link
Contributor Author

commented Oct 12, 2017

@joscollin fixed, please check.

@yuriw

This comment has been minimized.

Copy link
Contributor

commented Oct 12, 2017

@yuriw yuriw merged commit a8e91ce into ceph:master Oct 13, 2017

5 checks passed

Docs: build check OK - docs built
Details
Signed-off-by all commits in this PR are signed
Details
Unmodified Submodules submodules for project are unmodified
Details
make check make check succeeded
Details
make check (arm64) make check succeeded
Details

@yaozongyou yaozongyou deleted the yaozongyou:wip-radosgw-admin-bi-fix branch Oct 14, 2017

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.