8000 feat(ci): add example container image build step for nightly clickhouse metrics job by janzak-dev · Pull Request #1056 · google/xmc · GitHub
[go: up one dir, main page]

Skip to content

feat(ci): add example container image build step for nightly clickhouse metrics job - #1056

Merged
janzak-dev merged 1 commit into
mainfrom
feat/nightly-ci-build-example-images
Oct 2, 2026
Merged

janzak-dev merged 1 commit into
mainfrom
feat/nightly-ci-build-example-images

Conversation

@janzak-dev
@janzak-dev janzak-dev commented Oct 2, 2026 •
Copy link
Copy Markdown
Collaborator

Summary

Introduces a dedicated prerequisite CI job build-example-images in nightly-examples-e2e.yml that builds and pushes container images required by nightly example workloads before executing the test matrix, starting with clickhouse-metrics-job.

Stack Context: Layer 1 of 3 (Prerequisite Image Build Step)

Note

This PR is Layer 1 (Base) of a 3-PR stack (GitHub Stack #1058):

Motivation & Context

Nightly example verification workloads that execute custom container images (such as clickhouse_metrics_job) require those images to be present in Artifact Registry. Running a build-and-push step prior to the matrix execution ensures the worker containers are up to date and eliminates missing image failures.

Key Changes

  • Prerequisite Job: Added build-example-images to .github/workflows/nightly-examples-e2e.yml running on arc-runner-set with Workload Identity Federation authentication and Artifact Registry Docker auth.
  • ClickHouse Worker Image: Builds and pushes us-docker.pkg.dev/${PROJECT_ID}/xmc/clickhouse-metrics-job:latest using examples/clickhouse_metrics_job/Dockerfile.
  • Workflow Dependency Graph: Configured run-and-verify-examples to depend on build-example-images, and updated report-nightly-failure to alert on failure in build-example-images.

Assumptions

None.

Verification & Testing

  • Live E2E Workflow Run: Run #36995644267 — Build Example Container Images succeeded in 34s, followed by 100% green execution of Run & Verify: ClickHouse Metrics Job (1m17s) and Run & Verify: Simple Job (3 Multi-Unit Batch) (2m34s).
  • Pre-commit checks passed (./.githooks/pre-commit).
  • Bazel tests passed (bazel test //tools/nightly_tests/...).

Related Issues / Bugs

Part of the nightly E2E verification test suite stack (#1058).

TAG=agy
CONV=acf882de-1561-404c-ba2e-ae0940451165

@gemini-code-assist
Copy link
Copy Markdown

Note

Gemini is unable to generate a review for this pull request due to the file types involved not being currently supported.

@github-actions
github-actions Bot requested a review from andrzejfudro October 2, 2026 10:05
@janzak-dev
janzak-dev marked this pull request as draft October 2, 2026 10:07
@janzak-dev
janzak-dev added this pull request to stack #1058 October 2, 2026 10:15
@janzak-dev
janzak-dev marked this pull request as ready for review October 2, 2026 10:37
@github-actions
github-actions Bot requested a review from VLukiantsev October 2, 2026 10:37
@VLukiantsev
VLukiantsev removed their request for review October 2, 2026 10:38
@janzak-dev
janzak-dev added this pull request to the merge queue Oct 2, 2026
Merged via the queue into main with commit c9bbc02 Oct 2, 2026
22 checks passed
@janzak-dev
janzak-dev deleted the feat/nightly-ci-build-example-images branch October 2, 2026 10:55

This branch was previously deployed

1 inactive deployment
xmc-nightly — 574fc2a2 Deployed Oct 2, 2026 by janzak-dev via Run & Verify: Simple Job (3 Multi-Unit Batch) #36
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.

2 participants

0