Skip to content

chore/aws sdk upgrade#1733

Merged
gambol99 merged 2 commits intodevelopfrom
chore/aws_sdk_upgrade
Jan 5, 2026
Merged

chore/aws sdk upgrade#1733
gambol99 merged 2 commits intodevelopfrom
chore/aws_sdk_upgrade

Conversation

@gambol99
Copy link
Member

@gambol99 gambol99 commented Jan 5, 2026

  • fix(eks): resolve test failures after AWS SDK v2 migration
  • feat: upgraded to the latest aws sdk

gambol99 and others added 2 commits January 5, 2026 18:55
Fixed test compilation errors in the EKS preloader package that occurred
after migrating from AWS SDK Go v1 to v2:

- Removed dot import causing import cycle in loader_test.go
- Updated boolean fields to use direct values instead of pointers
  (EndpointPrivateAccess and EndpointPublicAccess)

The AWS SDK v2 uses direct bool values instead of *bool for these fields,
which is a breaking change from v1.

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>
@gambol99 gambol99 force-pushed the chore/aws_sdk_upgrade branch from 177ecdd to 7c7a065 Compare January 5, 2026 18:55
@gambol99 gambol99 merged commit ba89294 into develop Jan 5, 2026
9 checks passed
@gambol99 gambol99 deleted the chore/aws_sdk_upgrade branch January 5, 2026 20:50
gambol99 added a commit that referenced this pull request Jan 22, 2026
* fix(eks): resolve test failures after AWS SDK v2 migration

Fixed test compilation errors in the EKS preloader package that occurred
after migrating from AWS SDK Go v1 to v2:

- Removed dot import causing import cycle in loader_test.go
- Updated boolean fields to use direct values instead of pointers
  (EndpointPrivateAccess and EndpointPublicAccess)

The AWS SDK v2 uses direct bool values instead of *bool for these fields,
which is a breaking change from v1.

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>

* feat: upgraded to the latest aws sdk

---------

Co-authored-by: Claude Sonnet 4.5 <noreply@anthropic.com>
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.

1 participant