Skip to content

feat: api generation with goa - #10

Merged
jason-lynch merged 1 commit into
mainfrom
jason-lynch/api-scaffolding
Jan 23, 2025
Merged

feat: api generation with goa#10
jason-lynch merged 1 commit into
mainfrom
jason-lynch/api-scaffolding

Conversation

@jason-lynch

@jason-lynch jason-lynch commented Jan 15, 2025

Copy link
Copy Markdown
Member

This commit adds an api module that uses Goa to generate HTTP client and server code. The API specification so far is incomplete but does have some of the core entities and operations.

Some of the generated examples in the OpenAPI spec are wonky, such as repeating the same example for the ArrayOf properties. Hopefully I can fix these in a followup commit.

All of the code in the gen directory is generated based on the spec in the design directory.

This commit adds an `api` module that uses Goa to generate HTTP client
and server code. The API specification so far is incomplete but does
have some of the core entities and operations.

Some of the generated examples in the OpenAPI spec are wonky, such as
repeating the same example for the `ArrayOf` properties. Hopefully I
can fix these in a followup commit.
@jason-lynch
jason-lynch merged commit 9dd789c into main Jan 23, 2025
@jason-lynch
jason-lynch deleted the jason-lynch/api-scaffolding branch January 23, 2025 15:04
AntTheLimey added a commit that referenced this pull request Jul 20, 2026
CheckImageExists relied solely on a registry DistributionInspect, so a
locally-built image never pushed to a registry (or a private registry
without resolvable auth) failed spec validation. Add a local Docker
daemon ImageInspectWithRaw fallback: registry first, then local cache,
error only if both miss.

The two lookups get independent timeout budgets so a slow-failing
registry lookup cannot starve the local check of its context.

Finding #10 of the ColdFront single-node trial remediation.
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.

2 participants