Thanks to visit codestin.com
Credit goes to Github.com

Skip to content

Conversation

@sam0044
Copy link
Collaborator

@sam0044 sam0044 commented Nov 13, 2025

No description provided.

@sam0044 sam0044 changed the title src/clyso/ceph/ai/rgw: added rgw subcommands for otto draft: src/clyso/ceph/ai/rgw: added rgw subcommands for otto Nov 13, 2025
@sam0044 sam0044 force-pushed the rgw-module branch 8 times, most recently from a6fdbc3 to 331ed88 Compare November 13, 2025 19:27
@sam0044 sam0044 changed the title draft: src/clyso/ceph/ai/rgw: added rgw subcommands for otto src/clyso/ceph/ai/rgw: added rgw subcommands for otto Nov 13, 2025
@JoshuaGabriel
Copy link
Collaborator

just pasting the cli changes

(otto) jblanch@ubuntu-dev:~/github/otto$ otto rgw -h
usage: otto rgw [-h] {find-missing,incomplete-multipart-list,user-df,user-quota} ...

options:
  -h, --help            show this help message and exit

subcommands:
  RGW subcommands

  {find-missing,incomplete-multipart-list,user-df,user-quota}
    find-missing        Search for rgw objects missing rados objects in data pool
    incomplete-multipart-list
                        List incomplete multipart uploads
    user-df             Calculate user's usage stats
    user-quota          List user's quota

Copy link
Collaborator

@JoshuaGabriel JoshuaGabriel left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

test

@JoshuaGabriel
Copy link
Collaborator

sorry for the multiple reviews, I was testing some pr review tool

@JoshuaGabriel
Copy link
Collaborator

the CLI command looks good overall, haven't had a chance to actually test the commands. It would be nice if you tested this out yourself already if we can get some example outputs under docs/, or we can do this together when we get a rgw cluster.

also would be nice to get some tests but not sure what that would look like, maybe some of the json output then pipe it into the commands?

@sam0044
Copy link
Collaborator Author

sam0044 commented Nov 17, 2025

@JoshuaGabriel I do have a rgw cluster on my work proxmox, I can try testing this out. Should I add tests in this PR or create a separate one after this is merged?

@JoshuaGabriel
Copy link
Collaborator

@JoshuaGabriel I do have a rgw cluster on my work proxmox, I can try testing this out. Should I add tests in this PR or create a separate one after this is merged?

add it to this one

@sam0044 sam0044 force-pushed the rgw-module branch 3 times, most recently from 0724bb3 to bc6d5b5 Compare November 18, 2025 23:28
@JoshuaGabriel
Copy link
Collaborator

looks good, could you add some basic tests or some short docs on how to use it

@sam0044
Copy link
Collaborator Author

sam0044 commented Nov 19, 2025

yes will add those too

@sam0044 sam0044 merged commit 45b6b75 into main Nov 20, 2025
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants