Skip to content

Add initialization step logs - #414

Merged
RyanLee-Dev merged 1 commit into
mainfrom
codex/add-initialization-step-logs
Oct 4, 2026
Merged

RyanLee-Dev merged 1 commit into
mainfrom
codex/add-initialization-step-logs

Conversation

@RyanLee-Dev

@RyanLee-Dev RyanLee-Dev commented Oct 4, 2026 •

Copy link
Copy Markdown
Contributor

The ingress initialization command previously emitted only completion messages, making startup failures difficult to locate. It now uses the shared structured logger to record initialization steps, durations, metadata copies, credential creation or reuse, and the failing step. Logs include revision and trace identity without credential values or digests. Initialization remains offline and preserves existing restart behavior.

Updates deployment guidance and the English and Chinese operations guide.

Validation:

  • go test -race ./services/core/cmd/oac and go vet ./services/core/cmd/oac passed.
  • Documentation bundle checks, distribution manifest tests, translation checks, and git diff --check passed.
  • Local Linux amd64 ingress container checks with networking disabled passed for first initialization, restart, mismatched revision, and modified installation files; checked actual logs for credential leakage.
  • Independent diff review found no actionable issues.
  • The full distribution target did not pass locally: node tests require a Linux/service-user host setup absent on the macOS host and minimal test container. The locally installed golangci-lint was built with an older Go version and could not run.

View with [code]smith Autofix with [code]smith
Need help on this PR? Tag @codesmith-bot with what you need. Autofix is disabled.

@RyanLee-Dev
RyanLee-Dev merged commit 9ee8e01 into main Oct 4, 2026
19 checks passed
@RyanLee-Dev
RyanLee-Dev deleted the codex/add-initialization-step-logs branch October 4, 2026 04:06
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.

1 participant