Skip to content

fix(ci): keep the running API image when the web triggers a deploy - #47

Merged
jhosepmyr merged 1 commit into
developfrom
bugfix/deploy-keep-api-image
Oct 7, 2026
Merged

jhosepmyr merged 1 commit into
developfrom
bugfix/deploy-keep-api-image

Conversation

@jhosepmyr

Copy link
Copy Markdown
Contributor

Description

Follow-up to #46: when a push to main triggers the reqsai-infra MVP deploy, pass api_ref=keep so only the web is rebuilt and the running API image is left untouched. Before, the API ref defaulted to main, so the first web release would have redeployed the API from its main (still from July) and rolled it back.

Feature module / area: ci

Related issue / US: continuous deployment of the MVP (reqsai-infra PR #7)


Type of Change

  • feat — new feature or UI component
  • fix — bug fix
  • refactor — code change without behavior change
  • test — tests only
  • docs — documentation only
  • build / ci — build, dependencies, or CI/CD
  • chore — maintenance

Checklist

  • The PR targets develop (not main)
  • Branch name follows feature/*, bugfix/*, or hotfix/*
  • Commits follow Conventional Commits
  • bun run lint passes locally (ESLint + angular-eslint)
  • bun run test passes locally (Vitest) — not applicable, workflow-only change
  • bun run build passes locally (no type errors, no budget exceeded) — not applicable, workflow-only change
  • New components use ChangeDetectionStrategy.OnPush and Angular signals — not applicable
  • No localStorage/sessionStorage access for JWT tokens (use the auth store)
  • No bypassSecurityTrust* calls without explicit review
  • No secrets, credentials, or .env content committed
  • CHANGELOG.md updated under [Unreleased] — not needed for a one-line CI fix

How to Test

After the release to main, the "Deploy" workflow run in this repo triggers "Deploy MVP" in Kntro-Soft/reqsai-infra with web_ref=<sha> and api_ref=keep; the API container is not recreated.

@jhosepmyr
jhosepmyr merged commit cc9c8df into develop Oct 7, 2026
5 checks passed
@jhosepmyr jhosepmyr mentioned this pull request Oct 7, 2026
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