feat: aws package, so builder and signet_node are isolated to k8s resources

[rswanson]

No description provided.

[rswanson]

• feat: manual triggered release workflow #13
• fix: consistently being validated #10
• fix: dry constants #9
• feat: quincey #8
• feat: consistent labeling of all resources #5
• feat: aws package, so builder and signet_node are isolated to k8s resources #4 👈 (View in Graphite)
• feat: utils package for better DRY #3
• feat: signet-node #2
• main

This stack of pull requests is managed by Graphite. Learn more about stacking.

[Copilot]

Pull Request Overview

This PR extracts AWS IAM/KMS resource management out of the builder package into a dedicated pkg/aws module, removing inline IAM/KMS types and creation logic from pkg/builder and adding shared types, factory functions, and tests under pkg/aws.

• Removed inline IAM/KMS types, helpers, and tests from pkg/builder
• Added pkg/aws/types.go with policy document definitions
• Introduced pkg/aws/iam.go with CreateIAMResources and CreateKMSPolicy, plus one unit test

Reviewed Changes

Copilot reviewed 7 out of 7 changed files in this pull request and generated 2 comments.

Show a summary per file

File	Description
pkg/builder/types.go	Removed AWS IAM imports and type definitions
pkg/builder/helpers.go	Deleted CreateKMSPolicy helper
pkg/builder/helpers_test.go	Removed now-obsolete KMS policy unit tests
pkg/builder/builder.go	Stripped out IAM role/policy creation code
pkg/aws/types.go	Added shared IAM/KMS policy type definitions
pkg/aws/iam.go	New AWS IAM/KMS resource factory functions
pkg/aws/iam_test.go	Added test for KMS policy creation

Comments suppressed due to low confidence (2)

pkg/builder/builder.go:38

• IAM resource creation was removed from NewBuilder but no new call to aws.CreateIAMResources was added, so the component no longer provisions required IAM roles and policies. Consider invoking CreateIAMResources and integrating its outputs into the builder component.

// Create ConfigMap for environment variables

pkg/aws/iam.go:19

• CreateIAMResources is not covered by any unit tests. Consider adding tests to validate IAM role creation, policy generation, and attachment logic.

func CreateIAMResources(

[rswanson]

Merge activity

• May 29, 3:22 PM UTC: @rswanson merged this pull request with Graphite.