Repository navigation
Conversation
| imageService := appimage.NewService() | ||
| networkService := appnetwork.NewService() | ||
| volumeService := appvolume.NewService() | ||
| computeService := appcompute.NewServiceWithRuntime( |
There was a problem hiding this comment.
This router now acts as the composition root for default in-memory services. The important dependency direction is API -> app interfaces plus store implementations supplied here, not app -> store concrete types.
| clock.Wall(), | ||
| idgen.Random(), | ||
| ) | ||
| networkService := appnetwork.NewServiceWithRepositories( |
There was a problem hiding this comment.
Network has multiple repositories earlier than the other services because networks, subnets, and ports already have independent state. This is the representative wiring pattern for services that grow more than one resource store.
| @@ -45,6 +45,31 @@ func (h Handler) getVolume(w http.ResponseWriter, r *http.Request) { | |||
| }) | |||
| } | |||
There was a problem hiding this comment.
Update endpoints follow the existing handler convention rather than introducing a generic patch layer. That keeps this slice scoped to the compatibility calls currently needed.
c9c3ad2 to
45e072e
Compare
ff34ab3 to
a7acbbd
Compare
|
Closing without merge. The repository was reset to a new root that keeps only the retained design decisions (see |
1 participant
Summary
Main Review Points
Verification