Skip to content

feat(B-0855.1): add first-boot self-register service module#5416

Merged
AceHack merged 12 commits into
mainfrom
claim/codex-b0855-1-zeta-self-register-service-20260527
May 27, 2026
Merged

feat(B-0855.1): add first-boot self-register service module#5416
AceHack merged 12 commits into
mainfrom
claim/codex-b0855-1-zeta-self-register-service-20260527

Conversation

@AceHack
Copy link
Copy Markdown
Member

@AceHack AceHack commented May 27, 2026

Summary

  • add disabled-by-default zeta-self-register.service NixOS module for B-0855.1
  • import/expose the module from the cluster module surface
  • add source-level audit sentinels for first-boot ordering and env handoff

Checks

  • git diff --check
  • bun tools/ci/audit-installer-substrate.ts
  • bun test tools/ci/test-iter-54-install-flow.test.ts

Notes

  • nix-instantiate is not installed in this environment, so Nix parse/eval remains for CI/local Nix.
  • Draft until CI/review confirms the Nix service surface.

Lior and others added 2 commits May 27, 2026 03:10
…55.1

Co-Authored-By: Codex <noreply@openai.com>
…oot self-register service

Co-Authored-By: Codex <noreply@openai.com>
Copilot AI review requested due to automatic review settings May 27, 2026 07:19
…n PR #5416

Co-Authored-By: Codex <noreply@openai.com>
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Adds a disabled-by-default NixOS systemd service module to move node self-registration to post-install, first boot (B-0855.1), and strengthens CI/source audits to ensure the new substrate stays wired into the AI-cluster module surface.

Changes:

  • Add zeta-self-register NixOS module defining a zeta-self-register oneshot service gated by ConditionFirstBoot and ordered after network-online.target (and zeta-creds-restore.service when present).
  • Import/expose the new module via common.nix and full-ai-cluster/flake.nix module outputs.
  • Extend tools/ci/audit-installer-substrate.ts to require the new module file and assert key ordering/env-handoff sentinels.

Reviewed changes

Copilot reviewed 4 out of 4 changed files in this pull request and generated 1 comment.

Show a summary per file
File Description
tools/ci/audit-installer-substrate.ts Adds required-file + sentinel assertions for the new self-register module surface.
full-ai-cluster/nixos/modules/zeta-self-register.nix Introduces the disabled-by-default zeta-self-register first-boot oneshot systemd service module.
full-ai-cluster/nixos/modules/common.nix Imports zeta-self-register.nix so all node types share the module surface.
full-ai-cluster/flake.nix Exposes zeta-self-register in nixosModules outputs for reuse/consistency.
docs/claims/codex-b0855-1-zeta-self-register-service-20260527.md Adds a live work-claim marker for this effort.

Comment thread full-ai-cluster/nixos/modules/zeta-self-register.nix
@AceHack
Copy link
Copy Markdown
Member Author

AceHack commented May 27, 2026

CI failure inspection (Vera 2026-05-27T07:22Z): docker-nixos-install-sh-test fails during Docker build before the harness runs: /bin/sh: line 1: mkdir: command not found at tools/ci/dockerfiles/nixos-install-sh-test/Dockerfile:53. This PR does not modify that Dockerfile, and origin/main has the same lines 53-54, so this is not currently attributable to the B-0855.1 module patch. I am not rerunning it yet; next safe action is to wait for remaining checks and then decide whether this is an existing base-image/tooling blocker or needs a separate fix.

…register path defaults

Co-Authored-By: Codex <noreply@openai.com>
@AceHack
Copy link
Copy Markdown
Member Author

AceHack commented May 27, 2026

Codex/Vera opened Docker harness repair PR #5427 for the docker-nixos-install-sh-test blocker on this draft branch.

Local verification on the repair branch passed:

bun tools/ci/docker-nixos-install-sh-test.ts

Keeping this PR draft-blocked until #5427 lands or CI confirms the harness fix.

Co-Authored-By: Codex <noreply@openai.com>
Copilot AI review requested due to automatic review settings May 27, 2026 09:02
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 5 out of 5 changed files in this pull request and generated 1 comment.

Comment thread full-ai-cluster/nixos/modules/zeta-self-register.nix Outdated
Lior and others added 2 commits May 27, 2026 05:10
Co-Authored-By: Codex <noreply@openai.com>
Co-Authored-By: Codex <noreply@openai.com>
Copilot AI review requested due to automatic review settings May 27, 2026 09:24
Co-Authored-By: Codex <noreply@openai.com>
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 5 out of 5 changed files in this pull request and generated 2 comments.

Comment thread full-ai-cluster/nixos/modules/zeta-self-register.nix Outdated
Comment thread .github/workflows/build-ai-cluster-iso.yml Outdated
Lior and others added 2 commits May 27, 2026 05:37
Co-Authored-By: Codex <noreply@openai.com>
Address the unresolved operational review finding on the B-0855.1 service by replacing the first-boot-only gate with a marker-path gate and failure retry/backoff. Update the installer substrate audit so the retry semantics remain checked in CI.

Co-Authored-By: Codex <noreply@openai.com>
Copilot AI review requested due to automatic review settings May 27, 2026 09:48
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 5 out of 5 changed files in this pull request and generated 2 comments.

Comment thread .github/workflows/build-ai-cluster-iso.yml Outdated
Comment thread .github/workflows/build-ai-cluster-iso.yml Outdated
Address current Copilot workflow wording findings by matching the cosign bundle comment to the runner-temp output path and pointing verification guidance at the workflow run step summary rather than nonexistent workflow comments.

Co-Authored-By: Codex <noreply@openai.com>
@AceHack AceHack marked this pull request as ready for review May 27, 2026 10:10
Copilot AI review requested due to automatic review settings May 27, 2026 10:10
@chatgpt-codex-connector
Copy link
Copy Markdown

You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard.

Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 5 out of 5 changed files in this pull request and generated 2 comments.

Comment thread full-ai-cluster/nixos/modules/zeta-self-register.nix Outdated
Comment thread .github/workflows/build-ai-cluster-iso.yml Outdated
Co-Authored-By: Codex <noreply@openai.com>
@AceHack AceHack merged commit 815f1cf into main May 27, 2026
35 checks passed
@AceHack AceHack deleted the claim/codex-b0855-1-zeta-self-register-service-20260527 branch May 27, 2026 10:28
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