Skip to content

ci: retire the frontend image build - #267

Merged
aleksdotbar merged 1 commit into
mainfrom
ci/retire-frontend-image-build
Aug 5, 2026
Merged

ci: retire the frontend image build#267
aleksdotbar merged 1 commit into
mainfrom
ci/retire-frontend-image-build

Conversation

@aleksdotbar

@aleksdotbar aleksdotbar commented Aug 4, 2026

Copy link
Copy Markdown
Contributor

Frontend image builds belong to constructorfabric/insight, which owns the
sources under src/frontend/. Two publishers race the mutable latest tag;
this leaves one.

  • Image name and tag format unchanged — charts, values and environment
    overlays need no edit.
  • GHCR retention for the insight-front package stays here.
  • Comment-only edits in ci.yml, trivy.yml, ghcr-cleanup.yml drop
    references to the removed file.
  • ci.yml, semgrep.yml, trivy.yml, trufflehog.yml keep gating open PRs.

⚠️ Do not merge until the cutover is ready. Nothing publishes the frontend
image between this landing and the monorepo building it — latest freezes at
the last build.

Refs constructorfabric/insight#1818

Image builds, multi-arch manifest assembly, and the umbrella chart
dispatch are owned by constructorfabric/insight, which holds the
frontend sources under src/frontend/.

The image name and tag format are unchanged, so charts, values files
and environment overlays need no edit. Retention for the insight-front
package stays here; the published manifest is scanned by the monorepo's
trivy-images.yml.

Refs constructorfabric/insight#1818

Signed-off-by: Aleksandr Barkhatov <pm@aleks.bar>
@aleksdotbar
aleksdotbar requested a review from a team as a code owner August 4, 2026 12:31
@coderabbitai

coderabbitai Bot commented Aug 4, 2026

Copy link
Copy Markdown

Review Change Stack

📝 Walkthrough

Walkthrough

The .github/workflows/docker.yml file is deleted, removing Docker build, scan, and publish steps from this repository. Comments in ci.yml, ghcr-cleanup.yml, and trivy.yml are updated to reference the external constructorfabric/insight workflow for image publishing and scanning instead of docker.yml.

Changes

Workflow documentation update after docker.yml removal

Layer / File(s) Summary
Remove docker.yml workflow
.github/workflows/docker.yml
Deletes the entire Docker build, GHCR authentication, caching, Trivy scanning, and umbrella chart dispatch workflow.
Update workflow comments to reference external publisher
.github/workflows/ci.yml, .github/workflows/ghcr-cleanup.yml, .github/workflows/trivy.yml
Comments in these files now reference the constructorfabric/insight publishing workflow and its trivy-images.yml job instead of the deleted docker.yml.

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

Possibly related PRs

Suggested reviewers: ktursunov

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
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.
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title clearly summarizes the primary change: retiring the frontend image build from this repository.
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch ci/retire-frontend-image-build

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

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

@aleksdotbar
aleksdotbar marked this pull request as draft August 4, 2026 12:33
@aleksdotbar
aleksdotbar marked this pull request as ready for review August 5, 2026 06:42
@aleksdotbar
aleksdotbar merged commit a2d4643 into main Aug 5, 2026
13 checks passed
@aleksdotbar
aleksdotbar deleted the ci/retire-frontend-image-build branch August 5, 2026 06:42
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.

2 participants