RepoMirrors/ceph
1
0
Fork 0
mirror of https://github.com/ceph/ceph synced 2025-01-19 01:21:49 +00:00
Code Issues Packages Projects Releases Wiki Activity

rgw: remove comments

Browse Source 
Signed-off-by: Yehuda Sadeh <yehuda@redhat.com>
This commit is contained in:
Yehuda Sadeh 2015-11-30 13:57:56 -08:00
parent eda44cd39c
commit 42305044d5
2 changed files with 1 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
src/rgw/rgw_acl_s3.cc
View File

@ -516,7 +516,7 @@ int RGWAccessControlPolicy_S3::rebuild(RGWRados *store, ACLOwner *owner, RGWAcce
ldout(cct, 0) << "ERROR: src_grant.get_id() failed" << dendl;
return -EINVAL;
}
email = u.id; // XXX none of that .to_str() stuff here, but why?
email = u.id;
ldout(cct, 10) << "grant user email=" << email << dendl;
if (rgw_get_user_info_by_email(store, email, grant_user) < 0) {
ldout(cct, 10) << "grant user email not found or other error" << dendl;

2
src/rgw/rgw_bucket.cc
View File

@ -168,7 +168,6 @@ int rgw_bucket_sync_user_stats(RGWRados *store, const string& tenant_name, const
int rgw_link_bucket(RGWRados *store, const rgw_user& user_id, rgw_bucket& bucket, time_t creation_time, bool update_entrypoint)
{
int ret;
// XXX Actually, should we use user_id.tenant when creating bucket?
string& tenant_name = bucket.tenant;
string& bucket_name = bucket.name;
@ -1770,7 +1769,6 @@ public:
if (!exists || old_bci.info.bucket.bucket_id != bci.info.bucket.bucket_id) {
/* a new bucket, we need to select a new bucket placement for it */
// XXX not sure if this is correct -- stolen from Radoslaw; what about get()?
string tenant_name;
string bucket_name;
parse_bucket(entry, tenant_name, bucket_name);