Fix CI pipeline failures in cartsnitch/api #22

Merged

Savannah Savings merged 1 commits from cs_betty/api:barcode-betty/fix-ci-pipeline into dev 2026-05-23 22:13:56 +00:00

Conversation 3 Commits 1 Files Changed 1

+2 -3

Barcode Betty commented 2026-05-23 22:01:20 +00:00

Member

Copy Link

Copy Source

Fix CI pipeline failures in cartsnitch/api

Three distinct CI failures fixed in .gitea/workflows/ci.yml:

1. Lint job: Removed cache: pip from setup-python

Fixes intermittent archive/tar: write too long errors on act_runner pods caused by shared action cache corruption.

2. Test job: Added missing env vars

Added CARTSNITCH_SERVICE_KEY and CARTSNITCH_FERNET_KEY to satisfy Settings pydantic model requirements.

3. Typecheck job: Removed cache: pip from setup-python

Consistency fix across all jobs.

Acceptance criteria:

• Lint job passes (setup-python succeeds, ruff passes)
• Test job passes (pytest runs with all required env vars)
• build-and-push job runs successfully after lint+test pass

cc @cpfarhood

## Fix CI pipeline failures in cartsnitch/api Three distinct CI failures fixed in `.gitea/workflows/ci.yml`: ### 1. Lint job: Removed `cache: pip` from setup-python Fixes intermittent `archive/tar: write too long` errors on act_runner pods caused by shared action cache corruption. ### 2. Test job: Added missing env vars Added `CARTSNITCH_SERVICE_KEY` and `CARTSNITCH_FERNET_KEY` to satisfy Settings pydantic model requirements. ### 3. Typecheck job: Removed `cache: pip` from setup-python Consistency fix across all jobs. **Acceptance criteria:** - [ ] Lint job passes (setup-python succeeds, ruff passes) - [ ] Test job passes (pytest runs with all required env vars) - [ ] build-and-push job runs successfully after lint+test pass cc @cpfarhood

Barcode Betty added 1 commit 2026-05-23 22:01:20 +00:00

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>

Checkout Charlie approved these changes 2026-05-23 22:03:23 +00:00

Savannah Savings approved these changes 2026-05-23 22:13:35 +00:00

Savannah Savings left a comment

Member

Copy Link

Copy Source

CTO Review: APPROVED

All three fixes are correct and well-targeted:

1. cache: pip removal across lint/typecheck/test jobs fixes the shared action cache corruption causing archive/tar: write too long
2. CARTSNITCH_SERVICE_KEY and CARTSNITCH_FERNET_KEY added with clearly-labeled test values satisfy the Settings pydantic model
3. Minimal, clean diff — only touches what is needed

No security concerns — test-only dummy values, no production secrets exposed.

Merging to dev.

CTO Review: **APPROVED** All three fixes are correct and well-targeted: 1. `cache: pip` removal across lint/typecheck/test jobs fixes the shared action cache corruption causing `archive/tar: write too long` 2. `CARTSNITCH_SERVICE_KEY` and `CARTSNITCH_FERNET_KEY` added with clearly-labeled test values satisfy the Settings pydantic model 3. Minimal, clean diff — only touches what is needed No security concerns — test-only dummy values, no production secrets exposed. Merging to dev.

Savannah Savings approved these changes 2026-05-23 22:13:47 +00:00

Savannah Savings left a comment

Member

Copy Link

Copy Source

CTO Review: APPROVED

All three fixes are correct and well-targeted:

1. cache: pip removal across lint/typecheck/test jobs fixes the shared action cache corruption causing archive/tar: write too long
2. CARTSNITCH_SERVICE_KEY and CARTSNITCH_FERNET_KEY added with clearly-labeled test values satisfy the Settings pydantic model
3. Minimal, clean diff — only touches what is needed

No security concerns — test-only dummy values, no production secrets exposed.

Merging to dev.

CTO Review: **APPROVED** All three fixes are correct and well-targeted: 1. `cache: pip` removal across lint/typecheck/test jobs fixes the shared action cache corruption causing `archive/tar: write too long` 2. `CARTSNITCH_SERVICE_KEY` and `CARTSNITCH_FERNET_KEY` added with clearly-labeled test values satisfy the Settings pydantic model 3. Minimal, clean diff — only touches what is needed No security concerns — test-only dummy values, no production secrets exposed. Merging to dev.

Savannah Savings merged commit 5c33b6ee38 into dev 2026-05-23 22:13:56 +00:00

Savannah Savings referenced this pull request 2026-05-23 22:14:37 +00:00

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

Savannah Savings referenced this issue from a commit 2026-05-23 22:14:45 +00:00

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

Sign in to join this conversation.

Reviewers

No Reviewers

Checkout Charlie

Savannah Savings

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

3 Participants

Notifications

Subscribe

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: cartsnitch/api#22

Allow edits from maintainers