Project

General

Profile

Actions

Bug #57706

closed

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

Added by wang kevin over 1 year ago. Updated 7 months ago.

Status:
Can't reproduce
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.


Files

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

Updated by Casey Bodley over 1 year ago

  • Assignee set to Ali Maredia
Actions #2

Updated by J. Eric Ivancich over 1 year 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'
Actions #3

Updated by Ali Maredia over 1 year 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.

Actions #4

Updated by Ali Maredia over 1 year 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.

Actions #5

Updated by Casey Bodley about 1 year ago

bump ^

Actions #6

Updated by Ali Maredia 7 months ago

  • Status changed from Need More Info to Can't reproduce
Actions

Also available in: Atom PDF