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

doc: Correcting the remove bucket example and adding bucket link/unlink examples #12460

Merged
merged 1 commit into from Apr 26, 2017

Conversation

Projects
None yet
2 participants
@umullan
Contributor

umullan commented Dec 13, 2016

Signed-off-by: Uday Mullangi umullangi@walmartlabs.com

@umullan umullan changed the title from doc: Correcting the bucket unlink example with correct syntax to doc: Correcting the remove bucket example and adding bucket link/unlink examples Dec 13, 2016

@tchaikov

This comment has been minimized.

Contributor

tchaikov commented Apr 26, 2017

@umullan could you update your Signed-off-by line so it contains your name properly. see https://github.com/ceph/ceph/blob/master/SubmittingPatches.rst#1-sign-your-work thanks.

@tchaikov

aside from the signed-off-by line. lgtm.

@tchaikov tchaikov self-assigned this Apr 26, 2017

@umullan

This comment has been minimized.

Contributor

umullan commented Apr 26, 2017

Updated Signed-off-by in required format.

doc: Correcting the bucket unlink example with correct syntax
Signed-off-by: Uday Mullangi <umullangi@walmartlabs.com>

@tchaikov tchaikov merged commit b833ba7 into ceph:master Apr 26, 2017

2 of 4 checks passed

arm build Build triggered. sha1 is merged.
Details
default Build started sha1 is merged.
Details
Signed-off-by all commits in this PR are signed
Details
Unmodifed Submodules submodules for project are unmodified
Details

@umullan umullan deleted the umullan:bucketunlink branch Apr 26, 2017

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment