Project

General

Profile

Bug #57706

When creating a new user, if the 'uid' is not provided, error reported as 'Permission denied'

Added by wang kevin 2 months ago. Updated 27 days ago.

Status:
Need More Info
Priority:
Normal
Assignee:
Target version:
-
% Done:

0%

Source:
Tags:
Backport:
Regression:
No
Severity:
3 - minor
Reviewed:
Affected Versions:
ceph-qa-suite:
Pull request ID:
Crash signature (v1):
Crash signature (v2):

Description

When creating a new user, if the 'uid' does not contain an error reporting 'Permission denied', we think it is reasonable to report an error related to parameter errors.

Image.png View (4.2 KB) wang kevin, 09/29/2022 01:20 AM

History

#1 Updated by Casey Bodley 2 months ago

  • Assignee set to Ali Maredia

#2 Updated by J. Eric Ivancich 2 months ago

  • Subject changed from When creating a new user, if the 'uid' does not contain an error reporting 'Permission denied' to When creating a new user, if the 'uid' is not provided, error reported as 'Permission denied'

#3 Updated by Ali Maredia 27 days ago

On a branch close to the master branch from a vstart cluster when I try this same scenario I see:

[ali@acadia build]$ ./bin/radosgw-admin -c ceph.conf user create --display-name=test3 --access-key=test3 --secret-key=test3
2022-11-09T17:01:21.651-0500 7f60307a89c0 -1 WARNING: all dangerous and experimental features are enabled.
2022-11-09T17:01:21.657-0500 7f60307a89c0 -1 WARNING: all dangerous and experimental features are enabled.
2022-11-09T17:01:21.657-0500 7f60307a89c0 -1 WARNING: all dangerous and experimental features are enabled.
user.init failed: (22) Invalid argument

Having an 'Invalid Argument' instead of a 'Permission Denied' seems closer to feedback that could help whomever is trying to create a user understand the mistake they made.

Providing even more fine grained feedback like: 'uid not present, add --uid={user name}' in the error message could be helpful but would be not trivial to implement.

#4 Updated by Ali Maredia 27 days ago

  • Status changed from New to Need More Info

Hi Kevin Wang,

Could I get what version of Ceph this issue occurred on? The issue does seem to be resolved in the main branch but I can look to back port a fix to an older version of Ceph.

Also available in: Atom PDF