Skip to content

ci: drop the application scaffolding workflow - #12

Merged
agustincelentano merged 1 commit into
mainfrom
ci/drop-app-scaffolding-workflow
Sep 14, 2026
Merged

agustincelentano merged 1 commit into
mainfrom
ci/drop-app-scaffolding-workflow

Conversation

@agustincelentano

Copy link
Copy Markdown
Contributor

Why

ci.yml (ci-nullplatform) treats this repository as a nullplatform application:

- run: np build start
- run: docker build -t main .
- run: np asset push --type docker-image --source main

That is the workflow a scaffolded application repository gets. It worked while the root Dockerfile was the http-echo placeholder, because there was something to build.

#9 replaced that placeholder with Dockerfile.aurora-postgres-server and Dockerfile.aurora-postgres-db, so every push to main now fails:

ERROR: failed to build: failed to solve: failed to read dockerfile:
open Dockerfile: no such file or directory

Why removing rather than fixing

A service repository does not publish itself as an application asset. The images it ships are the two worker images, and those are built and registered by release.yml.

services-postgresql-rds, services-s-3 and services-dynamo-db carry no equivalent workflow — this one is scaffolding that outlived its Dockerfile.

ci.yml treats this repository as a nullplatform application: np build start,
docker build -t main ., np asset push. That made sense while the root Dockerfile
was the http-echo placeholder, and it is what scaffolded repositories get.

It is not what a service repository does. #9 replaced the placeholder with
Dockerfile.aurora-postgres-{server,db}, so 'docker build .' now fails with
'failed to read dockerfile: open Dockerfile: no such file or directory' on every
push to main. The images this repository actually publishes are built by
release.yml.

services-postgresql-rds, services-s-3 and services-dynamo-db carry no equivalent
workflow.

@sebastiancorrea81 sebastiancorrea81 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

@agustincelentano
agustincelentano merged commit a25cb64 into main Sep 14, 2026
6 checks passed
@agustincelentano
agustincelentano deleted the ci/drop-app-scaffolding-workflow branch September 14, 2026 19:47
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.

3 participants