Skip to content

Fake data plane installs no longer fail at startup on the --preview flag - #2884

Merged
Max Smythe (maxsmythe) merged 1 commit into
agent-substrate:mainfrom
haiyanmeng:fake-data-plane-preview-flag
Oct 19, 2026
Merged

Max Smythe (maxsmythe) merged 1 commit into
agent-substrate:mainfrom
haiyanmeng:fake-data-plane-preview-flag

Conversation

@haiyanmeng

@haiyanmeng haiyanmeng commented Oct 19, 2026 •

Copy link
Copy Markdown
Collaborator

ate-setup passes a --preview argument to every substrate binary it installs. Neither fake-atelet nor fake-workersync defined that flag, so both exited at startup with unknown flag: --preview, and a --fake-data-plane install never came up.

This defines --preview in both binaries and initializes the preview gates from it, the same way the other control-plane binaries do.

  • Tests pass
  • Appropriate changes to documentation are included in the PR (none needed)

ate-setup gives every substrate binary it installs a --preview argument,
and neither fake binary defined the flag, so both exited at start with
"unknown flag: --preview" and no --fake-data-plane install came up.
Define it in both and initialize the preview gates from it, as the other
control-plane binaries do.

@maxsmythe Max Smythe (maxsmythe) left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

LGTM

@maxsmythe
Max Smythe (maxsmythe) added this pull request to the merge queue Oct 19, 2026
Merged via the queue into agent-substrate:main with commit 5114b77 Oct 19, 2026
15 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area/benchmarking kind/bug Something isn't working / bugfixes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants