Skip to content

feat(ci): add migration-summary.md artifact to migration-upgrade-with-flows - #273

Merged
lice-reis merged 1 commit into
mainfrom
feat/migration-summary-upgrade-with-flows
May 18, 2026
Merged

feat(ci): add migration-summary.md artifact to migration-upgrade-with-flows#273
lice-reis merged 1 commit into
mainfrom
feat/migration-summary-upgrade-with-flows

Conversation

@lice-reis

Copy link
Copy Markdown
Collaborator

Summary

Adds a structured summary markdown file (`/tmp/migration-summary.md`) to the artifact of `migration-upgrade-with-flows.yml`. The file is generated at the end of every run (`if: always()`), so it documents both successful runs and failures.

What the summary contains

  • Outcome (`success` / `failure`), run number, date, trigger
  • Source and target image digests (read from the existing `/tmp/latest-digest.txt` and `/tmp/nightly-digest.txt`)
  • Database backend (PostgreSQL via GHA service)
  • Deployment shape (`docker run` + `LANGFLOW_LOAD_FLOWS_PATH` bind mount of the flow JSON)
  • The preloaded flow name (`env.FLOW_NAME`)
  • A bullet list of what the run verified
  • List of artifacts produced

Implementation

  • New step `Generate migration summary` runs before `Upload logs` with `if: always()`.
  • Writes to `/tmp/migration-summary.md` via inline bash heredoc (no new helpers).
  • The existing upload-artifact step now also includes `/tmp/migration-summary.md` in its `path:`.

This is the first of three companion PRs adding the same summary pattern to:

Test plan:

  • Trigger workflow manually after merge; confirm `migration-summary.md` appears in artifact and contains the expected fields

🤖 Generated with Claude Code

…-flows

Generates a /tmp/migration-summary.md at the end of every run (success or
failure) and includes it in the existing artifact upload. The summary
captures: outcome, source/target image digests, database backend,
deployment shape (LANGFLOW_LOAD_FLOWS_PATH bind mount), preloaded flow
name, and a checklist of what the run verified.

Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
lice-reis added a commit that referenced this pull request May 18, 2026
Generates /tmp/migration-summary.md at the end of every matrix entry
(postgresql / sqlite / docker-compose), uploaded with the existing logs.
The summary adapts to the matrix value: includes deploy mode, witness
flow id, user count, OPENAI_API_KEY availability, and a checklist of
what was verified end-to-end.

Same pattern as #273 (migration-upgrade-with-flows) and #274
(migration-test, both jobs). The structure stays consistent across all
migration workflows so the team can review runs uniformly.

Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
@lice-reis
lice-reis merged commit 87d626e into main May 18, 2026
2 checks passed
@Victor-w-Madeira
Victor-w-Madeira deleted the feat/migration-summary-upgrade-with-flows branch May 19, 2026 15:57
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