Skip to content

Commit 5b3bc09

Browse files
committed
improve agent to clarify no changes detected
1 parent d3b6574 commit 5b3bc09

1 file changed

Lines changed: 22 additions & 5 deletions

File tree

.github/workflows/dockerfile-readiness.md

Lines changed: 22 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -143,7 +143,19 @@ For each Java version (8, 11, 17, 21, 25, 26, etc.) and each package type
143143

144144
### Step 4: Determine readiness per version
145145

146+
First, check whether the version has **any changes at all** compared to upstream.
147+
A version has **no updates** if ALL of the following are true for every entry:
148+
- The Java version string in the tags is identical to upstream
149+
- The set of architectures is identical to upstream
150+
- No entries were added or removed
151+
152+
A version with no updates must be marked **⏭️ No Updates** — do NOT mark it as
153+
"Ready to Ship". This is critical: "Ready to Ship" implies the version was updated
154+
and the update is complete, which is misleading when nothing changed. Use this
155+
status to signal to reviewers that this version was not touched by the PR.
156+
146157
A version is **Ready to Ship** only if ALL of the following are true:
158+
- There is at least one change from upstream (version bump, GitCommit change, new entry, or architecture change)
147159
- Every distro/arch entry that exists upstream also exists locally (nothing dropped unintentionally)
148160
- For every entry in both manifests, the local architectures include all architectures listed upstream (no regressions)
149161
- The Java version in tags is consistent across all entries for that version
@@ -171,16 +183,21 @@ Output a clear markdown report. Use this structure:
171183

172184
| Version | Status | Java Version | Missing Distros | Missing Arches | Notes |
173185
|---------|--------|-------------|-----------------|----------------|-------|
174-
| 8 | ✅ Ready | 8u492-b08 ||| Version bumped from 8u482-b08 |
186+
| 8 | ⏭️ No Updates | 8u482-b08 ||| Unchanged from upstream |
175187
| 11 | ⚠️ Partial | 11.0.30_7 || noble: riscv64 | New arch not yet built |
176-
| 21 | ✅ Ready | 21.0.7_6 ||| |
188+
| 21 | ✅ Ready | 21.0.7_6 ||| Version bumped from 21.0.6_7 |
177189
| 25 | ❌ Not Ready | 25.0.2_10 | alpine/3.23 (jre) | noble: s390x | First release, ubi9-minimal skipped (deprecated) |
178190

179191
### Version Details
180192

181-
#### JDK 8 — ✅ Ready to Ship
193+
#### JDK 8 — ⏭️ No Updates
194+
195+
No changes detected — Java version, architectures, and GitCommit are all
196+
identical to upstream. This version was not updated in this PR.
197+
198+
#### JDK 21 — ✅ Ready to Ship
182199

183-
**Java Version:** `8u492-b08` (upstream: `8u482-b08` — version bump)
200+
**Java Version:** `21.0.7_6` (upstream: `21.0.6_7` — version bump)
184201

185202
| Distro | JDK Arches | JRE Arches | Status |
186203
|--------|-----------|-----------|--------|
@@ -189,7 +206,7 @@ Output a clear markdown report. Use this structure:
189206
| ... | ... | ... | ... |
190207

191208
**Changes from upstream:**
192-
- Version bumped from 8u482-b08 to 8u492-b08
209+
- Version bumped from 21.0.6_7 to 21.0.7_6
193210
- GitCommit changed (Dockerfiles updated)
194211

195212
#### JDK 25 — ❌ Not Ready (new version)

0 commit comments

Comments
 (0)