feat(xtra-test): consolidate TEST onto a single t3.medium node - #280
Merged
Conversation
TEST does not require HA (fail-fast by design). Collapse the two-node setup (system + app) into a single t3.medium node group. - Enable VPC CNI prefix delegation (new enable_prefix_delegation flag, default false so prod/sandbox are unchanged) and set kubelet maxPods=110 via an AL2023 launch template, so one t3.medium can host the full workload. - Remove the system node group; pin ingress-nginx and metrics-server to nodepool=app and scale them to 1 replica. - Drop cluster-autoscaler (pointless on a fixed 1-node group). - Ignore RDS engine_version changes (auto_minor_version_upgrade is on, so RDS bumps the minor version out of band) to stop plan drift. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
rohit-joy
approved these changes
Sep 2, 2026
arielr-lt
added a commit
that referenced
this pull request
Sep 3, 2026
…281) Mirror the TEST consolidation for SANDBOX. SANDBOX does not require HA (fail-fast by design); collapse the two-node setup (system + app) into a single t3.medium node group. - Enable VPC CNI prefix delegation (enable_prefix_delegation flag, default false so prod is unchanged) and set kubelet maxPods=110 via an AL2023 launch template, so one t3.medium can host the full workload. - Remove the system node group; pin ingress-nginx and metrics-server to nodepool=app and scale them to 1 replica. - Drop cluster-autoscaler (pointless on a fixed 1-node group). - Ignore RDS engine_version changes (auto_minor_version_upgrade is on) to stop plan drift. Note: the shared modules/eks change is duplicated from #280 and will de-dupe on rebase once #280 merges to main. Co-authored-by: Ariel Rolfo <arielr-lt+username@users.noreply.github.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Per client request, reduce the TEST environment to a single VM (no HA — TEST should surface failures fast). This collapses the two-node setup (separate
system+appnode groups) into one consolidated t3.medium node group.This change is already applied to the live
ctdl-xtra-testcluster (in two phases, rolling); this PR brings the committed IaC in line with the running state.Changes
Node consolidation
enable_prefix_delegationmodule flag (defaultfalse→ prod/sandbox unchanged). Enabled on TEST to raise per-node pod capacity.maxPods: 110so one t3.medium can host the full workload (default cap is 17).systemnode group; singleappnode group pinned tomin=max=desired=1.ingress-nginxandmetrics-serverfromnodepool: system→nodepool: app, scaled to 1 replica.cluster-autoscaler(nothing to scale on a fixed single node); removed its manifest and install step.