Skip to content

Commit c7958df

Browse files
committed
docs: clarify service pre-registration ordering and add missing manifest fields
Signed-off-by: Abhinav Singh <abhinavsingh717073@gmail.com>
1 parent 06b58b8 commit c7958df

1 file changed

Lines changed: 5 additions & 0 deletions

File tree

docs/design/multi-agent-runtime-proposal.md

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -370,6 +370,9 @@ func (s *Server) createSandboxGroup(
370370
}
371371
372372
// Create a Headless Service for this role to provide a stable DNS endpoint.
373+
// NOTE: The service is intentionally created before injectDependencyEndpoints()
374+
// and registered in createdServices immediately, so that if the subsequent
375+
// injection step fails, the deferred rollback will still clean up this service.
373376
svcName, svcDNS, err := s.createHeadlessServiceForRole(
374377
ctx, mar.Namespace, groupSessionID, role.Name, sandbox,
375378
)
@@ -612,6 +615,8 @@ type CreateAgentGroupResponse struct {
612615
```go
613616
type AgentGroupManifest struct {
614617
GroupSessionID string `json:"groupSessionId"`
618+
StartupPolicy string `json:"startupPolicy"`
619+
Namespace string `json:"namespace"`
615620
Roles []AgentGroupRoleState `json:"roles"`
616621
CreatedAt time.Time `json:"createdAt"`
617622
}

0 commit comments

Comments
 (0)