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

add bucket usage collector #225

Open
zhucan opened this issue Nov 7, 2022 · 4 comments
Open

add bucket usage collector #225

zhucan opened this issue Nov 7, 2022 · 4 comments

Comments

@zhucan
Copy link

zhucan commented Nov 7, 2022

[root@rook-node01 /]# radosgw-admin usage show --bucket test-bucket5 --uid os-user-ce1dd0c1-f552-46da-9153-c0a785348b0e --categories="put_obj,get_obj" --show-log-entries=false
{
    "summary": [
        {
            "user": "os-user-ce1dd0c1-f552-46da-9153-c0a785348b0e",
            "categories": [
                {
                    "category": "get_obj",
                    "bytes_sent": 425099031,
                    "bytes_received": 0,
                    "ops": 3,
                    "successful_ops": 3
                },
                {
                    "category": "put_obj",
                    "bytes_sent": 0,
                    "bytes_received": 604471055,
                    "ops": 42,
                    "successful_ops": 42
                }
            ],
            "total": {
                "bytes_sent": 425099031,
                "bytes_received": 604471055,
                "ops": 45,
                "successful_ops": 45
            }
        }
    ]
}
@baergj
Copy link
Contributor

baergj commented Nov 8, 2022

👋 I'm going to say the same thing about this as #217 - this could maybe be added behind a enable/disable flag, as the cardinality of this data can get rather high.

Patches welcome, as always!

@zhucan
Copy link
Author

zhucan commented Nov 9, 2022

the usage is printed by cmd "radosgw-admin usage show xxxx", not "radosgw-admin bucket stats".

@baergj
Copy link
Contributor

baergj commented Nov 9, 2022

@zhucan I understand, and that wasn't my point; the issue is that anything that's per bucket can result in a very high number of prometheus metrics being collected each scrape, and this can quickly get folks into trouble, which is why I think it should be behind a flag if it gets implemented.

@zhucan
Copy link
Author

zhucan commented Nov 9, 2022

Thanks for your explain. @baergj

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

No branches or pull requests

2 participants