Skip to content

fix(28-21): drop non-existent google_batch_job resource; keep fleet spec as docs#94

Merged
helloiamvu merged 1 commit into
mainfrom
claude/relaxed-bhabha-d105ee
Jul 3, 2026
Merged

fix(28-21): drop non-existent google_batch_job resource; keep fleet spec as docs#94
helloiamvu merged 1 commit into
mainfrom
claude/relaxed-bhabha-d105ee

Conversation

@helloiamvu

Copy link
Copy Markdown
Member

What

infra/batch.tf declared resource "google_batch_job" "weather_backfill", but google_batch_job is not a real resource type in the hashicorp/google or hashicorp/google-beta provider (verified against the pinned 6.50.0 binary — it has google_storage_batch_operations_job and google_dataproc_batch, but no google_batch_job). GCP Cloud Batch is submit-only and has no first-class Terraform resource.

So tofu -chdir=infra validate failed with:

The provider hashicorp/google-beta does not support resource type "google_batch_job".

…blocking any apply of the Phase 28 platform. Pre-existing (merged in #92), independent of the phase28/deploy-runtime-layer branch.

Fix (option a — the workflow is already the submit path)

  • Removed the phantom google_batch_job resource.
  • Preserved the shard / Spot / duration / secret / marker-bucket spec as a reviewable comment block. The real submit path is run-weather-backfill.ymlgcloud batch jobs submit --config (the correct API for submit-only Cloud Batch).
  • Lockstep fix: documented task_count = 65 (Kalshi∪Polymarket roster of 66 minus the non-satellite HKO) to match the workflow's TASK_COUNT=65. The committed file had drifted to 66.
  • Kept the durable google_storage_bucket.backfill_progress + its ..._iam_member.backfill_progress_rw — those are real standing resources.

Verification

Offline: tofu -chdir=infra init -backend=false + tofu -chdir=infra validate (provider hashicorp/google 6.50.0).

  • Before: 4 errors — batch.tf:72 (google_batch_job) + three cloud_run.tf max_instance_count.
  • After: batch.tf errors = 0. tofu fmt -check does not flag batch.tf.

tofu validate is not yet fully green: the only remaining errors are the three cloud_run.tf top-level scaling { max_instance_count } misplacements — a separate task, intentionally not touched here to avoid conflicting with that work. Once both land, validate goes green.

🤖 Generated with Claude Code

…pec as docs

google_batch_job is not a real resource type in the hashicorp/google or
google-beta provider (verified vs pinned 6.50.0) — Cloud Batch is submit-only,
so `tofu -chdir=infra validate` failed with "provider hashicorp/google-beta
does not support resource type google_batch_job", blocking any Phase 28 apply.
Pre-existing (merged in #92), independent of the phase28/deploy-runtime-layer
branch.

Remove the phantom resource and preserve the shard/Spot/duration/secret/
marker-bucket spec as a reviewable comment. run-weather-backfill.yml is the
real submit path (`gcloud batch jobs submit --config`). Document task_count=65
(Kalshi∪Polymarket roster minus non-satellite HKO) to match the workflow's
TASK_COUNT=65 — the committed file had drifted to 66. The durable progress
bucket + its IAM binding stay as standing resources.

Eliminates the batch.tf validate error; the remaining cloud_run.tf
max_instance_count errors are a separate task.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
@helloiamvu helloiamvu requested a review from Tarabcak July 3, 2026 19:56
@github-actions

github-actions Bot commented Jul 3, 2026

Copy link
Copy Markdown

Parity ticket gate: PASSED

parity-ticket-check: PR does not touch parity-trigger surface; gate skipped.

See CROSS-SDK-SYNC.md §2 for the workflow.

@helloiamvu helloiamvu merged commit 74d4f17 into main Jul 3, 2026
11 checks passed
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