Skip to content

Conversation

janitbidhan
Copy link
Member

@janitbidhan janitbidhan commented Feb 7, 2024

Identifies buckets with relative increase in cost on enabling the soft-delete. The relative increase in cost of using soft delete is calculated by combining the storage/v2/deleted_bytes metric with the existing storage/v2/total_byte_seconds metric.

Relative cost of each bucket = ('soft delete retention duration'
× 'deleted bytes' / 'total bytes seconds' )
x 'cost of storing in storage class'
x 'ratio of storage class'.

Description

Fixes #11196

Note: Before submitting a pull request, please open an issue for discussion if you are not associated with Google.

Checklist

* Combines storage/v2/deleted_bytes and storage/v2/total_byte_seconds for analysis.
* Formula for relative cost:
   ('soft delete retention duration' × 'deleted bytes' / 'total bytes')
   x 'cost of storing in storage class' x 'ratio of storage class'
@janitbidhan janitbidhan requested review from a team as code owners February 7, 2024 21:53
Copy link

snippet-bot bot commented Feb 7, 2024

Here is the summary of changes.

You are about to add 1 region tag.

This comment is generated by snippet-bot.
If you find problems with this result, please file an issue at:
https://github.com/googleapis/repo-automation-bots/issues.
To update this comment, add snippet-bot:force-run label or use the checkbox below:

  • Refresh this comment

@product-auto-label product-auto-label bot added samples Issues that are directly related to samples. api: storage Issues related to the Cloud Storage API. labels Feb 7, 2024
Copy link
Contributor

@engelke engelke left a comment

Choose a reason for hiding this comment

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

I'm not going to demand changes, as, except for the copyright date, none of these suggestions are essential. I'll leave it to you do decide which of them to include. Request a re-review if you want after making changes, but I don't insist on it.

@janitbidhan janitbidhan requested a review from engelke February 12, 2024 22:04
Copy link
Contributor

@engelke engelke left a comment

Choose a reason for hiding this comment

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

LGTM again. Thanks for the changes.

@janitbidhan janitbidhan reopened this Feb 12, 2024
@engelke engelke merged commit 207aaa4 into GoogleCloudPlatform:main Feb 12, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
api: storage Issues related to the Cloud Storage API. samples Issues that are directly related to samples.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Estimating storage cost impact of soft delete feature in storage service.
3 participants