docs(release): make changelog check B find changed image build files

[Harshit Rawat (harshitrwt)]

What this PR does

Check B of the changelog pre-publish verification in docs/release.md lists changed image build files with git diff --name-only $PREV..$NEW -- 'images/*/Dockerfile'. That pathspec never matches anything: there is no images/ directory at the repository root, since image build files live under packages/*/*/images/. The check therefore printed nothing on every release and passed without looking at anything. It also skipped Containerfile, which first-party images such as otelcol-tracing and the dashboard console use.

This PR changes the pathspec to '*Dockerfile' '*Containerfile', which matches both file names at any depth. I checked it in a scratch repository: the old pathspec returned no files for a changed Dockerfile and Containerfile, and the new one returned both. Every Dockerfile and Containerfile currently in the repository lives under packages/*/*/images/, so the broader pathspec does not pick up files from vendored charts.

Closes #4892

Screenshots

N/A: documentation-only change, no UI impact.

Downstream repositories

• No downstream repository is affected by this change
• cozystack/website - follow-up:
• cozystack/terraform-provider-cozystack - follow-up:
• cozystack/ansible-cozystack - follow-up:
• cozystack/ccp - follow-up:
• cozystack/talm - follow-up:
• cozystack/cozyhr - follow-up:
• cozystack/cozy-proxy - follow-up:
• cozystack/cozystack-telemetry-server - follow-up:
• cozystack/external-apps-example - follow-up:
• cozystack/examples - follow-up:
• cozystack/community - follow-up:

Release note

docs(release): make changelog verification check B find changed image build files by matching Dockerfile and Containerfile at any depth

Summary by CodeRabbit

• Chores
  • Changelog verification now recognizes changes to Dockerfile and Containerfile files anywhere in a diff.

[coderabbitai]

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

• Configuration used: Repository: cozystack/cozystack/.coderabbit.yaml
• Review profile: CHILL
• Plan: Advanced
• Run ID: 58428e42-69ba-4609-80cd-0441b718ef80

📥 Commits

Reviewing files that changed from the base of the PR and between aed7fa8 and ec378cc.

📒 Files selected for processing: 1

• docs/release.md

Included review availability: This review used your included allowance. Your plan provides up to 8 included reviews per hour; 7 remain after this review.

📝 Walkthrough

The changelog verification command now searches the diff for Dockerfile and Containerfile paths at any directory depth.

Changes: Changelog verification

Layer / File(s)	Summary
Build-file path matching docs/release.md	The changed-file check now matches Dockerfile and Containerfile paths anywhere in the diff.

Priority: ⬇️ Low

Estimated code review effort: 2 (Simple) | ~5 minutes

Change: Bug fix · Severity of issue fixed: Low

Suggested reviewers: myasnikovdaniil

Merge Risk: ⚪ Minimal · up to ec378

The changelog check now searches for Dockerfile and Containerfile paths at any depth. No actionable merge risk is apparent.

Pre-merge checks: 5

Check name	Status	Explanation
Description Check		Check skipped - CodeRabbit’s high-level summary is enabled.
Title check		The title clearly and concisely describes the main change: updating changelog Check B to find changed image build files.
Linked Issues check		The change satisfies issue #4892. In docs/release.md, Check B replaces 'images/*/Dockerfile' with '*Dockerfile' '*Containerfile'. This matches both required image build file names at any depth. …
Out of Scope Changes check		The whole-PR diff contains one documentation change in docs/release.md. The change directly implements issue #4892 and adds no unrelated source, test, or configuration changes.
Docstring Coverage		No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 0…

Full details: Linked Issues check

Explanation

The change satisfies issue #4892. In docs/release.md, Check B replaces 'images/*/Dockerfile' with '*Dockerfile' '*Containerfile'. This matches both required image build file names at any depth. The issue specifies no additional coding or automated-test requirement.

Full details: Docstring Coverage

Explanation

No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 0 files. (1 skipped: 1 unsupported.)

✨ Finishing Touches

🧪 Generate unit tests (beta)

• Create a new PR

• Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

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

[myasnikovdaniil]

LGTM, thanks. I ran the new pathspec against the last 13 stable tag ranges, the old one returns nothing on every range and the new one returns the changed Dockerfile and Containerfile files (40 of them on v1.5.4..v1.6.0), no false positives.

The first path in the same loop is dead too (packages/core/talos/values.yaml is empty), I fixed it separately in #4959.