fix(ci): simplify Push step to match auth pattern (CAR-1362) #53

2026-06-10T04:09:25Z

Barcode Betty commented

2026-06-10 04:09:25 +00:00

fix(ci): simplify Push step to match auth pattern (CAR-1362)

The Push Docker image step is failing post-merge of CAR-1362 with buildx
unknown error after layers push successfully. The pre-existing failure
was masked by the cache export error (CAR-1362 took the cache out of the
way; the push failure became visible).

Simplify the Push step to match the proven-green cartsnitch/auth/ci.yml
pattern: drop file: ./Dockerfile (default is Dockerfile; identical
semantics) and build-args: APT_CACHE_BUST=... (only used to bust apt
cache in stage 1 of multi-stage build; not strictly needed for the
rebuilt image). Keep if: github.event_name == "push" so the job is
skipped on pull_request events (PR CI then runs lint/typecheck/test
without the build-and-push cost).

Diff: 4 lines removed from .gitea/workflows/ci.yml Push step.

This is a config simplification only — no app code, no build context,
no test changes. Auth repo has been using this exact shape successfully
throughout; api should match.

Refs:

CAR-1362 (parent — cache removal)
CAR-1357 (this PR's identifier per spec parity)
CAR-1356 (REGISTRY_TOKEN prerequisite)

cc @cpfarhood

fix(ci): simplify Push step to match auth pattern (CAR-1362) The Push Docker image step is failing post-merge of CAR-1362 with buildx `unknown` error after layers push successfully. The pre-existing failure was masked by the cache export error (CAR-1362 took the cache out of the way; the push failure became visible). Simplify the Push step to match the proven-green `cartsnitch/auth/ci.yml` pattern: drop `file: ./Dockerfile` (default is `Dockerfile`; identical semantics) and `build-args: APT_CACHE_BUST=...` (only used to bust apt cache in stage 1 of multi-stage build; not strictly needed for the rebuilt image). Keep `if: github.event_name == "push"` so the job is skipped on `pull_request` events (PR CI then runs lint/typecheck/test without the build-and-push cost). Diff: 4 lines removed from `.gitea/workflows/ci.yml` Push step. This is a **config simplification only** — no app code, no build context, no test changes. Auth repo has been using this exact shape successfully throughout; api should match. Refs: - [CAR-1362](/CAR/issues/CAR-1362) (parent — cache removal) - [CAR-1357](/CAR/issues/CAR-1357) (this PR's identifier per spec parity) - [CAR-1356](/CAR/issues/CAR-1356) (REGISTRY_TOKEN prerequisite) cc @cpfarhood

Barcode Betty added 1 commit 2026-06-10 04:09:26 +00:00

fix(ci): simplify Push step to match auth pattern (CAR-1362)

CI / lint (pull_request) Successful in 5s

Details

CI / typecheck (pull_request) Successful in 20s

Details

CI / test (pull_request) Successful in 22s

Details

CI / build-and-push (pull_request) Has been skipped

Details

354e26295c

The Push Docker image step is failing post-merge of CAR-1362 with
buildx "unknown" error after layers push successfully. The pre-existing
failure was masked by the cache export error.

Simplify the Push step to match the proven-green cartsnitch/auth/ci.yml
pattern: drop `file: ./Dockerfile` (default) and `build-args:`
(APT_CACHE_BUST is only used to bust apt cache in stage 1 of multi-
stage build, not needed for the rebuilt image). Keep `if: github.event_name
== "push"` to skip on pull_request events.

Diff: 4 lines removed from .gitea/workflows/ci.yml Push step.

Co-authored-by: Paperclip <noreply@paperclip.ing>

Barcode Betty merged commit a3a01eefe2 into dev

2026-06-10 04:10:58 +00:00

Barcode Betty referenced this issue from a commit

2026-06-10 04:16:23 +00:00

fix(ci): remove GHA cache + simplify Push to match auth (CAR-1357, CAR-1362)

Barcode Betty referenced this pull request

2026-06-10 04:16:47 +00:00

fix(ci): remove GHA cache + simplify Push to match auth (CAR-1357, CAR-1362) #54

Sign in to join this conversation.

1 Participants

Notifications

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: cartsnitch/api#53