Update quay.io/coreos/etcd Docker tag to v3.7.1 - #220
Conversation
|
PR author is in the excluded authors list. |
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: eb93f98276
ℹ️ About Codex in GitHub
Codex has been enabled to automatically review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
When you sign up for Codex through ChatGPT, Codex can also answer questions or update the PR, like "@codex address that feedback".
| services: | ||
| etcd: | ||
| image: quay.io/coreos/etcd:v3.5.15 | ||
| image: quay.io/coreos/etcd:v3.6.8 |
There was a problem hiding this comment.
Keep etcd log filter tag aligned with service image
After changing the service image to quay.io/coreos/etcd:v3.6.8, the failure handler in the Wait for etcd step still runs docker ps -q --filter ancestor=quay.io/coreos/etcd:v3.5.15 (line 277), so when readiness fails it will not match the running container and emits no etcd logs. This regresses CI diagnosability exactly in the scenario where logs are needed most (startup/health failures).
Useful? React with 👍 / 👎.
eb93f98 to
d5ec0cc
Compare
|
Important Review skippedBot user detected. To trigger a single review, invoke the ⚙️ Run configurationConfiguration used: Organization UI Review profile: ASSERTIVE Plan: Pro Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
✨ Finishing Touches🧪 Generate unit tests (beta)
✨ Simplify code
Comment |
d5ec0cc to
f52f7c3
Compare
f52f7c3 to
8eb126a
Compare
Code Review SummaryStatus: 1 Issue Found | Recommendation: Address before merge Overview
Fix these issues in Kilo Cloud Issue Details (click to expand)WARNING
Files Reviewed (2 files)
Previous Review Summaries (2 snapshots, latest commit 03cd8cd)Current summary above is authoritative. Previous snapshots are kept for context only. Previous review (commit 03cd8cd)Status: 1 Issue Found | Recommendation: Address before merge Overview
Fix these issues in Kilo Cloud Issue Details (click to expand)WARNING
Files Reviewed (2 files)
Previous review (commit b41a6f4)Status: 2 Issues Found | Recommendation: Address before merge Overview
Fix these issues in Kilo Cloud Issue Details (click to expand)No new inline-commentable issues. The CI log-filter mismatch is already covered by an existing review comment, and the remaining version-drift finding is outside the diff. Other Observations (not in diff)Issues found outside changed diff hunks that cannot receive inline comments:
Files Reviewed (5 files)
Reviewed by gpt-5.5-2026-04-23 · Input: 37K · Output: 3.9K · Cached: 125.4K |
8eb126a to
4012d0f
Compare
4012d0f to
b66a64f
Compare
7e78b0d to
e632aa1
Compare
e632aa1 to
85141e9
Compare
85141e9 to
4543fc4
Compare
4543fc4 to
b41a6f4
Compare
b41a6f4 to
03cd8cd
Compare
03cd8cd to
5aa7b98
Compare
5aa7b98 to
b27695e
Compare
This PR contains the following updates:
v3.5.15→v3.7.1v3.5.21→v3.7.1Configuration
📅 Schedule: (UTC)
🚦 Automerge: Disabled by config. Please merge this manually once you are satisfied.
♻ Rebasing: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.
🔕 Ignore: Close this PR and you won't be reminded about these updates again.
This PR was generated by Mend Renovate. View the repository job log.