Skip to content

feat(runners): add workload logs rpc#131

Merged
vitramir merged 2 commits into
mainfrom
noa/issue-33-2
Apr 23, 2026
Merged

feat(runners): add workload logs rpc#131
vitramir merged 2 commits into
mainfrom
noa/issue-33-2

Conversation

@casey-brooks
Copy link
Copy Markdown
Contributor

Summary

  • add StreamWorkloadLogs RPC to RunnersService
  • add SERVICE_TYPE_RUNNERS enum value for service identities

Testing

  • buf lint

Refs agynio/runners#33

@casey-brooks casey-brooks requested a review from a team as a code owner April 23, 2026 18:01
@github-actions
Copy link
Copy Markdown

github-actions Bot commented Apr 23, 2026

The latest Buf updates on your PR. Results from workflow buf-pr / buf (pull_request).

BuildFormatLintBreakingUpdated (UTC)
✅ passed✅ passed✅ passed✅ passedApr 23, 2026, 6:22 PM

@casey-brooks
Copy link
Copy Markdown
Contributor Author

Test & Lint Summary

  • buf lint (no errors)

@casey-brooks
Copy link
Copy Markdown
Contributor Author

Test & Lint Summary (update)

  • buf lint (no errors)

Copy link
Copy Markdown

@noa-lucent noa-lucent left a comment

Choose a reason for hiding this comment

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

LGTM.

  • RunnersService.StreamWorkloadLogs added using the canonical runner.v1 request/response types.
  • SERVICE_TYPE_RUNNERS added to ziti management enum without disturbing reserved slots.

(Static review only; relying on CI/buf lint.)

@vitramir vitramir merged commit 99da041 into main Apr 23, 2026
1 check passed
casey-brooks added a commit that referenced this pull request Apr 25, 2026
* feat(runners): add log stream rpc

* style(api): format runners proto
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.

3 participants