Skip to content

fix(freshness-deploy): use file:// for historical-cron put-targets#343

Merged
cipher813 merged 2 commits into
mainfrom
fix/freshness-deploy-historical-target-json
May 28, 2026
Merged

fix(freshness-deploy): use file:// for historical-cron put-targets#343
cipher813 merged 2 commits into
mainfrom
fix/freshness-deploy-historical-target-json

Conversation

@cipher813
Copy link
Copy Markdown
Owner

Summary

deploy.sh --bootstrap was tripping on the put-targets shorthand when the Input contains JSON. From PR #339 the historical cron's target carries Input={"mode":"historical"}; the shorthand form Id=N,Arn=...,Input={...} confuses argparse on the embedded quotes + comma.

Caught live 2026-05-28 when --bootstrap re-run partial-fired:

ParamValidation: Error parsing parameter '--targets': Expected: '=', received: '"' for input

Switch to a temp-file JSON spec via file://.

Live state

The historical EB cron IS wired live in AWS (target attached manually via direct CLI during this session with the same JSON). First daily firing at 04:00 UTC tomorrow. This commit ensures the next operator who runs --bootstrap from main doesn't trip the same trap.

Verification

aws events list-targets-by-rule --rule alpha-engine-freshness-monitor-historical-cron \
  --query 'Targets[].{Id:Id,Input:Input}'
[{"Id": "1", "Input": "{\"mode\":\"historical\"}"}]

bash deploy.sh --bootstrap --dry-run
DRY: aws events put-targets --rule alpha-engine-freshness-monitor-historical-cron \
     --targets file:///tmp/tmp.KUaF4o3som --region us-east-1

Composes with

🤖 Generated with Claude Code

cipher813 and others added 2 commits May 28, 2026 09:07
deploy.sh --bootstrap was tripping on the put-targets shorthand when
the Input contains JSON. From PR #339 the historical cron's target
carries Input={"mode":"historical"}; the shorthand form
Id=N,Arn=...,Input={...} confuses argparse on the embedded quotes +
comma.

Caught live 2026-05-28 when --bootstrap re-run partial-fired:
  ParamValidation: Error parsing parameter '--targets': Expected:
  '=', received: '"' for input

Switch to a temp-file JSON spec via file://. The shorthand for the
original 15-min cron stays unchanged because that target has no Input
payload.

The historical EB cron is wired live in AWS — target attached
manually via direct CLI during this session with the same JSON.
First daily firing at 04:00 UTC. This commit ensures the next
operator who runs --bootstrap from main doesn't trip the same trap.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
@cipher813 cipher813 merged commit 83168a3 into main May 28, 2026
1 check passed
@cipher813 cipher813 deleted the fix/freshness-deploy-historical-target-json branch May 28, 2026 16:15
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