Skip to content

benchmarking: rename --lifecycle-mode values to sync/async and default to async - #3097

Merged
Zoe Zhao (zoez7) merged 1 commit into
agent-substrate:mainfrom
chw120:default-lifecycle-mode-pause
Oct 22, 2026
Merged

Zoe Zhao (zoez7) merged 1 commit into
agent-substrate:mainfrom
chw120:default-lifecycle-mode-pause

Conversation

@chw120

Copy link
Copy Markdown
Collaborator

Rename the --lifecycle-mode choices from suspend and pause to sync and async, and switch the default in lifecycle_mode.py and the boomer agentsession, ateenv, glutton, durdir, and sweperf user classes to async (DURABILITY_MODE_ASYNCHRONOUS) to match Control.SuspendActor's default (#2813).

  • Tests pass
  • Docs updated

…t to async

Rename the --lifecycle-mode choices from suspend and pause to sync and async, and switch the default in lifecycle_mode.py and the boomer agentsession, ateenv, glutton, durdir, and sweperf user classes to async (DURABILITY_MODE_ASYNCHRONOUS) to match Control.SuspendActor's default.

@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

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

nit: test names still say pause, can we make that _async? Max Smythe (@maxsmythe)

@chw120

Copy link
Copy Markdown
Collaborator Author

nit: test names still say pause, can we make that _async? Max Smythe (Max Smythe (@maxsmythe))

I thought about renaming them, but the test name is used as the GCS output prefix and BigQuery table name for the dashboard—renaming it would break historical data continuity unless we also migrate the GCS folders and recreate the BigQuery tables, so I left the names as-is.

@zoez7
Zoe Zhao (zoez7) added this pull request to the merge queue Oct 22, 2026
@github-merge-queue
github-merge-queue Bot removed this pull request from the merge queue due to failed status checks Oct 22, 2026
@zoez7
Zoe Zhao (zoez7) added this pull request to the merge queue Oct 22, 2026
Merged via the queue into agent-substrate:main with commit e88ffcd Oct 22, 2026
15 checks passed
@chw120
Chenyi Wang (chw120) deleted the default-lifecycle-mode-pause branch October 22, 2026 16:21
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.

4 participants