SDP-2091 chore: update cloudformation stacks#1140
Conversation
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: ba174c1e09
ℹ️ About Codex in GitHub
Codex has been enabled to automatically review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
When you sign up for Codex through ChatGPT, Codex can also answer questions or update the PR, like "@codex address that feedback".
There was a problem hiding this comment.
Pull request overview
Updates the cloudformation/eks deployment stack templates and accompanying operator-facing manifests/docs to make SDP’s EKS deployment flow more consistent, secure, and usable (including newer runtimes and Kubernetes version), while pruning legacy/dead config.
Changes:
- Network stack: adds bring-your-own-VPC CIDR support, improves AZ selection, and standardizes resource tags.
- Keys + EKS stacks: reworks key generation/derivation logic, bumps Lambda runtimes, tightens Secrets Manager IAM scope, and bumps EKS Kubernetes version.
- Operator assets: refreshes the EKS deployment README plus example Helm values and ExternalSecret manifests (dev/prod).
Reviewed changes
Copilot reviewed 12 out of 12 changed files in this pull request and generated 4 comments.
Show a summary per file
| File | Description |
|---|---|
| cloudformation/eks/sdp-network-eks.yaml | Adds existing-VPC CIDR parameter, updates AZ selection, standardizes tags, and adjusts VPC CIDR output behavior. |
| cloudformation/eks/sdp-keys.yaml | Removes legacy keys stack template (superseded by EKS-focused template). |
| cloudformation/eks/sdp-keys-eks.yaml | Reworks secrets/key generation logic, bumps Node runtime, and rationalizes secrets created for EKS deployments. |
| cloudformation/eks/sdp-eks.yaml | Bumps Kubernetes version, updates IRSA/IAM scoping for Secrets Manager, and removes unused/broken resources. |
| cloudformation/eks/sdp-database-eks.yaml | Prunes redundant Secrets Manager secrets and aligns parameter paths with namespace/env. |
| cloudformation/eks/README.md | Updates EKS deployment guide, adds clearer steps/snippets, and aligns operator workflow with the updated stacks. |
| cloudformation/eks/helm/values-testnet.yaml | Updates example values (image versions, secret mappings, and config placement) for testnet. |
| cloudformation/eks/helm/values-mainnet.yaml | Fixes mainnet secret paths and updates example values (image versions, config placement). |
| cloudformation/eks/helm/sdp-secrets-prod.yaml | Adds a prod ExternalSecret manifest mapping required keys from Secrets Manager. |
| cloudformation/eks/helm/sdp-secrets-dev.yaml | Updates dev ExternalSecret manifest (API version + prunes orphaned entries + adds admin account). |
| cloudformation/eks/helm/cluster-issuer.yaml | Replaces hard-coded email with a placeholder. |
| .github/copilot-instructions.md | Adds deployment/config-specific review reminders for future PRs. |
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: d2af47c182
ℹ️ About Codex in GitHub
Codex has been enabled to automatically review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
When you sign up for Codex through ChatGPT, Codex can also answer questions or update the PR, like "@codex address that feedback".
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 2b4330aaac
ℹ️ About Codex in GitHub
Codex has been enabled to automatically review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
When you sign up for Codex through ChatGPT, Codex can also answer questions or update the PR, like "@codex address that feedback".
|
Codex usage limits have been reached for code reviews. Please check with the admins of this repo to increase the limits by adding credits. |
What
Updated the cloudformation/eks deployment templates, supporting Helm/ExternalSecret manifests and the deployment README.
A summary of notable changes:
For all stacks and templates: removed redundant/dead parameters, resources and code blocks.
sdp-network-eks.yaml(network stack)sdp-keys-eks.yaml(secrets stack)sdp-eks.yaml(EKS cluster stack)helm/values-testnet.yaml/helm/values-mainnet.yamlhelm/sdp-secrets-dev.yamlsdp-keys-eks.yamlthat have values now imported.README.md.github/copilot-instructions.mdWhy
The
cloudformation/eksfolder has a mix of deployment blocking bugs, inconsistency between keys stack/External Secrets/Helm values, and a README with broken commands.These changes make the EKS deployment guide functional, internally consistent, more secure and better documented for prospective SDP operators.
Known limitations
The below are noted in the README:
Checklist
SDP-1234: Add new featureorChore: Refactor package xyzformat. The Jira ticket code was included if available.CHANGELOG.mdis updated (if applicable)Contract WASM Artifactsworkflow and open a PR to update the WASMs ondev