Skip to content

Inventory of S3 bucket storage usage by types#727

Open
ValentinLeTallec wants to merge 3 commits intoBoavizta:mainfrom
ValentinLeTallec:s3-storage-usage
Open

Inventory of S3 bucket storage usage by types#727
ValentinLeTallec wants to merge 3 commits intoBoavizta:mainfrom
ValentinLeTallec:s3-storage-usage

Conversation

@ValentinLeTallec
Copy link
Contributor

Hello,

This should resolve #420.

To express storage size, I went with size_bytes: u64 but tell me if you prefer another unit.

@ValentinLeTallec
Copy link
Contributor Author

ValentinLeTallec commented Jul 23, 2025

Tests are failing on the api version, but I don't think it should be solved in this PR, so I created #729.

@ValentinLeTallec
Copy link
Contributor Author

ValentinLeTallec commented Jul 24, 2025

For information, the minimal permission I required to run an inventory were :

{
    "Version": "2012-10-17",
    "Statement": [
        {
            "Sid": "CloudScannerWatch",
            "Effect": "Allow",
            "Action": [
                "ec2:DescribeInstances",
                "cloudwatch:GetMetricStatistics",
                "cloudwatch:ListMetrics"
            ],
            "Resource": "*"
        }
    ]
}

I started from no permission and added them one by one from the error messages.

I did not need "cloudwatch:DescribeAlarms" (as mentioned in #402 (comment)) neither before nor after creating an alarm.

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.

Add inventory of S3 buckets

1 participant