Skip to content

Remove keikoproj/aws-sdk-go-cache dependency#550

Draft
tekenstam wants to merge 1 commit intomasterfrom
remove-aws-sdk-go-cache
Draft

Remove keikoproj/aws-sdk-go-cache dependency#550
tekenstam wants to merge 1 commit intomasterfrom
remove-aws-sdk-go-cache

Conversation

@tekenstam
Copy link
Copy Markdown
Member

Summary

  • Removes github.com/keikoproj/aws-sdk-go-cache dependency, which relies on AWS SDK Go v1 (EOL July 2025)
  • Removes the CacheConfig field from RollingUpgradeReconciler and all associated cache setup/flush logic
  • Also removes the transitive github.com/karlseguin/ccache/v2 dependency

The cache provided minimal value — only 60s TTLs on DescribeAutoScalingGroups and DescribeLaunchTemplates. AWS SDK v1's built-in retry mechanism with exponential backoff handles throttling without custom caching.

Closes #544

Test plan

  • go build ./... passes (verified locally)
  • go test ./... passes
  • CI checks pass

The aws-sdk-go-cache library depends on AWS SDK Go v1 which reaches
end-of-life in July 2025. The cache provides minimal value (60s TTLs
on DescribeAutoScalingGroups and DescribeLaunchTemplates only), and
the AWS SDK's built-in retry mechanism with exponential backoff
adequately handles throttling without custom caching.

Closes #544

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Signed-off-by: Todd Ekenstam <todd_ekenstam@intuit.com>
@tekenstam tekenstam requested review from a team as code owners March 23, 2026 17:57
@tekenstam tekenstam marked this pull request as draft March 24, 2026 00:07
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.

chore: remove aws-sdk-go-cache dependency

1 participant