-
Notifications
You must be signed in to change notification settings - Fork 29
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
core: adding 'radosgw-admin' command support #296
Conversation
23c393a
to
61aef7d
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Mainly some small doc suggestions
cmd/commands/radosgw.go
Outdated
// RadosgwCmd represents the radosgw command | ||
var RadosgwCmd = &cobra.Command{ | ||
Use: "radosgw-admin", | ||
Short: "call a 'radosgw-admin' CLI command with arbitrary args", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Short: "call a 'radosgw-admin' CLI command with arbitrary args", | |
Short: "call a 'radosgw-admin' CLI command", |
docs/radosgw-admin.md
Outdated
## Examples | ||
|
||
```bash | ||
kubectl rook-ceph radosgw-admin user create --display-name="johnny rotten" --uid=johnny |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
rotten? This suggestion might be boring though :)
kubectl rook-ceph radosgw-admin user create --display-name="johnny rotten" --uid=johnny | |
kubectl rook-ceph radosgw-admin user create --display-name="my user" --uid=myuser |
f45cc80
to
0d26a17
Compare
0d26a17
to
872f1c7
Compare
adding support for 'radosgw-admin' command in the cli tool and also adding docs and ci. Signed-off-by: subhamkrai <[email protected]>
872f1c7
to
e89ebf7
Compare
adding support for 'radosgw-admin' command in the cli tool and also adding docs and ci.
Issue resolved by this Pull Request:
Resolves #253
Checklist: