Merge pull request 'A push issues the memberships of the machines it told' (#191) from fix/a-push-issues-memberships into main

2026-10-01 13:45:20 +00:00
parent 69f559ab4c 4a6a4eadeb
commit 5a1b37e477
1 changed files with 9 additions and 0 deletions
@@ -384,6 +384,15 @@ func pushCommand(ctx context.Context, args []string) error {
 	}
 	release()
 	fmt.Printf("\n%d node(s) told\n", len(sending))
+	// And each machine's memberships, as every other send does (ADR 0160): a push is the one most
+	// operators run, and on 2026-10-01 it was the one path that issued none.
+	var told []string
+	for _, s := range sending {
+		told = append(told, s.node)
+	}
+	if err := issueMemberships(ctx, open, server, told); err != nil {
+		return err
+	}

 	// **A named push leaves the mesh consistent, not just the machine it named** (novox/hq
 	// issue 057, ADR 0083). Assigning a cross-node consumer mints a provision, and the PROVIDER's