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: Let the object stat command be shown in the usage #13291

Merged
merged 1 commit into from Feb 21, 2017

Conversation

prallabh
Copy link
Contributor

@prallabh prallabh commented Feb 7, 2017

Fixes: http://tracker.ceph.com/issues/19013
Signed-off-by: Pavan Rallabhandi PRallabhandi@walmartlabs.com

Copy link
Member

@theanalyst theanalyst left a comment

Choose a reason for hiding this comment

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

a corresponding test needs to be added in test/cli/radosgw-admin

Copy link
Member

@theanalyst theanalyst left a comment

Choose a reason for hiding this comment

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

lgtm

Copy link
Member

@yehudasa yehudasa left a comment

Choose a reason for hiding this comment

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

@prallabh it should be its and not it's

@prallabh
Copy link
Contributor Author

prallabh commented Feb 8, 2017

@yehudasa addressed

@cbodley
Copy link
Contributor

cbodley commented Feb 15, 2017

will this need backport?

@prallabh
Copy link
Contributor Author

@cbodley not sure if that question was for me, but I don't mind backporting once this gets merged in master.

@cbodley
Copy link
Contributor

cbodley commented Feb 20, 2017

@prallabh i opened a tracker issue so we can track the backport. can you please add Fixes: http://tracker.ceph.com/issues/19013 to your commit message?

Fixes: http://tracker.ceph.com/issues/19013
Signed-off-by: Pavan Rallabhandi <PRallabhandi@walmartlabs.com>
@prallabh
Copy link
Contributor Author

prallabh commented Feb 21, 2017

@cbodley thanks for the tracker, have updated the commit message.

@cbodley cbodley merged commit 70d1f90 into ceph:master Feb 21, 2017
@prallabh prallabh deleted the obj_stat branch March 2, 2017 10:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
5 participants