Skip to content

Develop#19

Merged
Cro22 merged 2 commits into
mainfrom
develop
May 9, 2026
Merged

Develop#19
Cro22 merged 2 commits into
mainfrom
develop

Conversation

@Cro22

@Cro22 Cro22 commented May 9, 2026

Copy link
Copy Markdown
Owner

No description provided.

Cro22 and others added 2 commits May 9, 2026 12:54
The GitHub Actions runner forwards inputs as `INPUT_<NAME>` env vars
with dashes preserved verbatim — `plan-file` becomes `INPUT_PLAN-FILE`,
not `INPUT_PLAN_FILE` — because only spaces are converted to
underscores. POSIX parameter expansion can't reference names with `-`
(the dash is the default-value operator), so the previous reads of
`${INPUT_PLAN_FILE}` always saw an empty string and the entrypoint
exited with "plan-file input is required" before anything else ran.

Caught by the in-repo self-test workflow added on this branch.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
fix(action): read INPUT_* env vars via printenv to handle dashes
@github-actions

github-actions Bot commented May 9, 2026

Copy link
Copy Markdown

💰 Cloud Cost Impact

Net monthly change: +$7.59 🔴

The entire cost increase stems from the new aws_instance.self_test. If this instance is for a light-duty workload, consider a smaller instance class to optimize costs. The estimate is compute-only, as root block device costs are not included and could add to the total.

Top movers by cost impact

Resource Action Δ Monthly Confidence
aws_instance.self_test 🆕 create +$7.59 low
📋 Full breakdown (1 priced, 0 skipped)

Created (1)

  • aws_instance.self_test — +$7.59
    • Compute: +$7.59
⚠️ Assumptions and caveats
  • Net cost increase this plan
  • Operating system assumed Linux (plan does not specify) (applies to: aws_instance.self_test)
  • Pricing assumes On-Demand (no Reserved Instances or Savings Plans) (applies to: aws_instance.self_test)
  • Root block device unknown, compute-only estimate (applies to: aws_instance.self_test)

Generated by CloudOracle · Confidence: low

@Cro22
Cro22 merged commit 1d5ff3b into main May 9, 2026
5 checks passed
Cro22 added a commit that referenced this pull request Jun 9, 2026
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