Skip to content

ci: look up Dataproc images in a separate job so the release matrices survive - #351

Merged
tnazarew merged 1 commit into
mainfrom
ci/fix-masked-release-matrices
Nov 4, 2026
Merged

tnazarew merged 1 commit into
mainfrom
ci/fix-masked-release-matrices

Conversation

@tnazarew

@tnazarew tnazarew commented Nov 4, 2026

Copy link
Copy Markdown
Collaborator

Fixes the scheduled release check broken by #348 (run #836).

Cause: #348 added GCP auth (GCP_SA_KEY) to initialize_workflow to look up the newest Dataproc images. The key is a multi-line secret and GitHub masks each line, including the bare { and }. Every JSON matrix output of that job was then dropped (Skip output 'spark_matrix' since it may contain secret), so stage-openlineage-jars got no versions and hive-dataproc failed on fromJson: empty input.

Fix:

  • New find-dataproc-images job (the only one with the GCP key) runs scripts/get_dataproc_images.sh and outputs a brace-free JSON list of [family, image] pairs.
  • initialize_workflow no longer authenticates. select_components.sh compares those pairs with releases.json (env DATAPROC_IMAGES).

Tested locally with the real gcloud lookup: the output has no braces, the detected changes match #836 (2-1-deb11, 2-1-ubu20, 2-2-ubu22), the matrices come out as expected, and empty input gives []. That GitHub keeps the output when the secret is loaded in that job still needs a real run (manual dispatch on this branch).

🤖 Generated with Claude Code

@tnazarew
tnazarew marked this pull request as ready for review November 4, 2026 17:37
… survive

The release check authenticated to GCP in initialize_workflow to find the newest
Dataproc images. GCP_SA_KEY is a multi-line secret and GitHub masks each of its
lines, including the bare "{" and "}", so every JSON matrix output of that job was
dropped ("Skip output 'spark_matrix' since it may contain secret") and the Spark and
Hive jobs got empty matrices (run #836).

The lookup now runs in its own find-dataproc-images job, which outputs a brace-free
list of [family, image] pairs that select_components.sh compares with releases.json.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Signed-off-by: tnazarew <tomasz.nazarewicz@getindata.com>
@tnazarew
tnazarew force-pushed the ci/fix-masked-release-matrices branch from 54c0497 to b8fb188 Compare November 4, 2026 19:31
@tnazarew
tnazarew merged commit cf62b06 into main Nov 4, 2026
10 checks passed
@tnazarew
tnazarew deleted the ci/fix-masked-release-matrices branch November 4, 2026 19:32
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