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

jewel: rgw: radosgw-admin: EEXIST messages for create operations #9268

Merged
merged 1 commit into from Jul 12, 2016

Conversation

theanalyst
Copy link
Member

@theanalyst theanalyst commented May 23, 2016

currently for any admin operations like user create etc. you would
always see:

`RGWZoneParams::create(): error creating default zone params: (17) File
Exists`

in stdout as the debug level is set to 0, which doesn't make much sense
for an end user, so skip the error message, callers of the function handle
the error message anyway, so we skip it here

Fixes: http://tracker.ceph.com/issues/15720
Signed-off-by: Abhishek Lekshmanan <abhishek@suse.com>
(cherry picked from commit 2ace9d7)
@theanalyst theanalyst self-assigned this May 23, 2016
@theanalyst theanalyst added this to the kraken milestone May 23, 2016
@theanalyst theanalyst modified the milestones: jewel, kraken May 23, 2016
@theanalyst theanalyst added rgw and removed core labels May 23, 2016
@theanalyst theanalyst changed the title radosgw-admin: EEXIST messages for create operations jewel: radosgw-admin: EEXIST messages for create operations May 23, 2016
@ghost
Copy link

ghost commented May 31, 2016

test this please

@ghost
Copy link

ghost commented May 31, 2016

test this please (bluefs mount crash and /tmp/virtualenv leftovers)

@ghost
Copy link

ghost commented Jun 29, 2016

@cbodley does this backport look good to merge ? It passed the rgw suite http://tracker.ceph.com/issues/16344#note-6 (except for one failure which I believe to be unrelated).

ghost pushed a commit that referenced this pull request Jun 30, 2016
…reate operations

Reviewed-by: Loic Dachary <ldachary@redhat.com>
ghost pushed a commit that referenced this pull request Jul 1, 2016
…reate operations

Reviewed-by: Loic Dachary <ldachary@redhat.com>
@cbodley cbodley merged commit bb94997 into ceph:jewel Jul 12, 2016
@ghost ghost changed the title jewel: radosgw-admin: EEXIST messages for create operations jewel: rgw: radosgw-admin: EEXIST messages for create operations Aug 25, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
2 participants