Skip to content

Feat/improve s3 it - #59

Merged
anavirginianery merged 17 commits into
devfrom
feat/improve-s3-it
Feb 9, 2026
Merged

anavirginianery merged 17 commits into
devfrom
feat/improve-s3-it

Conversation

@anavirginianery

Copy link
Copy Markdown

Improvements to Heuristic for Missing Intelligent-Tiering Recommendation

This PR introduces enhancements to the heuristic used to determine when an S3 bucket should receive a Missing Intelligent-Tiering recommendation.

Previous Behavior

Previously, the system only verified whether a bucket stored objects in the Standard storage class while exhibiting infrequent access patterns.

New Logic

The heuristic has been expanded to evaluate two conditions:

  • Storage Class vs Access Pattern Mismatch
    Detects inconsistencies between how objects are stored and how they are actually accessed.
    Examples:

    • Infrequent access but objects stored in Standard
    • Frequent access but stored in Archival tiers
  • Cost Efficiency Check
    Compares the projected Intelligent-Tiering cost with the current storage cost.
    If Intelligent-Tiering is cheaper, the bucket becomes a candidate.

Buckets satisfying both criteria are now recommended for Intelligent-Tiering.

Dynamic Pricing Integration

This PR also adds support for dynamic pricing by:

  • Querying ClickHouse for historical expenses related to S3 usage.
  • Calling the cloud provider’s pricing API to fetch up-to-date Intelligent-Tiering price details.

@marianezei marianezei left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

lgtm

Comment thread bumiworker/bumiworker/modules/recommendations/s3_intelligent_tiering.py Outdated
@anavirginianery
anavirginianery merged commit e7cd9a2 into dev Feb 9, 2026
1 check passed
@gabrielecalafange
gabrielecalafange deleted the feat/improve-s3-it branch March 24, 2026 13:10
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.

2 participants