Skip to content

chore: remove automated AMI builds from release workflow - #1203

Merged
MicBun merged 1 commit into
mainfrom
removeDuplicateWorkflow
Oct 8, 2025
Merged

chore: remove automated AMI builds from release workflow#1203
MicBun merged 1 commit into
mainfrom
removeDuplicateWorkflow

Conversation

@MicBun

@MicBun MicBun commented Oct 8, 2025

Copy link
Copy Markdown
Contributor

The release workflow supposed to be only on release.
The AMI build should be triggered manually as the image is handled via latest tag

image

Summary by CodeRabbit

  • Chores
    • Reduced permissions in the release workflow by removing write access.
    • Removed the job that triggered downstream builds on tag releases, simplifying the pipeline.
    • Release process now runs only core steps; downstream AMI/Docker builds no longer auto-trigger automatically.
    • No impact to application behavior or user-facing features.

@coderabbitai

coderabbitai Bot commented Oct 8, 2025

Copy link
Copy Markdown

Caution

Review failed

The pull request is closed.

Walkthrough

Removed actions: write permission from the release workflow and deleted the trigger-builds job that dispatched AMI and Docker build workflows on tag releases.

Changes

Cohort / File(s) Summary
Release workflow cleanup
.github/workflows/release.yaml
Removed permissions.actions: write. Deleted trigger-builds job and its steps that dispatched AMI and Docker build workflows on tag releases.

Sequence Diagram(s)

sequenceDiagram
  autonumber
  actor Developer as Developer
  participant GH as GitHub Release Workflow

  rect rgb(235, 245, 255)
  note over GH: Previous flow (removed)
  Developer->>GH: Push tag / create release
  GH--x GH: trigger-builds job (dispatch AMI & Docker workflows) [Removed]
  end

  rect rgb(240, 255, 240)
  note over GH: Current flow
  Developer->>GH: Push tag / create release
  GH-->>GH: Continue remaining release steps (no AMI/Docker dispatch)
  end
Loading

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Possibly related PRs

Suggested labels

type: chore

Suggested reviewers

  • outerlook

Poem

A tag once rang—off went the builds,
But now the bells are softly stilled.
I thump my paws: less hop, more chill,
Permissions trimmed, the queue stands still.
Carrot commits, neat and tight—hip-hop, ship right! 🥕🐇

✨ Finishing touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch removeDuplicateWorkflow

📜 Recent review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between a0770fa and 4e04189.

📒 Files selected for processing (1)
  • .github/workflows/release.yaml (0 hunks)

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 and usage tips.

@MicBun MicBun self-assigned this Oct 8, 2025
@MicBun
MicBun requested a review from outerlook October 8, 2025 23:36
@holdex

holdex Bot commented Oct 8, 2025

Copy link
Copy Markdown

Time Submission Status

Member Status Time Action Last Update
MicBun ✅ Submitted 2h Update time Oct 8, 2025, 11:37 PM

@MicBun
MicBun merged commit 02061e8 into main Oct 8, 2025
4 of 5 checks passed
@MicBun
MicBun deleted the removeDuplicateWorkflow branch October 8, 2025 23:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant