Skip to content

Add gRPC definition for Python - #673

Merged
gaius-qi merged 2 commits into
mainfrom
feature/python
Aug 5, 2026
Merged

Add gRPC definition for Python#673
gaius-qi merged 2 commits into
mainfrom
feature/python

Conversation

@gaius-qi

@gaius-qi gaius-qi commented Aug 5, 2026

Copy link
Copy Markdown
Member

Description

This pull request adds support for building and publishing Python gRPC bindings for the Dragonfly API. It introduces automation for generating Python protobuf code, updates the release workflow to publish to PyPI, and adds necessary documentation and licensing for the new Python package.

Python gRPC bindings and packaging:

  • Added a new python/dragonfly_api package with an __init__.py file and included an Apache 2.0 LICENSE and a README.md with installation and usage instructions. [1] [2] [3]

Build and release automation:

  • Added a hack/protoc-python.sh script and corresponding Makefile targets to generate Python gRPC protobuf code from proto files, including fixes for import paths. [1] [2] [3] [4]
  • Updated the GitHub Actions release workflow to publish the Python package to PyPI automatically.

Other changes:

  • Bumped the Rust crate version to 2.3.0 in Cargo.toml.
  • Removed an unused import in proto/scheduler.proto.

Related Issue

Motivation and Context

Signed-off-by: Gaius <gaius.qi@gmail.com>
@gaius-qi gaius-qi self-assigned this Aug 5, 2026
@gaius-qi
gaius-qi requested review from a team as code owners August 5, 2026 03:26
@gaius-qi gaius-qi added the enhancement New feature or request label Aug 5, 2026
@gaius-qi
gaius-qi enabled auto-merge (squash) August 5, 2026 03:38
Removes the duplicate LICENSE from the python directory and instead copies the root LICENSE at build time, keeping a single source of truth.

Signed-off-by: Gaius <gaius.qi@gmail.com>

@BraveY BraveY left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

LGTM

@gaius-qi
gaius-qi merged commit c2a181f into main Aug 5, 2026
6 checks passed
@gaius-qi
gaius-qi deleted the feature/python branch August 5, 2026 03:49
@gaius-qi
gaius-qi restored the feature/python branch August 5, 2026 04:33
@gaius-qi
gaius-qi deleted the feature/python branch August 5, 2026 04:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants