Skip to content

chore(image): drop the hand-written base-image label - #327

Merged
chodeus merged 1 commit into
mainfrom
chore/drop-stale-base-label
Oct 26, 2026
Merged

chodeus merged 1 commit into
mainfrom
chore/drop-stale-base-label

Conversation

@chodeus

@chodeus chodeus commented Oct 26, 2026 •

Copy link
Copy Markdown
Owner

The final stage labels org.opencontainers.image.base.name="alpine:3.24", a hand-written copy of the FROM tag. Renovate bumps only the FROM line, so the label goes stale at the next Alpine release; nothing reads it, so it goes rather than being kept in sync by hand. Same fix as chodeus/spotify-to-plex#39, where it had already fallen behind.

Summary by CodeRabbit

  • Chores
    • Updated container image metadata by removing a base-image label. This change does not alter application features or behavior.
    • No user-facing interface, workflow, or capability changes are included in this update. Users can continue using the application as before; the update only affects metadata associated with the container image.

base.name repeated the FROM tag by hand, and Renovate bumps only the FROM line, so it goes stale on the next Alpine release.
@coderabbitai

coderabbitai Bot commented Oct 26, 2026 •

Copy link
Copy Markdown

Review in Change Stack →

No actionable comments were generated in the recent review.

Recent review info
Run configuration
Commits

Reviewing files that changed from the base of the PR and between d0694ee and 16df8e3.

Files selected for processing: 1
  • Dockerfile
Files with no reviewable changes: 1
  • Dockerfile

Included review availability: This review used your included allowance. 0 included reviews remain after this review. Your included PR review attempts over the past 7 days set your current allowance at 1 review per hour.

Walkthrough

The Dockerfile no longer sets the org.opencontainers.image.base.name label to alpine:3.24.

Changes: Docker image metadata

Layer / File(s) Summary
Remove base image name label
Dockerfile
Removes the org.opencontainers.image.base.name label.

Priority: Low

Estimated code review effort: 1 (Trivial) | ~3 minutes

Change: Other

Merge Risk: Minimal · up to 16df8

No merge-blocking issue is identified with removing the redundant image label.

Pre-merge checks: Passed 4
Check name Status Explanation
Description Check Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check Passed The title uses the Conventional Commits form with type chore and scope image. The diff removes one Dockerfile label, a metadata-only change that does not alter runtime behaviour, so chore is an accura…
Linked Issues check Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check Passed Check skipped because no linked issues were found for this pull request.
Full details: Title check

Explanation

The title uses the Conventional Commits form with type chore and scope image. The diff removes one Dockerfile label, a metadata-only change that does not alter runtime behaviour, so chore is an accurate type. The title names the main change, dropping the hand-written base-image label, and it matches the PR objectives. It is 52 characters, so it stays within a typical length limit.

Finishing Touches
🧪 Generate unit tests (beta)
  • Commit to this branch
  • Create a new PR
  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

Comment @coderabbitai help to get the list of available commands.

@chodeus

chodeus commented Oct 26, 2026

Copy link
Copy Markdown
Owner Author

@coderabbitai full review

@coderabbitai

coderabbitai Bot commented Oct 26, 2026 •

Copy link
Copy Markdown
✅ Action performed

Full review finished.

@chodeus
chodeus marked this pull request as ready for review October 26, 2026 23:54
@chodeus
chodeus merged commit 3279b78 into main Oct 26, 2026
15 checks passed
@chodeus
chodeus deleted the chore/drop-stale-base-label branch October 26, 2026 23:54
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