fix(freshness-deploy): use file:// for historical-cron put-targets#343
Merged
Conversation
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>
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
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
deploy.sh --bootstrapwas tripping on theput-targetsshorthand when the Input contains JSON. From PR #339 the historical cron's target carriesInput={"mode":"historical"}; the shorthand formId=N,Arn=...,Input={...}confuses argparse on the embedded quotes + comma.Caught live 2026-05-28 when
--bootstrapre-run partial-fired: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
--bootstrapfrom main doesn't trip the same trap.Verification
Composes with
🤖 Generated with Claude Code