Skip to content

fix: empty zeam_global_flags default for devnet3 image#148

Merged
ch4r10t33r merged 1 commit intomainfrom
fix/zeam-global-flags-empty-default
Mar 23, 2026
Merged

fix: empty zeam_global_flags default for devnet3 image#148
ch4r10t33r merged 1 commit intomainfrom
fix/zeam-global-flags-empty-default

Conversation

@ch4r10t33r
Copy link
Copy Markdown
Contributor

Problem

Merged #147 added --console-log-level debug before node. The published blockblaz/zeam:devnet3 image is built from an older Zeam that does not define that top-level CLI option, so the node exits with:
Unknown option '--console-log-level'.

Fix

  • Default no global flags in zeam-cmd.sh; opt in with ZEAM_GLOBAL_FLAGS when using a Zeam binary that supports them (e.g. local zig-out/bin/zeam).
  • Ansible: zeam_global_flags: "" by default; override in inventory when the image supports log flags.

Verification (reasoning)

  • Empty zeam_global_flags in bash: zeam $zeam_global_flags node word-splits to zeam then node (no unknown argv).
  • Ansible: empty Jinja expands to nothing; docker run ... IMAGE node remains valid (extra whitespace between args is fine).
  • Confirmed main still had hard-coded --console-log-level debug after client-cmds: default zeam console log level to debug #147 squash; this commit matches the follow-up that was on zeam-default-console-debug but not in the merge.

@ch4r10t33r ch4r10t33r marked this pull request as ready for review March 23, 2026 23:12
@ch4r10t33r ch4r10t33r merged commit 043b42d into main Mar 23, 2026
4 checks passed
@ch4r10t33r ch4r10t33r deleted the fix/zeam-global-flags-empty-default branch March 23, 2026 23:16
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