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: add apis to support ragweed #13645

Merged
merged 2 commits into from Mar 4, 2017

Conversation

Projects
None yet
2 participants
@yehudasa
Copy link
Member

yehudasa commented Feb 25, 2017

  • new management api that returns object layout
  • new management api that retrieves zone config (pools)
rgw: rest api to read zone config params
Signed-off-by: Yehuda Sadeh <yehuda@redhat.com>
@oritwas
Copy link
Contributor

oritwas left a comment

looks good , I would separate out the rgw_obj fixes to a separate commit or even pr

@@ -508,7 +508,8 @@ int rgw_build_object_policies(RGWRados *store, struct req_state *s,
}
s->object_acl = new RGWAccessControlPolicy(s->cct);

rgw_obj obj(s->bucket, s->object);

This comment has been minimized.

Copy link
@oritwas

oritwas Feb 26, 2017

Contributor

Maybe we should fix those separately? we will need to backport those

@@ -5500,7 +5501,8 @@ void RGWSetAttrs::execute()
if (op_ret < 0)
return;

rgw_obj obj(s->bucket, s->object);
rgw_obj obj(s->bucket, s->object.name);

This comment has been minimized.

Copy link
@oritwas

oritwas Feb 26, 2017

Contributor

same here

@yehudasa yehudasa force-pushed the yehudasa:wip-rgw-support-ragweed branch from a3ae47c to 2768583 Feb 27, 2017

rgw: new rest api to retrieve object layout
Signed-off-by: Yehuda Sadeh <yehuda@redhat.com>
@yehudasa

This comment has been minimized.

Copy link
Member Author

yehudasa commented Mar 3, 2017

@oritwas separated, anything else, or is it good to go?

@oritwas

oritwas approved these changes Mar 4, 2017

@oritwas oritwas merged commit 60bb2a8 into ceph:master Mar 4, 2017

3 checks passed

Signed-off-by all commits in this PR are signed
Details
Unmodifed Submodules submodules for project are unmodified
Details
default Build finished.
Details
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.