Address review comments.

Signed-off-by: Revital Sur <eres@il.ibm.com>

Author: revit13

.github/actions/docker-build-and-push/action.yml

@@ -33,6 +33,10 @@ inputs:
     required: false
     description: Git commit SHA to embed via COMMIT_SHA build arg
     default: ''
+  build-ref:
+    required: false
+    description: Human-readable build ref embedded via BUILD_REF build arg. Falls back to tag when unset.
+    default: ''
   additional-tags:
     required: false
     description: Additional image tags to push
@@ -66,6 +70,6 @@ runs:
         build-args: |
           LDFLAGS=-s -w
           COMMIT_SHA=${{ inputs.commit-sha || 'unknown' }}
-          BUILD_REF=${{ inputs.tag || 'unknown' }}
+          BUILD_REF=${{ inputs.build-ref || inputs.tag || 'unknown' }}
         cache-from: type=gha,scope=${{ inputs.image-name }}
         cache-to: type=gha,mode=max,scope=${{ inputs.image-name }}

.github/workflows/ci-coordinator.yaml

@@ -42,6 +42,7 @@ jobs:
               - scripts/**
               - hack/**
               - test/**
+              - config/coordinator/**
               - .github/actions/docker-build-and-push/action.yml
               - .github/actions/e2e-runner-setup/action.yml
               - .github/workflows/ci-coordinator.yaml
@@ -133,6 +134,7 @@ jobs:
           push: 'false'
           buildx-outputs: type=docker,dest=${{ runner.temp }}/${{ matrix.image.archive }}.tar
           commit-sha: ${{ github.sha }}
+          build-ref: ${{ github.head_ref || github.ref_name }}
 
       - name: Pull ${{ matrix.image.name }} image
         if: ${{ matrix.image.source == 'pull' }}

.github/workflows/ci-pr-checks.yaml

@@ -259,6 +259,7 @@ jobs:
           push: 'false'
           buildx-outputs: type=docker,dest=${{ runner.temp }}/${{ matrix.image.archive }}.tar
           commit-sha: ${{ github.sha }}
+          build-ref: ${{ github.head_ref || github.ref_name }}
 
       - name: Pull ${{ matrix.image.name }} image
         if: ${{ matrix.image.source == 'pull' }}
@@ -322,6 +323,7 @@ jobs:
           push: 'false'
           buildx-outputs: type=docker,dest=${{ runner.temp }}/${{ matrix.image.archive }}.tar
           commit-sha: ${{ github.sha }}
+          build-ref: ${{ github.head_ref || github.ref_name }}
 
       - name: Pull ${{ matrix.image.name }} image
         if: ${{ matrix.image.source == 'pull' }}

Dockerfile.coordinator

@@ -25,10 +25,11 @@ RUN go mod download
 # Copy the go source
 COPY cmd/ cmd/
 COPY pkg/ pkg/
+COPY version/ version/
 COPY config/coordinator/ config/coordinator/
 
 RUN CGO_ENABLED=0 GOOS=${TARGETOS} GOARCH=${TARGETARCH} go build \
-    -ldflags="${LDFLAGS}" \
+    -ldflags="${LDFLAGS} -X github.com/llm-d/llm-d-router/version.CommitSHA=${COMMIT_SHA} -X github.com/llm-d/llm-d-router/version.BuildRef=${BUILD_REF}" \
     -o bin/coordinator ./cmd/coordinator/...
 
 # Runtime stage

Makefile.coord.mk

@@ -196,7 +196,7 @@ test-unit: image-build-builder
 	$(BUILDER_RUN) "go test -v -race $(TEST_PACKAGES)"
 
 .PHONY: test-e2e-coordinator-run
-test-e2e-coordinator-run: image-pull ## Ensure images are present, then run coordinator e2e tests
+test-e2e-coordinator-run: image-pull ## Run coordinator e2e tests against pre-loaded builder/coordinator/epp images (CI entry point; use test-e2e-coordinator locally)
 	@printf "\033[33;1m==== Running Coordinator End to End Tests ====\033[0m\n"
 	$(CONTAINER_RUNTIME) run $(BUILDER_RUN_FLAGS) $(BUILDER_E2E_FLAGS) \
 		$(BUILDER_IMAGE) test/coordinator/scripts/run_e2e_coordinator.sh

README.coord.md

@@ -307,6 +307,8 @@ make -f Makefile.coord.mk test-e2e-coordinator
 
 This creates a temporary Kind cluster named `e2e-coordinator-tests`, runs the coordinator e2e suite against it, and deletes the cluster on completion.
 
+`test-e2e-coordinator` is the local entry point: it builds the builder, coordinator, and epp images first. CI invokes `test-e2e-coordinator-run` instead, which expects those images to be already built and loaded into the container daemon (`image-pull` fetches only the simulator). Running `test-e2e-coordinator-run` directly without prebuilt images will use stale or missing ones.
+
 **Keeping the cluster on failure**
 
 Set `E2E_KEEP_CLUSTER_ON_FAILURE=true` to preserve the cluster when any test fails. This is useful for inspecting pod logs, events, or cluster state after a failure.

cmd/coordinator/main.go

@@ -29,6 +29,7 @@ import (
 	ctrl "sigs.k8s.io/controller-runtime"
 
 	logutil "github.com/llm-d/llm-d-router/pkg/common/observability/logging"
+	"github.com/llm-d/llm-d-router/version"
 
 	"github.com/llm-d/llm-d-router/pkg/coordinator/config"
 	"github.com/llm-d/llm-d-router/pkg/coordinator/gateway"
@@ -48,6 +49,8 @@ func main() {
 	logutil.InitSetupLogging()
 	log := ctrl.Log.WithName("coordinator")
 
+	log.Info("coordinator build", "commit-sha", version.CommitSHA, "build-ref", version.BuildRef)
+
 	cfg, err := config.Load(*configPath)
 	if err != nil {
 		log.Error(err, "failed to load config")