From 3450646ce5a3018400ed7f744bfb0761b4ad6909 Mon Sep 17 00:00:00 2001 From: mkultraWasHere Date: Tue, 4 Aug 2026 23:02:51 -0400 Subject: [PATCH 1/3] chore(config): restore generic dreadgoad.yaml defaults PR #344 committed a working dreadgoad.yaml from a one-off deployment: env, provider and region pinned to that deployment, a bespoke environment block, and variant overrides grafted onto the shared staging and test environments. The checked-in config is a template every user starts from, so it should not carry one deployment's state. Restores env: staging, provider: aws, the commented-out region, and idle_timeout: 1200, and drops the deployment-specific variant blocks. Keeps azure in the provider comment list, since that provider support is real. Co-Authored-By: Claude Opus 5 (1M context) --- dreadgoad.yaml | 23 +++++------------------ 1 file changed, 5 insertions(+), 18 deletions(-) diff --git a/dreadgoad.yaml b/dreadgoad.yaml index 6d0d2e0c..14c2d1fe 100644 --- a/dreadgoad.yaml +++ b/dreadgoad.yaml @@ -1,12 +1,12 @@ # DreadGOAD CLI Configuration -env: dreadindex -provider: azure # Provider: aws, azure, proxmox, ludus -region: centralus +env: staging +provider: aws # Provider: aws, azure, proxmox, ludus +# region: us-west-2 # Override provider region (default: from inventory) instance_profile: WarpgateImageBuilderInstanceProfile # IAM instance profile for EC2 Image Builder debug: false max_retries: 3 retry_delay: 30 -idle_timeout: 3600 +idle_timeout: 1200 # log_dir: ~/.ansible/logs/goad # project_root: /path/to/DreadGOAD # Auto-detected if omitted @@ -51,22 +51,9 @@ environments: variant_name: variant-1 vpc_cidr: "10.0.0.0/16" staging: - variant: true - variant_source: ad/GOAD - variant_target: ad/GOAD-dreadindex - variant_name: dreadindex + variant: false vpc_cidr: "10.1.0.0/16" prod: vpc_cidr: "10.2.0.0/16" test: - variant: true - variant_source: ad/GOAD - variant_target: ad/GOAD-dreadindex - variant_name: dreadindex vpc_cidr: "10.8.0.0/16" - dreadindex: - variant: true - variant_source: ad/GOAD - variant_target: ad/GOAD-dreadindex - variant_name: dreadindex - vpc_cidr: "10.1.0.0/16" From 0f44d5539318a41d78da82f0f049d7af3b784641 Mon Sep 17 00:00:00 2001 From: mkultraWasHere Date: Tue, 4 Aug 2026 23:11:38 -0400 Subject: [PATCH 2/3] docs(config): correct the region comment in dreadgoad.yaml The comment described the top-level region as an override defaulting to the inventory. Both halves were wrong: ResolveRegion treats it as the lowest-precedence fallback behind --region/DREADGOAD_REGION and the active environment's region, and the inventory is only consulted by ResolveRegionWithInventory, which just two SSM-adjacent call sites use. Co-Authored-By: Claude Opus 5 (1M context) --- dreadgoad.yaml | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/dreadgoad.yaml b/dreadgoad.yaml index 14c2d1fe..4e7ea02f 100644 --- a/dreadgoad.yaml +++ b/dreadgoad.yaml @@ -1,7 +1,7 @@ # DreadGOAD CLI Configuration env: staging provider: aws # Provider: aws, azure, proxmox, ludus -# region: us-west-2 # Override provider region (default: from inventory) +# region: us-west-2 # Fallback when environments..region is unset; --region / DREADGOAD_REGION win instance_profile: WarpgateImageBuilderInstanceProfile # IAM instance profile for EC2 Image Builder debug: false max_retries: 3 From b888b9340907aabd1beb32430df7a0b3a49a0ba9 Mon Sep 17 00:00:00 2001 From: mkultraWasHere Date: Tue, 4 Aug 2026 23:27:33 -0400 Subject: [PATCH 3/3] chore(config): comment out the hardcoded instance_profile default WarpgateImageBuilderInstanceProfile is a Dreadnode-specific IAM profile that no Terraform in this repo creates, yet the template shipped it as an active default. ami build resolves flag -> cfg.InstanceProfile -> "" and stamps any non-empty value onto every AMI target, and none of the warpgate templates set one themselves, so a new user's build pointed at a profile that does not exist in their account. Commenting it out restores viper's "" default, which lets the instance factory create its own profile. Anyone who needs the named profile can uncomment it or pass --instance-profile. Co-Authored-By: Claude Opus 5 (1M context) --- dreadgoad.yaml | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/dreadgoad.yaml b/dreadgoad.yaml index 4e7ea02f..cb610c79 100644 --- a/dreadgoad.yaml +++ b/dreadgoad.yaml @@ -2,7 +2,7 @@ env: staging provider: aws # Provider: aws, azure, proxmox, ludus # region: us-west-2 # Fallback when environments..region is unset; --region / DREADGOAD_REGION win -instance_profile: WarpgateImageBuilderInstanceProfile # IAM instance profile for EC2 Image Builder +# instance_profile: MyImageBuilderInstanceProfile # IAM instance profile for EC2 Image Builder; empty lets the build create one debug: false max_retries: 3 retry_delay: 30