ci: fix stuck on migration runs #278

2026-06-11T23:28:47Z

talksik commented

2026-06-11 23:28:47 +00:00

(Migrated from github.com)

Summary by CodeRabbit

Chores
- Updated migration deployment configuration across CI/CD workflows and build system to streamline the deployment process.

## Summary by CodeRabbit * **Chores** * Updated migration deployment configuration across CI/CD workflows and build system to streamline the deployment process.

coderabbitai[bot] commented

2026-06-11 23:28:59 +00:00

(Migrated from github.com)

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro Plus

Run ID: 0bb2a72a-a262-4a19-a9e8-928f1f2bee66

📥 Commits

Reviewing files that changed from the base of the PR and between 9e3a30f74e and 3689f6e203.

📒 Files selected for processing (2)

.github/workflows/_deploy.yml
go/Makefile

📝 Walkthrough

Walkthrough

The PR removes the --tail flag from Skaffold migration commands across deployment automation. The reusable deploy workflow and Makefile migration targets are updated to run skaffold run -p migrations without the tail option, and the Makefile targets now explicitly set the SKAFFOLD_DEFAULT_REPO environment variable.

Changes

Migration Skaffold Configuration

Layer / File(s)	Summary
Migration Skaffold command updates `.github/workflows/_deploy.yml`, `go/Makefile`	The migrations deployment step removes the `--tail` flag from Skaffold invocations. The Makefile targets now set `SKAFFOLD_DEFAULT_REPO` inline with the appropriate repository configuration for dev and prod environments.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~5 minutes

Poem

A tail once trailed through every run,
But hops are quicker without one!
Now migrations dash, so clean and bright—
No streaming logs to slow the flight. 🐰✨

🚥 Pre-merge checks | ✅ 5

✅ Passed checks (5 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	The title 'ci: fix stuck on migration runs' accurately describes the main change: removing the --tail flag from Skaffold migration commands to resolve issues causing runs to hang.
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.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing Touches

🧪 Generate unit tests (beta)

Create PR with unit tests
Commit unit tests in branch fix-ci-stuck

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

[![Review Change Stack](https://storage.googleapis.com/coderabbit_public_assets/review-stack-in-coderabbit-ui.svg)](https://app.coderabbit.ai/change-stack/flowy-live/llink/pull/278?utm_source=github_walkthrough&utm_medium=github&utm_campaign=change_stack)  No actionable comments were generated in the recent review. 🎉 <details> <summary>ℹ️ Recent review info</summary> <details> <summary>⚙️ Run configuration</summary> **Configuration used**: defaults **Review profile**: CHILL **Plan**: Pro Plus **Run ID**: `0bb2a72a-a262-4a19-a9e8-928f1f2bee66` </details> <details> <summary>📥 Commits</summary> Reviewing files that changed from the base of the PR and between 9e3a30f74ee1ce961dde1e1df2a083183dfa34ba and 3689f6e203d0749cc812a96e2f6bccba0c6cca09. </details> <details> <summary>📒 Files selected for processing (2)</summary> * `.github/workflows/_deploy.yml` * `go/Makefile` </details> </details> ---  <details> <summary>📝 Walkthrough</summary> ## Walkthrough The PR removes the `--tail` flag from Skaffold migration commands across deployment automation. The reusable deploy workflow and Makefile migration targets are updated to run `skaffold run -p migrations` without the tail option, and the Makefile targets now explicitly set the `SKAFFOLD_DEFAULT_REPO` environment variable. ## Changes **Migration Skaffold Configuration** |Layer / File(s)|Summary| |---|---| |**Migration Skaffold command updates** `.github/workflows/_deploy.yml`, `go/Makefile`|The migrations deployment step removes the `--tail` flag from Skaffold invocations. The Makefile targets now set `SKAFFOLD_DEFAULT_REPO` inline with the appropriate repository configuration for dev and prod environments.| ## Estimated code review effort 🎯 2 (Simple) | ⏱️ ~5 minutes ## Poem > A tail once trailed through every run, > But hops are quicker without one! > Now migrations dash, so clean and bright— > No streaming logs to slow the flight. 🐰✨ </details>   <details> <summary>🚥 Pre-merge checks | ✅ 5</summary> <details> <summary>✅ Passed checks (5 passed)</summary> | Check name | Status | Explanation | | :------------------------: | :------- | :---------------------------------------------------------------------------------------------------------------------------------------------------------------------------------- | | Description Check | ✅ Passed | Check skipped - CodeRabbit’s high-level summary is enabled. | | Title check | ✅ Passed | The title 'ci: fix stuck on migration runs' accurately describes the main change: removing the --tail flag from Skaffold migration commands to resolve issues causing runs to hang. | | 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. | </details> ✏️ Tip: You can configure your own custom pre-merge checks in the settings. </details>   <details> <summary>✨ Finishing Touches</summary> <details> <summary>🧪 Generate unit tests (beta)</summary> - [ ]  Create PR with unit tests - [ ]  Commit unit tests in branch `fix-ci-stuck` </details> </details>   --- Comment `@coderabbitai help` to get the list of available commands and usage tips.

Sign in to join this conversation.

1 Participants

Notifications

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: FlowyLabs/llink#278