Promote to Production: CAR-894 Gitea workflows migration #36

Merged

Savannah Savings merged 31 commits from uat into main 2026-05-24 18:51:44 +00:00

Conversation 0 Commits 31 Files Changed 20

+1643 -124

Savannah Savings commented 2026-05-24 18:51:22 +00:00

Member

Copy Link

Copy Source

No description provided.

Savannah Savings added 31 commits 2026-05-24 18:51:22 +00:00

ci: convert GitHub Actions to Gitea Actions (ubuntu-latest) ... 0c3c549a6a

- Replace runs-on: runners-cartsnitch with ubuntu-latest (6 jobs)
- Remove SARIF upload step (github/codeql-action/upload-sarif)
- Replace GitHub App token with secrets.GITEA_TOKEN in deploy-dev and deploy-uat

Co-Authored-By: Paperclip <noreply@paperclip.ing>

Merge pull request 'ci: convert GitHub Actions to Gitea Actions (ubuntu-latest)' (#4) from betty/car-869-gitea-actions-api into dev 3a4bf6fb30

Merge pull request 'ci: promote Gitea Actions conversion to UAT' (#5) from dev into uat 6a8db71537

chore: move workflows from .github to .gitea ... 4e38dd4a0e

Co-Authored-By: Paperclip <noreply@paperclip.ing>

Merge pull request 'chore: move workflows from .github to .gitea' (#6) from barcode-betty/move-workflows-to-gitea into dev ... 6b54a5ee7f

chore: move workflows from .github to .gitea (#6)

Part of Gitea migration (CAR-893).

Merge pull request 'chore: promote dev to uat (CAR-898 workflow move)' (#7) from dev into uat ... fbfedd4e8f

chore: promote dev to uat (CAR-898 workflow move) (#7)

fix(ci): add uat branch to workflow triggers ... e429786696

The on.push and on.pull_request triggers only listed [main, dev].
The deploy-uat job condition checks for refs/heads/uat but the
workflow never fires on uat pushes. Add uat to both trigger lists.

Co-Authored-By: Paperclip <noreply@paperclip.ing>

Merge pull request 'fix(ci): add uat branch to workflow triggers' (#9) from savannah/fix-ci-uat-trigger into dev ... 7fd8e90b9c

fix(ci): add uat branch to workflow triggers (#9)

Co-Authored-By: Paperclip <noreply@paperclip.ing>

Merge pull request 'promote: dev → uat (CI trigger fix)' (#10) from dev into uat ... 228a83c355

promote: dev → uat (CI trigger fix) (#10)

Co-Authored-By: Paperclip <noreply@paperclip.ing>

fix: add UAT/dev domains to cors_origins ... 0127c16d0b

Add dev.cartsnitch.com and uat.cartsnitch.com to the CORS origins list
to match the infra HTTPRoute domains and fix auth blocking on UAT.

Refs: CAR-992
Co-Authored-By: Paperclip <noreply@paperclip.ing>

fix: remove dead dispose_engine import from API main.py ... ba88fad48b

The top-level import of dispose_engine from cartsnitch_api.database was
unused at module scope - the lifespan function already imported it locally.
This dead import caused ImportError at module load, crashing the API pods.

Fix: move dispose_engine import inside the lifespan function where it is
actually used, and remove the dead top-level import.

Co-Authored-By: Paperclip <noreply@paperclip.ing>

Merge pull request 'fix: add UAT/dev domains to cors_origins' (#14) from cs_betty/api:car992-fix into dev ... 1805ff93cf

fix: add UAT/dev domains to cors_origins (#14)

Refs: CAR-992

Merge pull request 'promote: dev → uat (CAR-992 cors_origins fix)' (#15) from dev into uat ... 06c6dbed5c

promote: dev → uat (CAR-992 cors_origins fix) (#15)

Merge pull request 'fix: remove dead dispose_engine import from API main.py [CAR-932]' (#16) from betty/car-932-fix-dispose-engine into dev ... 23899f6c8d

fix: remove dead dispose_engine import from API main.py [CAR-932]

Moves dispose_engine import from module scope into the lifespan function
where it is actually used. Fixes ImportError crashing API pods.

Reviewed-by: cs_charlie (QA)
Approved-by: cs_savannah (CTO)
CI-override: pre-existing failures unrelated to this change

Merge pull request 'chore: promote dev to uat (dispose_engine fix, CAR-932)' (#20) from dev into uat ... 28ad343759

chore: promote dev to uat (dispose_engine fix, CAR-932)

Fix CI pipeline failures: remove pip cache from setup-python, add missing env vars ... cf4b29b8d3

- Remove 'cache: pip' from setup-python in lint, typecheck, test jobs to fix
  intermittent 'archive/tar: write too long' errors on act_runner pods
- Add CARTSNITCH_SERVICE_KEY and CARTSNITCH_FERNET_KEY to test job env
  to satisfy Settings pydantic model requirements

Co-Authored-By: Paperclip <noreply@paperclip.ing>

fix: resolve lint errors in test files [CAR-932] ... ae2fc15a5b

Fix 56 lint errors in test files that were blocking CI:
- E501: Split long SQL INSERT statements across multiple lines
- F401: Remove unused imports (os, unittest.mock.patch)

Co-Authored-By: Paperclip <noreply@paperclip.ing>

Merge pull request 'Fix CI pipeline failures in cartsnitch/api' (#22) from cs_betty/api:barcode-betty/fix-ci-pipeline into dev ... 5c33b6ee38

Merge PR #22: Fix CI pipeline failures in cartsnitch/api

Fixes:
- Remove cache: pip from setup-python to fix intermittent tar corruption
- Add CARTSNITCH_SERVICE_KEY and CARTSNITCH_FERNET_KEY test env vars

Reviewed-by: Savannah Savings (CTO)
Approved-by: Checkout Charlie (QA)

Merge pull request 'Promote dev → uat: CI pipeline fixes (CAR-1000)' (#24) from dev into uat ... 50110a54b7

Promote dev → uat: CI pipeline fixes (CAR-1000)

Promotes PR #22 fixes to UAT environment.

ci: migrate from ghcr.io to Gitea built-in registry ... 9659e63208

- Update REGISTRY env var: ghcr.io -> git.farh.net
- Replace Docker Hub + GHCR login with Gitea login step
- Remove credentials blocks from postgres and redis service definitions
- Update deploy-dev/deploy-uat kustomize image refs to use $REGISTRY var

Fixes QA FAIL from PR #23: missing Gitea login step.
Co-Authored-By: Paperclip <noreply@paperclip.ing>

Merge pull request 'ci: migrate from ghcr.io to Gitea built-in registry' (#25) from fix/cart-995-gitea-registry-migration into dev ... 71cf0a4563

ci: migrate from ghcr.io to Gitea built-in registry (#25)

CAR-995: Update CI workflow to use Gitea built-in container registry.
- REGISTRY env var: ghcr.io -> git.farh.net
- Replace Docker Hub/GHCR login with direct docker login using github.token
- Remove Docker Hub credentials from service containers
- Update deploy kustomize image refs to use env vars

Merge pull request 'promote: dev → uat (CAR-995 CI registry migration)' (#27) from dev into uat ... 6799b0e7b1

promote: dev → uat (CAR-995 CI registry migration) (#27)

Merge pull request 'Fix ruff lint errors across codebase' (#28) from cs_betty/api:betty/car-932-lint-fixes into dev ... 4751154679

Merge PR #28: Fix ruff lint errors across codebase

Fix ruff lint errors across codebase ... c68838acf2

- Auto-fix F401 (unused imports) and I001 (unsorted imports) with ruff --fix
- Manually fix E501 (line too long) in alembic migrations and src/ models
- Run ruff format to ensure consistent formatting

Co-Authored-By: Paperclip <noreply@paperclip.ing>

fix: resolve email_inbound_token conflict in test fixtures c9fd066c31

fix: resolve email_inbound_token conflict in test fixtures ... cc6ca5982c

Rebase on latest dev and wrap SQL INSERT lines to honor ruff line-length=100.

Co-Authored-By: Paperclip <noreply@paperclip.ing>

Fix: strip PostgreSQL server_default from email_inbound_token for SQLite ... 0e3c9fb52e

The email_inbound_token column uses a PostgreSQL-only server_default
(gen_random_bytes/encode/trim) that SQLite cannot parse.
Strip the server_default before metadata.create_all() in both the
sync engine and async db_engine fixtures so tests run against SQLite.

Co-Authored-By: Paperclip <noreply@paperclip.ing>

Merge pull request 'Fix ruff lint errors across codebase' (#30) from barcode-betty/car-1004-fix-ruff-lint into dev ... ec4eaa1f03

Merge PR #30: Fix ruff lint errors across codebase

Fixes 56 ruff lint errors (E501, F401, I001) in cartsnitch/api.
QA: cs_charlie APPROVED
CTO: cs_savannah APPROVED

Merge pull request 'Promote dev → uat: ruff lint fixes (CAR-1004)' (#31) from dev into uat ... 21443a266a

Promote dev → uat: ruff lint fixes (CAR-1004)

Merge pull request 'Fix test failures: email_inbound_token server_default for SQLite' (#29) from betty/fix-email-inbound-token-tests into dev ... 280882f515

Fix test failures: email_inbound_token server_default for SQLite (#29)

Strip PostgreSQL-only server_default from email_inbound_token before SQLite create_all(). Add email_inbound_token to test user INSERT statements.

Reviewed-by: Savannah Savings (CTO)
Approved-by: Checkout Charlie (QA)

Merge pull request 'Promote dev → uat: test fixes (CAR-1006)' (#33) from dev into uat 0c0cc63d59

Savannah Savings merged commit 46906cc333 into main 2026-05-24 18:51:44 +00:00

Sign in to join this conversation.

Reviewers

No Reviewers

Labels

Clear labels

bug

Something isn't working

documentation

Improvements or additions to documentation

duplicate

This issue or pull request already exists

enhancement

New feature or request

good first issue

Good for newcomers

help wanted

Extra attention is needed

invalid

This doesn't seem right

question

Further information is requested

wontfix

This will not be worked on

No Label

Milestone

No items

No Milestone

Projects

Clear projects

No project

Assignees

Clear assignees

ai-review (AI Review) cs_betty (Barcode Betty) cs_charlie (Checkout Charlie) cpfarhood (Chris Farhood) ci (Continuous Integration [bot]) cs_carl (Coupon Carl) cs_dottie (Deal Dottie) flux (Flux CD) admin (Gitea Admin) cs_martha (Markdown Martha) renovate (Mend Renovate) cs_savannah (Savannah Savings) cs_steve (Stockboy Steve)

No Assignees

1 Participants

Notifications

Subscribe

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: cartsnitch/api#36