Skip to content

Conversation

@yuandrew
Copy link
Contributor

@yuandrew yuandrew commented Oct 24, 2025

NOTE: This PR needs a server patch release that includes sending this information in namespace capabilities.

What was changed

Call DescribeNamespace and gate worker heartbeating from the worker_heartbeat Capability

Why?

So older servers without this support don't get pinged and emit server error logs for unimplemented

Checklist

  1. Closes

  2. How was this tested:

  1. Any docs updates needed?

Note

Gate worker heartbeating via DescribeNamespace capabilities; shut down worker if unsupported/unimplemented; update tests and CLI override.

  • Core/Worker Heartbeats:
    • Call describe_namespace at startup and proceed only if namespace_info.capabilities.worker_heartbeats is true; otherwise log warning and shut down worker.
    • On record_worker_heartbeat returning UNIMPLEMENTED, shut down worker before exiting.
  • Tests:
    • Update heartbeat test to mock DescribeNamespace with worker_heartbeats=true; adjust imports accordingly.
  • Tooling:
    • Set CLI_VERSION_OVERRIDE to server-1-29-1 in .cargo/config.toml.

Written by Cursor Bugbot for commit e7499eb. This will update automatically on new commits. Configure here.

@yuandrew yuandrew requested a review from a team as a code owner October 24, 2025 22:06
cursor[bot]

This comment was marked as outdated.

cursor[bot]

This comment was marked as outdated.

Copy link
Member

@Sushisource Sushisource left a comment

Choose a reason for hiding this comment

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

This makes sense to me but looks like you'll need to update a mocks etc to fix tests

@yuandrew
Copy link
Contributor Author

Added mock, need to wait for next server release for integration tests to pass, now that we're disabling heartbeating for from this feature that is added in the next release

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